Anruf ist mehrdeutiger Fehler - Würfelspiel

Question

Hallo, ich versuche, ein Würfelspiel zu machen und immer wieder diesen Fehler zu bekommen, nicht sicher, was es ist oder ob etwas mit meinem Algorithmus nicht stimmt. Jede Hilfe wäre willkommen. Thanks :) Es zeigt einen Fehler:

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

Sie haben zwei Methoden mit derselben Signatur:

private void UpdateGameStatus(int sumDice) 

und

private int UpdateGameStatus(int sumDice) 

Der Compiler kann nicht wissen, was man was meinen Sie. Sie haben unterschiedliche Rückgabetypen, aber diese sind nicht Teil der Signatur.

Answer 2

Die Fehlermeldung zeigt Ihnen, was falsch ist: Sie geben einen mehrdeutigen Anruf an UpdateGameStatus. Sie haben zwei Methoden, beide mit dem Namen UpdateGameStatus, die beide einen Ganzzahlwert haben. Warum hast du nicht nur eine Version, die die Summe der Würfel zurückgibt und auch den Spielstatus setzt? Das würde dein Problem lösen. Sie vervielfältigen hier unnötig Code.

Bearbeiten: Ich habe auch festgestellt, dass Ihre erste Methode nicht wirklich den Status des Spiels überhaupt aktualisieren, also wenn Sie beide behalten möchten, in Betracht ziehen, diese umzubenennen.

Bearbeiten 2: Um zusammenzufassen, was in den Kommentaren erwähnt wurde (und über die Fehlermeldung hinausgehen), scheinen Sie einige unnötige Code-Duplizierung zu haben. Denken Sie darüber nach, was diese Methode wirklich tun soll. Soll der Text auf "you lose!" gesetzt werden? Möchten Sie die gamestatus Variable (1, 0 oder -1) verfolgen? Möchten Sie, dass die Variable gamestatus zurückgegeben wird? Finden Sie genau heraus, was diese Methode tun/zurückgeben soll und versuchen Sie, den doppelten Code zu entfernen.