GridView Adapter.getView() не задава правилно избраното състояние

Имам GridView, който попълвам с персонализиран подклас на ArrayAdapter. Този адаптер връща бутони, които съм персонализирал да могат да се избират (вижте Android ImageButton с избрано състояние?). Това работи досега и щракването върху бутоните ги избира (което се вижда с фон на селектора).

Проблемът е: не мога да задам тези бутони в избрано състояние от самото начало. Те просто се показват неизбрани, когато за първи път стартирам View.

Създадох прост тестов проект, за да илюстрирам проблема:

package com.example.buttonselection;

import android.app.Activity;
import android.content.Context;
import android.os.Bundle;
import android.view.View;
import android.view.View.OnClickListener;
import android.view.ViewGroup;
import android.widget.ArrayAdapter;
import android.widget.Button;
import android.widget.GridView;

public class MainActivity extends Activity {

    public class SelectButtonAdapter extends ArrayAdapter<String> {

        public SelectButtonAdapter(Context context) {
            super(context, 0);
        }

        @Override
        public View getView(int position, View convertView, ViewGroup parent) {
            String name = getItem(position);

            View rowView = convertView;
            if (rowView == null || !(rowView instanceof Button)) {
                rowView = new Button(getContext());
                ((Button)rowView).setOnClickListener(new OnClickListener() {
                    public void onClick(View button) {
                        if (button.isSelected()){
                            button.setSelected(false);
                        } else {
                            button.setSelected(true);
                        }
                    }
                });
            }

            Button button = (Button)rowView;
            button.setText(name);
            button.setBackgroundResource(R.drawable.button_selection);

            button.setSelected(true); // this does not work

            return button;
        }
    }

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        GridView grid = (GridView)findViewById(R.id.gridview);

        SelectButtonAdapter adapter = new SelectButtonAdapter(this);
        adapter.add("One");
        adapter.add("Two");
        adapter.add("Three");
        grid.setAdapter(adapter);
    }
}

Поради това дори не мога да възстановя състоянието на бутоните, които запазих с onSaveInstanceState. Как мога да разреша или заобиколен този проблем?

Благодарен съм за всяка помощ!

РЕДАКТИРАНЕ: тук е моят button_selection.xml, въпреки че това трябва да е добре, тъй като избирането на бутоните по-късно работи добре.

<?xml version="1.0" encoding="utf-8"?>
<selector xmlns:android="http://schemas.android.com/apk/res/android">

    <item android:state_pressed="true">
        <shape>

            <solid android:color="@color/bet_button_pressed" />

            <stroke
                android:width="2dip"
                android:color="@color/white" />

        </shape>
    </item>

    <item android:state_selected="true">
        <shape>

            <solid android:color="@color/bet_button_selected" />

            <stroke
                android:width="2dip"
                android:color="@color/white" />

        </shape>
    </item>

        <item>
        <shape>

            <gradient
                android:angle="90"
                android:startColor="@color/bet_button_dark_green"
                android:endColor="@color/bet_button_light_green"
                android:centerX="0.5"
                android:centerY="0.5" />

            <stroke
                android:width="2dip"
                android:color="@color/white" />

        </shape>
    </item>
</selector>

person cit    schedule 22.09.2012    source източник
comment
можем ли да видим button_selection чертеж?   -  person Sherif elKhatib    schedule 22.09.2012
comment
Разбира се, но не мисля, че е важно, тъй като избира и показва бутоните правилно по-късно, като ги щракне/избере.   -  person cit    schedule 22.09.2012
comment
Точно! Работя върху това от няколко часа и не мога да намеря решение. Бих казал, че това е грешка в API. Може ли някой да измисли заобиколно решение?   -  person cit    schedule 22.09.2012


Отговори (2)


Е, намерих решение за тази грешка. Просто съхранявам външно дали е избран бутон и замествам ondraw-метода на бутона, за да задам правилното състояние всеки път, когато бъде изчертан. Това има допълнителното предимство, че селекциите могат да се поддържат много по-лесно.

Ето заобиколното решение (просто доказателство за концепцията, моят производствен код е по-сложен):

package com.example.buttonselection;

import java.util.HashSet;
import java.util.Set;

import android.app.Activity;
import android.content.Context;
import android.graphics.Canvas;
import android.os.Bundle;
import android.view.View;
import android.view.View.OnClickListener;
import android.view.ViewGroup;
import android.widget.ArrayAdapter;
import android.widget.Button;
import android.widget.GridView;

public class MainActivity extends Activity {

    public class SelectButtonAdapter extends ArrayAdapter<String> {

        public SelectButtonAdapter(Context context) {
            super(context, 0);
        }

        @Override
        public View getView(int position, View convertView, ViewGroup parent) {
            String name = getItem(position);

            View rowView = convertView;
            if (rowView == null || !(rowView instanceof Button)) {
                rowView = new Button(getContext()) {
                    @Override
                    protected void onDraw(Canvas canvas) {
                        setSelected(selectedButtons.contains(getText()));
                        super.onDraw(canvas);
                    }
                };
                ((Button)rowView).setOnClickListener(new OnClickListener() {
                    public void onClick(View button) {
                        String text = ((Button)button).getText().toString();

                        if(selectedButtons.contains(text)) {
                            selectedButtons.remove(text);
                        } else {
                            selectedButtons.add(text);
                        }
                        button.invalidate();
                    }
                });
            }

            Button button = (Button)rowView;
            button.setText(name);
            button.setBackgroundResource(R.drawable.button_selection);

            return button;
        }

    }
    private Set<String> selectedButtons = new HashSet<String>();

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        GridView grid = (GridView)findViewById(R.id.gridview);

        SelectButtonAdapter adapter = new SelectButtonAdapter(this);
        adapter.add("One");
        adapter.add("Two");
        adapter.add("Three");
        grid.setAdapter(adapter);

        // this selects the first button from the start
        selectedButtons.add("One");
    }

}

Към странична бележка: Човек би си помислил, че версия 4.1 API няма да съдържа толкова очевидни грешки. Времето, загубено за това, е просто разочароващо и не мотивира да се развива повече за тази система.

person cit    schedule 23.09.2012

Имам същия проблем. Като заобиколно решение използвах runnable, за да настроя изгледа да бъде избран.

за твоя случай:

        @Override
        public View getView(int position, View convertView, ViewGroup parent) {
           String name = getItem(position);

           View rowView = convertView;
           if (rowView == null || !(rowView instanceof Button)) {
               rowView = new Button(getContext());
               ((Button)rowView).setOnClickListener(new OnClickListener() {
                    public void onClick(View button) {
                        if (button.isSelected()){
                            button.setSelected(false);
                        } else {
                            button.setSelected(true);
                        }
                   }
               });
            }

            Button button = (Button)rowView;
            button.setText(name);
            button.setBackgroundResource(R.drawable.button_selection);

            final Handler handler = new Handler();
            handler.postDelayed(new Runnable() {
                @Override
                public void run() {
                    button.setSelected(true);
                }
            }, 50);

            return button;
        }

50 е просто произволно число, което избирам да използвам, можете да го промените както искате.

Това заобиколно решение работи за мен и аз лично го харесвам по-добре, отколкото да заменя метода onDraw.

Дано това помогне на някого.

person Shinta S    schedule 02.09.2014