CheckBox дважды меняет значение

У меня есть Android ListView, элементы которого отмечены флажком.

Флажок установлен по умолчанию. После снятия флажка он должен быть удален из списка.

Проблема в том, что onCheckedChanged срабатывает дважды: когда я нажимаю на флажок, чтобы снять его (с isChecked false), и после удаления элемента (с isChecked true).

Это соответствующий код моего ArrayAdapter:

public View getView(final int position, View convertView, ViewGroup parent) {
    ViewHolder holder;
    if (convertView == null) {
        convertView = mInflater.inflate(R.layout.item, parent, false);
        holder = new ViewHolder();
        holder.check = (CheckBox) convertView.findViewById(R.id.check);
        convertView.setTag(holder);
    } else {
        holder = (ViewHolder) convertView.getTag();
    }
    final Object item = this.getItem(position);
    holder.check.setOnCheckedChangeListener(new CheckBox.OnCheckedChangeListener() {
        @Override
        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
            if (!isChecked) {
                remove(item); // This somehow calls onCheckedChanged again
            }
        }
    });
    return convertView;
}

Что я делаю неправильно?


android listview checkbox android-arrayadapter
person hpique    schedule 12.10.2010    source источник
comment
Что делает метод remove(), поскольку это не метод ListAdapter?   -  person CommonsWare    schedule 12.10.2010
comment
Я использую ArrayAdapter. Согласно исходному коду Android, метод удаления удаляет элемент из массива и уведомляет об изменении данных.   -  person hpique    schedule 12.10.2010

Ответы (3)


arrow_upward
6
arrow_downward

Я столкнулся с той же проблемой, похоже, это ошибка Android, это двойное выполнение метода onCheckChanged.

Мое решение: реализовать onClickListener вместо onCheckedChangedListener.

Что-то вроде этого:

private final class CheckUpdateListener implements OnClickListener {

    private Group parent;
    private boolean isChecked;

    private CheckUpdateListener(Group parent) {

        this.parent = parent;

    }

    @Override
    public void onClick(View box) {
        this.isChecked = !parent.isChecked();

        parent.setChecked(isChecked);

        notifyDataSetChanged();

    }

}
person Andrei Catinean    schedule 07.10.2011
comment
Да, я думаю, что слушать onCheckedChange — это излишество и в большинстве случаев это неправильно. Однако основная проблема для меня заключалась в том, чтобы получить правильную перспективу, т. е. представление не то же самое, что просматриваемый объект данных. - person rwst; 01.02.2014

arrow_upward
5
arrow_downward

У меня была аналогичная проблема, и я просто решил ее правильно, не избегая использования предполагаемого OnCheckedChangeListener.

Код проблемы

holder.someCheckBox.setChecked(false);
holder.someCheckBox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
    @Override
    public void onCheckedChanged(CompoundButton compoundButton, boolean checked) {
       ...
    }
});

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

Но прослушиватель начал запускать событие on-check ровно дважды только после принудительного пересоздания адаптера или с помощью notifyDataSetChanged.

Решение

Очистите прослушиватель перед установкой состояния проверки, даже во время создания здесь.

holder.someCheckBox.setOnCheckedChangeListener(null);
holder.someCheckBox.setChecked(false);
holder.someCheckBox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
    @Override
    public void onCheckedChanged(CompoundButton compoundButton, boolean checked) {
       ...
    }
});

Теперь любые остаточные прослушиватели не будут запускаться при установке начального значения флажка.

person Cord Rehn    schedule 29.07.2016

arrow_upward
0
arrow_downward

Я сталкиваюсь с той же проблемой.

После некоторых исследований я обнаружил, что в RadioGroup есть похожее поведение для setCheckChangeListener.

Тегирование все портит, похоже на ошибку.

Мой обходной путь заключался в том, чтобы установить для тега значение null в конце прослушивателя. Это сработало для меня.

private OnCheckedChangeListener checkBoxitemClickListener = new OnCheckedChangeListener() {

    public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {


        if(isChecked)
        {               
            //do smth
        }else
        {
            //do smth
        }   

        buttonView.setTag(null);                        
    }
};

Если вам нужно обновить свой ListView позже, не забудьте вызвать «notifyDataSetChanged» на вашем адаптере.

Ваше здоровье

person ehanoc    schedule 21.06.2011