android的ListView点击事件问题？

问题描述

当我在写一个android应用的时候遇到了这样一个需求：

有个ListView，里面的item都有图片和文字，当一个item被点击的时候显示在此item上显示图片和一个进度条，当另一个item被点击的时候隐藏上一次被电击item的图片和进度条，显示当前被点击item的图片和进度条

我的有关代码是这样的，用的是simpleAdapter

我的思路是就这样的，先把ListView的所有item遍历，隐藏我该隐藏的控件，然后显示当前被点击item的要显示的控件（progressbar和playing_effect）

不过问题出现了，点击之后的效果特别卡，不知道怎么回事点击40-50次之后就出现手机内存不足的情况，难道我的思路一开始是错误的吗？还是需要优化？耗内存是怎么回事啊

谢谢

public class MainActivity extends AppCompatActivity { ArrayList<Boolean> itemState = new ArrayList<>(); int checkedItemIndex = -1; @Override protected void onCreate(Bundle savedInstanceState) {super.onCreate(savedInstanceState);setContentView(R.layout.activity_main);for (int s=0;s<20;s++){ itemState.add(false);}class AudioListAdapter extends SimpleAdapter{ public AudioListAdapter(Context context, List<? extends Map<String, ?>> data, int resource, String[] from, int[] to) {super(context, data, resource, from, to); } @Override public View getView(int position, View convertView, ViewGroup parent) {View v = super.getView(position, convertView, parent);//if (itemState.get(position)){ // 设置点击状态 v.findViewById(R.id.progressBar).setVisibility(View.VISIBLE); v.findViewById(R.id.playing_effect).setVisibility(View.VISIBLE);}else { // 设置未点击状态 v.findViewById(R.id.progressBar).setVisibility(View.GONE); v.findViewById(R.id.playing_effect).setVisibility(View.GONE);}//return v; }}final ArrayList<HashMap<String, Object>> listItem = new ArrayList<HashMap<String, Object>>();for (int i = 0; i < 10; i++){ HashMap<String,Object> map = new HashMap<String, Object>(); map.put('song','song name '+i); map.put('singer','singer '+i); map.put('currentTime','01:3'+i); map.put('allTime','04:5'+i); listItem.add(map);}final ListView listView = (ListView)findViewById(R.id.listView);final AudioListAdapter audioListAdapter = new AudioListAdapter(this,listItem,R.layout.item,new String[]{'song','singer','currentTime','allTime'},new int[]{R.id.song,R.id.singer,R.id.currentTime,R.id.allTime});listView.setAdapter(audioListAdapter);// onclicklistView.setOnItemClickListener(new AdapterView.OnItemClickListener() { @Override public void onItemClick(AdapterView<?> parent, View view, int position, long id) {//if (checkedItemIndex>=0){ itemState.set(checkedItemIndex,false);}checkedItemIndex = position;itemState.set(position,true);audioListAdapter.notifyDataSetChanged();// }}); } @Override public void onStart(){super.onStart(); }}

上面我把所有代码给贴出来的，我按楼下回复里的想法去做的，我发现结果同样很卡，内存耗得更快，大神们看一下吧

谢谢

问题解答

楼上两位已经说的比较清楚了，自己维护一个列表，然后在getView中根据列表的状态绘制不同的View。之前你的方法需要遍历并创建所有itemView的实例，无论是对cpu还是内存都有很大的压力，所以会卡，在item数量未知的前提下最好不要遍历，保存一个上次点击的index就行了。随便写了些伪代码，大概是这样的

ArrayList<Boolean> list = new ArrayList<>(); int listClickedItemIndex = -1; public View getView(int position, View convertView, ViewGroup parent) {if(list.get(position)){ //设为点击状态}else{ //设为未点击状态}return convertView; } public void onItemClick(int position){ if(listClickedItemIndex >= 0){ list.set(listClickedItemIndex,false); }listClickedItemIndex = position;list.set(position,true);adapter.notifyDataSetChanged(); }

----------------------------分割线---------------------你getView部分的代码有问题，在这里你应该创建/复用convertView

我说下我的思路吧：把item的点击状态都存在adapter中，每次点击只要去更新adapter中具体item的某个属性就行了。接着在getView里根据这个属性的不同做不同的处理。

把显示/隐藏的逻辑写在listView的adapter的getView方法，点击的时候调用notifyDataSetChanged就可以了

3楼正解，自己新建一个boolean类型的list用以保存每个item的状态，每次点击先将下标currentitemindex的item属性改为false，将下标position的item属性设为true。然后更新curreentitemindex.最后刷新adapter