关于在哪里放置 Try 和 Catch 语句的问题

Question

我使用 try 和 catch 语句作为保持代码运行而不崩溃的简单方法（我会将所有内容包装在一次大尝试中）。最近，我想开始更正确地使用 try 和 catch 语句。举个例子，我有以下问题：

public class Ninja{
    Ninja(){
    }

    public void ThrowShirikin(int numberOfShirikins){
        try{
            if(numberOfShirikins == 0){
                throw new System.ArgumentException("Invalid number of shirikins");
            }

            //Throw shirikin
        }
        catch(ArgumentException e){
            MessageBox.Show(e.Message);
        }
    }
}

在上面的 Ninja 类中，我的 ThrowShirikin 方法的全部内容都包含在 try 循环中。由于输入错误只有一次机会（在本例中，当 numberOfShirikins == 0 时），因此 try 循环中不应该只包含检查此错误的代码行吗？见下文：

public class Ninja{
    Ninja(){
    }

    public void ThrowShirikin(int numberOfShirikins){
        bool errorsExist = false;
        try{
            if(numberOfShirikins == 0){
                errorsExist = true;
                throw new System.ArgumentException("Invalid number of shirikins");
            }
        }
        catch(ArgumentException e){
            MessageBox.Show(e.Message);
        }

        if(!errorsExist){
            //Throw shirikin
        }
    }
}

^但是我这里的东西似乎有点笨重。关于我如何理解 try catch 语句的使用有什么建议和意见吗？谢谢！

编辑：

或者我可以这样做，这样如果 numberOfShirikins 的值无效，则 //Throw shirikin 代码永远不会执行？：

public class Ninja{
    Ninja(){
    }

    public void ThrowShirikin(int numberOfShirikins){
        try{
            if(numberOfShirikins == 0){
                throw new System.ArgumentException("Invalid number of shirikins");
                return;
            }
        }
        catch(ArgumentException e){
            MessageBox.Show(e.Message);
        }

        //Throw shirikin
    }
}

Answer 1

没有真正的理由在相同的方法中捕获异常。没有：

public class Ninja{
    Ninja(){
    }

    public void ThrowShirikin(int numberOfShirikins){
        if(numberOfShirikins == 0){
            MessageBox.Show(e.Message);
            return;
        }

        //...
    }
}

Answer 2

方法不应该捕获它自己抛出的异常 - 生成它们很好，但要明白它们很昂贵并且首选正常的流程控制（更容易推理等等）。

此外，在上面的示例中，errorsExist 永远不会设置为 true - 应该将其放在 catch 块中，而不是直接放在抛出异常的位置下。这将终止正常的程序流程。

重述一下，方法看起来像这样。

public void ThrowShirikin(int numberOfShirikins) {
    if(numberOfShirikins == 0){
        throw new System.ArgumentException("Invalid number of shirikins", "numberOfShirikins");
    }

    //Throw shirikin
}

由调用该方法的类来决定如何处理异常。

Answer 3

两者都不是一个好主意。抛出异常并以相同的方法捕获它并显示该类的表单对话框是没有意义的。

只有应包含 GUI 代码的类才是 GUI 中的类。

在你的情况下，我也不会使用异常，我会使用布尔返回值：

public bool ThrowShirikin(int numberOfShirikins){
            if(numberOfShirikins == 0){
                return false;
            }
            //throw
            return true;
    }

异常应该用于特殊情况，而不是一般情况。

另外，请注意，第二种方式中的 errorsExist = true; 代码永远不会被调用，因为它位于 throw 语句下方。

Answer 4

当你抛出异常时，你基本上是在说：“这是一种异常情况，我不知道如何处理（或者更高层的另一种方法会更好地了解应该如何处理）”

那么该方法是什么你刚刚发布的内容是：“当我被要求扔 0 个手里剑时，我不知道该怎么办。实际上我撒谎了，我将显示一个消息框”

要么一起跳过异常并显示消息（并不理想，但比抛出的更好），或者只是抛出异常并在 UI 代码的更高位置捕获它。如果您想专门处理这一特定异常，您可能需要创建自己的异常类型。

Answer 5

IMO - 异常实际上是代码中的异常情况 - 即无法预期的情况和/或错误可能在真正的异常情况下出错的情况。

如果上面显示的代码片段，有一个 numberOfShirikins == 0 不是一个异常条件，它是一个正常的参数检查

所以，我将重构代码：

public class Ninja{
    Ninja(){
    }

    public void ThrowShirikin(int numberOfShirikins){
        bool errorsExist = false;
        try{
            if(numberOfShirikins == 0){ showUserMessage("Invalid number of shirikins", MessageType.Information); }
        }
        catch(ArgumentException e){
            showUserMessage(e.Message, MessageType.Exception, e);
        }

        if(!errorsExist){
            //Throw shirikin
        }
    }
   private showUserMessage(string message, MessageType type, Exception ex = null){
        MessageBox.Show(message);
   }

}

一旦完成，你会发现捕获异常实际上没有任何附加值，所以我们也许可以编写一个通用的异常处理程序来显示异常并显示异常。让你的代码正常流通。

现在，我的代码如下所示：

public class Ninja{
        Ninja(){
        }

        public void ThrowShirikin(int numberOfShirikins){
                if(numberOfShirikins == 0){ showUserMessage("Invalid number of shirikins", MessageType.Information); return; }
                else{ /* do something here */ }
        }

另一方面，如果 numberOfShirikins == 0 是一个异常条件，那么我将编写一个自定义异常，代码如下：

public class Ninja{
        Ninja(){
        }

        public void ThrowShirikin(int numberOfShirikins){
                if(numberOfShirikins == 0){ throw NumberOfShirikinsException(); }
                else{ /* do something here */ }
        }

Answer 6

如果你能避免抛出异常，你就应该这样做。它们花费大约 1000-4000 个时钟周期。

Answer 7

当您抛出像 ArgumentException 这样的异常时，您不应该在方法中捕获它们。根据您的方法将此工作交给客户。

Answer 8

看起来 try/catch 的存在只是为了捕获您选择创建和抛出的异常，然后捕获它只是为了向用户弹出一个消息框。然后您根据错误情况选择继续或不继续。

消除try和catch。消除消息框。 Do 为无效参数抛出异常。让函数的调用者决定是否要捕获以及如何处理。除了确定哪些内容有效、哪些内容无效之外，您的 Ninja 类不应做出这些决定。

if (numberOfShirikins == 0)
    throw new ArgumentException("...");

// rest of your shirikin throwing code

Answer 9

你不应该以这种方式捕获异常。只捕获您实际上可以处理的异常。

public class Ninja{
    Ninja(){
    }

    public void ThrowShirikin(int numberOfShirikins){
            if(numberOfShirikins <= 0){
                throw new System.ArgumentException("Invalid number of shirikins");
            }
        //Throw shirikin
    }
}

我建议您开始编程时不要使用 try/catch。然后修复您看到的任何异常，不要隐藏它们。