2014-11-06 52 views
0

我正面临着一个非常奇怪的行为,我在自定义适配器中的复选框及其标签上设置了onClickListener。 问题是,当我检查第一个问题的答案时,也检查第三个问题上的同一个项目。复选框似乎是同步的,但我不明白为什么。setOnClickListener getView不良行为

以下是一些需要了解的屏幕。

1rst question

3rd question

这里是我的适配器getView():

public View getView(final int position, View convertView, ViewGroup parent) 
{ 
    View vi = convertView; 

    if(convertView == null) 
    { 
     vi = inflater.inflate(R.layout.question_item, null); 
    } 

    Qcm qcm = qcms.get(position); 
    List<Proposition> propositions = qcm.getPropositionsList(); 

    TextView numQuestion = (TextView)vi.findViewById(R.id.question_number_item); 
    TextView question = (TextView)vi.findViewById(R.id.question_text); 
    LinearLayout[] labels = new LinearLayout[5]; 
    labels[0] = (LinearLayout)vi.findViewById(R.id.question_label1); 
    labels[1] = (LinearLayout)vi.findViewById(R.id.question_label2); 
    labels[2] = (LinearLayout)vi.findViewById(R.id.question_label3); 
    labels[3] = (LinearLayout)vi.findViewById(R.id.question_label4); 
    labels[4] = (LinearLayout)vi.findViewById(R.id.question_label5); 

    final CheckBox[] checkBoxes = new CheckBox[5]; 
    checkBoxes[0] = (CheckBox)vi.findViewById(R.id.question_check1); 
    checkBoxes[1] = (CheckBox)vi.findViewById(R.id.question_check2); 
    checkBoxes[2] = (CheckBox)vi.findViewById(R.id.question_check3); 
    checkBoxes[3] = (CheckBox)vi.findViewById(R.id.question_check4); 
    checkBoxes[4] = (CheckBox)vi.findViewById(R.id.question_check5); 

    numQuestion.setText(String.valueOf(position+1)); 
    question.setText(Html.fromHtml(qcm.getQuestion())); 

    for(int i=0; i<5; i++) 
    { 
     final int cpt = i; 


     checkBoxes[i].setOnClickListener(new View.OnClickListener() { 
      @Override 
      public void onClick(View v) { 
       // Checked[i] -> !Checked[i] 
       QcmListAdapter.this.checked[position][cpt] = !QcmListAdapter.this.checked[position][cpt]; 
       APIHelper.log('d', "position="+position); 
       // TODO : Faire la methode PlaySerieFragment.saveSession(); 
       //QcmListAdapter.this.activity.getFragmentManager().getFragment(QcmListAdapter.this.); 
      } 
     }); 


     labels[i].setOnClickListener(new View.OnClickListener() { 
      @Override 
      public void onClick(View v) { 
       checkBoxes[cpt].performClick(); 
       APIHelper.log('d', "onClick:cpt="+cpt); 
      } 
     }); 

     Proposition p = propositions.get(i); 
     p.setFormattedView(this.activity, labels[i], (i + 1)); 

    } 

    return vi; 

} 

,这里是一个项目的观点:我怀疑这

<?xml version="1.0" encoding="utf-8"?> 
<RelativeLayout 
xmlns:android="http://schemas.android.com/apk/res/android" 
android:id="@+id/question_block" 
android:layout_width="fill_parent" 
android:layout_height="wrap_content" 
android:padding="10dp"> 

<RelativeLayout android:id="@+id/question" android:layout_width="match_parent" android:layout_height="wrap_content"> 
    <TextView 
      android:id="@+id/question_number_item" 
      android:layout_width="50dp" 
      android:layout_height="50dp" 
      android:layout_marginRight="10dp" 
      android:text="X" 
      android:textSize="30sp" 
      android:textColor="@android:color/white" 
      android:gravity="center" 
      android:background="@drawable/question_number" /> 

    <TextView 
      android:id="@+id/question_text" 
      android:layout_width="wrap_content" 
      android:layout_height="wrap_content" 
      android:paddingTop="5dp" 
      android:text="Question 1 : Texte descriptif de la question tellement long" 
      android:textSize="16sp" 
      android:textColor="@color/MPQ_blue" 
      android:layout_toRightOf="@id/question_number_item"/> 
</RelativeLayout> 

<RelativeLayout android:id="@+id/question_item1" android:layout_width="fill_parent" android:layout_height="wrap_content" android:layout_below="@id/question" android:layout_marginTop="16dp"> 

    <CheckBox android:id="@+id/question_check1" android:layout_width="wrap_content" android:layout_height="wrap_content" android:gravity="start" android:button="@drawable/play_qcm_checkbox" android:text="" android:paddingTop="13dp"/> 
    <LinearLayout android:orientation="vertical" android:id="@+id/question_label1" android:layout_width="fill_parent" android:layout_height="wrap_content" android:layout_toRightOf="@id/question_check1"></LinearLayout> 

</RelativeLayout> 
<RelativeLayout android:id="@+id/question_item2" android:layout_width="fill_parent" android:layout_height="wrap_content" android:layout_below="@id/question_item1" android:layout_marginTop="8dp"> 

    <CheckBox android:id="@+id/question_check2" android:layout_width="wrap_content" android:layout_height="wrap_content" android:gravity="start" android:button="@drawable/play_qcm_checkbox" android:text="" android:paddingTop="13dp"/> 
    <LinearLayout android:orientation="vertical" android:id="@+id/question_label2" android:layout_width="fill_parent" android:layout_height="wrap_content" android:layout_toRightOf="@id/question_check2"></LinearLayout> 

</RelativeLayout> 
<RelativeLayout android:id="@+id/question_item3" android:layout_width="fill_parent" android:layout_height="wrap_content" android:layout_below="@id/question_item2" android:layout_marginTop="8dp"> 

    <CheckBox android:id="@+id/question_check3" android:layout_width="wrap_content" android:layout_height="wrap_content" android:gravity="start" android:button="@drawable/play_qcm_checkbox" android:text="" android:paddingTop="13dp"/> 
    <LinearLayout android:orientation="vertical" android:id="@+id/question_label3" android:layout_width="fill_parent" android:layout_height="wrap_content" android:layout_toRightOf="@id/question_check3"></LinearLayout> 

</RelativeLayout> 
<RelativeLayout android:id="@+id/question_item4" android:layout_width="fill_parent" android:layout_height="wrap_content" android:layout_below="@id/question_item3" android:layout_marginTop="8dp"> 

    <CheckBox android:id="@+id/question_check4" android:layout_width="wrap_content" android:layout_height="wrap_content" android:gravity="start" android:button="@drawable/play_qcm_checkbox" android:text="" android:paddingTop="13dp"/> 
    <LinearLayout android:orientation="vertical" android:id="@+id/question_label4" android:layout_width="fill_parent" android:layout_height="wrap_content" android:layout_toRightOf="@id/question_check4"></LinearLayout> 

</RelativeLayout> 
<RelativeLayout android:id="@+id/question_item5" android:layout_width="fill_parent" android:layout_height="wrap_content" android:layout_below="@id/question_item4" android:layout_marginTop="8dp"> 

    <CheckBox android:id="@+id/question_check5" android:layout_width="wrap_content" android:layout_height="wrap_content" android:gravity="start" android:button="@drawable/play_qcm_checkbox" android:text="" android:paddingTop="13dp"/> 
    <LinearLayout android:orientation="vertical" android:id="@+id/question_label5" android:layout_width="fill_parent" android:layout_height="wrap_content" android:layout_toRightOf="@id/question_check5"></LinearLayout> 

</RelativeLayout> 

</RelativeLayout> 

回答

2

是因为视图正在被重用。 Android在列表视图中重新使用视图以提高性能,因为当您单击相同的视图时会重新使用该视图来显示选中的状态。

您可以删除周围convertView

的if语句if(convertView == NULL) ..

但是,这将是determental性能,如果你有很多的项目。

否则,你应该使用普通viewHolder方法: http://developer.android.com/training/improving-layouts/smooth-scrolling.html

希望这有助于。

+0

有趣的是,我没有想到它是这样重复使用的。 有没有什么办法强制重新创建视图的getView()中的任何发生? 我的意思是,删除if语句只是给我一个崩溃,因为convertView为null。 – 2014-11-06 14:50:43

+1

对不起,我的错是一个不好的解释。删除if,但不包含其内容。 所以你只需要这一行: vi = inflater.inflate(R.layout.question_item,null); – James 2014-11-06 15:13:58

+0

这是我首先想到的,但每次滚动时都会重新创建视图,例如,创建的每个视图上的复选框状态都不是持久的。 – 2014-11-06 15:21:00