在开发中经常会遇到这样的问题,做了一个列表,给列表的每一项添加了按钮监听事件,但是在列表的数据很多的时候经常会出现点击后错乱的问题。对于这种问题,我们在程序中可能都有自己的解决办法,但是你也许第一次发现这个问题的时候会跟我之前一样手足无措。

    那么现在我们可以分析一下这种问题的根本原因。

    首先,我们来看一下一个出错的BaseAdapter。

package com.example.listdelectdemo;import java.util.ArrayList;import android.content.Context;import android.view.LayoutInflater;import android.view.View;import android.view.View.OnClickListener;import android.view.ViewGroup;import android.widget.BaseAdapter;import android.widget.Button;import android.widget.TextView;import android.widget.Toast;public class MyDataAdapter extends BaseAdapter {	private Context mContext;	private ArrayList
 mStrings; private LayoutInflater mInflater; private String mStrData; public MyDataAdapter(Context c, ArrayList
 s) { mContext = c; mStrings = s; mInflater = LayoutInflater.from(c); } @Override public int getCount() { return mStrings.size(); } @Override public Object getItem(int position) { return mStrings.get(position); } @Override public long getItemId(int position) { return position; } @Override public View getView(int position, View convertView, ViewGroup parent) { MyViewHolder viewHolder = null; if (convertView == null) { convertView = mInflater.inflate(R.layout.item, null); viewHolder = new MyViewHolder(); viewHolder.item_button_test = (Button) convertView.findViewById(R.id.item_button_test); viewHolder.item_textView_content = (TextView) convertView.findViewById(R.id.item_textView_content); convertView.setTag(viewHolder); } else { viewHolder = (MyViewHolder) convertView.getTag(); } //这里拿出来数据集合里的当前这一项mStrData  mStrData = mStrings.get(position); viewHolder.item_textView_content.setText(mStrData); viewHolder.item_button_test.setText("点击"); //这里给item的button设置了点击监听事件 viewHolder.item_button_test.setOnClickListener(new OnClickListener() { @Override public void onClick(View v) {         //这里toast出来的mStrData却不是点击的那一项 Toast.makeText(mContext, "您点击了-" + mStrData, Toast.LENGTH_LONG).show(); } }); return convertView; } class MyViewHolder { TextView item_textView_content; Button item_button_test; }}

    然后,我们分析一下原因,相信老程序员都可以看出问题的所在:

mStrData = mStrings.get(position);

getView方法第一次被调用的时候,将集合中的当前项数据拿出来付给了成员变量mStrData,程序继续往下执行:

viewHolder.item_button_test.setOnClickListener(new OnClickListener() {			@Override			public void onClick(View v) {			        //这里toast出来的mStrData却不是点击的那一项				Toast.makeText(mContext, "您点击了-" + mStrData, Toast.LENGTH_LONG).show();			}		});

这里给item的按钮添加了监听事件,但是要注意:程序并不会回调监听事件中的

@Overridepublic void onClick(View v) {    //这里toast出来的mStrData却不是点击的那一项    Toast.makeText(mContext, "您点击了-" + mStrData, Toast.LENGTH_LONG).show();}

而是会继续回调getView方法。

等到列表即将被加载完成,也就是最后一次回调getView方法时,成员变量mStrData会被最后一次赋值,

那么,getView方法每回调一次,mStrData的值就会被重新赋一次。

然后,当我们点击按钮,就会回调监听的onClick方法,这时候执行toast:

Toast.makeText(mContext, "您点击了-" + mStrData, Toast.LENGTH_LONG).show();

此时的mStrData就是只能是最后一次赋的值了,出错就是必然的。

    那么,来看一下我的解决方法:

package com.example.listdelectdemo;import java.util.ArrayList;import android.content.Context;import android.view.LayoutInflater;import android.view.View;import android.view.View.OnClickListener;import android.view.ViewGroup;import android.widget.BaseAdapter;import android.widget.Button;import android.widget.TextView;import android.widget.Toast;public class MyDataAdapter extends BaseAdapter {	private Context mContext;	private ArrayList
 mStrings; private LayoutInflater mInflater; private String mStrData; public MyDataAdapter(Context c, ArrayList
 s) { mContext = c; mStrings = s; mInflater = LayoutInflater.from(c); } @Override public int getCount() { return mStrings.size(); } @Override public Object getItem(int position) { return mStrings.get(position); } @Override public long getItemId(int position) { return position; } @Override public View getView(int position, View convertView, ViewGroup parent) { MyViewHolder viewHolder = null; if (convertView == null) { convertView = mInflater.inflate(R.layout.item, null); viewHolder = new MyViewHolder(); viewHolder.item_button_test = (Button) convertView.findViewById(R.id.item_button_test); viewHolder.item_textView_content = (TextView) convertView.findViewById(R.id.item_textView_content); convertView.setTag(viewHolder); } else { viewHolder = (MyViewHolder) convertView.getTag(); } mStrData = mStrings.get(position); viewHolder.item_textView_content.setText(mStrData); viewHolder.item_button_test.setText("点击"); // 给item的按钮设置点击监听,创建一个监听的实现类,并传入当前的position viewHolder.item_button_test.setOnClickListener(new MyAdapterListener(position)); return convertView; } class MyViewHolder { TextView item_textView_content; Button item_button_test; } class MyAdapterListener implements OnClickListener { private int position; public MyAdapterListener(int pos) { position = pos; } @Override public void onClick(View v) { Toast.makeText(mContext, "您点击了-" + mStrings.get(position), Toast.LENGTH_LONG).show(); } }}

这时候就不会出现问题。不同的地方就在于给item的button添加点击事件的时候是每次都创建一个新的MyAdapterListener对象来实现这个监听。那么在new这个MyAdapterListener对象的时候给他传入一个position做为标记,这样每一个item都会有一个属于自己的监听类,我们就可以在这个监听类中做一些自己的逻辑处理,就不会出现错乱的问题。

    这个方案只能作为一个参考方案,有一个弊端就是在列表数据多的时候,会创建很多新的对象,而占用内存。那么大家有什么更好的方案可以分享分享。