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

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

首先，我们来看一下一个出错的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的按钮添加了监听事件，但是要注意：程序并不会回调监听事件中的@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 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都会有一个属于自己的监听类，我们就可以在这个监听类中做一些自己的逻辑处理，就不会出现错乱的问题。

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