java threads простая глупая программа

Question

Надеюсь, кто-то мне поможет. Я пытаюсь сделать светофор с java Threads. Есть два круга. Каждый круг представляет собой лампу (нить). Я хотел бы, чтобы каждый светильник работать х мс, а затем выключить и пусть другие лампы работы (опять же на несколько секунд и снова ..)

public void run() 
{ 
    while (true) 
    {   
     repaint(); 
     if (working) 
     { 
      System.out.println(name + " is WORKING "); 
      try 
      { 
       Thread.sleep(LAMP_WORKING_TIME); 
      } 
      catch (InterruptedException e) 
      { 
       e.printStackTrace(); 
      } 

      System.out.println(name + " has done working"); 

      synchronized (lock) 
      { 
       this.working = false; 
       lock.notifyAll(); 
      } 
     } 
     else 
     { 
      try 
      { 
       System.out.println(name + " is waiting.."); 

       synchronized (lock) 
       { 
        lock.wait(); 
        working = true; 
       } 

       System.out.println(name + " is WAKING UP AFTER WAIT"); 
      } 
      catch (InterruptedException e) 
      { 
       e.printStackTrace(); 
      } 
     } 
    } 
} 

Когда замок объявлен как свойство класса:

private final Object lock = new Object(); 

Я считаю, что моя проблема возникает с помощью методов wait/notify. Пожалуйста, помогите мне исправить это!

Answer 1

Возможно, вы захотите сделать свой lock обмен между экземплярами. Поэтому либо сделайте это static, либо передайте его в Runnables/Threads в качестве параметра вместо создания нового для каждого экземпляра, даже если это final.

Answer 2

Как будет выглядеть ваш код, он будет ждать LAMP_WORKING_TIME мс, задает значение false и следующий цикл сразу вернет его в true. Вы спите только тогда, когда это working истинно, поэтому ваш цикл «on» равен LAMP_WORKING_TIME, а цикл «off» - столько, сколько требуется, чтобы выпустить и снова запереть замок. Вероятно, менее 1 мс.