一些哈希表问题。从哈希表中删除对象

Question

我在从频道列表中删除客户端时遇到一些问题。

这是我目前的代码：

服务器端：

private Hashtable<String, ArrayList<String>> channels = new Hashtable<String, ArrayList<String>>();

    public synchronized void logMeOut(String username) throws RemoteException {
        for(Client c : clients){
            if(c.findName().equals(username)){
                clients.remove(c);
                disconnectAllChans(username);
                System.out.println(username + " removed from clientlist.");
            }
        }
        updateJListForOnlineUsers(); //Callback for other clients to update the userlist.
    }

  public void disconnectAllChans(String username) throws RemoteException{
    for(Enumeration e = channels.elements(); e.hasMoreElements();){
        if(channels.contains(username)){
            channels.remove(username);
        }
    }
    updateJListForUsersInChannel();
    System.out.println("User " + username + " left all channel");
}

我已经尝试了 if(channels.contains(username) 和 containsKey。它们似乎都不起作用。当我离开服务器时，运行注销方法，我猜

客户端会在该枚举循环中挂起：如果用户的频道列表为空，则客户端只会挂起。有

什么想法吗？

**

解决方案：

**

所以，是的，我想通了，但如果没有你们，我就不会这样做。谢谢，

我刚刚在 Brainzzy 发布的断开所有通道方法中运行了断开连接方法。结果如下：

@Override
public void disconnectChannel(String username, String channel) throws RemoteException{
    if(isUserInChannelX(username, channel)){
        channels.get(channel).remove(username);
        String message = "User " + username + " left the channel.";
        notifySelf(username, " You have left the channel " + channel);
        notifyChannelSystem(channel, "SYSTEM", message);
        updateJListForActiveChannels();
        if(channels.get(channel).isEmpty()){
            channels.remove(channel);
        } 
    }
}

public void disconnectAllChans(String username) throws RemoteException{
    for (String channel : channels.keySet()) {
    ArrayList<String> members = channels.get(channel);
        if (members.contains(username)) {
            disconnectChannel(username, channel);
            System.out.println("User " + username + " left channel " + channel);
        }
    }
    updateJListForUsersInChannel();
}

我觉得有点愚蠢^^谢谢大家！

Answer 1

您的代码中有一个无限循环：

for(Enumeration e = channels.elements(); e.hasMoreElements();){
    if(channels.contains(username)){
        channels.remove(username);
    }
}

您永远不会从枚举 e 中取出一个元素，因此 e.hasMoreElements 将始终返回 true。您可能想要更像这样的东西：

ArrayList<String> channel = null;
for(Enumeration e = channels.elements(); e.hasMoreElements(); channel = e.nextElement()){
    if(channel.contains(username)){
        channel.remove(username);
    }
}

作为旁注，您将遇到当前代码的并发修改错误。

Answer 2

将 for 替换为 foreach 循环，使您更难犯无限循环错误，就像您因未正确处理枚举器而陷入的错误一样：

for (String channel : channels.keySet()) {
    ArrayList<String> members = channels.get(channel);
    if (members.contains(username)) {
        members.remove(username);
    }
}

我认为这会做您想要的事情。