Этот пользовательский CursorAdapter для ListView правильно закодирован для Android?

Я никогда не был доволен кодом моего пользовательского CursorAdapter до сегодняшнего дня, я решил рассмотреть его и исправить небольшую проблему, которая беспокоила меня в течение длительного времени (достаточно интересно, ни один из пользователей моего приложения никогда не сообщал об этом проблема).

Вот небольшое описание моего вопроса:

Мой пользовательский CursorAdapter переопределяет newView() и bindView() вместо getView() как большинство примеров, которые я вижу. Я использую шаблон ViewHolder между этими двумя методами. Но моя главная проблема заключалась в том, что пользовательский макет, который я использую для каждого элемента списка, содержит ToggleButton.

Проблема заключалась в том, что состояние кнопки не сохранялось, когда просмотр элемента списка прокручивался вне поля зрения, а затем снова прокручивался назад. Эта проблема возникла из-за того, что cursor никогда не знал, что данные базы данных изменились при нажатии кнопки ToggleButton, и она всегда вытягивала одни и те же данные. Я попытался запросить курсор, щелкнув ToggleButton и решив проблему, но это было очень медленно.

Я решил эту проблему, и я публикую весь класс здесь для обзора. Я прокомментировал код полностью для этого конкретного вопроса, чтобы лучше объяснить мои решения по кодированию.

Этот код выглядит хорошо для вас? Не могли бы вы улучшить/оптимизировать или изменить его каким-то образом?

P.S: Я знаю, что CursorLoader является очевидным улучшением, но на данный момент у меня нет времени на обработку таких больших переписаний кода. Это то, что у меня есть в дорожной карте, хотя.

Здесь код:

public class NotesListAdapter extends CursorAdapter implements OnClickListener {

    private static class ViewHolder {
        ImageView icon;
        TextView title;
        TextView description;
        ToggleButton visibility;
    }

    private static class NoteData {
        long id;
        int iconId;
        String title;
        String description;
        int position;
    }

    private LayoutInflater mInflater;

    private NotificationHelper mNotificationHelper;
    private AgendaNotesAdapter mAgendaAdapter;

    /*
     * This is used to store the state of the toggle buttons for each item in the list
     */
    private List<Boolean> mToggleState;

    private int mColumnRowId;
    private int mColumnTitle;
    private int mColumnDescription;
    private int mColumnIconName;
    private int mColumnVisibility;

    public NotesListAdapter(Context context, Cursor cursor, NotificationHelper helper, AgendaNotesAdapter adapter) {
        super(context, cursor);

        mInflater = LayoutInflater.from(context);

        /*
         * Helper class to post notifications to the status bar and database adapter class to update
         * the database data when the user presses the toggle button in any of items in the list
         */
        mNotificationHelper = helper;
        mAgendaAdapter = adapter;

        /*
         * There no need to keep getting the column indexes every time in bindView() (as I see in
         * a few examples) so I do it once and save the indexes in instance variables
         */
        findColumnIndexes(cursor);

        /*
         * Populate the toggle button states for each item in the list with the corresponding value
         * from each record in the database, but isn't this a slow operation?
         */
        for(mToggleState = new ArrayList<Boolean>(); !cursor.isAfterLast(); cursor.moveToNext()) {
            mToggleState.add(cursor.getInt(mColumnVisibility) != 0);
        }
    }

    @Override
    public View newView(Context context, Cursor cursor, ViewGroup parent) {
        View view = mInflater.inflate(R.layout.list_item_note, null);

        /*
         * The ViewHolder pattern is here only used to prevent calling findViewById() all the time
         * in bindView(), we only need to find all the views once
         */
        ViewHolder viewHolder = new ViewHolder();

        viewHolder.icon = (ImageView)view.findViewById(R.id.imageview_icon);
        viewHolder.title = (TextView)view.findViewById(R.id.textview_title);
        viewHolder.description = (TextView)view.findViewById(R.id.textview_description);
        viewHolder.visibility = (ToggleButton)view.findViewById(R.id.togglebutton_visibility);

        /*
         * I also use newView() to set the toggle button click listener for each item in the list
         */
        viewHolder.visibility.setOnClickListener(this);

        view.setTag(viewHolder);

        return view;
    }

    @Override
    public void bindView(View view, Context context, Cursor cursor) {
        Resources resources = context.getResources();

        int iconId = resources.getIdentifier(cursor.getString(mColumnIconName),
                "drawable", context.getPackageName());

        String title = cursor.getString(mColumnTitle);
        String description = cursor.getString(mColumnDescription);

        /*
         * This is similar to the ViewHolder pattern and it need to access the note data when the
         * onClick() method is fired
         */
        NoteData noteData = new NoteData();

        /*
         * This data is needed to post a notification when the onClick() method is fired
         */
        noteData.id = cursor.getLong(mColumnRowId);
        noteData.iconId = iconId;
        noteData.title = title;
        noteData.description = description;

        /*
         * This data is needed to update mToggleState[POS] when the onClick() method is fired
         */
        noteData.position = cursor.getPosition();

        /*
         * Get our ViewHolder with all the view IDs found in newView()
         */
        ViewHolder viewHolder = (ViewHolder)view.getTag();

        /*
         * The Html.fromHtml is needed but the code relevant to that was stripped
         */
        viewHolder.icon.setImageResource(iconId);
        viewHolder.title.setText(Html.fromHtml(title));
        viewHolder.description.setText(Html.fromHtml(description));

        /*
         * Set the toggle button state for this list item from the value in mToggleState[POS]
         * instead of getting it from the database with 'cursor.getInt(mColumnVisibility) != 0'
         * otherwise the state will be incorrect if it was changed between the item view scrolling
         * out of view and scrolling back into view
         */
        viewHolder.visibility.setChecked(mToggleState.get(noteData.position));

        /*
         * Again, save the note data to be accessed when onClick() gets fired
         */
        viewHolder.visibility.setTag(noteData);
    }

    @Override
    public void onClick(View view) {
        /*
         * Get the new state directly from the toggle button state 
         */
        boolean visibility = ((ToggleButton)view).isChecked();

        /*
         * Get all our note data needed to post (or remove) a notification 
         */
        NoteData noteData = (NoteData)view.getTag();

        /*
         * The toggle button state changed, update mToggleState[POS] to reflect that new change
         */
        mToggleState.set(noteData.position, visibility);

        /*
         * Post the notification or remove it from the status bar depending on toggle button state
         */
        if(visibility) {
            mNotificationHelper.postNotification(
                    noteData.id, noteData.iconId, noteData.title, noteData.description);
        } else {
            mNotificationHelper.cancelNotification(noteData.id);
        }

        /*
         * Update the database note item with the new toggle button state, without the need to
         * requery the cursor (which is slow, I've tested it) to reflect the new toggle button state
         * in the list because the value was saved in mToggleState[POS] a few lines above
         */
        mAgendaAdapter.updateNote(noteData.id, null, null, null, null, visibility);
    }

    private void findColumnIndexes(Cursor cursor) {
        mColumnRowId = cursor.getColumnIndex(AgendaNotesAdapter.KEY_ROW_ID);
        mColumnTitle = cursor.getColumnIndex(AgendaNotesAdapter.KEY_TITLE);
        mColumnDescription = cursor.getColumnIndex(AgendaNotesAdapter.KEY_DESCRIPTION);
        mColumnIconName = cursor.getColumnIndex(AgendaNotesAdapter.KEY_ICON_NAME);
        mColumnVisibility = cursor.getColumnIndex(AgendaNotesAdapter.KEY_VISIBILITY);
    }

}

ОТВЕТЫ

Ответ 1

Ваше решение оптимально, я добавлю его в свое оружие:) Возможно, я попытаюсь немного оптимизировать вызовы в базу данных.

Действительно, из-за условий задачи существует только три решения:

  • Обновить только одну строку, указатель запроса и перерисовать все элементы. (Прямая, грубая сила).
  • Обновить строку, кешировать результаты и использовать кеш для элементов чертежа.
  • Загрузите результаты, используйте кеш для рисования элементов. И когда вы покидаете эту активность/фрагмент, затем передайте результаты в базу данных.

Для третьего решения вы можете использовать SparseArray для поиска изменений.

private SparseArray<NoteData> mArrayViewHolders;

public void onClick(View view) {
     //here your logic with NoteData. 
     //start of my improve
     if (mArrayViewHolders.get(selectedPosition) == null) {
        // put the change into array
        mArrayViewHolders.put(selectedPosition, noteData);
     } else {
        // rollback the change
        mArrayViewHolders.delete(selectedPosition);
     }
     //end of my improve
     //we don't commit the changes to database.
}

Еще раз: с самого начала этот массив пуст. Когда вы переключаете кнопку в первый раз (есть изменение), вы добавляете NoteData в массив. Когда вы переключаете кнопку второй раз (есть откат), вы удаляете NoteData из массива. И так далее.

Когда вы закончите, просто запросите массив и вставьте изменения в базу данных.

Ответ 2

То, что вы видите, - это просмотр повторного использования Android. Я не думаю, что вы делаете что-то неправильно, снова запросив курсор. Просто не используйте функцию cursor.requery().

Вместо этого сначала установите переключатель в значение false, а затем спросите курсор и включите его, если вам нужно.

Возможно, вы это делали, и я что-то не понял, но я не думаю, что вы должны иметь медленные результаты.

Псевдо-код:

getView(){
setToggleFalse();
boolean value = Boolean.valueOf(cursor.getString('my column'));
if (value){
   setToggleTrue();
}
}

Ответ 3

Я бы подождал, прежде чем отправиться в CursorLoader. Как кажется, производные CursorLoader не беспокоят CursorLoader.