Question

我正在开发一个小型服务器 - 客户端程序。我在这里的代码也在不同的程序中运行并且工作正常，但由于某些原因，在这个程序中没有。

因此，当用户想要登录时，他们键入他们的用户名和密码，然后将此消息发送为＆＃34; loginreq user：pass＆＃34;到服务器套接字。

虽然客户显然不是问题（我认为），但我会为您提供代码：

private static void loginreq(){
    String User = null;
    String Pass = null;
    try {
        User = bufferRead.readLine();
        WriteOut(ANSI_RESET+"Password: " + ANSI_GREEN);
        Pass = bufferRead.readLine();
        WriteOut(ANSI_RESET);
    } catch (IOException e1) {
        e1.printStackTrace();
    }
    int serverPort = 6880;
      String ip = Server;
      String data = "loginreq " + User + ":" + Pass;
    try{ 
        Socket s = new Socket(ip, serverPort); 
        DataInputStream input = new DataInputStream( s.getInputStream()); 
      DataOutputStream output = new DataOutputStream( s.getOutputStream());
      if(s.isConnected()){
          //Step 1 send length
          System.out.println("Length"+ data.length());
          output.writeInt(data.length());
          //Step 2 send length
          System.out.println("Writing.......");
          output.writeBytes(data); // UTF is a string encoding

          //Step 1 read length
          int nb = input.readInt();
          byte[] digit = new byte[nb];
          //Step 2 read byte
          for(int i = 0; i < nb; i++){
            digit[i] = input.readByte();
          }
          String st = new String(digit);
      System.out.println("Received: "+ st);
    } else {
        WriteOut("Failed to connect to the server: "+Server+ "No Server");
    }
    }
    catch (UnknownHostException e){ 
        WriteOut("Sock:"+e.getMessage());
    }
    catch (EOFException e){
        WriteOut("EOF:"+e.getMessage()); 
    }
    catch (IOException e){
        WriteOut("IO:"+e.getMessage());
    }
}

以上获取用户名和密码，并按照我之前的解释发送。

然后服务器做了它的魔力（或缺乏）服务器代码：

//Step 1 read length
          int nb = input.readInt();
          byte[] digit = new byte[nb];
          //Step 2 read byte
           String st =null;
          for(int i = 0; i < nb; i++){
            digit[i] = input.readByte();

           st = new String(digit);
          }
          ServerOut("Recieved : " + ANSI_CYAN +
            clientSocket.getInetAddress() + ANSI_RESET + " - " + st);
            if (st.startsWith("loginreq ")){ //login attempt
                ServerOut("[" + Server.getTimeNow() + "] "+getThreadInfoString() +" Proccessing login request from: " + clientSocket.getInetAddress());
                try(BufferedReader br = new BufferedReader(new FileReader("users.txt"))) {
                    st = st.replaceFirst("loginreq ", "");
                    for(String line; (line = br.readLine()) != null; ) {
                        if(st.equals(line)){
                            String[] userDat = st.split(":");
                            boolean duplicate = false;
                            if(!connected.isEmpty()){
                            for(User u : connected){
                                if (u.getUsername().equals(userDat[0])){
                                    duplicate = true;
                                    break;
                                } else {

                                }
                            }
                            }
                            if (duplicate){
                                ServerOut("[" + Server.getTimeNow() + "] "+getThreadInfoString() +" User rejected (duplicate) " + userDat[0] + " : " + clientSocket.getInetAddress());
                                clientReturn("User was already logged in!");
                            } else {
                                User user = new User(clientSocket.getInetAddress().toString(), userDat[0], (int) time);
                                connected.add(user);
                                ServerOut("[" + Server.getTimeNow() + "] "+getThreadInfoString() + " User: '" + userDat[0] + "' logged in with ip: " + clientSocket.getInetAddress());
                                clientReturn("Login accepted!");
                            }
                            break;
                        }
                    }
                } catch (Exception ex){
                    clientReturn("Server ran into an error!");
                    ex.printStackTrace();
                }
            }

这将读取客户端输入，如果它以＆＃34; loginreq＆＃34;开头，则删除它并分隔字符串，以便userDat [0]是用户名，userDat [1]是密码。到目前为止，它已经验证用户存在。然后，如果有用户连接（我刚启动服务器，所以没有任何），它会发现同一用户是否已经登录，由于某种原因似乎返回true。然后它返回到用户已经登录的客户端。这不仅是一个问题，而且使用ServerOut的初始启动代码正确地打印到控制台，现在它在新线程中似乎没有打印。我怀疑这可能是因为我使用的是Jansi控制台而是IDK。

对不起，这是一篇很长的帖子，但感谢你坚持下去！

Answer 1

问题似乎是connected列表包含不应包含任何条目的条目。

你需要弄清楚这是怎么发生的。查看创建列表对象的代码，并检查是否没有使用错误的东西预先填充它。列表不会神奇地获取条目¹。如果它是非空的，那么你的代码已经将条目放在那里......某处，某种程度上。

我还会添加一些日志记录，以查看connected包含的内容，以及“重复检查”代码之前的userDat[0]。

最后，既然你提到这段代码是多线程的：

检查只有一个connected并且所有线程都共享它。
检查您是否正确同步。

（我怀疑你没有正确同步。这不能直接解释你所描述的行为...... connected在任何人试图登录之前都是非空的......但是它可能导致其他问题。）

如果这没有帮助，那么您需要提供MCVE，以便我们可以＆gt;＆gt;运行＆lt;＆lt;你的代码可以弄清楚这里到底发生了什么。

^{1 - 虽然理论上可行，但我会忽略类库或JVM中错误的可能性。}

比较字符串不工作，空列表不为空

1 个答案: