Question

我使用套接字制作了一个基本服务器，并想添加一个简单的GUI来打开和关闭它。为了使GUI在服务器运行while循环时仍能正常工作，我为套接字创建了一个线程。现在我想到的是在该while循环中添加一个布尔值，该布尔值退出该循环并导致在按下GUI中的按钮时服务器停止运行。

现在的问题是，布尔值是在GUI线程中设置的，需要在服务器线程中调用。我读过有关将布尔值设置为volatile或使用AtomicBoolean的信息，但它们似乎都不起作用。从服务器线程调用布尔值时，可能需要寻找一些特别的东西吗？

这是我到目前为止编写的（简化）代码：

public class GUI {

    private static int port = 12345;    
    private static volatile boolean online = false;     

    public static void main(String[] args) {

        //GUI works so i left the code out
        //Basically generates a button giving it the actionlistener below

    }

    private static ActionListener getActionListener() {

        return new ActionListener() {

            @Override
            public void actionPerformed(ActionEvent e) {

                if(!online) {
                    online = true;

                    Thread serverThread = new Thread(new ServerThread());
                    serverThread.start();

                } else {
                    online = false;                 
                }               
            }
        };
    }

    public static boolean getOnlineState() {
        return online;
    }

    public static int getPort() {
        return port;
    }
}

以及包含服务器线程的类：

public class ServerThread implements Runnable {

    @Override
    public void run() {

        try {
            ServerSocket serSoc = new ServerSocket(GUI.getPort());
            Socket cliSoc = serSoc.accept();
            PrintWriter  out = new PrintWriter(cliSoc.getOutputStream(), true);
            BufferedReader in = new BufferedReader(new    InputStreamReader(cliSoc.getInputStream()));

            String input;
            while(GUI.getOnlineState()) {
                while((input = in.readLine()) != null) {
                    out.println(input);
                }
            }

            out.println("Stopping");
            cliSoc.shutdownInput();
            cliSoc.shutdownOutput();
            cliSoc.close();         
            serSoc.close();
            out.close();
            in.close();

        } catch(IOException e) {
            e.printStackTrace();
        }       
    }   
}

由于我是所有多线程技术的新手，所以如果您能告诉我一些其他错误，我将不胜感激。

Answer 1

嵌套循环是一个问题：

while(GUI.getOnlineState()) {
    while((input = in.readLine()) != null) {
        out.println(input);
    }
}

一旦进入内循环，它将继续循环直到输入流不再起作用，并且您将无法中断它。也许最好是完全摆脱外部循环并结合您的逻辑：

while(GUI.getOnlineState() && (input = in.readLine()) != null) {
    out.println(input);
}

其他不相关的问题是您对静态字段和方法的过度依赖，这可能适用于小型玩具程序，但随着程序变得更大甚至更多，可能会增加问题的风险。静态变量有增加不应连接的事物的连接性的风险，从而增加了代码的循环复杂性，并因此而带来了错误的风险。将您的顾虑隔离到私有实例字段中，并需要最少的公共方法。

Java：在线程之间共享和调用变量

1 个答案: