当 SQL Select 不返回任何内容时如何避免抛出错误

Question

我在某处读到，永远不应该将错误条件用作正常的程序流程。 对我来说非常有意义...但是

C# 应用程序位于 MySQL 数据库之上。 我需要将字符串值解析为两部分：ID 和值。 （原始数据来自泥盆纪数据库），然后根据查找表验证该值。 因此，几个原始字符串可能如下所示：

“6776 Purple People Eater”

“BIK Yellow Polka-Dot Bikini（当前正在使用）”

“DCP Deuce Coup”

因此，我的小实用程序将每个字符串解析为 ID 和描述基于第一个空间的索引（幸运的是，一致）。 然后，我将 ID 传递给查找，获取新值，然后我们就可以开始了。

不幸的是，TPTB 还决定我们不再需要臭黄色圆点比基尼（目前正在使用）。 因此，BIK 不返回行。 这是一个代码片段：

    foreach (string product in productTokens) {
      tempProduct = product.Trim();
      if (tempProduct.Length > 0) {
        if (tempProduct.Length < 10) {
          product_id = tempProduct;
        }
        else {
          int charPosition = tempProduct.IndexOf(" ");
          product_id = tempProduct.Substring(0, charPosition);
        }
        try {
          s_product = productAdapter.GetProductName(product_id).ToString();
        }
        catch (Exception e) {
          if (e.Message.ToString() == "Object reference not set to an instance of an object.") {
            s_product = "";
          }
          else {
            errLog.WriteLine("Invalid product ID " + e.Message.ToString());
            Console.WriteLine("Invalid product ID " + e.Message.ToString());
            throw;
          } //else
        } //catch
        if (s_product.Length > 0) {
          sTemp = sTemp + s_product + "; ";
        }
      } //if product.length > 0
    } //foreach product in productTokens

真的，真的很难看！ 特别是我在 catch 块中测试无效 ID 的部分。 一定有更好的方法来处理这个问题。

如果有人能帮助我，我将非常感激。

谢谢。

Answer 1

此时您不应该调用 ToString()，首先您应该检查返回的值是否为 null；

object productName = productAdapter.GetProductName(product_id);
if ( productName != null )
{
    s_product = productName.ToString();
}
else
{
    s_product = String.Empty;
}

Answer 2

除了 Rob 和 Marc 的建议之外，根据上面的代码，我建议进行另一个调整； 如果 productAdapter.GetProductName() 没有返回 String，在这种情况下调用 ToString() 的话，我会感到非常惊讶完全是多余的。 如果它确实已经返回了一个String，那么你的整个try/catch块就变成了一行：

s_product = productAdapter.GetProductName(product_id) ?? string.Empty;

另外，我认为提到已经返回一个的其他内容可能会很有用。 字符串 - Exception.Message。 因此，您在代码中调用 ToString() 的所有不同位置也是完全多余的。

另外，我建议使用实例方法 String.Split() 而不是 IndexOf 和 SubString 的组合：

product_id = tempProduct.Split(" ", 2)[1];

最后，决定使用哪种方法通过检查 Message 属性发现的 Exception 应该是最后的方案。 即使您确实需要在这里捕获NullReferenceException，您也应该明确地这样做：

catch (NullReferenceException) {
    s_product = "";
}
catch (Exception e) {
    // Log your invalid ID error condition here
}

PS：我也不太确定这个问题与 MySQL 有什么关系，因为在您的代码中没有任何 DB API 的证据。

Answer 3

不能简单地检查 GetProductName 是否返回 null 吗？

var productName = s_product = productAdapter.GetProductName(product_id);
if(productName == null) { ... do something }
else {
    string name = productName.ToString();
}