2016-11-08 94 views
1

我在我的應用程序中有一個listView。對於listView中的每個元素,它都有一個Switch。以前我在listView Adapter的getView方法中爲Switch實現了一個onClickListener。但是,我發現雖然它在用戶單擊交換機時工作,但當用戶在交換機上滑動時,它不起作用。然後我嘗試將onClickListener更改爲onCheckedChangeListener。當用戶在Switch上單擊或滑動時,它可以工作,但是,當我滾動ListView以使元素消失時,我發現最初選中的Switch未被選中。Android:在ListView適配器中切換OnCheckedChangeListener

你能幫忙嗎?下面是我的getView方法在ListView適配器代碼:

@Override 
public View getView(final int position, View convertView, final ViewGroup parent) { 
    class viewHolder { 
     public TextView tvSceneName; 
     public Switch swSelectSwitch; 
    } 
    final viewHolder holder; 

    final View v = convertView; 

    // Use ViewHolder to avoid findViewById each time the user scroll 
    if (convertView == null) { 
     holder = new viewHolder(); 
     convertView = mInflater.inflate(R.layout.scene_list_element, parent, false); 
     holder.tvSceneName = (TextView) convertView.findViewById(R.id.tvSceneName); 
     holder.swSelectSwitch = (Switch) convertView.findViewById(R.id.swScene); 
     convertView.setTag(holder); 
    } else { 
     holder = (viewHolder) convertView.getTag(); 
    } 


    if(holder.swSelectSwitch != null) { 

     // Set the switch to correct on/off status 
     holder.swSelectSwitch.setChecked(mDataSource.get(position).get_isOn()); 

     // Set up On checked change listener for the switch 
     holder.swSelectSwitch.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() { 
      @Override 
      public void onCheckedChanged(CompoundButton toggleButton, boolean isChecked) { 

       // Set the new status to arraylist 
       Switch sw = (Switch) v.findViewById(R.id.swScene); 
       MainActivity.sceneArrayList.get(position).set_isOn(sw.isChecked()); 

     }); 
    } 
} 

編輯: 在進一步的測試中,我發現有錯誤在我的新增onCheckChangedListener(因爲我從複製的一些代碼onClickListener)。另外我找到了一個解決方案,它是在我通過代碼設置Switch之前將onCheckChangedListener設置爲null。

if(holder.swSelectSwitch != null) { 

    // Set the switch to correct on/off status 
    holder.swSelectSwitch.setOnCheckedChangeListener(null); 
    holder.swSelectSwitch.setChecked(mDataSource.get(position).get_isOn()); 

    // Set up On checked change listener for the switch 
    holder.swSelectSwitch.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() { 
     @Override 
     public void onCheckedChanged(CompoundButton toggleButton, boolean isChecked) { 

      // Set the new status to arraylist 
      mDataSource.get(position).set_isOn(isChecked); 

    }); 
} 
+1

您正在使用兩個列表:

你應該mDataSource在發生變化時更新狀態。他們有相同的參考?如果沒有,你正在更新一個不同的列表,所以在你滾動之後清楚爲什麼你的複選框沒有更新。其他的事情是你爲什麼不使用onCheckedChanged監聽器的參數,而不是使用findviewbyId – Alex

+0

@Alex你是對的,我發現我應該使用isChecked參數。我使用findviewbyId,因爲我只是從onClickedListner複製代碼。 onClickedListener沒有這個參數。列表是一樣的。事實上,我不清楚如何使用listView,當它有很多組件和數據源是複雜的。 – eepty

回答

0

您應該刪除行if(holder.swSelectSwitch != null) {: 滾動時,意見不被破壞,但回收的,所以當getView被滾動後調用,convertView不爲空。當convertView不爲空時,您將重新使用您已創建的持有者並放入視圖標記中。

1

ViewHolder模式的要點在於View正在循環使用。

你從mDataSource獲取的選中狀態,但沒有更新,所以當View得到回收的Switch返回其原始(未選中)狀態。

@Override 
public void onCheckedChanged(CompoundButton toggleButton, boolean isChecked) { 

    // Set the new status to arraylist 
    MainActivity.sceneArrayList.get(position).set_isOn(isChecked); 

    // updating mDataSource 
    mDataSource.get(position).set_isOn(isChecked); 
}