Question

我有一个自定义SimpleAdapter关联的列表视图。我创建了自定义SimpleAdapter，因为我想将不同的事件处理程序与包含的textview和imageview相关联，而不是列表项小部件的其余部分。所以，基本上我有两个不同的事件处理程序，用于列表项的两个部分。

我的SimpleAdapter自定义是：

class ClickableButtonListAdapter extends SimpleAdapter {

    private static class ViewHolder {
        TextView text;
        ImageView image;
    }

    public ClickableButtonListAdapter(Context context,
        List<? extends Map<String, ?>> data, int resource, String[] from, int[] to) {
        super(context, data, resource, from, to);
    }

    @SuppressWarnings("unchecked")
    @Override
    public View getView(final int position, View convertView, ViewGroup parent) {
        final View view = super.getView(position, convertView, parent);
        ViewHolder holder = (ViewHolder) view.getTag();
        if(holder == null) {
            holder = new ViewHolder();
            holder.text = (TextView) view.findViewById(R.id.comments);
            holder.image = (ImageView) view.findViewById(R.id.arrow);
            view.setTag(holder);
            final Context context = view.getContext();
            final HashMap<String, String> article = (HashMap<String,String>) getItem(position);
            OnClickListener listener = new OnClickListener() {
                @Override
                public void onClick(View view) {
                    String item_id = article.get("item_id");
                    Intent intent = new Intent(context, HNewsCommentsActivity.class);
                    intent.putExtra("item_id", item_id);
                    context.startActivity(intent);
                }
            };
            holder.text.setOnClickListener(listener);
            holder.image.setOnClickListener(listener);
        }
        return view;
    }
}

稍后，在我的onCreate活动中，我将我的自定义SimpleAdapter关联起来：

    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
        try {
            final ArrayList<HashMap<String, String>> articles = getHNewsFeed();
            final SimpleAdapter adapter = new ClickableButtonListAdapter(this,
                    articles, R.layout.article,
                    new String[] {"title", "urlShort", "score", "comments", "item_id"},
                    new int[] {R.id.title, R.id.url, R.id.score, R.id.comments, R.id.item_id}
                    );
            final ListView l = (ListView) findViewById(android.R.id.list);
            l.setAdapter(adapter);
            l.setOnItemClickListener( new OnItemClickListener() {
                @Override
                public void onItemClick(AdapterView<?> parent, View v, int position, long id) {
                    final HashMap<String, String> article = articles.get(position);
                    String url = article.get("url");
                    Intent intent = new Intent(Intent.ACTION_VIEW, Uri.parse(url));
                    startActivity(intent);
                }
            });
        } catch (Exception e) {
            Log.w(TAG, e.getMessage());
            Toast.makeText(this, e.getMessage(), Toast.LENGTH_SHORT).show();
        }
    }

我的问题是这些与包含的textview和imageview相关联的事件处理程序（我在自定义SimpleAdapter的getView中初始化）似乎得到了错误的数据源项，并在单击时显示不同文章的数据。但是，listview项单击处理程序获取正确的数据源项。有人可以帮我指出为什么我与自定义SimpleAdapter的getView中的textview和imageview关联的事件处理程序无法正常工作？我的理解是SimpleAdapter的'getItem（position）'方法应该返回正确的数据源项。但出于某种原因，似乎并没有这样做。

Answer 1

你的逻辑有误。适配器的getView方法被调用很多次，一个View可以在不同的时间段显示不同的项目。

它是如何运作的：

当ListView想要一个项目时，它从Adapter类调用方法getView（如果该视图应该显示给用户或仅用于计算视图项目的大小，则无关紧要）。 ListView传递convertView参数 - 这是此适配器在一段时间之前返回的旧视图，现在不需要。可以重复使用它而不是再次膨胀（超级方法就像这样工作）。这是你的代码问题。如果将旧视图传递给getView方法，则它已经具有标记对象。而你的（持有人== null）== false。因此，代表一篇文章的视图会打开另一篇文章。

尝试改变你的逻辑来修复这个错误。最好的方法是只创建一次holder，每次都设置视图监听器。

希望这会有所帮助。

EDIT。以下是解决问题的两种方法。

@SuppressWarnings("unchecked")
@Override
public View getView(final int position, View convertView, ViewGroup parent) {
    final View view = super.getView(position, convertView, parent);
    ViewHolder holder = (ViewHolder) view.getTag();
    if(holder == null) {
        holder = new ViewHolder();
        holder.text = (TextView) view.findViewById(R.id.comments);
        holder.image = (ImageView) view.findViewById(R.id.arrow);
        view.setTag(holder);
    }

    final Context context = view.getContext();
    final HashMap<String, String> article = (HashMap<String,String>) getItem(position);
    OnClickListener listener = new OnClickListener() {
        @Override
        public void onClick(View view) {
            String item_id = article.get("item_id");
            Intent intent = new Intent(context, HNewsCommentsActivity.class);
            intent.putExtra("item_id", item_id);
            context.startActivity(intent);
        }
    };
    holder.text.setOnClickListener(listener);
    holder.image.setOnClickListener(listener);

    return view;
}

第二，我更爱。感谢Vinay S Shenoy的这个版本。

@SuppressWarnings("unchecked")
@Override
public View getView(final int position, View convertView, ViewGroup parent) {
    final HashMap<String, String> article = (HashMap<String,String>) getItem(position);

    final View view = super.getView(position, convertView, parent);
    ViewHolder holder = (ViewHolder) view.getTag();
    if(holder == null) {
        holder = new ViewHolder();
        holder.text = (TextView) view.findViewById(R.id.comments);
        holder.image = (ImageView) view.findViewById(R.id.arrow);
        view.setTag(holder);
        final Context context = view.getContext();
        OnClickListener listener = new OnClickListener() {
            @Override
            public void onClick(View view) {
                String item_id = view.getTag();
                //String item_id = article.get("item_id");
                Intent intent = new Intent(context, HNewsCommentsActivity.class);
                intent.putExtra("item_id", item_id);
                context.startActivity(intent);
            }
        };
        holder.text.setOnClickListener(listener);
        holder.image.setOnClickListener(listener);
    }

    holder.text.setTag(article.get("item_id"));
    holder.image.setTag(article.get("item_id"));

    return view;
}

Answer 2

问题是因为文章对象是在OnClickListener之外创建的。

您需要做的是将item_id作为标记保存在TextView和ImageView中。使用ID，tag_holder和tag_item_id在res / strings.xml文件中添加两个字符串。然后，像这样更改你的getView（）。

 @Override
    public View getView(final int position, View convertView, ViewGroup parent) {
        final View view = super.getView(position, convertView, parent);
        ViewHolder holder = (ViewHolder) view.getTag(R.string.tag_holder);
        if(holder == null) {
            holder = new ViewHolder();
            holder.text = (TextView) view.findViewById(R.id.comments);
            holder.image = (ImageView) view.findViewById(R.id.arrow);
            view.setTag(R.string.tag_holder, holder);
            final Context context = view.getContext();

            OnClickListener listener = new OnClickListener() {
                @Override
                public void onClick(View view) {
                    String item_id = (String)view.getTag(R.string.tag_item_id);
                    //String item_id = article.get("item_id");
                    Intent intent = new Intent(context, HNewsCommentsActivity.class);
                    intent.putExtra("item_id", item_id);
                    context.startActivity(intent);
                }
            };
            holder.text.setOnClickListener(listener);
            holder.image.setOnClickListener(listener);
        }

         final HashMap<String, String> article = (HashMap<String,String>) getItem(position);

            holder.text.setTag(R.string.tag_item_id, article.get("item_id");
            holder.image.setTag(R.string.tag_item_id, article.get("item_id");
        return view;
    }

此外，我建议不要为每个列表项创建新的OnClickListener（），因为此OnClick侦听器仅依赖于View标签。相反，在适配器的构造函数中实例化您的侦听器，并对所有List项使用相同的实例。

Android SimpleAdapter错误的数据项与列表行相关联

2 个答案: