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

Question

我使用套接字制作了一个基本服务器，并想添加一个简单的 GUI 来打开和关闭它。为了在服务器运行 while 循环时使 GUI 仍然工作，我为套接字创建了一个线程。现在我在想的是向那个 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);

                }

            }

由于我对所有这些多线程的东西都不熟悉，因此如果您能告诉我，我会很高兴地发现其他一些错误。

Answer 1

嵌套循环是个问题：while(GUI.getOnlineState()) {    while((input = in.readLine()) != null) {        out.println(input);    }}一旦进入内循环，它将继续循环，直到输入流不再工作，您将无法摆脱它。也许更好的是完全摆脱外部循环并结合您的逻辑：while(GUI.getOnlineState() && (input = in.readLine()) != null) {    out.println(input);}其他不相关的问题是您对静态字段和方法的过度依赖，这可能适用于小型玩具程序，但随着您的程序变得更大并且可能有更多错误，这可能会增加问题。静态可能会增加不应该连接的事物的连接性，增加代码的圈复杂度，从而增加出现错误的风险。将您的关注点隔离到私有实例字段和所需的最少公共方法中。