Try - Catch 块中未处理的异常

Question

我已经查看了与此问题相关的其他问题，但我没有看到对我有帮助的答案（或者可能更多到我理解的程度）。

此代码：

   public static Bitmap GetLibraryObjectImage(Guid guid) {
            try {
                string tempPath = GetLibraryObjectImagePath(guid);
                if (tempPath != string.Empty) {
                    var bytes = File.ReadAllBytes(tempPath);
                    var ms = new MemoryStream(bytes);
                    return (Bitmap)Image.FromStream(ms);
                }
            }
            catch {
                return (Bitmap)Image.FromFile(Application.StartupPath + @"\na.bmp");
            }

            return (Bitmap)Image.FromFile(Application.StartupPath + @"\na.bmp");
        }

旨在处理找不到图像文件或以其他方式无效的任何情况。我还没有确定任何特定的异常类型，希望它能捕获任何内容。

对于我的一位用户，它抛出了这个异常：

参数无效。
在 System.Drawing.Image.FromStream(Stream 流、布尔值 useEmbeddedColorManagement、布尔值 validateImageData） 在 System.Drawing.Image.FromStream(Stream 流)
在 ScruffyDuck.AirportDesignEditor.Helpers.U.GetLibraryObjectImage(Guid 指南）

我之前在处理图像的 try-catch 块的其他情况下见过这种情况。我认为上面的 try-catch 会得到一切，包括非托管异常，但也许我错了。

我不知道导致异常的情况，但我真的不希望我的应用程序崩溃并烧毁，即使我试图解决问题。它被我的全局异常处理程序捕获，但当然那时已经有点晚了。

非常感谢您对避免这种情况的任何见解

Answer 1

正如 @KeithS 在评论中提到的，堆栈跟踪显示异常源自 Image.FromStream 方法。您是否已验证导致该调用的所有代码是否正确返回预期数据？文件路径必须存在（否则 File.ReadAllBytes 会引发异常），并且 MemoryStream 构造函数返回有效流（否则会引发异常），但是有没有可能内存流中的数据实际上不是图像？

另外，您可能想尝试使用 File.OpenRead 方法。这会返回一个 FileStream，因此您可以将其直接传递到 Image.FromStream 方法。这看起来类似于以下代码。 （我将其更改为使用 using 语句，并且只有一个 return 语句。）

public static Bitmap GetLibraryObjectImage(Guid guid) 
{ 
   Bitmap bitmap = null;

   try
   { 
      string tempPath = GetLibraryObjectImagePath(guid); 
      if (!String.IsNullOrEmpty(tempPath) 
      {
         using (var stream = File.OpenRead(tempPath))
         {
            bitmap = (Bitmap)Image.FromStream(stream);
         }
      } 
   } 
   catch
   { 
      bitmap = (Bitmap)Image.FromFile(Application.StartupPath + @"\na.bmp"); 
   }

   if (bitmap == null)
   {
      bitmap = (Bitmap)Image.FromFile(Application.StartupPath + @"\na.bmp"); 
   }

   return bitmap;
}