自定义CursorAdapter中是否正确实现了ViewHolder模式？

Question

这是我的自定义CursorAdapter：

public class TasksAdapter extends CursorAdapter implements Filterable {

    private final Context context;

    public TasksAdapter(Context context, Cursor c) {
        super(context, c);
        this.context = context;
    }

    /**
     * @see android.widget.CursorAdapter#newView(android.content.Context, android.database.Cursor, android.view.ViewGroup)
     */
    @Override
    public View newView(Context context, Cursor cursor, ViewGroup parent) {
        LayoutInflater inflater = LayoutInflater.from(context);
        View v = inflater.inflate(android.R.layout.simple_list_item_checked, parent, false);        

        ViewHolder holder = new ViewHolder();
        holder.textview = (CheckedTextView)v.findViewById(android.R.id.text1);
        v.setTag(holder);

        return v;
    }

    /**  
     * @see android.widget.CursorAdapter#bindView(android.view.View, android.content.Context, android.database.Cursor)
     */
    @Override
    public void bindView(View view, Context context, Cursor cursor) {

        ViewHolder holder = (ViewHolder)view.getTag();
        int titleCol = cursor.getColumnIndexOrThrow(Tasks.TITLE);
        int completedCol = cursor.getColumnIndexOrThrow(Tasks.COMPLETED);

        String title = cursor.getString(titleCol);
        boolean completed = Util.intToBool(cursor.getInt(completedCol));

        holder.textview.setText(title);
        holder.textview.setChecked(completed);
    }

    /**
     * @see android.widget.CursorAdapter#runQueryOnBackgroundThread(java.lang.CharSequence)
     */
    @Override
    public Cursor runQueryOnBackgroundThread(CharSequence constraint) {

        StringBuffer buffer = null;
        String[] args = null;

        if (constraint != null) {
            buffer = new StringBuffer();
            buffer.append("UPPER (");
            buffer.append(Tasks.TITLE);
            buffer.append(") GLOB ?");
            args = new String[] { "*" + constraint.toString().toUpperCase() + "*" };
        }

        Cursor c = context.getContentResolver().query(Tasks.CONTENT_URI,
            null, (buffer == null ? null : buffer.toString()), args,
            Tasks.DEFAULT_SORT_ORDER);

        c.moveToFirst();
        return c;
    }

    /**
     * @see android.widget.CursorAdapter#convertToString(android.database.Cursor)
     */
    @Override
    public CharSequence convertToString(Cursor cursor) {
        final int titleCol = cursor.getColumnIndexOrThrow(Tasks.TITLE);
        String title = cursor.getString(titleCol);
        return title;
    }

    static class ViewHolder {
        CheckedTextView textview;
    }

}

这是否符合ViewHolder模式的限制？我不确定，因为这是一个CursorAdapter，其中没有getView。如果有任何问题或建议，请指出。

- Mohit Deshpande

Answer 1

CursorAdapter 不会在每次需要新行时都调用 newView 方法；如果它已经有一个 View，它将调用 bindView 方法，因此实际上重复使用了创建的视图。

话虽如此，就像评论中 Joseph 指出的那样，您仍然可以使用 ViewHolder 以避免重复调用 findViewById 方法。

如果您仍然关注效率，请查看 SimpleCursorAdapter 实现，其中使用了一个 WeakHashMap（一种 WeakReferences 映射表）：

WeakHashMap<View, View[]> mHolders = new WeakHashMap<View, View[]>();

- Cristian

---

那么我可以像View.findViewById(int)这样进行昂贵的调用，而我的应用程序不会出现滞后吗？ - Mohit Deshpande

findViewById(int)并没有你想象的那么耗费资源。它只会返回一个引用（如果存在的话）。ViewHolder技术的存在是为了解决其他类型的问题：不要创建比实际需要更多的视图（因此，它避免了过度的视图膨胀，这是很耗费资源的）。 - Cristian

@Cristian，谢谢。我使用了newView和bindView来使一个类继承SimpleCursorAdapter。请看下面的代码。 - Maksim Dmitriev

@Cristian，你又错了！从Commonsware书籍这里中可以找到一个不使用ViewHolder但使用convertView的getView()实现。如你所见，当convertView非空时，无需充气：视图会被回收利用。因此，为避免充气，你需要使用convertView。而ViewHolder则是用于避免许多findViewById调用，正如同一本Commonsware书籍这里中所述。 - GVillani82

我现在明白了。避免通货膨胀是由convertView实例提供的，而ViewHolder仅用作避免“findViewById”调用的一种方式。感谢你让我明白这一点！ - Cristian

Answer 2

如果您覆盖了 newView() 和 bindView()，则在 getView() 中不需要执行额外的操作。 CursorAdapter 已经实现了 getView() 方法，并将委托给 newView() 和 bindView() 来强制执行行回收。

在 ListView 滚动时可能会频繁调用 findViewById()，这可能会降低性能。即使 Adapter 返回了一个已经填充好的可重用视图，您仍然需要查找元素并更新它们。为了避免这种情况，可以使用 ViewHolder 模式。

以下是一个天气应用程序实现 ViewHolder 模式的示例：

public class ForecastAdapter extends CursorAdapter {

    public ForecastAdapter(Context context, Cursor cursor, int flags) {
        super(context, cursor, flags);
    }

    @Override
    public View newView(Context context, Cursor cursor, ViewGroup parent) {
        View view = LayoutInflater.from(context).inflate(
                R.layout.list_item_forecast, parent, false);
        ViewHolder viewHolder = new ViewHolder(view);
        view.setTag(viewHolder);
        return view;
    }

    @Override
    public void bindView(View view, Context context, Cursor cursor) {
        ViewHolder viewHolder = (ViewHolder) view.getTag();

        long date = cursor.getLong(ForecastFragment.COL_WEATHER_DATE);
        viewHolder.dateView.setText("Today");

        String weatherForecast =
                cursor.getString(ForecastFragment.COL_WEATHER_DESC);
        viewHolder.descriptionView.setText(weatherForecast);

        double high = cursor.getFloat(ForecastFragment.COL_WEATHER_MAX_TEMP);
        viewHolder.highTempView.setText("30");

        double low = cursor.getFloat(ForecastFragment.COL_WEATHER_MIN_TEMP);
        viewHolder.lowTempView.setText("24");

        int weatherConditionId =
                cursor.getInt(ForecastFragment.COL_WEATHER_CONDITION_ID);
        viewHolder.iconView.setImageResource(R.drawable.ic_snow);
    }

    /** Cache of the children views for a list item. */
    public static class ViewHolder {
        public final ImageView iconView;
        public final TextView dateView;
        public final TextView descriptionView;
        public final TextView highTempView;
        public final TextView lowTempView;

        public ViewHolder(View view) {
            iconView =
                    (ImageView) view.findViewById(R.id.item_icon);
            dateView =
                    (TextView) view.findViewById(R.id.item_date_textview);
            descriptionView =
                    (TextView) view.findViewById(R.id.item_forecast_textview);
            highTempView =
                    (TextView) view.findViewById(R.id.item_high_textview);
            lowTempView =
                    (TextView) view.findViewById(R.id.item_low_textview);
        }
    }
}

- Yogesh Umesh Vaity

Answer 3

我使用一个类扩展了SimpleCursorAdapter，并实现了newView和bindView方法，但没有使用ViewHolder模式。

    private class CountriesAdapter extends SimpleCursorAdapter {

            private LayoutInflater mInflater;

            public CountriesAdapter(Context context, int layout, Cursor cursor, String[] from,
                    int[] to, LayoutInflater inflater) {
                super(getActivity(), layout, cursor, from, to, CURSOR_ADAPTER_FLAGS);
                mInflater = inflater;
            }

            @Override
            public View newView(Context context, Cursor cursor, ViewGroup parent) {
                return mInflater.inflate(R.layout.countries_list_row, parent, false);
            }

            @Override
            public void bindView(View rowView, Context context, Cursor cursor) {

                TextView tvCountry = (TextView) rowView.findViewById(R.id.countriesList_tv_countryName);
                TextView tvOrgs = (TextView) rowView.findViewById(R.id.countriesList_tv_orgNames);
                ImageView ivContinent =
                        (ImageView) rowView.findViewById(R.id.countriesList_iv_continentName);

                // TODO: set texts of TextViews and an icon here
                }

            }
    }

- Maksim Dmitriev