c# - 从引发的异常中恢复或返回

Question

多年以来，我的主要语言是Perl，而且我定期验证用户输入的内容没有问题。现在，我正在使用大量的C#，并且希望朝着验证用户输入并从引发的异常中恢复/返回的引发/捕获样式迁移。我正在使用一种非常幼稚(即愚蠢)的方法来执行此操作，并且迫切需要转移到更成熟，更少愚蠢的东西上。我复制了一个从提示返回整数的函数。我正在使用可怕的GOTO语句从用户错误中恢复。什么是更好的方法呢？

谢谢，CC。

private static int GetInput(string v)
{
    begin:
    Console.Write(v);
    string strradius = Console.ReadLine();
    int intradius;
    try
    {
        intradius = int.Parse(strradius);
        if (intradius < 1)
            throw new ArgumentOutOfRangeException();
    }
    catch (ArgumentNullException)
    {
        Console.WriteLine("You must enter a value.");
        goto begin;
    }
    catch (FormatException)
    {
        Console.WriteLine("You must enter a valid number.");
        goto begin;
    }
    catch (ArgumentOutOfRangeException)
    {
        Console.WriteLine("Your number is out of range");
        goto begin;
    }
    catch (Exception ex)
    {
        Console.WriteLine(ex);
        goto begin;
    }
    finally
    {
        Console.WriteLine("Okay");
    }
    return intradius;
} 

Answer 1

首先，从来没有一个关于何时使用goto的好的经验法则。确实，除了少数极少数特殊情况外，您永远都不想使用它。

接下来，对于您的问题，使用异常来验证输入通常是一个坏主意。正如大多数人指出的那样，这很昂贵。应该使用异常来处理特殊情况，因此我实际上根本不会使用它们。

相反，您可以使用do-while循环，并在用户输入错误输入时重复执行。一旦获得正确的输入，就可以退出循环。如果发生异常，则不应真正继续该过程。要么在外部处理它(即方法内部没有try-catch)，否则，如果您必须执行try-catch，则只需打印一条消息并退出该方法即可。但是我不会对这种方法使用异常处理。将返回类型实际更改为bool也是一个好主意，因此您可以通过返回类型向外界指示该方法是否成功。您使用out参数实际返回转换后的int。

private static bool GetInput(string msg, out int converted)
{
    bool result = false;
    converted = 0;
    do
    {
        Console.Write(msg);
        string str = Console.ReadLine();
        result = int.TryParse(str, out converted);
        if (result && converted < 1)
        {
            Console.WriteLine("Your number is out of range");
            result = false;
        }
        if (!result && string.IsNullOrEmpty(str))
        {
            Console.WriteLine("You must enter a value.");
        }
        if (!result && !string.IsNullOrEmpty(str))
        {
            Console.WriteLine("You must enter a valid number.");
        }
    } while (!result);

    return result;
}