0 
 private void SetConnection(string id, string classCode)
    {
        try
        {
            _connection = new SqlConnection { ConnectionString = Settings.Default.CurrentConnection };
            _connection.Open();
            while (_connection.State == ConnectionState.Connecting || _connection.State == ConnectionState.Closed)
                Thread.Sleep(1000);

            _command = new SqlCommand(Settings.Default.EligibilityBenefitSP, _connection);
            if (_command != null) _command.CommandType = CommandType.StoredProcedure;
            _command.Parameters.Add("@ClassCode", SqlDbType.NVarChar).Value = classCode;
            _command.Parameters.Add("@Id", SqlDbType.NVarChar).Value = id;
        }
        catch (Exception e)
        {
            throw new Exception(e.Message + " " + Settings.Default.EligibilityBenefitSP);
        }

    }

   public Collection<EligibilityClassBenefit> ExtractEligibilityClassBenefit(string id, string classCode)
    {
        SetConnection(id, classCode);
        Collection<EligibilityClassBenefit> eclassBene = new Collection<EligibilityClassBenefit>();
        SqlDataReader reader = null;
        try
        {
            _command.CommandTimeout = 420;
            if (_connection.State == ConnectionState.Open)
                reader = _command.ExecuteReader(CommandBehavior.CloseConnection);
            else
                throw new Exception("Connection Closed");

                /* no data */
                if (!reader.HasRows) return null;

                while (reader.Read())
                {
                    EligibilityClassBenefit eligibilityClassBenefit = new EligibilityClassBenefit
                    {
                        EffectiveDate                = reader["EffectiveDate"].ToString(),
                        EndDate                      = reader["EndDate"].ToString(),
                        InitialEffectiveDate         = reader["InitialEffectiveDate"].ToString(),
                        IsAdministrativeServicesOnly = reader["IsAdministrativeServicesOnly"].ToString(),
                        EffectiveProvision           = reader["EffectiveProvision"].ToString(),
                        ProbationPeriod              = reader["ProbationPeriod"].ToString(),
                        UnderwritingType             = ExtractUnderwritingType(id),
                        ProbationPeriodUnit          = reader["ProbationPeriodUnit"].ToString(),
                        StateOfIssue                 = reader["StateOfIssue"].ToString(),
                    };
                    BenefitData benefitData = new BenefitData();
                    eligibilityClassBenefit.Benefit = benefitData.ExtractBenefit(reader, id, classCode);

                    EligibilityClassBenefitBusinessLevelData eligibilityLevelData = new EligibilityClassBenefitBusinessLevelData();
                    eligibilityClassBenefit.EligibilityClassBenefitBusinessLevelNodes = eligibilityLevelData.ExtractBenefitBusinessLevel(reader);

                    eclassBene.Add(eligibilityClassBenefit);
            }
            return eclassBene;
        }
        catch (Exception e)
        {
            throw new Exception(e.InnerException.Message + e.InnerException.StackTrace);
        }
        finally
        {
            //if (_connection.State == ConnectionState.Open) _connection.Close();
            if (reader != null) reader.Close();
            _command.Dispose();
        }
    }

上面是一个包含一般异常捕获的代码示例，但是当我运行这个程序时，它会随机中断并抛出未处理的异常到应用程序日志，并带有 .net 运行时错误空引用异常。

一点背景知识……这是一个控制台应用程序，它会在午夜自动在应用程序服务器上运行。它针对不同的 SQL Server 2008 机器执行存储过程。我们过去常常在执行维护任务时连接被 sql server 删除时得到这些错误，现在不再是这种情况了。我需要得到一个更具体的错误。我不明白为什么它绕过了 catch 子句，只是抛出了一个未处理的运行时异常。这是什么意思？它发生在任意数量的代码点，而不仅仅是这一点。这只是最后一个爆炸的例子

4

1 回答 1

2

当您捕获异常时，您也将它们抛出以由调用者处理。现在，您发布的代码中没有入口点，因此很难看到此代码段之外发生了什么。

但是，大胆猜测一下，我对 NullRef 异常起源的建议是您执行此操作的位置：e.InnerException.Message.

该InnerException属性很可能为 null，这将导致 NullRef 异常。然而，这并不是真正的例外。由于上述错误，导致程序最终进入异常处理程序的真正异常被隐藏了。

如果要包含来自 InnerException 的消息，首先检查它是否为空。

编辑：

这样做：

catch (Exception e)
{
    throw new Exception(e.InnerException.Message + e.InnerException.StackTrace);
}

捕获任何异常并将其重新抛出以进行处理。如果调用代码未处理异常，即未将调用包装在 try-catch 块中，则异常将被视为运行时未处理。

实际上，做你正在做的事情根本没有意义。除非您打算对问题采取措施，否则不要捕获异常。您在这里所做的只是弄乱了调用者的 StackTrace，因为您正在重新抛出一个新异常。如果您觉得由于某种原因必须切入并重新投掷，您应该这样做：

catch (Exception e)
{
    throw new Exception("I have a good reason for interrupting the flow", e);
}

请注意，异常实例是在构造函数中为重新抛出的异常传递的。这将最终成为内部异常。

关于您的异常策略，这也是非常不必要的：

if (_connection.State == ConnectionState.Open)
    reader = _command.ExecuteReader(CommandBehavior.CloseConnection);
else
    throw new Exception("Connection Closed");

如果连接关闭，该ExecuteReader方法已经抛出一个比你抛出的更具体的. 如果您打算对此做点什么，请稍后捕获更具体的异常。现在，您将异常用作程序逻辑的一部分，这不是一个好习惯。InvalidOperationExceptionException

于 2013-09-16T12:36:41.573 回答