循环遍历列表元组并添加（如果不匹配）

Question

我正在用C＃编写游戏服务器，每次有人用他的用户名发送登录消息时，我将他添加到ID为IPENDPOINT，USERNAME的元组列表中。 我想检查客户端是否与已连接的客户端没有相同的用户名，所以我试图循环访问列表并获取list.Item3'这是用户名'，并使用String.Equals（list.Item3，username）检查用户名是否存在。 我的问题是当我循环，我的代码只比较列表中的第一个元组，如果它是真的，然后发送错误消息，如果没有，然后发送接受消息，当列表有+1客户端它只比较第一个也接受即使元组2或3 ...具有该用户名。 我尝试过很多办法，这是最后的代码，我想：

for (int i = 0; i < clientsInfoList.Count; i++) 
{ 
    bool isUsed; 
    if (String.Equals(clientsInfoList[i].Item3, Username)) 
    { 
     isUsed = true; 
    } 

    if (isUsed) 
    { 
     Console.WriteLine("Username is already used!"); 
     udpServer.Send(Encoding.ASCII.GetBytes("REFUSED"), Encoding.ASCII.GetByteCount("REFUSED"), remoteEP); 
     break; 
    }    
    else if(!isUsed) 
    { 
     clientsInfoList.Add(new Tuple<int, IPEndPoint, string>(id, remoteEP, Username)); 
     Console.WriteLine("Username has been added to the list :)"); 
     udpServer.Send(Encoding.ASCII.GetBytes("ACCEPTED"), Encoding.ASCII.GetByteCount("ACCEPTED"), remoteEP);      
    } 
} 

我试过很多其他方法，但我不能获得验证。 在此先感谢。

Answer 1

你把所有东西放在一个循环中。为什么？您应该仅使用循环来检查用户名是否被使用，并将消息发送给它。

首先，你想强制你的循环工作，只要它应该。您可以在它之前声明isUsed并将其添加到条件中。然后，在循环中，只检查名称是否已被使用并更改变量的值。循环将遍历所有客户端，或者在遇到与您的条件匹配的名字时结束。当循环结束时，您应该根据结果决定发送哪个命令。

bool isUsed = false; 
for (int i = 0; i < clientsInfoList.Count && !isUsed; i++) 
{ 
    isUsed = String.Equals(clientsInfoList[i].Item3, Username); 
} 

if (isUsed) 
{ 
    Console.WriteLine("Username is already used!"); 
    udpServer.Send(Encoding.ASCII.GetBytes("REFUSED"), Encoding.ASCII.GetByteCount("REFUSED"), remoteEP); 
} 
else 
{ 
    clientsInfoList.Add(new Tuple<int, IPEndPoint, string>(id, remoteEP, Username)); 
    Console.WriteLine("Username has been added to the list :)"); 
    udpServer.Send(Encoding.ASCII.GetBytes("ACCEPTED"), Encoding.ASCII.GetByteCount("ACCEPTED"), remoteEP); 

} 

Answer 2

为什么要将整个逻辑放在for循环中？在循环内只放置下面的代码。

  bool isUsed; 
      if (String.Equals(clientsInfoList[i].Item3, Username)) 
      { 
       isUsed = true; 
      } 

它现在将验证元组中的所有元素并执行剩余的逻辑。

Answer 3

你要对这个完全错误的方式，如果我是你，我会做，第一件事是摆脱的元组，创建一个包含你所需要的3个属性的类。

而且在这样的情况下，尽量用一个HashSet而不是列表，他们是更好的优化，将帮助你的游戏运行速度更快，如果你有大量的客户。

达到你的要求最彻底的方法是使用LINQ，这里就是我会重写你上面写的代码。

public class Game 
{ 
    public Game() 
    { 
     this.Clients = new HashSet<Clients>(); 
    } 

    public HashSet<Client> Clients { get; set;} 

    public void OnClientConnect(Client newClient) 
    { 
     // Are there any clients with the username that the newUser is attempting to use? 
     bool usernameIsFree = this.Clients.Any(clients => clients.UserName == newClient.UserName); 

     if (usernameIsFree) 
     { 
      this.Clients.Add(newClient); 
      Console.WriteLine("Username has been added to the list :)"); 
      // UDP stuff here... 
      return; 
     } 

     Console.WriteLine("Username is already used!"); 
     // UDP stuff here 
    } 
} 
public class Client 
{ 
    public int ClientId { get; set; } 
    public IPEndPoint IPEndPoint { get; set; } 
    public string UserName { get; set; } 
} 

Answer 4

你可以做

var isUsed = clientsInfoList.Any(info => info.Item3 == Username); 
if (isUsed) 
{ 
    // ... 
} 
else 
{ 
    // ... 
} 

我假定这两个Item3（的Tuple<T1, T2, T3>T3）和Username有编译时型string。