这是 goto 语句的一个很好的用法吗？

问题描述

我正在编写一个简单的 C# 控制台应用程序，在该应用程序中，如果 UserId 的输入无效，我将通过使用 GoTo 语句返回标签来解决问题。

但我不确定我是否正确使用了该语句。我想解决是否有更好的方法来解决问题，即使不使用 GoTo 语句？

[skip ci]

我将对其他字段（例如 Pin、AccountType、Balance、AccountStatus 等）使用相同的方法，因此我想确保在扩展其使用之前以正确的方式执行此操作。

解决方法

一个简单的方法是使用 if/else if/else 结构：

public void CreateAccount()
{
    Console.WriteLine("----- Create New Account -----\n" +
        "Enter following account information:\n");

    string userid = null;
    while (true)
    {
        Console.WriteLine("Enter a UserID (Alphanumeric; no special characters): ");
        userid = Console.ReadLine();
        if (!ValidUserID(userid))
        {
            Console.WriteLine("Userid can only contain A-Z,a-z & 0-9. Try again");
        }
        // if the condition above isn't matched,we'll check this condition
        else if (data.IsUserInFile(userid))
        {
            Console.WriteLine("Userid already exists. Try again");
        }
        // if that isn't matched either,then we have an acceptable userid and can exit the loop
        else
        {
            break;
        }
    }
}

请注意，我已将 userid 声明移出循环，因为您可能需要在循环之后使用它。

或者，您可以进一步分解代码。读取 userId 的整个方法可以移出到它自己的方法中：

public string ReadUserId()
{
    while(true)
    {
        // Prompt and read answer
        Console.WriteLine("Enter a UserID (Alphanumeric; no special characters): ");
        string userId = Console.ReadLine();
        
        // If the id is valid,return it and leave the method (ending the loop)
        if (ValidUserId(userId))
        {
            return userId;
        }
        
        // If we got here,then the id is invalid and we should inform the user to try again
        Console.WriteLine("Userid can only contain A-Z,a-z & 0-9. Try again");
    }
}

现在在 main 方法中，我们可以在需要获取 userId 时引用该方法。检查用户是否已经存在的循环现在可以单独实现以读取用户输入，因此不再需要担心两种不同的情况：

public void CreateAccount()
{
    Console.WriteLine("----- Create New Account -----\n" +
        "Enter following account information:\n");

    string userId = null;
    bool isNewUser = false;
    while (!isNewUser)
    {       
        // Use our other method to read the user id
        userId = ReadUserId();
    
        if (data.IsUserInFile(userId))
        {
            Console.WriteLine("Userid already exists. Try again");
            continue;
        }
        
        isNewUser = true;
    }
}