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

Question

我有一个与自定义 SimpleAdapter 关联的 ListView 。我制作了自定义 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 Activity 中，我像这样关联我的自定义 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

你的逻辑有问题。 Adapter的getView方法被多次调用，一个View可以在不同的时间段展示不同的item。

那么它是如何工作的:

当 ListView 需要一个项目时，它会调用 Adapter 类中的方法 getView(无论此 View 应该显示给用户还是仅用于计算 View 项目的大小都无关紧要)。 ListView 传递 convertView 参数 - 这是此适配器一段时间前返回的旧 View ，现在不需要了。可以重复使用它而不是再次膨胀( super 方法是这样工作的)。这是您的代码问题。如果将旧 View 传递给您的 getView 方法，则它已经具有标记对象。而你的 (holder == null) == false。因此代表一篇文章的 View 会打开另一篇文章。

尝试改变你的逻辑来修正这个错误。最好的方法是只创建一次 holder，每次都设置 view listeners。

希望这会有所帮助。

编辑。这里有两种解决问题的方法。

@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;
}