如果对象存在，则更新数组列表

Question

需要一些关于如何更新列表的帮助。我有一个类，它定义了字符串名称，字符串状态，float lat和float long的对象。

每次玩家移动一台服务器向我发送这些变量的异步消息时，唯一改变的可以是状态经纬度和长度，这是我想要“覆盖”的那个。这个名字是独一无二的，所以我猜测我必须根据这个来搜索，因为这是永远不会改变的事情。我想更新现有的对象以防止一个玩家出现多次。

我的（非功能性）到目前为止的代码：

if(players.size()==0){ ///// 1. 
         Player p = new Player(); 
         p.name = split[2]; 
         p.status = split[3]; 
         p._lat = Float.valueOf(split[4]); 
         p._long = Float.valueOf(split[5]); 
         players.add(p); 
        }//This works 


        else{ ///// 2. 
        for(int i = 0; i < players.size(); i++){ ///// 3. 
         if(players.get(i).name.equals(split[2])){ 
          Player p = new Player(); 
          p.name = split[2]; 
          p.status = split[3]; 
          p._lat = Float.valueOf(split[4]); 
          p._long = Float.valueOf(split[5]); 
          players.add(p); 
          players.add(i, p); 
         } 
         else{ ///// 4. 
          Player p = new Player(); 
          p.name = split[2]; 
          p.status = split[3]; 
          p._lat = Float.valueOf(split[4]); 
          p._long = Float.valueOf(split[5]); 
          players.add(p); 
         } 
        }//This doesnt 
        } 

这可能是一些明显的，但我真的不知道如何查找和替换的对象。我之后的功能是：

1. 检查列表是否为空，在这种情况下，无法在其中添加播放器，因此只需添加一个。
2. 如果列表存在循环，并查找对象的名称。
3. 如果名称存在，只需再次创建相同的播放器，但其更新值位于同一索引处。
4. 该列表存在但名称未找到，因此只需添加播放器即可。

我的代码到目前为止让我在一个永无止境的循环我猜，因为应用程序停止响应。 split变量是从异步调用中分离出来的一个词，这是我通过打印检查的名称。

Answer 1

，而不是

players.add(p); 
players.add(i, p); 

使用

players.set(i, p) 

Answer 2

嗯，我终于把dmons的建议，并提出了HashMap的，我不知道，如果它的好做法，但它似乎与此，而不是工作：

if(players.size()==0){ 
         Player p = new Player(); 
         p.name = split[2]; 
         p.status = split[3]; 
         p._lat = Float.valueOf(split[4]); 
         p._long = Float.valueOf(split[5]); 
         players.add(p); 
         hash.put(split[2], p); 
        } 
        else{ 
         Player p = new Player(); 
         p.name = split[2]; 
         p.status = split[3]; 
         p._lat = Float.valueOf(split[4]); 
         p._long = Float.valueOf(split[5]); 
         players.add(p); 
         hash.put(split[2], p); 
         } 

随着哈希映射在oncreate中被实例化并接受一个字符串键，这是一个名字和一个播放器对象。

Answer 3

你可以使用indexOf来知道p对象是否存在于玩家内部。如果不存在或者p对象存在于玩家内部，则返回-1。

Player p = new Player(); 
    p.name = split[2]; 
    p.status = split[3]; 
    p._lat = Float.valueOf(split[4]); 
    p._long = Float.valueOf(split[5]); 

    if(players.size()==0){ 
     players.add(p); 
    } 
    else{ 
     int index=players.indexOf(p); 
     if (index==-1){ 
      players.add(p); 
     } 
     else{ 
      players.set(index,p); 
     } 
    } 

诀窍是在您的Player类中添加此方法，该方法覆盖indexOf上使用的等号方法。这样，您可以为两个Player对象设置相同的必要条件（仅限您的案例中的名称）。

@Override 
public boolean equals(Object o) { 
    if (o == this) { 
     return true; 
    } 
    if (o == null) { 
     return false; 
    } 
    if (o.getClass() == this.getClass()) { 
     String a = this.name; 
     String b = ((Player) o).name; 
     return a.equalsIgnoreCase(b); 
    } 
    return false; 
}