問題描述

當(dāng)我在寫一個android應(yīng)用的時候遇到了這樣一個需求：

有個ListView，里面的item都有圖片和文字，當(dāng)一個item被點(diǎn)擊的時候顯示在此item上顯示圖片和一個進(jìn)度條，當(dāng)另一個item被點(diǎn)擊的時候隱藏上一次被電擊item的圖片和進(jìn)度條，顯示當(dāng)前被點(diǎn)擊item的圖片和進(jìn)度條

我的有關(guān)代碼是這樣的，用的是simpleAdapter

我的思路是就這樣的，先把ListView的所有item遍歷，隱藏我該隱藏的控件，然后顯示當(dāng)前被點(diǎn)擊item的要顯示的控件（progressbar和playing_effect）

不過問題出現(xiàn)了，點(diǎn)擊之后的效果特別卡，不知道怎么回事點(diǎn)擊40-50次之后就出現(xiàn)手機(jī)內(nèi)存不足的情況，難道我的思路一開始是錯誤的嗎？還是需要優(yōu)化？耗內(nèi)存是怎么回事啊

謝謝

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)){ // 設(shè)置點(diǎn)擊狀態(tài) v.findViewById(R.id.progressBar).setVisibility(View.VISIBLE); v.findViewById(R.id.playing_effect).setVisibility(View.VISIBLE);}else { // 設(shè)置未點(diǎn)擊狀態(tài) 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(); }}

上面我把所有代碼給貼出來的，我按樓下回復(fù)里的想法去做的，我發(fā)現(xiàn)結(jié)果同樣很卡，內(nèi)存耗得更快，大神們看一下吧

謝謝

問題解答

樓上兩位已經(jīng)說的比較清楚了，自己維護(hù)一個列表，然后在getView中根據(jù)列表的狀態(tài)繪制不同的View。之前你的方法需要遍歷并創(chuàng)建所有itemView的實(shí)例，無論是對cpu還是內(nèi)存都有很大的壓力，所以會卡，在item數(shù)量未知的前提下最好不要遍歷，保存一個上次點(diǎn)擊的index就行了。隨便寫了些偽代碼，大概是這樣的

ArrayList<Boolean> list = new ArrayList<>(); int listClickedItemIndex = -1; public View getView(int position, View convertView, ViewGroup parent) {if(list.get(position)){ //設(shè)為點(diǎn)擊狀態(tài)}else{ //設(shè)為未點(diǎn)擊狀態(tài)}return convertView; } public void onItemClick(int position){ if(listClickedItemIndex >= 0){ list.set(listClickedItemIndex,false); }listClickedItemIndex = position;list.set(position,true);adapter.notifyDataSetChanged(); }

----------------------------分割線---------------------你getView部分的代碼有問題，在這里你應(yīng)該創(chuàng)建/復(fù)用convertView

我說下我的思路吧：把item的點(diǎn)擊狀態(tài)都存在adapter中，每次點(diǎn)擊只要去更新adapter中具體item的某個屬性就行了。接著在getView里根據(jù)這個屬性的不同做不同的處理。

把顯示/隱藏的邏輯寫在listView的adapter的getView方法，點(diǎn)擊的時候調(diào)用notifyDataSetChanged就可以了

3樓正解，自己新建一個boolean類型的list用以保存每個item的狀態(tài)，每次點(diǎn)擊先將下標(biāo)currentitemindex的item屬性改為false，將下標(biāo)position的item屬性設(shè)為true。然后更新curreentitemindex.最后刷新adapter