基本数据记录器的实现

Question

在我的代码中，我有一个名为DataLogger的对象。顾名思义，它负责将数据记录到文件中。

类的骨架如下所示：

class DataLogger {
   private BufferedWriter mFileWriter = ...
   private boolean mRunning = false;
   //constructor
   public DataLogger() {
   //init stuff
   }
   public void onEvent(..) {
   //...
   if(mRunning) {
      mFileWriter.write(...);
   }
   /...
   }
   public void start() {
   //startdata logging
   mRunning = true;
   }
   public void stop() {
   //stop data logging
   mRunning = false;
   //closing File writer etc.
   }
}

因此对象在onEvent方法中接收Events。每次出现新事件时，数据都会记录到一个文件中。数据记录器应使用start()方法启动，并使用stop()方法停止。我在这段代码中看到的问题是，如果DataLogger的客户端/调用方调用stop()，则可能是一个事件在同一时刻刚刚到达，然后将被记录下来，尽管用户以前调用过stop()。如果数据流已经关闭，这甚至可能导致崩溃。我想我需要在这里添加一些同步/锁定代码，对吗？扩展代码的最佳方式是什么？

Answer 1

最直接的解决方案是同步onEvent（）和stop（）。这样，两个方法中只有一个可以同时调用，从而解决最初的问题。

public void onEvent(..) {
public void start() {
public void stop() {

成为

public synchronized void onEvent(..) {
public synchronized void start() {
public synchronized void stop() {