Android中ListView的item按钮监听错乱问题解决办法

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

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

首先，我们来看一下一个出错的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<String> mStrings;
	private LayoutInflater mInflater;
	private String mStrData;

	public MyDataAdapter(Context c, ArrayList<String> 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的按钮添加了监听事件，但是要注意：程序并不会回调监听事件中的

@Override
public 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<String> mStrings;
	private LayoutInflater mInflater;
	private String mStrData;

	public MyDataAdapter(Context c, ArrayList<String> 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都会有一个属于自己的监听类，我们就可以在这个监听类中做一些自己的逻辑处理，就不会出现错乱的问题。

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