线程之间的共享数据不对

Question

我的程序有问题。 和数据在多个线程之间共享。 它应该在每个Thread添加到它之后给出总和。

public class ThreadPractise implements Runnable
{
    Integer num;
    String name;
    volatile Integer commonSum;
    Object lock;

    public ThreadPractise(Integer i,String threadName ,Integer sum, Object sharedLock) {
        num = i;
        name = threadName;
        commonSum =sum;
        lock = sharedLock;
    }

    @Override
    public void run() {
        Integer cube = calculate(num);
        update(cube);
        System.out.println(Thread.currentThread().getName()+" has commonSum after update as "+commonSum);
    }

    public void update(Integer cube) {
        synchronized (lock) {
            this.commonSum = this.commonSum + cube;
            System.out.println(Thread.currentThread().getName()+" has commonSum as "+commonSum);
        }
    }
    public int calculate(Integer num2) {
        return (num2*num2*num2);
    }

    public static void main(String[] args) {

        Integer sum = new Integer(0);
        Object lock = new Object();
        for(int i=1;i<=3;i++)
        {
            Thread t = new Thread(new ThreadPractise(i, "Thread"+i, sum,lock));
            t.start();
            try {
                t.join();
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }

        System.out.println("Sum of 1^3+2^3+3^3 = "+sum);

    }

}

但是总和总是0.当我改为AtomicInteger它的工作但我需要这个实现的问题。 还请告诉我，我正在做的线程锁定是否正确。

Answer 1

您的问题与并发性无关。 看看你的main方法。 它有一个局部变量sum 。 永远不会更新此局部变量。 而是，您正在更改ThreadPractise实例的num字段引用的对象。 此值使用sum的值初始化，但是一旦替换num的值，其值就不再与sum相关，因为该值只是被替换但未更新。 因此，打印的值保持为0 ，这是最初分配给sum 。

你做的只是简单地说：

Integer a = 0;
Integer b = a;
b = 1;                 // does not change a
System.out.println(a); // prints '0'

但是在Java中，如果不为此变量赋值，则无法更改变量引用的值。

相反，使用AtomicInteger您将一个可变实例传递给您的线程。 您现在正在更新此实例所代表的值，而不是替换num引用的值。 这样， num和sum变量保持相同，因为它们引用相同的对象。 对于不可变的Integer ，这种方法是不可能的。 如果Integer类有一个类似setValue(int)的方法，那么你正在做的是sum和num会同样引用同一个对象。

Answer 2

Integer是不可变的。 您不能将Integer传递给方法，并期望以这种方式获得更新版本。

如果您不想使用AtomicInteger （可更新，甚至以线程安全的方式，这将是我的首选解决方案），您需要从每个线程检索部分总和或让线程更新公共累加器某处（例如通过使commonSum静态）。

Answer 3

Integer对象是不可变的。 虽然Thread中存在的Integer实例最初与您在main方法中声明的实例相同，但每次其值更改时，它都会替换为新实例，即每次添加到commonSum 。 所有线程都将保存对不同Integer对象的引用，并且main方法中的实例将不受影响。

您必须将Integer包装到某种对象中，以便能够在main方法和线程之间共享它。 AtomicInteger就是这样做的，作为额外的奖励，它将为您完成所有同步。