多线程 Java Web 服务器中的内存泄漏

Question

我有一个多线程 Java Web 服务器，它接受来自客户端的请求。

RAM中的每个请求都会为这个请求分配内存，但在socket关闭后不会被清除。仅在重新启动 Web 服务器后才会进行清理。

如果是内存泄漏，那么会发生在代码的哪一部分呢？

我的代码：

import java.io.IOException;
import java.net.ServerSocket;
import java.net.Socket;

public class Main {
    public static void main(String[] args) {
        try (ServerSocket server = new ServerSocket(80)) {
            Socket socket = server.accept();
            while (true) {
                new Thread(new Client(socket)).start();
            }
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
}

class Client implements Runnable {
    private Socket socket;

    Client(Socket socket) {
        this.socket = socket;
    }

    @Override
    public void run() {
        try {
            System.out.println("run");
        } finally {
            try {
                socket.close();
                System.out.println("Socket closed: " + socket);
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
    }
}

Answer 1

在while(true) 循环中创建线程绝不是一个好主意。
默认情况下，每个创建的 Thread 都会分配； 1mb 用于 64 位 jvm，128 kb 用于 32 位 jvm 的堆栈（这是用于 linux 操作系统，其他详细信息 here）。尝试ExecutorService 处理这类事情。线程是宝贵的资源。

Answer 2

为每个传入的连接启动一个新线程是非常低效的，并且可能是内存泄漏的原因。

正确的做法是使用一个线程池，其中有固定数量的可重复使用的线程。

还值得指出的是，线程消耗大量的本地内存（Java 堆外）而堆内存非常少，因此不会触发垃圾回收，而垃圾回收又会调用终结器来释放本地端。

编辑： 此外，代码并没有按照您的想法执行。将 accept 置于循环之外意味着代码将一遍又一遍地为同一个连接启动线程，而不是为每个连接创建一个新线程。