問題描述

當我在寫一個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