带有 onclicklistener 问题的 Arrayadapter

Question

这是我的自定义 arrayadapter 类，我想在单击它时更改每一行中按钮的图像资源，它可以工作，但另一行中的另一个按钮也会更改。感谢您的帮助！

public class Coursadapter extends ArrayAdapter<String> {

    Context context;
    int layoutResourceId;
    ArrayList<String> data = null;
    WeatherHolder holder;

    public Coursadapter(Context context, int layoutResourceId,
            ArrayList<String> data) {
        // super(context, layoutResourceId, data, coeff);
        super(context, layoutResourceId, data);
        this.layoutResourceId = layoutResourceId;
        this.context = context;
        this.data = data;

    }

    public View getView(int position, View convertView, ViewGroup parent) {
        View row = convertView;

        if (row == null) {
            LayoutInflater inflater = ((Activity) context).getLayoutInflater();
            row = inflater.inflate(layoutResourceId, null);

            holder = new WeatherHolder();
            holder.name = (TextView) row.findViewById(R.id.item_cours_name);
            holder.b = (ImageButton) row.findViewById(R.id.button);
            holder.b.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    // TODO Auto-generated method stub
                    WeatherHolder w = (WeatherHolder) v.getTag();
                    w.b.setImageResource(R.drawable.butgreen);
                }
            });
            row.setTag(holder);

        } else {
            holder = (WeatherHolder) row.getTag();
        }

        holder.b.setTag(holder);
        String name1 = data.get(position);
        holder.name.setText(name1);

        return row;
    }

    static class WeatherHolder {

        TextView name;
        ImageButton b;
    }
}

Answer 1

您可以声明变量 int 来保存所选项目的位置和

public void onItemClick(AdapterView<?> adapter, View arg1, int position, long id) {
            mSelectedItem = position;
            mAdapter.notifyDataSetChanged();
    }

并在您的自定义适配器 getView 方法中检查位置并更改按钮的图像

public View getView(int position, View convertView, ViewGroup parent) {
        if (position == mSelectedItem) {
        // set your image
    }

看我和我一起尝试这段代码并工作，在你的活动中实现 OnClickListener

public class MainActivity extends Activity  implements OnClickListener

在你的 getView 方法中

public View getView(int position, View convertView, ViewGroup parent) {
        View row = convertView;

        if (row == null) {
            LayoutInflater inflater = ((Activity) context)
                    .getLayoutInflater();
            row = inflater.inflate(layoutResourceId, null);

            holder = new WeatherHolder();
            holder.name = (TextView) row.findViewById(R.id.item_cours_name);
            holder.b = (ImageButton) row.findViewById(R.id.button);
            holder.b.setOnClickListener(MainActivity.this);

并实现Onclick方法设置资源

@Override
public void onClick(View v) {
    WeatherHolder w = (WeatherHolder) v.getTag();
    w.b.setImageResource(android.R.drawable.star_big_on);
}

Answer 2

您的代码的问题是您的 OnClickListener 设置在错误的位置。

考虑getView() 中的 if/else 语句会发生什么。如果传递给该方法的convertView 为空，则可能是第一次创建 View（以及包含它的 ListView） - 例如，您正在启动包含该 ListView 的活动。如果它不为 null，则表示您正在回收一个曾经但现在不在屏幕上的视图。

如果您仔细观察，您会发现您的 OnClickListener 仅在第一种情况下设置 - 一个全新的视图。一旦你稍微滚动一下，你的监听器将永远不会正确地附加到那些来自屏幕外的视图。

更好的getView() 方法是：

public View getView(int position, View convertView, ViewGroup parent) {
    View row = convertView;

    if (row == null) {
        LayoutInflater inflater = ((Activity) context).getLayoutInflater();
        row = inflater.inflate(layoutResourceId, null);

        holder = new WeatherHolder();
        holder.name = (TextView) row.findViewById(R.id.item_cours_name);
        holder.b = (ImageButton) row.findViewById(R.id.button);

        row.setTag(holder);

    } else {
        holder = (WeatherHolder) row.getTag();
    }

    String name1 = data.get(position);
    holder.name.setText(name1);

    holder.b.setOnClickListener(new OnClickListener() {
            @Override
            public void onClick(View v) {                    
                ((ImageButton)v).setImageResource(R.drawable.butgreen);
            }
        });

    return row;
}

编辑：在我下面的评论的另一个修复中添加。

Answer 3

发生这种情况的原因是适配器在滚动时回收视图。

例如，假设您总共有 15 行，并且一次有 5 行适合屏幕。 if(row == null) 部分仅对前 5 行执行。当您向下滚动到第 6 行时，它使用第 1 行的视图。这就是为什么你应该在 if-else 子句之后设置 View 的内容。所以现在，当第 6 行进入屏幕时，行不为空。它拥有第一行的视图。当它执行 holder.name.setText(name1); 时，它将第 1 行的名称替换为第 6 行的名称。同样，第 7 行使用第 2 行的视图，依此类推。

现在，假设您单击第一行的按钮。第一行的视图发生变化， w.b.setImageResource(R.drawable.butgreen);设置第一行按钮的图片资源。当您滚动到第 6 行时，它使用第 1 行的视图。但是您仅将 holder.name 设置为第 6 行的名称。第一行按钮的图像资源将保留在那里，从而导致您的问题。

解决方案：创建另一个 ArrayList 并将按钮的状态存储在其中。因此，在您的情况下，我假设按钮的颜色为“红色”或“绿色”。您可以在每个位置存储“红色”或“绿色”，表示每个按钮当前的颜色（您可以将此 ArrayList 初始化为红色）。然后，当每行加载时，您可以设置图像资源以及该行的名称。看看下面的代码。我已将 cmets 添加到我添加的行中。

public class Coursadapter extends ArrayAdapter<String> {

Context context;
int layoutResourceId;
ArrayList<String> data = null;
WeatherHolder holder;
ArrayList<String> colors; // Stores the color of each button

public Coursadapter(Context context, int layoutResourceId, ArrayList<String> data) {
    // super(context, layoutResourceId, data, coeff);
    super(context, layoutResourceId, data);
    this.layoutResourceId = layoutResourceId;
    this.context = context;
    this.data = data;
    this.colors = new ArrayList<>(Collections.nCopies(data.size(), "red")); // initialize all the buttons to red color
}

public View getView(final int position, View convertView, ViewGroup parent) {
    View row = convertView;


    if(row == null)
    {
        LayoutInflater inflater = ((Activity)context).getLayoutInflater();
        row = inflater.inflate(layoutResourceId, parent, false);

        holder = new WeatherHolder();
        holder.name = (TextView)row.findViewById(R.id.item_cours_name);
        holder.b=(ImageButton) row.findViewById(R.id.button);
        holder.b.setTag(holder);
        row.setTag(holder);
    }
    else
    {
        holder = (WeatherHolder)row.getTag();
    }

    if(colors.get(position).equals("red")) {
        holder.b.setImageResource(R.drawable.butred); // set the colors of the button to red or whatever you need
    } else {
        holder.b.setImageResource(R.drawable.butgreen); // set the colors of the button to green
    }

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

        @Override
        public void onClick(View v) {
            // TODO Auto-generated method stub
            WeatherHolder w = (WeatherHolder) v.getTag();
            //w.b.setImageResource(R.drawable.butgreen);
            colors.set(position, "green"); // update the ArrayList colors
        }
    });
    String name1 = data.get(position);
    holder.name.setText(name1);


    return row;
}

static class WeatherHolder
{

    TextView name;
    ImageButton b;
}
}