捕获预期的例外情况

Question

我有代码看起来像这样，因为我检查某些数据是否是图像的唯一可靠方法是实际尝试加载它像图像一样。

    static void DownloadCompleted(HttpConnection conn) {
        Image img;
        HtmlDocument doc;

        try {
            img = Image.FromStream(conn.Stream);
        } catch {
            try {
                doc = new HtmlDocument();
                doc.Load(conn.Stream);
            } catch { return; }
            ProcessDocument(doc);
            return;
        }
        ProcessImage(img);
        return;
    }

哪个看起来很糟糕！

处理这些情况的好方法是什么？ 你基本上被迫使用if语句之类的异常？

Answer 1

你的逻辑结构是

if( /* Loading Image Fails */ )
    /* Try Loading HTML */

所以我会尝试以这种方式读取代码。 引入辅助方法可能是最干净的（尽管通常令人讨厌）。

bool LoadImage()
{
    Image img;
    try 
    {
        img = Image.FromStream(conn.Stream);
    } 
    catch( NotAnImageException /* or whatever it is */ )
    {
        return false;
    }

    ProcessImage(img);
    return true;
}


bool LoadDocument()
{
   // etc
}

所以你可以写

if( !LoadImage() )
   LoadDocument();

或者扩展到：

if( !LoadImage() && !LoadDocument() )
{
     /* Complain */
}

Answer 2

我认为空的捕获块，你编码它们的方式，总的来说是一个非常糟糕的主意。 你在那里吞下一个例外。 没有人会知道你编码它的方式有些不对劲。

我的政策是，如果我无法处理并从异常中恢复，我应该简单地允许它将调用堆栈冒泡到可以处理它的地方。 这可能仅仅意味着记录错误，但它比隐藏错误更好。

如果未经检查的异常是.NET中的规则，我建议您在单个方法中必须使用一个try / catch块。 我同意 - 方法中的多个try / catch块是丑陋和混乱的。

我不确定你的代码是做什么的。 看起来你正在使用异常作为逻辑：“如果没有抛出异常，则处理并返回一个图像;如果抛出异常，则处理并返回一个HTML文档。” 我认为这是一个糟糕的设计。 我将它重构为两个方法，一个用于图像和HTML，而不是使用异常而不是“if”测试。

Answer 3

与duffymo的答案相结合，如果你正在处理

• 文件输入/输出，使用IOException
• 网络套接字，使用SocketException
• XML，使用XmlException

这将使得异常更加整洁，因为你不允许它冒泡到通用的catch-all异常。 它可能很慢，如果你使用泛型catch-all异常，因为堆栈跟踪将冒泡并占用内存作为Exception属性的引用，InnerException会嵌套为结果。

无论哪种方式，制作您的代码以显示异常并在必要时记录它或重新抛出它以捕获所有异常...永远不要假设代码不会失败，因为有足够的内存，大量的磁盘等等那是在脚下射击自己。

如果要构建一组唯一的类和API，则要设计自己的Exception类，请继承ApplicationException类。

编辑：我更了解OP正在做什么...我建议以下，更清晰的实现，注意我如何检查流，看看是否有HTML标签或DOC标签作为XHTML的一部分...

static void DownloadCompleted(HttpConnection conn) {
        Image img;
        HtmlDocument doc;
        bool bText = false;
        using (System.IO.BinaryReader br = new BinaryReader(conn.Stream)){
            char[] chPeek = br.ReadChars(30);
            string s = chPeek.ToString().Replace(" ", "");
            if (s.IndexOf("<DOC") > 0 || s.IndexOf("<HTML") > 0){
                // We have a pure Text!
                bText = true;
            }
        }
        if (bText){
            doc = new HtmlDocument();
            doc.Load(conn.Stream);
        }else{
            img = Image.FromStream(conn.Stream);
        }
}

如果您愿意，可以根据conn.Stream距离来增加长度， conn.Stream指示html标签的位置......

希望这会有所帮助，最好的问候，汤姆。

Answer 4

我喜欢Eric的例子的想法，但我发现实现很难看：在if工作并且有一个工作方法返回一个布尔看起来非常难看。

我的选择是返回图像的方法，以及类似的方法

Image LoadImage(HttpConnection conn)
{
    try 
    {
        return Image.FromStream(conn.Stream);
    } 
    catch(NotAnImageException)
    {
        return null;
    }
}

并以原始方法完成工作：

static void DownloadCompleted(HttpConnection conn)
{
    Image img = LoadImage(conn);

    if(img != null)
    {
         ProcessImage(img);
         return;
    }

    HtmlDocument doc = LoadDocument(conn);

    if(doc != null)
    {
         ProcessDocument(doc)
         return;
    }

    return;
}

Answer 5

无需尝试猜测 HTTP下载的内容类型，这就是您在此处捕获异常的原因。 HTTP协议定义了Content-Type标头字段，该字段为您提供下载内容的MIME类型 。

此外，不可搜索的流只能读取一次。 要多次读取流的内容，您必须将其复制到 MemoryStream并在每次使用Seek（0，SeekOrigin.Begin）读取会话之前重置它。

Answer 6

回答你的确切问题：你根本不需要一个catch块，try / finally块在没有catch块的情况下工作正常。

try
{
    int i = 0;
}
finally {}