Java - Логическое состояние не меняется

Question

У меня этот код здесь:

if (event.getSource() == Square0) 
     { 

      if (PlayerOneTurn == true) Square0.setBackground(Color.red); 
      if (PlayerOneTurn == true) PlayerOneTurn = false ; 

      if (PlayerOneTurn == false) Square0.setBackground(Color.blue) ; 

     } 

Если это не ясно, я хочу фон краснеть и состояние PlayerOneTurn, чтобы изменить ложное, так что потом она меняется синий, когда я нажму снова. Это работает, но if (PlayerOneTurn == true) PlayerOneTurn = false ;, похоже, не изменяет значение переменной. Я использую полностью неправильное утверждение или что-то не хватает?

Answer 1

Использовать else if и аналогичные конструкции.

Вы перезаписать значение PlayerOneTurn в строке 3.

Также убедитесь, что вы вызываете перерисовки при изменении визуальных.

Answer 2

Вы меняете цвет на красный и с первой if и изменить его обратно в синий с третьей if заявление, Измените код, изменив if к if-else как этот

if (PlayerOneTurn == true) 
{ 
    Square0.setBackground(Color.red); 
    PlayerOneTurn = false; 
} 
else 
{ 
    Square0.setBackground(Color.blue) ; 
    PlayerOneTurn = true; 
} 

Answer 3

использовать еще, если

if (PlayerOneTurn) { 
Square0.setBackground(Color.red); 
PlayerOneTurn = false; 
} 
else 
{ 
Square0.setBackground(Color.blue) 
} 

Answer 4

В настоящее время ваш код устанавливает цвет в красный цвет, а затем устанавливает PlayerOneTurn на false, а затем снова устанавливает цвет в синий, потому что, ну, PlayerOneTurn является n ow false.

Что вы хотите

if (event.getSource() == Square0) { 

     if (PlayerOneTurn == true) { 
      Square0.setBackground(Color.red); 
      PlayerOneTurn = false ; 
     } else { 
      Square0.setBackground(Color.blue) ; 
      PlayerOneTurn = true; 
     } 
    } 

Или, более идиоматические относительно булевы:

if (event.getSource() == Square0) { 

     if (PlayerOneTurn) { 
      Square0.setBackground(Color.red); 
     } else { 
      Square0.setBackground(Color.blue) ; 
     } 

     PlayerOneTurn = !PlayerOneTurn; // True becomes false and false becomes true 
    }