Изменить фон ListView - странное поведение

У меня проблема с изменением фона представления в ListView.

Что мне нужно:
Изменить фоновое изображение строки onClick ()

Что происходит на самом деле:
Фон изменяется (выбирается) после нажатия, например, первой записи. Но после прокрутки вниз выбирается и 8-я запись. Прокрутите назад к началу, первый больше не выбран. Вторая запись выбрана. Продолжайте прокрутку, и она продолжает прыгать ...

Что я делаю в коде:
У меня есть каналы, и onClick () я переключаю атрибут канала boolean selected а потом меняю фон. Я делаю это только onClick (), поэтому я не понимаю, почему это на самом деле происходит и с другими записями. Я заметил одну вещь: кажется, это только часть «рисования», потому что элемент, который выбирается «самим», по-прежнему имеет значение selected на false

Я думаю, что это похоже, как-то связано с повторным использованием представлений в пользовательских ListAdapters getView (...)

Код onClick () в ListActivity:

@Override
    protected ViewHolder createHolder(View v) {

        // createHolder will be called only as long, as the ListView is not
        // filled

        TextView title = (TextView) v
                .findViewById(R.id.tv_title_channel_list_adapter);
        TextView content = (TextView) v
                .findViewById(R.id.tv_content_channel_list_adapter);

        ImageView icon = (ImageView) v
                .findViewById(R.id.icon_channel_list_adapter);

        if (title == null || content == null || icon == null) {
            Log.e("ERROR on findViewById",
                    "Couldn't find Title, Content or Icon");
        }
        ViewHolder mvh = new MyViewHolder(title, content, icon);

        // We make the views become clickable
        // so, it is not necessary to use the android:clickable attribute in
        // XML

        v.setOnClickListener(new ChannelListAdapter.OnClickListener(mvh) {

            public void onClick(View v, ViewHolder viewHolder) {
                // we toggle the enabled state and also switch the the
                // background
                MyViewHolder mvh = (MyViewHolder) viewHolder;
                Channel ch = (Channel) mvh.data;
                ch.setSelected(!ch.getSelected()); // toggle

                if (ch.getSelected()) {
                    v.setBackgroundResource(R.drawable.row_blue_selected);
                } else {
                    v.setBackgroundResource(R.drawable.row_blue);
                }
                // TESTING
                Log.d("onClick() Channel", "onClick() Channel: "
                        + ch.getTitle() + " selected: " + ch.getSelected());
            }
        });
return mvh;
    }

Код getView (...):

@Override
public View getView(int position, View view, ViewGroup parent) {
    ViewHolder holder;

    // When view is not null, we can reuse it directly, there is no need
    // to reinflate it.
    // We only inflate a new View when the view supplied by ListView is
    // null.
    if (view == null) {
        view = mInflater.inflate(mViewId, null);

        // call own implementation
        holder = createHolder(view);

        // TEST
        // we set the holder as tag
        view.setTag(holder);

    } else {
        // get holder back...much faster than inflate
        holder = (ViewHolder) view.getTag();
    }

    // we must update the object's reference
    holder.data = getItem(position);

            // <EDIT SOLUTION>

    if(getItem(position).get_id() == channelList.get(position).get_id()){
        if(getItem(position).getSelected())
        {
            view.setBackgroundResource(R.drawable.row_blue_selected);
        }
        else{
            view.setBackgroundResource(R.drawable.row_blue);
        }
    }

            // </EDIT SOLUTION>

    // call the own implementation
    bindHolder(holder);

    return view;
}

Я действительно был бы признателен за любую идею, как это решить! :)

Если потребуется дополнительная информация, сообщите мне.

Заранее спасибо!

7
задан Artem Russakovskii 21 July 2011 в 23:43
поделиться