ArrayList заполняется вместо замены индекса java

Question

Я пытаюсь заменить значение в моем строчном массиве, но мой аррайалист просто добавляет вместо замены.

temp - это строка, которая берет значение из буферизатора. player_status_line - мой строковый архаист.

Я не вижу, что я делаю неправильно. Я проверяю размер массива, и если он содержит имя пользователя (getName [2]), я устанавливаю новое значение в одно и то же место.

Это то, что я сделал, но почему-то все еще сохраняет все. Пробовал пройти код, но не вижу проблемы.

Код редактируется *

while ((checkIfPlayer = bReader.readLine()) != null) { 
    // Arraylist 
    String temp = checkIfPlayer.trim(); 
    System.out.println(temp); 
    if (temp.contains("ZOMBIE") || temp.contains("HUMAN")) { 
     String[] getName = temp.split(" "); 
     String checkName = getName[2]; 
     if (!player_status_list.contains(checkName)) { 
      player_status_list.add(temp); 
     } else if (player_status_list.contains(checkName)) { 
      for (int i = 0; i < player_status_list.size(); i++) { 
       player_status_list.set(i, temp); 
      } 
     } 
     DrawTheMap(); 
    } 
} 

Answer 1

чек неверны по адресу:

if (!player_status_list.contains(checkName)) { 
    player_status_list.add(temp); 
} else if (player_status_list.contains(checkName)) { 
    for (int i = 0; i < player_status_list.size(); i++) { 
     player_status_list.set(i, temp); 
    } 
} 

Вот разбивка со встроенными комментариями:

//temp = "4711 PLAYER joseph HUMAN 30.1 30.1"; 
//checkName = "joseph" 

//This is checking if the list contains: "joseph" 
if (!player_status_list.contains(checkName)) { 
    //If it doesn't have it we are adding: "4711 PLAYER joseph HUMAN 30.1 30.1"; 
    //This doesn't seem right!!! 
    player_status_list.add(temp); 

    //This will never happen because list is always adding the temp variable not the name 
} else if (player_status_list.contains(checkName)) { 
    for (int i = 0; i < player_status_list.size(); i++) { 
     player_status_list.set(i, temp); 
    } 
} 

Попробуйте следующее:

while ((checkIfPlayer = bReader.readLine()) != null) { 
    // Arraylist 
    String temp = checkIfPlayer.trim(); 
    System.out.println(temp); 
    if (temp.contains("ZOMBIE") || temp.contains("HUMAN")) { 
     String[] getName = temp.split(" "); 
     String checkName = getName[2]; 
     boolean added = false; 
     for(int i = 0; i < player_status_list.size(); i++) { 
      if(player_status_list.get(i).contains(checkName)) { 
       player_status_list.set(i, temp); 
       added = true; 
      } 
     } 
     if(!added) { 
      player_status_list.add(temp); 
     } 
     DrawTheMap(); 
    } 
} 

Входы я тестировал (Очевидно, я закомментировал DrawTheMap() часть)

ASYNC PLAYER joseph HUMAN 30.0 30.0 
ASYNC PLAYER joseph HUMAN 50.0 80.0 
ASYNC PLAYER BOB ZOMBIE 30.0 30.0 
ASYNC PLAYER GIL ZOMBIE 30.0 30.0 

Это результирующее player_status_list я получаю:

ASYNC PLAYER joseph HUMAN 50.0 80.0 
ASYNC PLAYER BOB ZOMBIE 30.0 30.0 
ASYNC PLAYER GIL ZOMBIE 30.0 30.0 

Answer 2

В ваш цикл, сначала проверьте, чтобы увидеть Если player_status_list.get (i) == checkName и только затем замените переменную в этом индексе с помощью checkName.

Вы также можете упростить весь код, отправленный выполнив:

if !containsname 
    add name 
else 
    find index at the name (maybe use forloop) 
    replace at index with name