简单添加方法线程安全吗？

Question

有了这个简单的类，用另外的方法：

class A { 
    public Integer add (int a, int b){ 
    return a+b; 
    } 
} 

是线程安全的或不..？它对我来说看起来很安全，但大多数人回答不，任何人都可以解释为什么？

Answer 1

它完全是线程安全的，因为所有变量都是本地的。

Answer 2

只有当你有一些共享状态的手段并且你没有任何锁定或同步，即修改一个共享变量（类级别变量）时，你应该关心线程安全性。
这里没有线程安全的问题。
而在这种特殊情况下每个变量是局部的，该位置不会被线程共享的每个函数调用都会对堆栈单独分配他们连同他们的局部变量你不应该打扰反正:)

Answer 3

实际上，这种方法不是线程安全的，但它需要您了解一些关于Integer类的内部，以了解原因。让我们来看看一些代码，产生相同的字节码：

class A { 
    public Integer add (int a, int b){ 
    // auto boxing hidden in OP's implementation 
    return Integer.valueOf(a+b); 
    } 
} 

足够小的值Integer s的高速缓存和阵列中抬起头来。使用反射可以访问该数组并更改其元素。 这些更改不会同步，因此如果您更改这些元素，则可能会从另一个线程更改方法的结果。

下面的代码应该演示大多数Java VM上的问题：您的方法中存在竞争条件。在大多数情况下，它会打印4S和5S：

import java.lang.reflect.Field; 

class A { 

    public Integer add(int a, int b) { 
     return a + b; 
    } 

    private static volatile boolean cont = true; 

    public static void main(String[] args) throws NoSuchFieldException, IllegalArgumentException, IllegalAccessException, InterruptedException { 
     final A a = new A(); 

     new Thread(() -> { 
      while(cont) { 
       for (int i = 0; i < 100; i++) { 
        // print result of add method 
        System.out.println(a.add(2,2)); 
       } 
      } 
     }).start(); 

     // give other thread time to start 
     Thread.sleep(1); 

     // mess around with the internals of Integer 
     Class cache = Integer.class.getDeclaredClasses()[0]; 
     Field c = cache.getDeclaredField("cache"); 
     c.setAccessible(true); 
     Integer[] array = (Integer[]) c.get(cache); 
     array[132] = array[133]; 

     cont = false; 
    } 
} 

然而，在大多数情况下，没有人与周围的Integer的内部食堂。如果Integer类中的数组永远不会被修改，则由方法返回的Integer对象包装的值始终是正确的，因为Integer.valueOf所使用的共享状态永远不会被修改。因此在这种情况下它将是线程安全的。