0

Подскажите, пожалуйста, почему while{thread.sleep(1)} работает, а while{} нет. Разве программа не должна компилироваться также?

public class Just_tell_me_why {
static int [] list = new int[51];
public static void main(String[] args) throws InterruptedException {
    Counter counter = new Counter();
    counter.start();
    while (list[50]!=50){

    }

    for (int x:list) System.out.print(x + " ");

}
public static class Counter extends Thread{
    public void run(){
        for (int i = 0; i < list.length; i++) {
            list[i]=i;
            try {
                Thread.sleep(1);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }

     }
    }
}
3
  • 1
    @Disabled77 а как вы поняли, что цикл не работает?
    – Barmaley
    22 окт 2014 в 16:22
  • При запуске программы это строчка не работает - for (int x:list) System.out.print(x + " ");
    – Disabled
    23 окт 2014 в 16:17
  • @Disabled77, так что все-таки происходит? Программа не завершается и "висит" в while (list[50] != 50); часами? И эта ситуация стабильно воспроизводится?
    – avp
    24 окт 2014 в 20:38

2 ответа 2

1

Переменные, доступ к которым осуществляется из разных потоков, должны быть объявлены, как volatile.

В вашем случае

volatile static int [] list = new int[51];
19
  • @Алексей Шаповалов: Что толку, что ссылка на массив объявлена как volatile? Это не сделает volatile элементы массива.
    – VladD
    23 окт 2014 в 20:25
  • @Алексей Шаповалов: Дубликаты объекта не будут создаваться никогда в любом случае без явного вызова new. Вы можете привести ссылку на конкретный абзац документации, который гарантирует, что объявление ссылки на массив как volatile гарантирует happens-before для элементов массива?
    – VladD
    23 окт 2014 в 21:10
  • @Алексей Шаповалов: Ещё раз: а где именно там про элементы массива? Там я вижу лишь консистентность вида разных потоков на ссылку на массив, но не на его элементы.
    – VladD
    23 окт 2014 в 21:47
  • 1
    @avp: Автор в комментарии к вопросу пишет, что строка вывода после цикла «не срабатывает». Это, судя по всему, означает, что у него цикл крутится бесконечно. --- Для сложных технических проблем четырёх комментариев явно недостаточно.
    – VladD
    25 окт 2014 в 22:33
0

В данном случае, он работает и кушает циклы процессора в пустую, пока истинно условие самого цикла. Работу цикла можно проверить добавлением всего лишь одной строки: System.out.println("I'm working, dude!");

5
  • при завершении цикла, должен сработать: for (int x:list) System.out.print(x + " "); В свою очередь, если добавить в цикл любой код(как например в нашем случае "System.out.println("I'm working, dude!")"), то цикл начнет работать
    – Disabled
    23 окт 2014 в 16:16
  • @Disabled77: Э, не. Вы должны засинхронизировать обращение к переменной из разных потоков. Иначе оптимизатор имеет право вас наказать за недостаточное курение документации и непонимание Java Memory Model.
    – VladD
    23 окт 2014 в 16:30
  • Можете, пожалуйста, кинуть ссылку почитать? Т.к. я об этом совсем ничего не знаю.
    – Disabled
    23 окт 2014 в 17:14
  • @Disabled77: docs.oracle.com/javase/tutorial/essential/concurrency/sync.html Вкратце: окружайте доступ к разделяемой между потоками переменной блоками synchronized(о). Не забудьте, что этот самый o обязан быть одним и тем же во всех потоках, которые работают с переменной. (Например, может подойти Counter.class.)
    – VladD
    23 окт 2014 в 17:36
  • @Disabled77: Вот ещё: en.wikipedia.org/wiki/Java_Memory_Model (неожиданно, да?)
    – VladD
    23 окт 2014 в 21:21

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.