我有一个 friend 说，在 Java Web 应用程序的上下文中，所有静态方法都应该同步。真的吗？我已经阅读了许多其他关于此的堆栈溢出页面。我逐渐相信，只有在满足以下条件时才需要同步:

1. 多线程(如在带有线程池的 Sevlet 容器中)
2. 单个类加载器
3. 线程间的共享数据，无论是Session数据还是静态成员数据。
4. 共享数据必须是可变的。只读数据可以共享。

基于此我认为应该同步静态成员，而不是静态方法。

import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;

public class ThreadTest {

    static String staticString = "";

    // This static method is safe b/c it only uses local data.
    // It does not use any shared mutable data.
    // It even uses a string builder.
    static String safeStaticMethod(String in) {
        // This also proves that StringBuilder is safe
        // When used locally by a thread.
        StringBuilder sb = new StringBuilder();
        sb.append("Hello: ");
        sb.append(in);
        return sb.toString();
    }

    // This static method is not safe b/c it updates and reads
    // shared mutable data among threads.
    // Adding synchronized will make this safe.
    static String unsafeStaticMethod(String in) {
        staticString = in;
        StringBuffer sb = new StringBuffer();
        sb.append("Hello: ");
        sb.append(staticString);
        return sb.toString();
    }

    public static void main(String[] args) {
        ThreadTest test = new ThreadTest();
        test.staticMethodWithLocalData();
        test.staticMethodWithStaticData();
    }

    public void staticMethodWithLocalData() {

        ExecutorService executor = Executors.newFixedThreadPool(2);
        final int iterations = 100000;

        executor.submit(new Runnable() {

            @Override
            public void run() {
                for (int index = 0; index < iterations; ++index) {
                    if (!safeStaticMethod("Thread1").equals("Hello: Thread1")) {
                        System.out.println("safeStaticMethod at " + index);
                    }
                }
            }
        });

        executor.submit(new Runnable() {

            @Override
            public void run() {
                for (int index = 0; index < iterations; ++index) {
                    if (!safeStaticMethod("Thread2").equals("Hello: Thread2")) {
                        System.out.println("safeStaticMethod at " + index);
                    }
                }
            }
        });
    }

    public void staticMethodWithStaticData() {

        ExecutorService executor = Executors.newFixedThreadPool(2);
        final int iterations = 100000;

        executor.submit(new Runnable() {

            @Override
            public void run() {
                for (int index = 0; index < iterations; ++index) {
                    if (!unsafeStaticMethod("Thread1").equals("Hello: Thread1")) {
                        System.out.println("unsafeStaticMethod at " + index);
                    }
                }
            }
        });

        executor.submit(new Runnable() {

            @Override
            public void run() {
                for (int index = 0; index < iterations; ++index) {
                    if (!unsafeStaticMethod("Thread2").equals("Hello: Thread2")) {
                        System.out.println("unsafeStaticMethod at " + index);
                    }
                }
            }
        });
    }
}

这段代码是否证明了这一点？

编辑:这只是我为了证明这一点而破解的一些一次性代码。

最佳答案

不，并不是所有的静态方法都需要同步。据我所知，你的 list 基本上是完整的。静态方法要么特别小心

1. 访问一个可变的静态成员，或者
2. 获取对可修改对象的引用。

我认为不用说 1(首先要有线程)是一个先决条件，因为没有线程 synchronize 是没有意义的。

我从来没有听说过 2，所以我不确定它是否是一个考虑因素。

关于Java:是否所有静态方法都需要同步？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/14862506/