Listview-更改项目的背景色

Question

我试图更改用户已经在列表视图中单击的行的背景色，为此，我使用的是保存在SharedPreferences中的arraylist，以使被单击的行的ID继续，问题是颜色几乎是随机变化的。

码：

    private static class ItemHolder {
        public TextView TXTTitle, TXTArtist, TXTid, TXTLikes;
        public RelativeLayout back;
    }

    @Override
    public View getView(final int position, View view, ViewGroup root) {
        ItemHolder holder = new ItemHolder();

        if (view == null) {
            view = inflater.inflate(R.layout.list_item, root, false);

            RelativeLayout _back = (RelativeLayout) view.findViewById(R.id.bg_item);

            holder.back = _back;

            // paint oldies
            Set<String> _set = new HashSet<String>();
            _set = Prefs.getStringSet("arrOfOldies", _set);
            for(int i = 0; i <= _set.size()  ; i++){
                if(_set.contains(IDs.get(position))){
                //oldie
                            holder.back.setBackgroundColor(Color.parseColor("#D0D0D0"));
                    }
            }

            view.setTag(holder);
        } else {
                holder = (ItemHolder) view.getTag();       
        }
    }

我认为这与构建Listview的方式有关，有什么问题以及如何解决？

Answer 1

我想补充一下马尼托巴省的答案，如果从未选择过该项目，您还必须重置背景。 理解使用convertView和项目持有者的意义在于重用视图进行优化。 如果您从从未为某项目选择的项目中重用视图，则背景颜色仍将保留在此处。

尝试：

@Override
public View getView(final int position, View view, ViewGroup root)
{
    ItemHolder holder = null;

    if (view == null)
    {
        view = inflater.inflate(R.layout.list_item, root, false);
        holder = new ItemHolder();

        holder.back = (RelativeLayout) view.findViewById(R.id.bg_item);
        view.setTag(holder);
    }
    else
    {
        holder = (ItemHolder) view.getTag();
    }

    // paint oldies
    Set<String> _set = new HashSet<String>();
    _set = Prefs.getStringSet("arrOfOldies", _set);
    for(int i = 0; i <= _set.size()  ; i++)
    {
        if(_set.contains(IDs.get(position)))
        {
            holder.back.setBackgroundColor(Color.parseColor("#D0D0D0"));
        }
        else 
        {
            //add this else clause.
            holder.back.setBackgroundColor( YOUR_DEFAULT_BACKGROUND_COLOR ); 
        }
    }
}

Answer 2

您犯了一个小错误，请在获取现有持有人之后移动background部分。 这是要使用的干净代码。

private static class ItemHolder
{
    public TextView TXTTitle, TXTArtist, TXTid, TXTLikes;
    public RelativeLayout back;

    @Override
    public View getView(final int position, View view, ViewGroup root)
    {
        ItemHolder holder = null;

        if (view == null)
        {
            view = inflater.inflate(R.layout.list_item, root, false);
            holder = new ItemHolder();

            holder.back = (RelativeLayout) view.findViewById(R.id.bg_item);
            view.setTag(holder);
        }
        else
        {
            holder = (ItemHolder) view.getTag();
        }

        // paint oldies
        Set<String> _set = new HashSet<String>();
        _set = Prefs.getStringSet("arrOfOldies", _set);
        if(_set.contains(IDs.get(position)))
        {
            //oldie
            holder.back.setBackgroundColor(Color.parseColor("#D0D0D0"));
        }else{
            holder.back.setBackgroundColor(Color.parseColor("#ffffff"));
        }
    }
}

Answer 3

你的逻辑是错误的。 您没有在代码中考虑回收ListViews。 同样，您无需检查_set是否包含您的商品，因为您的商品中有很多对象。 您应该重构代码以反映以下几点：

• 您的最大问题是，除非视图为null，否则永远不要设置holper变量，并且它的null只能是屏幕上同时适合列表项的次数-当它们被回收时，颜色会变得很奇怪。 解决此问题，它将解决。 这是一些ListView回收信息
• 将_set移出getView函数
• 摆脱for循环迭代_set的值