java - 避免静态集合的 ConcurrentModification 的良好做法

Question

考虑玩家类...当玩家加入游戏(创建对象)时，它会检查是否已加入同名玩家...

public class Player {

    private static List<Player> players = new ArrayList<>();
    private String name;

    public Player(String name) {
        this.name = name;

        for (Player otherPlayer : players) { // Iterating static field
            if (otherPlayer.name.equalsIgnoreCase(name)) {
                otherPlayer.quit("Somebody with the same name joined the game");
            }
        }
    }

    public void quit(String message) {
        players.remove(this); // Modifying static field
        Server.disconnect(this, message);
    }
}

我知道 Iterator 可以处理这个问题，但我们并不总是知道外部方法中的公共(public)静态字段会发生什么情况以及何时使用 foreach 以及何时改用 Iterator ...

这个问题有什么好的做法吗？

Answer 1

第一个也是更重要的良好做法称为关注点分离。如:Player 类应该模拟单个玩家。

您在一个地方混合作为玩家的职责和管理整个玩家对象集。不要那样做!

这两件事根本不属于一起。从这个意义上讲:应该有一个 PlayerManager 类，例如它知道所有玩家。并且也忘记像这样使用 static 字段。因为这会在你的类的不同方面之间产生超紧密的耦合。例如，当您需要多个 玩家列表时会发生什么情况？如果您有太多玩家想要根据某些属性将他们组织到桶中怎么办？

除此之外，直接的答案是:与其立即从列表中删除对象，不如将它们收集到第二个playersToBeDeleted 列表中。并且迭代第一个列表后，只需使用 players.removeAll(playersToBeDeleted) 为例。

并谈论良好实践:仔细考虑您是否真的想使用 Lists - 或者 Set 是否不是更好的选择。列表总是暗示顺序，糟糕的是，它们允许重复添加相同 对象。而集合免费为您提供“独特的元素”语义!