不，在这样的循环中执行thread.join()不是一个好主意。
thread.join()将导致当前线程进入等待状态，直到变量thread引用的线程结束。因此，循环将挂在那里。不接受新套接字。
在接受套接字之后，应该创建一个新线程，并且套接字应该被移交给新线程进行进一步处理。一旦接受了套接字并创建了ClientSocket，就会从积压队列中删除连接请求。如果该循环进入等待状态，则积压队列也将被填满，并且连接请求也可能被拒绝。
接受新连接的当前线程应该继续，所以它不应该使用join。它应该创建一个新线程，并尽快继续接受新的连接请求，以便积压队列在大多数时间内也保持相对空。
只有当服务器正在关闭时，循环才应该中断（不接受新连接），然后在循环之外，它可以等待正在运行的线程完成（完成与已经连接的客户端的通信），然后退出。

启动一个新线程然后立即join它是没有任何意义的。创建一个新线程的目的是让新线程可以做它的事情，而调用者 * 并发 * 做一些其他事情：

Runnable r = ...;
Thread t = new Thread(r);
t.start();
doSomeOtherThingConcurrentlyWithTheNewThread();
t.join();

如果调用者不想做其他事情，那么就没有理由创建新线程。你可以通过这样做来保存一些CPU周期，它会有完全相同的效果：

Runnable r = ...;
r.run();