Android ListView的一个坑,你可掉进去过?
需要的功能很简单,就是一个带checkbox的列表,提交时需要知道用户选择了那些项目,如下图:
使用SimpleAdapter作为数据适配器,重写SimpleAdapter.ViewBinder的方法,这样用比自定义Adapter要方便点,代码如下
datas定义是private List<Map<String, Object>> datas=null;
其中让Map中保存一项自我引用(my)绑定到checkBox
private Map<String, Object> populateMap(String lblNo, Map<String, Object>... maps) { Map<String, Object> map = null; if (maps.length > 0) { map = maps[0]; } else { map = new HashMap<String, Object>(); } map.put("lblNo", lblNo); map.put("my", map); map.put("checked", true); return map; }
private void bindAdapter(){ int[] to = new int[] { R.id.lblNo,R.id.ckbIt }; String[] from = new String[] { "lblNo","my"}; adapter = new SimpleAdapter(this, datas, R.layout.activity_post_list_item, from, to); // =======添加删除事件======= SimpleAdapter.ViewBinder binder = new SimpleAdapter.ViewBinder() { private boolean supressEvent=false; @Override public boolean setViewValue(View view, Object data, String textRepresentation) { final Object d = data; if (view instanceof CheckBox) { final Map map=(Map)d; CheckBox ckb = (CheckBox) view; Log.d("T", "->Map"+map); //ckb.setTag(map); supressEvent=true; //需要避免在这里触发OnCheckedChange事件监听处理 ckb.setChecked((Boolean)map.get("checked")); supressEvent=false; ckb.setOnCheckedChangeListener(new OnCheckedChangeListener() { @Override public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { Log.d("T", "onCheckedChanged->Map"+map +" ischecked:" +isChecked); if(supressEvent)return; //Map map=(Map)buttonView.getTag(); map.put("checked", isChecked); } }); return true; //返回true表示不需要父类进行默认设置 //参考http://www.cnblogs.com/carmanloneliness/p/3500832.html //http://www.cnblogs.com/carmanloneliness/p/3500832.html } return false; } }; adapter.setViewBinder(binder); // =======End======= lv.setAdapter(adapter); }
刚开始时,没加上面红色注释那一句,发现运行时程序行为总不合要求。
后来发现是ckb在执行setChecked时会触发OnCheckedChange处理程序,
而SimpleAdapter采用的也是控件重用机制,就是当列表往上下拖时,那些被拖出屏幕外的控件会重用(绑定新的数据,参考代码里给的那链接),由于使用了final在执行ckb.setChecked((Boolean)map.get("checked")); 触发该控件的OnCheckedChange处理程序,而这个处理程序指向的数据项是前一次绑定的那行即前一次调用setViewValue传入的数据,这样就可能导致datas中的某个数据被意外修改,进而引起程序行为的不确定。
解决办法就是在执行ckb.SetChecked时做个标记,而事件处理程序根据这个标记排除拖动列表产生消息。