0

ダイアログのリストにsetMultiChoiceItemsを使用します。[OK]をクリックすると、コードは

 .setPositiveButton(R.string.alert_remove, new DialogInterface.OnClickListener(){
public void onClick(DialogInterface dialog, int which){
String[] res = null;
for(int i=-1,l=usrCats.length; ++i<l;){
    if(selections[i])
       res[i] = usrCats[i].toString();
}
if(res != null && res.length>0)
   dbManager.removeUserShoppingCategories(res);
}       
})

アイテムが選択されているかどうかを確認したいif(selections[i]ので、選択されている場合は、文字列に名前を追加します

にnullポインタアクセスがありres[i] = usrCats[i].toString()、最後のifステートメントにデッドコード警告があります。

私は最後のifを使用して、メソッドを呼び出すための選択があったかどうかを確認しています。

私は何が間違っているのですか？

4

1 に答える 1

1 
        .setPositiveButton(R.string.alert_remove, new DialogInterface.OnClickListener(){
        public void onClick(DialogInterface dialog, int which){
        String[] res = null; // you initialised it with null
        for(int i=-1,l=usrCats.length; ++i<l;){ /*this line and the below line are condition 
    statemnts whose code may not run. so there are chances that your **res** will 
    remain initialised with null*/
            if(selections[i])
               res[i] = usrCats[i].toString();
        }
        if(res != null && res.length>0)// so here it shows a dead code warning.
           dbManager.removeUserShoppingCategories(res);
        }       
        })
于 2012-02-03T11:46:17.467 に答える