如何更改 goto 语句以改进代码？

问题描述

我正在编写一个 C# Windows 窗体应用程序。我有一个带有随机单词的标签，例如“计算机”。用户看到被覆盖的词，在这个例子中：“--------”。用户必须通过逐个字母的猜测来猜测这个词是什么。我创建了按钮提示。该按钮负责显示一个随机字母。这是我创建的代码：

private void btnHint_Click(object sender,EventArgs e)
{
    repeat:
    Random rnd = new Random(); 
    int rand = rnd.Next(corrArr.Length); 
    char letter = corrArr[rand]; 

    if (letters.Contains(letter.ToString())) //check if the random letter is already used
    {
        goto repeat; 
    }
    else
    {
        word.Text = "";

        var idx = correct.IndexOf(letter);
        wordArr[idx] = Convert.tochar(letter);

        foreach (var item in wordArr)
        {
            word.Text += item.ToString();
        }
        count++;
        
        if (wordLen == count)
        {
            this.BackColor = Color.LimeGreen; 
            result.Text = "Win!!";
            tb.Enabled = false; 
            btnClick.Visible = false; 
        }
    }
    letters.Add(letter.ToString());
}

通过 goto 语句，我可以再次绘制一个字母，如果前一个已经被使用。

我听说我不应该像代码开头那样使用 goto 语句，但程序运行良好。如何改进代码？

解决方法

离开 if/else 语句并使用 Do While 循环

tutorialsteacher

Random rnd = new Random(); 
char letter;
do
{
      int rand = rnd.Next(corrArr.Length); 
      letter = corrArr[rand]; 
} while (letters.Contains(letter.ToString()));

 word.Text = "";

 var idx = correct.IndexOf(letter);

 wordArr[idx] = Convert.ToChar(letter);

当然还有更多需要改进...;)

,

在你的随机字母仍然包含在已经使用的字母中时，围绕你检查并重复它。

private void btnHint_Click(object sender,EventArgs e) {
    Random rnd = new Random();
    char letter;
    while (letters.Contains((letter = corrArr[rnd.Next(corrArr.Length)]).ToString()))
      ; //the loops body is left empty intentionally

    ...
}

更好的方法是，收集一组可用的字母，每次您随机选择一个，将其从可用字母的集合中删除。这样，您就不必进行任何检查或循环

var availLetters = "abcdef....";

int rand = rnd.Next(availLetters.Length);
char letter = availLetters[rand];
availLetters = availLetters.Substring(0,rand) + availLetters.Substring(rand+1);

...