从列表中抛出抛出异常时抛出IndexOutOfBoundsException：Java-> Kotlin

Question

该代码在Java中工作，但在转换为Kotlin时不再有效。它抛出一个IndexOutOfBoundsException。

这里是原始的Java：

grid_view.setOnItemClickListener(new AdapterView.OnItemClickListener() { 
     @Override 
     public void onItemClick(AdapterView<?> adapterView, View view, int i, long l) { 
      String selectedItem = ((TextView) view.findViewById(R.id.tag_name)).getText().toString(); 
      for (int j = 0; j < itemList.size(); j++){ 
       String tempString = itemList.get(j); 
       if(tempString.equals(selectedItem)) { 
        Log.d("Update", "Removing: " + selectedItem); 
        itemList.remove(j); 
       } 
      } 
     } 
    }); 

这是新科特林：

grid_view!!.onItemClickListener = AdapterView.OnItemClickListener { _, _, _, _ -> 
      val selectedItem = (tag_name as TextView).text.toString() 
      itemList.indices.forEach { 
       val tempString = itemList[it] 
       if (tempString == selectedItem) { 
        Log.d("Update", "Removing: " + selectedItem) 
        itemList.removeAt(it) 
       } 
      } 
    } 

Answer 1

由于indices在第一时间被初始化，所以forEach操作indices后从未改变您从List删除元素，但size()是用Java改变，例如：

//       v--- `size` is changed after remove item from List 
for (int j = 0; j < itemList.size(); j++){ 
    //... 
} 

对于科特林易变的操作，请使用 MutableCollection#removeIf/MutableCollection#removeAll代替，例如：

itemList.removeIf { it == selectedItem } // java-8 

itemList.removeAll { it == selectedItem } // Kotlin 

和存在的Java代码错误的逻辑，例如：

for (int j = 0; j < itemList.size(); j++){ 
    String tempString = itemList.get(j); 
    if(tempString.equals(selectedItem)) { 
     itemList.remove(j--); 
     //    ^
     // you should minus the current j, if don't the next is skipped 
    } 
}