Android ListView条目IF判断失效：过期状态文本颜色设置异常

你遇到的是ListView View复用导致的状态残留问题——这是自定义Adapter时非常常见的一个坑！

问题根源 #

ListView会复用滑出屏幕的ItemView来显示新条目，如果你只在getExpired()返回true时设置文本颜色，却没有在返回false时重置回默认颜色，那么当一个标记为过期的条目被复用给未过期的条目时，颜色会保留下来，就出现了你看到的“明明返回false却变色”的情况；第一个条目异常也大概率是因为复用了Adapter内部缓存池里的View，残留了之前的状态。

修复后的getView示例代码 #

建议搭配ViewHolder模式（既提升性能又减少这类状态残留问题），修改后的代码需要同时处理两种状态：

@Override
public View getView(int position, View convertView, ViewGroup parent) {
    ViewHolder holder;
    // 初始化或复用ViewHolder
    if (convertView == null) {
        convertView = LayoutInflater.from(parent.getContext()).inflate(R.layout.your_item_layout, parent, false);
        holder = new ViewHolder();
        holder.itemText = convertView.findViewById(R.id.item_text);
        convertView.setTag(holder);
    } else {
        holder = (ViewHolder) convertView.getTag();
    }

    // 获取当前条目数据
    YourDataModel data = getItem(position);

    // 关键：同时处理两种状态，不要只设置过期的情况
    if (data.getExpired()) {
        // 过期时设置目标颜色，比如红色
        holder.itemText.setTextColor(Color.RED);
    } else {
        // 未过期时重置为默认颜色，比如黑色或者主题定义的文本色
        holder.itemText.setTextColor(Color.BLACK);
        // 如果用主题色，也可以这样写：
        // holder.itemText.setTextColor(ContextCompat.getColor(parent.getContext(), R.color.default_text_color));
    }

    // 设置条目文本内容
    holder.itemText.setText(data.getYourContent());

    return convertView;
}

// ViewHolder内部类，统一管理条目内的View
static class ViewHolder {
    TextView itemText;
}

核心注意事项 #

• 永远不要只设置“异常状态”的UI，必须对应设置“正常状态”的UI，彻底避免复用View时的状态残留。
• 坚持使用ViewHolder模式，不仅能提升列表滚动流畅度，还能避免重复findViewById的冗余操作，更便于统一控制View的状态。

内容的提问来源于stack exchange，提问作者Szmerd