Понимание того, почему небезопасно запускать поток внутри конструктора с точки зрения модели памяти Java

Согласно Параллелизм Java на практике опасно запускать поток в конструкторе класса. Причина в том, что это предоставляет указатель this другому потоку до того, как объект будет полностью построен.

Несмотря на то, что эта тема обсуждалась во многих предыдущих вопросах StackOverflow, мне все еще трудно понять, почему это так важно. В частности, я надеюсь выяснить, может ли запуск потока внутри конструктора привести к проблемам непротиворечивости памяти с точки зрения модели памяти Java.

Позвольте мне привести вам конкретный пример того, что я хочу сделать. (Желаемый результат для этого фрагмента кода — число 20, которое будет напечатано на консоли.)

private static class ValueHolder {

    private int value;
    private Thread thread;

    ValueHolder() {
        this.value = 10;
        thread = new Thread(new DoublingTask(this)); // exposing "this" pointer!!!
        thread.start();   // starting thread inside constructor!!!
    }

    int getValue() {
        return value;
    }

    void awaitTermination() {
        try {
            thread.join();
        } catch (InterruptedException ex) {}
    }

}

private static class DoublingTask implements Runnable {

    private ValueHolder valueHolder;

    DoublingTask(ValueHolder valueHolder) {
        this.valueHolder = valueHolder;
    }

    public void run() {
        System.out.println(Thread.currentThread().getName());
        System.out.println(valueHolder.getValue() * 2);  // I expect to print out 20...
    }

}

public static void main(String[] args) {
    ValueHolder myValueHolder = new ValueHolder();
    myValueHolder.awaitTermination();
}

Да, я знаю, что поток запускается до того, как мы возвращаемся из конструктора. Да, я знаю, что указатель this открыт для потока. И тем не менее, я убежден, что код правильный. Я считаю, что он всегда будет печатать число 20 на консоли.

• Задание this.value = 10бывает-преждеthread.start(). (Это потому, что this.value = 10 предшествует thread.start() в порядке выполнения программы.)
• Вызов thread.start() в основном потоке бывает-прежде запуск метода .run() во вновь созданном потоке. (Потому что thread.start() — это синхронизирующее действие.)
• Начало метода .run()бывает-прежде оператора печати System.out.println(valueHolder.getValue() * 2);. (Опять же, по порядку программы.)

Следовательно, согласно модели памяти Java оператор печати должен считывать правильно инициализированное значение valueHolder.value (а именно, 10). Таким образом, несмотря на то, что я проигнорировал совет Параллелизм Java на практике, я все же написал правильный фрагмент кода.

Я сделал ошибку? Что мне не хватает?

ОБНОВИТЬ: Основываясь на ответах и ​​комментариях, теперь я понимаю, что мой пример кода является функционально верен по указанным мною причинам. Однако писать код таким образом — плохая практика, потому что есть шанс, что другие разработчики в будущем добавят дополнительные операторы инициализации после запуска потока. Одна из ситуаций, когда такие ошибки вероятны, — это реализация подклассов этого класса.