Проблемы со статическими внутренними потоками при доступе к статическим переменным во внешних классах - PullRequest
0 голосов
/ 02 июня 2018

Эта проблема меня давно озадачила, пожалуйста, помогите мне, спасибо.Это мой код Java.

package com.concurrent.example;

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

/**
 * P683
 */
class CircularSet {
    private int[] array;
    private int len;
    private int index = 0;
    public CircularSet (int size) {
        array = new int[size];
        len = size;
        for (int i = 0; i < size; i++) {
            array[i] = -1;
        }
    }

    public synchronized void add(int i ) {
        array[index] = i;
        index = ++index % len;
    }

    public synchronized boolean contains(int val) {
        for (int i = 0; i < len; i++) {
            if(array[i] == val) {
                return true;
            }
        }
        return false;
    }
}

public class SerialNumberChecker {
    private static final int SIZE = 10;
    private static CircularSet serials = new CircularSet(1000);
    private static ExecutorService exec = Executors.newCachedThreadPool();
    private static int serial;
    static class SerialChecker implements Runnable {
        @Override
        public void run() {
            while(true) {
                //int serial;
                synchronized (serials) {
                     serial = SerialNumberGenerator.nextSerialNumber();
                }
                if (serials.contains(serial)) {
                    System.out.println("Duplicate: " + serial);
                    System.exit(0);
                }
                System.out.println(serial);
                serials.add(serial);
            }
        }
    }

    public static void main(String[] args) throws InterruptedException {
        for (int i = 0; i < SIZE; i++) {
            exec.execute(new SerialChecker());
            if (args.length > 0) {
                TimeUnit.SECONDS.sleep(new Integer(args[0]));
                System.out.println("No duplicates detected");
                System.exit(0);
            }
        }
    }
}

Он может остановиться, но когда я раскомментирую //int serial; Результат отличается, он не может остановиться. Почему эта временная переменная имеет другой результат, чем статическая переменнаявнешнего класса.Это причина использования потока?

Код SerialNumberGenerator:

public class SerialNumberGenerator {
    private static volatile int serialNumber = 0;
    public static int nextSerialNumber() {
        return serialNumber ++; //Not thread-safe
    }
}

1 Ответ

0 голосов
/ 02 июня 2018

При private static int serial все SerialNumberChecker s разделяют одинаковые serial.Например:

  1. Набор потоков1 serial = 1
  2. Набор потоков 2 serial = 2
  3. Набор потоков1 2 в CircularSet.
  4. Thread2 обнаружил его дубликат и завершил работу.

Однако, если вы объявите еще один int serial в методе run, он будет затенять private static int serial, что означает, что у всех потоков есть свои serial, и ониназначит и проверит это.Поскольку генерация serial находится в синхронизированном блоке, дубликатов не будет.

...