迭代器，列表迭代器和列表

Question

我遇到了一个我无法完全解决的问题。 如果你们可以帮助我，那就太好了=）

我有一个具有draw（）方法的Lobby.Class：

public void draw() {
    //USE ITERATOR TO DRAW
    Iterator<Player> itr = players.iterator();
    while(itr.hasNext()) {
        Player player = itr.next();
        int off = players.indexOf(player); //THE CORRECT HEIGHT
        player.draw(off);
    }
}  

请注意，这是服务器/客户端游戏

Lobby.Class也有一种称为addPlayer（）的方法

public void addPlayer(String _playerName, String _class, String _string_id) {

    int _id = Integer.parseInt(_string_id);

    //CREATE THE NEW PLAYER
    Player new_player = new Player(x, y, _id, _playerName, _class);

    //IF THIS PLAYER IS ME
    if(new_player.getID() == id) {

        me = new_player;

    } else {

        //TELL THE NEW PLAYER I EXIST
        ClientValue.getClient().iExist(_id);

                    //THIS WILL SEND TO CLIENT THAT WILL SEND TO THE SERVER
                    //THAT WILL LOOK FOR THE ID AND SEND TO CLIENT OF THAT ID
                    //AND THE CLIENT WILL SEND TO LOBBY UPDATE PLAYERS()
    }

    players.add(new_player);

    chat.appendChat(chat.joinString(_playerName, _class));
}

Lobby.Class也有1个称为updatePlayers（）的方法

public void updatePlayers(String _playerName, String _class, String _string_id) {

    //THIS IS CALLED WHEN THE SERVER TELLS THIS PLAYER
    //ABOUT OTHER PLAYERS

    int _id = Integer.parseInt(_string_id);

    //CREATE THE NEW PLAYER
    Player new_player = new Player(x, y, _id, _playerName, _class);
    players.add(new_player);

}

现在我的问题是球员名单，当我进入大厅addPlayer()将在同一时间运行在2个玩家加入，或者当updatePlayers()与运行addPlayer()

我得到的异常是： ConcurrentModificationException

我缺少两种方法，一种是在可能的情况下使用迭代器添加播放器，另一种是删除播放器 。 这样我就不会在通过或添加/删除播放器列表时收到修改播放器列表的错误。

我曾尝试过弄乱Iteror和ListIterator，但由于我以前从未使用过它们，因此我不确定该怎么做。 我需要一些有关如何执行此操作的建议。 提前非常感谢=）

解：

private List<Player> players = Collections.synchronizedList(new ArrayList<Player>());

    public synchronized void playersList(String cmd, Player _player) {

    Iterator<Player> itr = players.iterator();

    if(cmd.equals("draw")) {

        while(itr.hasNext()) {

            Player player = itr.next();
            int off = players.indexOf(player);
            player.draw(off);
        }

    } else if(cmd.equals("add")) {

        players.add(_player);

    } else if(cmd.equals("remove")) {

        players.remove(_player);
    }
}

Answer 1

您需要同步对列表的访问，一次只允许一个线程访问列表。

您可以在每个访问点周围使用synchronized块（包括整个paint方法），也可以使用Collections#synchronizedList将播放器列表包装在线程安全列表中，或者可以使用播放器的toArray方法来代替迭代器

Player[] arrayOfPlayers = players.toArray(new Player[players.size()]);

您仍将synchronized此调用，但是您只能锁定一行，而不是整个循环