静态变量和方法

Question

我遇到了一个像这样设置的类:

public class MyClass {

  private static boolean started = false;

  private MyClass(){
  }

  public static void doSomething(){
    if(started){
      return;
    }
    started = true;
    //code below that is only supposed to run
    //run if not started
  }
}

我对静态方法的理解是你不应该在它们中使用类变量,除非它们是常量,并且不要改变.相反,你应该使用参数.我的问题是,为什么通过执行MyClass.doSomething()多次调用时不会破坏.在我看来,它应该不起作用,但确实如此.它只会传递一次if语句.

所以任何人都可以向我解释为什么这不会破坏？

Answer 1

方法doSomething()和变量started都是静态的,因此只有一个变量副本,可以从中访问doSomething().第一次doSomething()被调用,started是假的,所以它设置started为真,然后......好吧,某事.它被调用的第二次和随后的时间started都是真的,所以它返回时不做任何事情.

Answer 2

没有理由使用静态变量不起作用.我不是说这是特别好的做法,但它会起作用.

应该发生的是:

1. 第一次打电话.该类初始化,启动为假.
2. doSomething被称为.if失败,代码绕过它.started设置为true,另一个代码运行.
3. 再次调用doSomething.if传递和执行停止.

需要注意的一点是,这里没有同步,所以如果在单独的线程上调用doSomething()非常接近,每个线程都可以读取为false,绕过if语句并执行工作,即存在竞争条件.

Answer 3

给出的代码不是线程安全的.使这段代码线程安全的简单方法是做类似的事情

public class MyClass {

  private static AtomicBoolean started = new AtomicBoolean(false);

  private MyClass(){
  }

  public static void doSomething(){
    boolean oldValue = started.getAndSet(true);
    if (oldValue)
      return;
    }

    //code below that is only supposed to run
    //run if not started
  }
}

这应该是线程安全的,因为AtomicBoolean getAndSet是同步的.

不可否认,如果您不使用线程,这不是问题(请注意,webapp可以使用相当多的线程来处理各种请求而您不知道这一点).