Question

需要一些有关如何更新列表的帮助。我有一个类，用字符串名称，字符串状态，浮点数和浮点数来定义对象。

每次玩家移动服务器时都会向我发送一个异步消息，这些变量只能是状态lat和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
                    }

这可能是显而易见的，但我真的不知道如何搜索和替换对象。我所追求的功能是：

检查列表是否为空，在这种情况下，不能有播放器，只需添加一个。
如果列表存在循环，则查找对象的名称。
如果名称存在，只需再次创建相同的播放器，但其更新值为相同的索引。
列表存在，但找不到名称，所以只需添加播放器。

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

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中实例化hashmap并获取一个字符串键，其中包含名称和Player对象。

Answer 3

您可以使用 indexOf 来了解玩家内部是否存在p对象。如果不存在，那将返回-1或者在玩家内部存在p对象的位置。

    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;
}

如果对象存在，则更新arraylist

3 个答案: