Java：修改和访问在构造函数中初始化的变量答案

【问题标题】：Java: Modify & access variable initialized in constructorJava：修改和访问在构造函数中初始化的变量
【发布时间】：2013-01-10 06:03:04
【问题描述】：

我有课。
我在该类的构造函数中初始化了一个变量。
我调用了一个包含while 循环的方法，并且每次都递增变量。
我写了一个测试来检查该方法被调用后的变量值（并通过一次while循环）。

public class ThreadGenerator implements Runnable {
  private int requests;
  private int limit;

  public ThreadGenerator() {
    requests = 0;
  }

  public void setRequestLimit(int anyLimit) {
    this.limit = anyLimit;
  }

  public void generateThread() {
    new Thread(this).start();
  }

  public void run() {
      while(requests < limit) {
          try {
              // do some stuff
              requests++;
              // close some stuff
          } catch (IOException e) {
            e.printStackTrace();
          }
      }
  }

  public int getRequests() {
      return requests; // calling this method from my tests always returns 0
  }

在我的测试中，当我创建此类的新实例，然后在该类上调用此方法时，它会正确运行并正确递增请求计数器。我已经尝试了几个打印语句来确保这一点。但是，如果我在测试中对 ThreadGenerator 对象调用 getRequests，它将返回 0，即初始化时的数量。

我的测试代码：

ThreadGenerator threadGenerator = new ThreadGenerator();
threadGenerator.setRequestLimit(1);
threadGenerator.generateThread();
assertEquals(threadGenerator.getRequests(), 1);

如何修改我在构造函数中初始化的这个变量并在我的测试套件中访问它？

【问题讨论】：

你能告诉我们你的测试代码吗？
generateThread() 是做什么的？

标签： java constructor scope variable-assignment

【解决方案1】：

请记住，仅仅因为您要求 Java 使用 Runnable 创建一个新线程，并不意味着会立即调用 run() 方法。 assertEquals 很可能在 run 第一次发生之前发生。

您可能希望返回线程并在生成的线程上的测试中调用join，这将确保线程运行直到它死亡，可能有一个短暂的超时。

/* in the system under test */
@VisibleForTesting Thread generateAndReturnThread() {
  Thread thread = new Thread(this);
  thread.start();
  return thread;
}

public void generateThread() {
  generateAndReturnThread();
}

/* in the test */
@Test public void yourTest() {
  ThreadGenerator threadGenerator = new ThreadGenerator();
  threadGenerator.setRequestLimit(1);

  // wait up to a second for thread to complete
  threadGenerator.generateThreadAndReturn().join(1000);

  assertEquals(threadGenerator.getRequests(), 1);
}

旁注：如果多个线程可能会修改requests，请考虑为您的请求类考虑AtomicInteger。这将有助于防止两个不同的线程同时修改requests 并相互覆盖。

【讨论】：

好吧，在阅读了你的第一句话之后，我在我的 run() 方法中添加了一个“这应该首先打印”语句，并在我去检查变量之前在我的测试中添加了一个“这应该先打印” requests。你是对的，他们的执行顺序与我预期的相反。
谢谢！ join() 就是这样做的。这允许线程完成，当我再次运行它时，它通过了，“这应该首先打印”语句实际上首先打印。我简直不敢相信所有的麻烦，只是因为他们没有按照我期望的顺序被调用！

【解决方案2】：

除了上面的答案，考虑将变量声明为易失性，这样你就可以在另一个线程增加变量后真正得到最新的值。

【讨论】：