如何让线程等待变量达到特定值（多线程 Java）

Question

我有一个接受客户端连接的服务器程序。这些客户端连接可以属于许多流。例如，两个或多个客户端可以属于同一个流。在这些流中，我必须传递一条消息，但我必须等到所有流都建立起来。为此，我维护了以下数据结构。

ConcurrentHashMap<Integer, AtomicLong> conhasmap = new ConcurrentHashMap<Integer, AtomicLong>();

Integer 是流 ID，Long 是客户端编号。为了使给定流的一个线程等待 AtomicLong 达到特定值，我使用了以下循环。实际上，流的第一个数据包将其放入流 ID 和要等待的连接数。对于每个连接，我都会减少等待的连接数。

while(conhasmap.get(conectionID) != new AtomicLong(0)){
       // Do nothing
}

但是这个循环阻塞了其他线程。根据这个 answer 它进行易失性读取。如何修改代码以等待给定流的正确线程，直到它达到特定值？

Answer 1

如果您使用的是 Java 8，CompletableFuture 可能非常适合。这是一个完整的、人为的示例，它正在等待 5 个客户端连接并向服务器发送消息（使用带有 offer/poll 的 BlockingQueue 进行模拟）。

在此示例中，当达到预期的客户端连接消息计数时，CompletableFuture 钩子完成，然后在您选择的任何线程上运行任意代码。

在此示例中，您没有任何复杂的线程等待/通知设置或繁忙的等待循环。

package so.thread.state;

import java.util.concurrent.*;
import java.util.concurrent.atomic.AtomicBoolean;
import java.util.concurrent.atomic.AtomicLong;

public class Main {

  public static String CONNETED_MSG = "CONNETED";
  public static Long EXPECTED_CONN_COUNT = 5L;

  public static ExecutorService executor = Executors.newCachedThreadPool();
  public static BlockingQueue<String> queue = new LinkedBlockingQueue<>();

  public static AtomicBoolean done = new AtomicBoolean(false);

  public static void main(String[] args) throws Exception {

    // add a "server" thread
    executor.submit(() -> server());

    // add 5 "client" threads
    for (int i = 0; i < EXPECTED_CONN_COUNT; i++) {
      executor.submit(() -> client());
    }

    // clean shut down
    Thread.sleep(TimeUnit.SECONDS.toMillis(1));
    done.set(true);
    Thread.sleep(TimeUnit.SECONDS.toMillis(1));
    executor.shutdown();
    executor.awaitTermination(1, TimeUnit.SECONDS);

  }

  public static void server() {

    print("Server started up");
    // track # of client connections established
    AtomicLong connectionCount = new AtomicLong(0L);

    // at startup, create my "hook"
    CompletableFuture<Long> hook = new CompletableFuture<>();
    hook.thenAcceptAsync(Main::allClientsConnected, executor);

    // consume messages
    while (!done.get()) {
      try {
        String msg = queue.poll(5, TimeUnit.MILLISECONDS);
        if (null != msg) {
          print("Server received client message");
          if (CONNETED_MSG.equals(msg)) {
            long count = connectionCount.incrementAndGet();

            if (count >= EXPECTED_CONN_COUNT) {
              hook.complete(count);
            }
          }
        }

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

    print("Server shut down");

  }

  public static void client() {
    queue.offer(CONNETED_MSG);
    print("Client sent message");
  }

  public static void allClientsConnected(Long count) {
    print("All clients connected, count: " + count);
  }


  public static void print(String msg) {
    System.out.println(String.format("[%s] %s", Thread.currentThread().getName(), msg));
  }
}

你会得到这样的输出

[pool-1-thread-1] Server started up
[pool-1-thread-5] Client sent message
[pool-1-thread-3] Client sent message
[pool-1-thread-2] Client sent message
[pool-1-thread-6] Client sent message
[pool-1-thread-4] Client sent message
[pool-1-thread-1] Server received client message
[pool-1-thread-1] Server received client message
[pool-1-thread-1] Server received client message
[pool-1-thread-1] Server received client message
[pool-1-thread-1] Server received client message
[pool-1-thread-4] All clients connected, count: 5
[pool-1-thread-1] Server shut down

Answer 2

你的表情：

conhasmap.get(conectionID) != new AtomicLong(0)

将始终为真，因为您正在比较永远不会相等的对象引用，而不是值。更好的表达方式是：

conhasmap.get(conectionID).longValue() != 0L)

，但是像这样在循环中没有等待/通知逻辑的循环并不是一个好习惯，因为它不断地使用 CPU 时间。相反，每个线程都应该在 AtomicLong 实例上调用 .wait()，并且当它递减或递增时，您应该在 AtomicLong 实例上调用 .notifyAll() 以唤醒每个等待线程以检查表达式。 AtomicLong 类在修改时可能已经在调用 notifyAll() 方法，但我不确定。

AtomicLong al = conhasmap.get(conectionID);
synchronized(al) {
    while(al.longValue() != 0L) {
        al.wait(100); //wait up to 100 millis to be notified
    }
}

在递增/递减的代码中，它看起来像：

AtomicLong al = conhasmap.get(conectionID);
synchronized(al) {
    if(al.decrementAndGet() == 0L) {
        al.notifyAll();
    }
}

我个人不会为此计数器使用 AtomicLong，因为您没有从 AtomicLong 的无锁行为中受益。只需使用 java.lang.Long 代替，因为无论如何您都需要在计数器对象上同步 wait()/notify() 逻辑。