尝试/捕获并返回值

Question

我有一个返回List 的方法。现在我想知道如何正确放置try/catch 块。如果我将 return 语句放在 try 中，我会收到错误

并非所有代码路径都返回值

如果我放在catch 之后（就像我现在正在做的那样），即使在Exception 之后，它也会返回products。那么最好的方法应该是什么？

方法如下：

public List<Product> GetProductDetails(int productKey)
{
    List<Product> products = new List<Product>();
    try
    {
       using (SqlConnection con = new SqlConnection(_connectionString))
       {
         SqlCommand cmd = new SqlCommand("usp_Get_ProductDescription", con);
         cmd.CommandType = CommandType.StoredProcedure;
         cmd.Parameters.AddWithValue("@riProductID", productKey);
         con.Open();
         using (SqlDataReader reader = cmd.ExecuteReader())
         {
           while (reader.Read())
           {
             Product product = new Product(reader["Name"].ToString(), reader["Code"].ToString());
             products.Add(product);
           }
         }
       }
     }
     catch { }
     return products;
}

Answer 1

目前，如果抛出异常，您不会返回任何内容。 使用try, catch, finally。 （更多官方信息请关注MSDN page）

try
{
    //try to execute this code
}
catch
{
    //execute this if an exception is thrown
}
finally
{
    //execute this code, after try/catch
}

因此，如果您将 return 语句放入 finally 部分，即使抛出异常，您也会返回您的列表...

Answer 2

删除完整的 Try 和 Catch 块。显然你无法处理 GetProductDetails 方法中的异常，所以让它们被抛出。

但是调用代码可以做出决定：

IList<Product> products = null;

try
{
    products = GetProductDetails(3);
}
catch(Exception ex)
{
    // Here you can make the decision whether you accept an empty list in case of retrieval errors.
    // It is the concern of this method, not of the ProductDetails method.
    // TODO: Use logging
    products = new List<Product>();
}

如果您必须使用GetProductDetails 方法在每个方法中编写此代码，我可以想象这感觉就像代码重复。但是考虑一下，当您有 X 实现时，您希望对无法获得产品详细信息做出不同的反应。您将不得不想出解决方法。您甚至可能会遇到难以解决的奇怪错误。

Answer 3

这取决于在特殊情况下应该发生什么。如果这可能由于某种原因而发生，而该原因还不足以让应用程序崩溃，或者如果您能够适当地处理该异常，那么您可以使用当前的方法 - 但是您绝对应该至少留下一条日志消息在包含已抛出错误的 catch 子句中：

catch (Exception e) 
{ 
    log.Info(e.Message);
}

通过这种方式，您可以获得列表中的所有结果，但导致任何异常的结果除外。您可以简单地继续处理您获得的所有结果并忽略那些错误（假设您以任何方式记录它们）。

如果这是一个非常意外的行为（这是异常的预期行为，这就是为什么它们被称为异常）你应该从你的方法中删除所有这些 try/catch 并处理方法之外的任何异常，正如 Maurice 已经提到的.