如何在值返回方法中使用 try catch 块？

Question

我正在检查注册表中上传的图像，我需要在其中使用 try catch 块。这是我的代码：

public bool CheckFileType(string FileName)
{
        string Ext = Path.GetExtension(FileName);
        switch (Ext.ToLower())
        {
            case ".gif":                   
                return true;
                break;
            case ".JPEG":                    
                return true;
                break;
            case ".jpg":                  
                return true;
                break;
            case ".png":                   
                return true;
                break;
            case ".bmp":                   
                return true;
                break;
            default:                  
                return false;
                break;
        }

}

请在这里建议我如何使用 try catch 块。

提前致谢。

Answer 1

这样会更好，

 public bool CheckFileType(string FileName)
 {
    bool result = false ;

    try
     {
      string Ext = Path.GetExtension(FileName);
      switch (Ext.ToLower())
      {
        case ".gif":                   
        case ".JPEG":                    
        case ".jpg":                  
        case ".png":                   
        case ".bmp":                   
            result = true;
            break;
       }

      }catch(Exception e)
      {
         // Log exception 
      }
      return result;
     }

Answer 2

有很多方法可以在返回值的方法中使用异常：

将 return 语句放在 try-catch 之外 例如：

T returnValue = default(T);
try
{
    // My code
}
catch 
{
    // Exception handling code
}
return returnValue;

在你的 catch 中添加一个 return 语句

try
{
    // My code
}
catch 
{
    // Handle exception
    return default(T);
}

抛出异常

您没有必须返回一个值，该方法只需要结束（例如，到达 return 语句或 throw 语句）。根据异常，返回值并不总是有效。

您应该仔细考虑何时以及如何捕获和处理异常：

1. 什么可能会失败？
2. 他们为什么/怎么会失败？
3. 失败时我该怎么办？

在你的情况下：

1. 唯一可能失败的语句是string Ext = Path.GetExtension(FileName);，根据documentation，如果FileName 包含，它可能会失败。 （请注意，GetExtension 不返回 null，即使 FileName 为 null）。
2. 如果用户提供的字符串包含这些无效字符，则可能会发生这种情况。
3. 如果发生这种情况，我想我们应该返回 false，表示路径无效（但这取决于应用程序）。

所以我可能会像这样处理异常：

public bool CheckFileType(string FileName)
{
    string Ext;
    try
    {
        Ext = Path.GetExtension(FileName);
    }
    catch (ArgumentException ex)
    {
        return false;
    }
    // Switch statement
}

请注意，我们只捕获我们预期的异常 (ArgumentException)，并且我们只将 try 语句放在我们希望从其中引发异常的语句周围。

事实上，尽可能避免抛出和捕获异常是个好主意——它们不仅会导致性能损失（如果在循环内调用此方法会导致严重问题），而且您可能会无意中捕获和处理异常您没有预料到的异常，掩盖了更严重的问题。

在这种情况下，我们可以通过检查FileName 是否包含任何无效字符来完全避免抛出异常：

public bool CheckFileType(string FileName)
{
    if (FileName == null)
    {
        return false;
    }
    if (FileName.IndexOfAny(System.IO.Path.GetInvalidPathChars()) >= 0)
    {
        return false;
    }
    // Your original method goes here
}

Answer 3

由于您实际上并未测试文件类型（仅测试文件名的扩展名），因此我首先要重命名该方法。您可以制作一个扩展方法来处理它：

public static bool HasImageExtension(this string fileName)
{
    try
    {
        if (fileName == null) return false;

        string[] validExtensions = new string[] { ".gif", ".jpg", ".jpeg", ".png", ".bmp" };

        string extension = Path.GetExtension(fileName);
        return validExtensions.Contains(extension);
    }
    // catch the specific exception thrown if there are 
    // invalid characters in the path
    catch (ArgumentException ex) 
    {
        // do whatever you need to do to handle 
        // the fact there are invalid chars
        throw; 
    }
}

然后您可以调用它，如下所示：

string fileName = "testFileName.jpg";
bool hasImageExtension = fileName.HasImageExtension();

Answer 4

这应该可行：

public bool CheckFileType(string FileName)
{
    try
    {
        string Ext = Path.GetExtension(FileName).ToLower();
        string[] okExt = ".gif|.jpg|.jpeg|.png|.bmp".Split('|');

        foreach(var item in okExt)
        {
            if(Ext == item)
                return true;
        }
        return false;
    }
    catch(Exception ex)
    {
        throw;
    }
}

请记住：永远不要捕获您不会处理的异常。（或至少重新抛出它们）