线程安全、静态方法和一些奇怪的代码

Question

我最近偶然发现了一段类似于下面的代码。这段代码确实很臭。看起来像单例，但不是因为没有私有构造函数。我确定这将有足够大的负载给它带来线程安全问题。专门给定的类实例。有人可以指出这段代码的线程安全问题吗？

public class AClass extends AnotherClass {

  public static final AClass instance = new AClass();

  public static SomeObject doSomethingThatCallsAService(Params params) {
       return methodThatCallsService(params, instance);
  }

  public static SomeObject methodThatCallsService(Params params, AClass instance) {
      -----call service here ---------
      instance.doSomethingElse();
  }

  private void doSomethingElse() {
      --- do some trivial work -----
  }
}

Answer 1

鉴于对象不携带状态，因此无需担心线程安全，无论有多少线程调用方法或引用单例对象。

类中的所有方法，包括静态方法，不使用任何共享数据。因此，无论是调用单例对象的方法还是传递实例，都无需访问要同步的任何内容。

正如代码所示，唯一可能可能需要同步的数据是methodThatCallsService 的参数中的params，并且只有在此方法修改数据并且多个线程持有引用的情况下到同一个Params 对象。

但就这个类而言，它是线程安全的，即使单例实现易受攻击。