在列表視圖中單擊項目時位置錯誤

Question

前提條件：-

• 我有一個復選框，並在我的列表視圖2個textviews。
• 我的ArrayList大小是2所以正在創建兩行。

功能：-如果我選中該復選框，則相應的文本視圖將被啟用，因此將變為可單擊狀態。

此邏輯工作正常，但問題是， 如果我選中第0個位置復選框，則第1個位置的textviews將被啟用

另一種情況是，即使當檢查的第一個復選框僅啟用了第一個TextView的，在TextView的的聽眾我打電話日期選擇器和設置日期，但日期設置為第二的TextView

  @Override
  public View getView(int position, View convertView, ViewGroup parent) {
    View rowView = convertView;
    ViewHolder viewHolder = null;
    if (rowView == null) {
      LayoutInflater inflater = context.getLayoutInflater();
      rowView = inflater.inflate(R.layout.rowlayout, null);
      viewHolder = new ViewHolder();
      viewHolder.textview1 = (TextView) rowView.findViewById(R.id.TextView11);
      viewHolder.textview2 = (TextView) rowView.findViewById(R.id.TextView12);
      viewHolder.checkbox = (CheckBox) rowView.findViewById(R.id.checkBox1);
      rowView.setTag(viewHolder);
    }
    else{
        holder = (ViewHolder) rowView.getTag();
    }
    viewHolder.textview1.setEnabled(false) //default textview is false
    viewHolder.textview2.setEnabled(false) //default textview is false

    viewHolder.checkbox.setOnCheckedChangeListener(new OnCheckedChangeListener()
    {
        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked)
        {
            if ( isChecked )
            {
                holder.textview1.setEnabled(true);
                holder.textview2.setEnabled(true);
            }
            else if ( !isChecked )
            {
                holder.textview1.setEnabled(false);
                holder.textview2.setEnabled(false);
            }

    }
});

// Textview's listeners here- showing toast in it. 
//When checking first checkbox, second position textviews are getting enabled

    return rowView;
  }

這是我的完整列表視圖custyom類： -

    package com.list.filter.datefilter;

import java.util.ArrayList;
import java.util.Date;

import android.app.Activity;
import android.app.FragmentManager;
import android.content.Context;
import android.util.DisplayMetrics;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.ArrayAdapter;
import android.widget.CheckBox;
import android.widget.CompoundButton;
import android.widget.CompoundButton.OnCheckedChangeListener;
import android.widget.ListView;
import android.widget.TextView;

import com.exzeo.jarvis.ApplicationController;
import com.exzeo.jarvis.R;
import com.exzeo.jarvis.ui.DatePickerFragment;
import com.exzeo.jarvis.ui.DatePickerFragment.onDateSet;
import com.exzeo.jarvis.workards.list.filter.datefilter.model.AdvanceDateFilter;

public class AdvanceDateFilterList extends ListView{
    private boolean tabletSize = getResources().getBoolean(R.bool.isTablet);
    private CustomFilterAdapter dataAdapter = null;
    private Context context;
    private OnCheckChangedClickListener changedClickListener;

    public AdvanceDateFilterList(Context context, ArrayList<AdvanceDateFilter> filterArray) {
        super(context);
        this.context=context;
        displayListView((Activity) context, filterArray);
    }

    public void setOnCheckChangedClickListener(OnCheckChangedClickListener listener) {
        this.changedClickListener = listener;
    }

    public interface OnCheckChangedClickListener {
        public void onClick(ArrayList<AdvanceDateFilter> filter/*, int position*/); 
        //      public void onDateChange(AdvanceDateFilter filter, int position);
    }

    @Override
    public void setId(int id) {
        super.setId(id);
    }

    @Override
    public void setLayoutParams(android.view.ViewGroup.LayoutParams params) {

        //TODO:set height dynamic
        if (dataAdapter == null) {
            // pre-condition
            return;
        }

        int width = ApplicationController.getInstance().getDeviceWidth();
        int height = ApplicationController.getInstance().getDeviceHeight();

        if(height<width)
            height=width;

        int totalHeight = 0;
        for (int i = 0; i < dataAdapter.getCount(); i++) {
            totalHeight +=height/10;
        }
        if(totalHeight>(height/4))
            totalHeight=height/4;
        params.height = totalHeight + (this.getDividerHeight() * (dataAdapter.getCount() - 1));
        /*
        if(tabletSize){

        for (int i = 0; i < dataAdapter.getCount(); i++) {
            totalHeight +=height/10;
        }
        if(totalHeight>(height/4))
            totalHeight=height/4;
        params.height = totalHeight + (this.getDividerHeight() * (dataAdapter.getCount() - 1));

        }else{
            for (int i = 0; i < dataAdapter.getCount(); i++) {
                if(tabletSize)
                totalHeight +=height/25;
                else
                    totalHeight +=height/18;
            }
            if(totalHeight>(height/3))
                totalHeight=height/3;
            params.height = totalHeight + (this.getDividerHeight() * (dataAdapter.getCount() - 1));


        }*/
        params.width = -1;
//      params.height = -1;
        super.setLayoutParams(params);
    }
    private void displayListView(Activity activity, ArrayList<AdvanceDateFilter> filterArray) {

        dataAdapter = new CustomFilterAdapter(activity, context, R.layout.advance_date_filter_row, filterArray);
        setAdapter(dataAdapter);

    }

    private class CustomFilterAdapter extends ArrayAdapter<AdvanceDateFilter> {

        private ArrayList<AdvanceDateFilter> filterList;
        private ViewHolder holder = null;
        private Activity activity;
        private int dateCheck = -1;
        FragmentManager fragmentManager;
        DatePickerFragment newFragment;
        public CustomFilterAdapter(Activity activity, Context context, int textViewResourceId,  ArrayList<AdvanceDateFilter> filterList) {
            super(context, textViewResourceId, filterList);
            this.filterList = new ArrayList<AdvanceDateFilter>();
            this.filterList.addAll(filterList);
            this.activity=activity;
            fragmentManager =((Activity) context).getFragmentManager();
            newFragment = new DatePickerFragment(context);
        }

        private class ViewHolder {
            CheckBox datefiltercheckbox;
            TextView fromDate, toDate;
        }

        @Override
        public int getCount() {
            return filterList.size();
        }

        @Override
        public View getView(final int position, View convertView, ViewGroup parent) {

            if (convertView == null) {
                LayoutInflater vi = (LayoutInflater) activity.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
                convertView = vi.inflate(R.layout.advance_date_filter_row, null);
                holder = new ViewHolder();

                holder.fromDate = (TextView) convertView.findViewById(R.id.fromdate);
                holder.toDate = (TextView) convertView.findViewById(R.id.todate);
                holder.datefiltercheckbox = (CheckBox) convertView.findViewById(R.id.datefiltercheckbox);
                convertView.setTag(holder);

            } else {
                holder = (ViewHolder) convertView.getTag();
            }

            holder.fromDate.setEnabled(false);
            holder.toDate.setEnabled(false);

            holder.fromDate.setText("From: "+filterList.get(position).getFromDate());
            holder.toDate.setText("To: "+filterList.get(position).getToDate());         

            if(filterList.get(position).getTypeOfDate().equals("WORKKARDLOSSDATEFILTER"))
                holder.datefiltercheckbox.setText("Loss Date");
            else if (filterList.get(position).getTypeOfDate().equals("WORKKARDREPORTEDDATEFILTER")) {
                holder.datefiltercheckbox.setText("Reported Date");
            }

            int checkBoxPaddingLeft = (int) (getResources().getDimension(R.dimen.rightDrawerFilterCheckbox)/getResources().getDisplayMetrics().density);
            holder.datefiltercheckbox.setPadding(checkBoxPaddingLeft, 0, 0, 0);

            holder.datefiltercheckbox.setOnCheckedChangeListener(null);
            holder.datefiltercheckbox.setChecked(filterList.get(position).isSelected());
            holder.datefiltercheckbox.setTag(filterList.get(position));

            if(filterList.get(position).isSelected()){
                holder.fromDate.setEnabled(true);
                holder.toDate.setEnabled(true);
                if(dateCheck == 0){
                    holder.fromDate.setText("From: "+filterList.get(position).getFromDate());
                }
                else{
                    holder.toDate.setText("To: "+filterList.get(position).getToDate());
                }

            }
            else{
                holder.fromDate.setEnabled(false);
                holder.toDate.setEnabled(false);
                if(dateCheck == 0){
                    holder.fromDate.setText("From: "+filterList.get(position).getFromDate());
                }
                else{
                    holder.toDate.setText("To: "+filterList.get(position).getToDate());
                }
            }
            newFragment.setDateListener(new onDateSet() {

                @Override
                public void onDate(int year, int monthOfYear, int dayOfMonth) {

                    if(dateCheck == 0){
                        filterList.get(position).setFromDate(dayOfMonth+"/"+monthOfYear+"/"+year);
                        holder.fromDate.setText("From: "+filterList.get(position).getFromDate());

                    }
                    else{
                        filterList.get(position).setToDate(dayOfMonth+"/"+monthOfYear+"/"+year);
                        holder.toDate.setText("To: "+filterList.get(position).getToDate());
                    }

                    //              changedClickListener.onDateChange(filterList.get(position), position);
                    changedClickListener.onClick(filterList);
                }
            });


            holder.datefiltercheckbox.setOnCheckedChangeListener(new OnCheckedChangeListener() {

                @Override
                public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                    if(isChecked){
                        holder.fromDate.setEnabled(true);
                        holder.toDate.setEnabled(true);
                    }
                    else{
                        holder.fromDate.setEnabled(false);
                        holder.toDate.setEnabled(false);
                    }
                    notifyDataSetChanged();
                    filterList.get(position).setSelected(isChecked);

                    changedClickListener.onClick(filterList);

                }
            });

            holder.fromDate.setOnClickListener(new View.OnClickListener() {

                @Override
                public void onClick(View v) {
                    dateCheck = 0;
                    newFragment.show(fragmentManager, "datePicker");
                    newFragment.setDateRange(new Date(filterList.get(position).getFromDate()).getTime(), 
                            new Date(filterList.get(position).getToDate()).getTime());


                }
            });

            holder.toDate.setOnClickListener(new View.OnClickListener() {

                @Override
                public void onClick(View v) {
                    dateCheck = 1;
                    newFragment.show(fragmentManager, "datePicker");
                    newFragment.setDateRange(new Date(filterList.get(position).getFromDate()).getTime(), 
                            new Date(filterList.get(position).getToDate()).getTime());
                }
            });

            return convertView;
        }

    }
}

Answer 1

看看這個關於在列表項中添加多個可點擊視圖的出色教程。 您問題的主要根源可能是列表視圖回收。 看看這個：

http://cyrilmottier.com/2011/11/23/listview-tips-tricks-4-add-several-clickable-areas/

Answer 2

如您在回答中所說，由於膨脹操作大量使用CPU，因此android正在回收視圖，因此，android嘗試減少膨脹操作量。 我真的不認為假設rowView為null根本不是個好主意。 我認為您需要做的是，在onChange偵聽器中，檢查包含要更改的文本的rowView是否與應更改的文本相同，否則，由於視圖已被更改，因此無需更改該文本。已回收並且不再可見。 當適配器再次需要該視圖時，將正確設置文本視圖。

Answer 3

我注意到，您在接口onCheckedChanged()上都選中了兩個復選框，可以使用一個參數來查看按下了哪個：

viewHolder.checkbox.setOnCheckedChangeListener(new OnCheckedChangeListener()
{
    public void onCheckedChanged(CompoundButton buttonView, boolean isChecked)
    {
        if ( isChecked )
        {  
          switch(buttonView.getId()){
           case R.id.checkbox1:
           holder.textview1.setEnabled(true);                 
           break;
           case R.id.checkbox2:
           holder.textview2.setEnabled(true);                
           break;
          }
        }
        else if ( !isChecked )
        {
            holder.textview1.setEnabled(false);
            holder.textview2.setEnabled(false);
        }
}
});

Answer 4

我已經遇到了這個問題。 我因為視圖被系統或其他東西重用。

只要將if (rowView == null)設為true 。 我的意思是改變

if (rowView == null) {
  LayoutInflater inflater = context.getLayoutInflater();
  rowView = inflater.inflate(R.layout.rowlayout, null);
  viewHolder = new ViewHolder();
  viewHolder.textview1 = (TextView) rowView.findViewById(R.id.TextView11);
  viewHolder.textview2 = (TextView) rowView.findViewById(R.id.TextView12);
  viewHolder.checkbox = (CheckBox) rowView.findViewById(R.id.checkBox1);
  rowView.setTag(viewHolder);
}
else{
    holder = (ViewHolder) rowView.getTag();
}

至

  LayoutInflater inflater = context.getLayoutInflater();
  rowView = inflater.inflate(R.layout.rowlayout, null);
  viewHolder = new ViewHolder();
  viewHolder.textview1 = (TextView) rowView.findViewById(R.id.TextView11);
  viewHolder.textview2 = (TextView) rowView.findViewById(R.id.TextView12);
  viewHolder.checkbox = (CheckBox) rowView.findViewById(R.id.checkBox1);
  rowView.setTag(viewHolder);