c# - 使用 Try Catch 转换值

Question

在下面的示例中，我从数据库返回一个值并将该值转换为 double 值，如果它由于为 NULL 而失败，则将 0 设置为默认值。

using (SqlCommand cmd = new SqlCommand(sql.ToString(), conn))
{
    try
    {
        this.value = Convert.ToDouble(cmd.ExecuteScalar());
    }
    catch (Exception)
    {
        this.value = 0;
    }
}

在这种情况下使用 Try Catch 是否会被视为不好的做法？处理这种情况的更好方法是什么？

Answer 1

By using a Try Catch in this instance, would this be considered bad practice?

当然。撇开其他不谈，如果任何 失败，您将返回 0，而不仅仅是返回值为 null 时。 (如果发现整张表都被删除了，你真的想继续当一切都很好吗？)

如果你想检测一个空返回值，你应该明确地这样做:

object result = cmd.ExecuteScalar();
return result == null ? 0d : (double) result;

如果存在不是 double 的非空返回值，这将仍然抛出(并且故意)。您应该知道您的查询应该返回什么类型，如果发现您得到了错误的类型则失败。