Звонок неоднозначная ошибка - игра в кости

Question

Привет, я пытаюсь сделать игру в кости и продолжать получать эту ошибку, не уверен, что это такое или что-то не так с моим алгоритмом. Любая помощь будет оценена по достоинству. Спасибо :) Это показывает ошибку:

The call is ambiguous between the following methods or properties: 'Form1.UpdateGameStatus(int)' and 'Form1.UpdateGameStatus(int)'

private int UpdateGameStatus(int sumDice) 
    { 

     if (sumDice == 2) 
     if (sumDice == 3) 
     if (sumDice == 14) 
      lblSum.Text = ("You lost!"); 
     if (sumDice == 9) 
     if (sumDice == 10) 
      lblSum.Text = ("You're winner!"); 
     if (sumDice == 0) 
      lblSum.Text = ("Roll again please!"); 
     return; 

    } 

    int point = 0; 
    int gamestatus = 0; 
    private void UpdateGameStatus(int sumDice) 
    { 
     while (sumDice == 4 || sumDice == 5 || sumDice == 6 || sumDice==||sumDice == 9 || sumDice == 10) 

      point = sumDice; 

     if (sumDice == point) 
      lblSum.Text = ("You're winner!"); 
     gamestatus = 1; 
     if (sumDice == 7) 
      lblSum.Text = ("You lost!"); 
     gamestatus = -1; 
     return; 
    } 
    private void btnRollDice_Click(object sender, EventArgs e) 
    { 

     int die1 = 0, die2 = 0; 
     RollDice(out die1, out die2); 
     UpdateGameStatus(sumDice);  
    } 
} 
}  

Answer 1

У вас есть два метода с той же подписью:

private void UpdateGameStatus(int sumDice) 

и

private int UpdateGameStatus(int sumDice) 

компилятор не может знать, какой из них ты имеешь в виду. У них разные типы возврата, но они не являются частью подписи.

Answer 2

Сообщение об ошибке сообщает вам, что не так: вы делаете двусмысленный вызов UpdateGameStatus. У вас есть два метода, оба из которых называются UpdateGameStatus, которые принимают целочисленное значение. Почему у вас нет только одной версии, которая возвращает сумму костей, а также устанавливает статус игры? Это решит вашу проблему. Вы дублируете код без необходимости здесь.

Редактировать: Я также заметил, что ваш первый метод фактически не обновляет статус игры, поэтому, если вы хотите сохранить оба, подумайте о переименовании этого.

Редактировать 2: Чтобы обобщить то, что было упомянуто в комментариях (и выйти за рамки только сообщения об ошибке), похоже, что у вас есть ненужное дублирование кода. Подумайте, что вы действительно хотите, чтобы этот метод делал. Вы хотите, чтобы текст был установлен на "you lose!"? Вы хотите, чтобы он отслеживал переменную gamestatus (1, 0 или -1)? Вы хотите, чтобы он возвращал переменную gamestatus? Выясните, что именно вы хотите, чтобы этот метод выполнял/возвращал и пытались удалить дублированный код.