Список с флажком, радиокнопкой, текстовым представлением и кнопкой не работает правильно в Android

Я создаю приложение для Android, ниже приведен пользовательский интерфейс моего приложения.

При нажатии кнопки отправки мне нужен выбранный флажок и значение переключателя.

Пример Linux не отмечен, отмечен cc(радиокнопка).

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

  1. Когда я прокручиваю список, радиокнопка автоматически выбирается или отменяется, и я не могу поддерживать состояние радиокнопки.
  2. При нажатии кнопки не появляется выбранный переключатель, а также флажок.

Ниже мой макет, а также программа Java. Предложите мне получить правильные значения.

изображение приложения

Main.xml

<ListView
    android:id="@+id/my_list"
    android:layout_width="fill_parent"
    android:layout_height="199dp" />

<Button
    android:id="@+id/submit"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:text="Submit"
    />

row.xml

<TextView
    android:id="@+id/label"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:text="@+id/label"
    android:layout_toRightOf="@+id/check"
    android:textSize="20sp" >
</TextView>

<CheckBox
    android:id="@+id/check"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:layout_alignParentLeft="true"
    android:layout_marginLeft="4dip"
    android:layout_marginRight="10dip"
    android:focusable="false"

    android:focusableInTouchMode="false" >
</CheckBox>

 <RadioGroup
     android:id="@+id/radioSex"
     android:layout_width="wrap_content"
     android:layout_height="fill_parent"
     android:layout_alignParentRight="true"
     android:layout_alignParentTop="true"
     android:orientation="horizontal" >

     <RadioButton
         android:id="@+id/to"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:layout_marginLeft="10dip"
         android:checked="true"
         android:text="To" />

     <RadioButton
         android:id="@+id/cc"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:layout_marginLeft="10dip"
         android:text="CC" />
 </RadioGroup>

MyAdaptor.java

public class MyAdapter extends ArrayAdapter<Model> {

    private final List<Model> list;
    private final Activity context;
    boolean checkAll_flag = false;
    boolean checkItem_flag = false;

    public MyAdapter(Activity context, List<Model> list) {
        super(context, R.layout.row, list);
        this.context = context;
        this.list = list;
    }

    static class ViewHolder {
        protected TextView text;
        protected CheckBox checkbox;

    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {

        ViewHolder viewHolder = null;
        if (convertView == null) {
            LayoutInflater inflator = context.getLayoutInflater();
            convertView = inflator.inflate(R.layout.row, null);
            viewHolder = new ViewHolder();
            viewHolder.text = (TextView) convertView.findViewById(R.id.label);
            viewHolder.checkbox = (CheckBox) convertView
                    .findViewById(R.id.check);

            viewHolder.checkbox
                    .setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {

                        @Override
                        public void onCheckedChanged(CompoundButton buttonView,
                                boolean isChecked) {
                            int getPosition = (Integer) buttonView.getTag();
                            list.get(getPosition).setSelected(
                                    buttonView.isChecked());
                        }
                    });
            convertView.setTag(viewHolder);
            convertView.setTag(R.id.label, viewHolder.text);
            convertView.setTag(R.id.check, viewHolder.checkbox);

        } else {
            viewHolder = (ViewHolder) convertView.getTag();
        }
        viewHolder.checkbox.setTag(position);

        viewHolder.text.setText(list.get(position).getName());
        viewHolder.checkbox.setChecked(list.get(position).isSelected());

        return convertView;
    }

MainActivity.java

public class MainActivity extends Activity implements OnItemClickListener {

    ListView listView;
    ArrayAdapter<Model> adapter;
    List<Model> list = new ArrayList<Model>();

    private RadioGroup radioCcToGroup;
    private RadioButton radioTypeButton;
    private Button btn;

    public void onCreate(Bundle icicle) {
        super.onCreate(icicle);
        setContentView(R.layout.main);

        listView = (ListView) findViewById(R.id.my_list);

        btn = (Button) findViewById(R.id.submit);

        btn.setOnClickListener(new View.OnClickListener() {
            int count = 0;

            @Override
            public void onClick(View view) {


                count = listView.getAdapter().getCount();
                for (int i = 0; i < count; i++) {
                    // here i am not able to get the records as getting on onItemClick of the listview
                }
            }
        });

        adapter = new MyAdapter(this, getModel());
        listView.setAdapter(adapter);
        listView.setOnItemClickListener(this);
    }

    @Override
    public void onItemClick(AdapterView<?> arg0, View v, int position, long arg3) {
        TextView label = (TextView) v.getTag(R.id.label);
        CheckBox checkbox = (CheckBox) v.getTag(R.id.check);
        Toast.makeText(v.getContext(),
                label.getText().toString() + " " + isCheckedOrNot(checkbox),
                Toast.LENGTH_LONG).show();

    }

    private String isCheckedOrNot(CheckBox checkbox) {
        if (checkbox.isChecked())
            return "is checked";
        else
            return "is not checked";
    }

    private List<Model> getModel() {
        list.add(new Model("Linux"));
        list.add(new Model("Windows7"));
        list.add(new Model("Suse"));
        list.add(new Model("Eclipse"));
        list.add(new Model("Ubuntu"));
        list.add(new Model("Solaris"));
        list.add(new Model("Android"));
        list.add(new Model("iPhone"));
        list.add(new Model("Java"));
        list.add(new Model(".Net"));
        list.add(new Model("PHP"));
        return list;
    }

Model.java

private String name;
    private boolean selected;

    public Model(String name) {
        this.name = name;
    }

    public String getName() {
        return name;
    }

    public boolean isSelected() {
        return selected;
    }

    public void setSelected(boolean selected) {
        this.selected = selected;
    }

person subodh    schedule 26.09.2012    source источник


Ответы (1)


У вас здесь две ошибки на вашем MyAdaptor.java:

  1. Когда вы прикрепляете onCheckedChangeListener, вы делаете это только тогда, когда необходимо создать новое представление, и вы забываете о случае, когда представление списка повторно использует представление. Вы должны setOnCheckedChangeListener снаружи if (convertView == null).

  2. Кажется, что onCheckedChangeListener вызывается также, когда прокрутка завершена (из-за viewHolder.checkbox.setChecked(list.get(position).isSelected());). Этого можно избежать, используя OnClickListener вместо viewHolder.checkbox или не вызывая viewHolder.checkbox.setChecked().

Вот мой код:

private class ViewHolder {
    protected TextView text;
    protected CheckBox checkbox;
    protected RadioGroup radioGroup;
}

public class MyAdapter extends ArrayAdapter<Model> {

    private final List<Model> list;
    private final Activity context;
    boolean checkAll_flag = false;
    boolean checkItem_flag = false;

    public MyAdapter(Activity context, List<Model> list) {
        super(context, R.layout.row, list);
        this.context = context;
        this.list = list;
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {

        ViewHolder viewHolder = null;
        if (convertView == null) {
            LayoutInflater inflator = context.getLayoutInflater();
            convertView = inflator.inflate(R.layout.row, null);

            viewHolder = new ViewHolder();
            viewHolder.text = (TextView) convertView.findViewById(R.id.label);
            viewHolder.checkbox = (CheckBox) convertView.findViewById(R.id.check);
            viewHolder.checkbox.setTag(position);
            viewHolder.radioGroup = (RadioGroup) convertView.findViewById(R.id.radioSex);
            viewHolder.radioGroup.setTag(position);

            convertView.setTag(viewHolder);
        } else {
            viewHolder = (ViewHolder) convertView.getTag();
        }

        viewHolder.text.setText(list.get(position).getName());
        viewHolder.checkbox.setChecked(list.get(position).isSelected());
        viewHolder.checkbox.setOnClickListener(new OnClickListener() {

            @Override
            public void onClick(View v) {
                CheckBox checkbox = (CheckBox) v;
                int getPosition = (Integer) checkbox.getTag();
                list.get(getPosition).setSelected(checkbox.isChecked());
            }
        });

        viewHolder.radioGroup.setOnCheckedChangeListener(new OnCheckedChangeListener() {

            @Override
            public void onCheckedChanged(RadioGroup group, int checkedId) {
                boolean isCcOrIsTo = (checkedId == R.id.cc);
                int getPosition = (Integer) group.getTag();
                list.get(getPosition).setCcOrIsTo(isCcOrIsTo);
            }
        });

        return convertView;
    }
}

Обратите внимание, что я также добавил некоторый контроль над радиогруппой, поэтому Model.java изменился:

public class Model {

    private String name;
    private boolean selected;
    private boolean isCcOrIsTo;

    public Model(String name) {
        this.name = name;
    }

    public String getName() {
        return name;
    }

    public boolean isSelected() {
        return selected;
    }

    public void setSelected(boolean selected) {
        this.selected = selected;
    }

    public boolean isCcOrIsTo() {
        return isCcOrIsTo;
    }

    public void setCcOrIsTo(boolean isCcOrIsTo) {
        this.isCcOrIsTo = isCcOrIsTo;
    }

    @Override
    public String toString() {
        String selectedString = selected ? "selected" : "not selected";
        String value = isCcOrIsTo ? "CC" : "To";
        return name+" -> "+selectedString+ " with value "+value;
    }
}

Наконец, вам не нужен onItemClick в MainActivity.java, чтобы проверить правильность значений при нажатии кнопки саммита:

public void onCreate(Bundle icicle) {
    super.onCreate(icicle);
    setContentView(R.layout.activity_main);

    listView = (ListView) findViewById(R.id.my_list);

    btn = (Button) findViewById(R.id.submit);

    btn.setOnClickListener(new View.OnClickListener() {

        @Override
        public void onClick(View view) {
            for (Model m : list) {
                Log.i("Stack1", m.toString());
            }
        }
    });

    ArrayAdapter<Model> adapter = new MyAdapter(this, getModel());
    listView.setAdapter(adapter);
}

Надеюсь, поможет ;)

person Kalem    schedule 26.09.2012
comment
Спасибо, @Kalem, но все же, если я прокручиваю список, он автоматически выбирается. Если я не выберу какой-либо переключатель, а затем прокрутка не будет работать, но в случае, если я выберу любой из них. - person subodh; 27.09.2012
comment
Странно, у меня не было этой проблемы, когда я тестировал, я посмотрю на это еще раз. Просто для ясности, вы заменили setOnCheckedChangeListener? Вы также можете оформить заказ, если он не пришел из : viewHolder.checkbox.setChecked(list.get(position).isSelected()); - person Kalem; 27.09.2012
comment
Я просто копирую и вставляю ваш код, но в случае флажка он работает нормально, даже если я получил все ответы правильно. но когда я выбираю любой из переключателей и прокручиваю, он выбирает более одного переключателя в пользовательском интерфейсе, но когда я отправляю страницу, она показывает только то, что ожидается, но в пользовательском интерфейсе отображается несколько отмеченных. - person subodh; 27.09.2012
comment
Он показывает более одного, выбранного в пользовательском интерфейсе, только в случае переключателя, а при отправке кнопки отвечает только тот, который я выбрал. - person subodh; 27.09.2012
comment
Все еще не удается воспроизвести ни на Gingerbread, ни на ICS. Убедитесь, что вы не добавляете слушателя, прежде чем устанавливать состояние RadioGroup viewHolder.radioGroup.check(); (если оно есть). Если мой ответ помог вам, пожалуйста, проверьте его;) - person Kalem; 27.09.2012
comment
спасибо и +1 за вашу поддержку, но все еще борюсь с проблемой. - person subodh; 27.09.2012
comment
@subodh Я столкнулся с той же проблемой, не могли бы вы сообщить, как вы решили эту проблему. - person Dory; 19.07.2013
comment
Вы делаете то же самое, что предложил Калем? - person subodh; 19.07.2013
comment
@Kalem: у меня те же проблемы. С помощью вашего метода .getTag() я получаю ту же самую переработанную позицию, а не «истинную» позицию. Так что я не понимаю, почему это было бы решением...? Android иногда очень сильно раздражает. :( - person Luis A. Florit; 20.10.2013
comment
@LuisA.Florit вы сделали viewHolder.radioGroup.setTag(position); по методу public View getView(int position, View convertView, ViewGroup parent)? - person Kalem; 28.10.2013
comment
@Калем: Да. Но ничего, я нашел ошибку в своем коде и решил проблему без тегов. Спасибо! - person Luis A. Florit; 28.10.2013