Tic Tac Toe игра winforms

Question

Я понимаю, что это работает и все, но есть ли лучший способ сделать это?

Button clicked = (Button)sender; 
     if (turn) 
     { 
      clicked.Text = player; 
     } 
     else { 
      clicked.Text = cpu; 
     } 

     turn = !turn; 

Answer 1

Когда вы нажимаете кнопку, выполняется обработчик события click. Это вызывает функцию tictactoe. Там ваш код выполняется сверху вниз, который сначала проверяет .Text первой кнопки в вашем массиве и настраивает ее. Затем закончится ваш If..Else. На следующем нажатии это повторяется. Это приводит к поведению.

Вместо этого вы должны наложить объект sender на Button и только работать над этим. Нет необходимости в целом блоке If..Else. Sender - объект, который поднял событие (здесь: нажата кнопка).

public void tictactoe(object sender, EventArgs e) 
{ 
    Button b = (Button)sender; 
    if (string.IsNullOrEmpty(b.Text)) { 
     b.Text = player; 
    } 
} 

После этого вы также должны перевернуть player от X к O или наоборот, но вы узнаете, как сделать это, я считаю. :-)

Answer 2

Изменение TicTacToe к этому:

public void tictactoe(object sender, EventArgs e) 
{ 
    var button = (Button)sender; 
    if (button.Text == "") 
     button.Text = player; 
}