当所有挂起的消息发送完毕后关闭套接字

Question

我有这种方法将消息写入套接字：

public void sendMessage(byte[] msgB) {
    try {
        synchronized (writeLock) {
            log.debug("Sending message (" + msgB.length + "): " + HexBytes.toHex(msgB));
            ous.write(HEADER_MSG);
            ous.writeInt(msgB.length);
            ous.write(msgB);
            ous.flush();
        }
    } catch (IOException e) {
        throw new RuntimeException(e);
    }
}

现在一个名为 Bob 的线程想要在某个不确定的时刻 X 关闭套接字，这意味着可能仍然有线程等待 writeLock 发送消息，甚至可能有一个线程正在写入消息。

我可以通过让 Bob 在关闭套接字之前获取 writeLock 来解决后者，但我仍然可能会丢失尚未开始发送的消息，因为据我所知 synchronized 是不公平的，Bob 可以在其他等待更长时间的线程之前获得锁。

我需要的是在 X 之前对 sendMessage 进行的所有调用都能正常完成其工作，而在 X 之后进行的调用会抛出错误。我该怎么做？

• 具体说明：Bob 是从套接字输入流读取的线程，X 是在该流上收到“关闭”消息的时间。

Answer 1

您可以在此处使用执行者。由于每个发送消息都是同步的（我假设在一个公共共享对象上），因此您可以使用线程限制。

static final ExecutorService executor = Executors.newSingleThreadExecutor();

public void sendMessage(byte[] msgB) {
    executor.submit(new Runnable() {
        public void run() {
            try {
                log.debug("Sending message (" + msgB.length + "): " + HexBytes.toHex(msgB));
                ous.write(HEADER_MSG);
                ous.writeInt(msgB.length);
                ous.write(msgB);
                ous.flush();
            } catch (IOException e) {
                throw new RuntimeException(e);
            }
        }
    });
}
public static void atMomentX(){
    executor.shutdown();
}

完成后，另一个线程可以调用 atMomentX()；

从 javadoc 中，关闭方法说：

启动有序关闭，其中
之前提交的任务有
已执行，但不会执行新任务
公认。调用没有
如果已经关闭则有额外效果
下来。

Answer 2

考虑使用单线程 ExecutorService< /code>执行消息写入。发送线程只是尝试通过调用execute(Runnable) 或submit(Callable) 来“发送”它们的消息。一旦您希望停止发送消息，请关闭 ExecutorService (shutdown()) 导致后续调用提交/执行导致 RejectedExecutionException 。

这种方法的优点是，与有多个线程等待写入消息本身相比，您只有一个 I/O 绑定线程，并且锁争用更少。这也是更好的关注点分离。

下面是一个简单的例子，它使问题更加面向对象：

public interface Message {
  /**
   * Writes the message to the specified stream.
   */
  void writeTo(OutputStream os);
}

public class Dispatcher {
  private final ExecutorService sender;
  private final OutputStream out;

  public Dispatcher() {
    this.sender = Executors.newSingleThreadExecutor();
    this.out = ...; // Set up output stream.
  }

  /**
   * Enqueue message to be sent.  Return a Future to allow calling thread
   * to perform a blocking get() if they wish to perform a synchronous send.
   */
  public Future<?> sendMessage(final Message msg) {
    return sender.submit(new Callable<Void>() {
      public Void call() throws Exception {
        msg.writeTo(out);
        return null;
      }
    });
  }

  public void shutDown() {
    sender.shutdown(); // Waits for all tasks to finish sending.

    // Close quietly, swallow exception.
    try {
      out.close();
    } catch (IOException ex) {
    }
  }
}

Answer 3

我想我可以用 ReentrantLock 设置为公平的。