尝试/捕获和返回值

Question

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

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

如果我放置在catch之后（就像我目前所做的那样），即使在异常之后，它也会返回产品。那么最好的方法应该是什么？

方法如下:

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

目前，如果抛出异常，则不会返回任何内容。使用＜code＞try、catch和finally＜code＞。（您可能想查看MSDN页面以了解更多官方信息。）

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

因此，如果将＜code＞return

Answer 2

这取决于在特殊情况下会发生什么。如果这种情况可能是由于某种还没有“糟糕到”让应用程序崩溃的原因而发生的，或者如果您能够适当地处理该异常，那么您可以使用当前的appraoch-但是您应该明确地在cat-子句中至少留下一条日志消息，其中包含已抛出的错误：

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

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

如果这是一个真正意外的行为(这是异常的预期行为，这就是它们被称为异常的原因)，你应该从你的方法中删除所有这些try/catch，并在方法之外处理任何异常，就像Maurice已经提到的那样。

Answer 3

删除完整的＜code＞Try＜code＞和＜code＞Catch＜code＞块。显然，您无法处理＜code＞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实现时，您希望对无法获得产品详细信息做出不同的反应。您将不得不提出解决方法。您甚至可能最终遇到难以排除故障的奇怪错误。