Question

我有一个练习：给定一个AtomicIntegers矩阵（初始化为0），我必须为每一行运行一个线程，并在我添加1的列上的每行的一个线程上减去1。所以最后矩阵应该保持不变。问题是矩阵发生了变化！这是代码：该对象采用矩阵，一个布尔值，它将告诉他对列或行进行操作，以及它必须操作的列/行的索引。

import java.util.concurrent.atomic.AtomicInteger;

public class FifthExerciseSafe implements Runnable{
    private Thread thread;
    private boolean onRows;//tells if the object operates on the rows or on the columns
    private AtomicInteger[][] matrix;
    private int index;

public FifthExerciseSafe(AtomicInteger[][] matrix, boolean onRows, int index){
    this.matrix = matrix;
    this.onRows = onRows;
    this.index = index;
}

public void start(){
    thread = new Thread(this);
    thread.start();
}

public boolean isOnRows() {
    return onRows;
}

public void setOnRows(boolean onRows) {
    this.onRows = onRows;
}

public AtomicInteger[][] getMatrix() {
    return matrix;
}

public void setMatrix(AtomicInteger[][] matrix) {
    this.matrix = matrix;
}

@Override
public void run() {
    if (this.onRows){
        for(int i = 0; i < matrix[index].length; i++){
            matrix[index][i].decrementAndGet();
        }
    } else {            
        for(int i = 0; i < matrix.length; i++){
            matrix[i][index].incrementAndGet();
        }
    }
}//run

public static void main(String args[]){
    AtomicInteger[][] m = new AtomicInteger[3][3];
    for (int i = 0; i < m.length; i++){
        for(int j = 0; j < m.length; j++){
            m[i][j]= new AtomicInteger(0);
        }
    }

    for(int i = 0; i < 3; i++){//I create 6 objects, 3 for columns and 3 for rows
        FifthExerciseSafe fes1 = new FifthExerciseSafe(m, true, i);
        FifthExerciseSafe fes2 = new FifthExerciseSafe(m, false, i);
        fes1.start();
        fes2.start();
    }
    for(int i = 0; i < m.length; i++){
        for(int j = 0; j < m.length; j++){
            System.out.print(m[i][j]+" ");
        }
    }
}//main
}

输出应为： 000000 但有时它是： -100-100-1-10

它只发生在带有Intel Atom的上网本上，在带有Core Duo的桌面上我还没有看到它，但我想知道它是否也会在那里发生。

Answer 1

你已经启动了线程，但是在继续打印矩阵之前，你永远不会等待它们完成。您需要为每个已启动的线程使用Thread.join，这将阻塞直到线程完成。只有这样才能安全地打印出结果。例如，在ArrayList中引入add您开始Thread的所有实例，然后在每个for循环中调用join实例

AtomicInteger不当行为

1 个答案: