Question

 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();
        }
    }

上面是一个代码示例，其中包含一般异常catch，但是当我运行此程序时，它将使用.net运行时错误空引用异常随机中断并将应用程序日志中的异常和未处理异常。

一点背景......这是一个在应用程序服务器上午夜自动运行的控制台应用程序。它针对不同的SQL Server 2008框执行存储过程。我们曾经在执行mainenace任务时由sql server丢弃连接时遇到这些错误，现在已不再是这种情况。我需要得到一个特定的错误。我不明白为什么它绕过catch子句并抛出一个未处理的运行时异常。这是什么意思？它发生在任何数量的代码点，而不仅仅是这一点。这只是爆炸的最后一个例子

Answer 1

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

但是，我猜测NullRef异常的起源是你做的事情：e.InnerException.Message。

InnerException属性可能为null，这将导致NullRef异常。然而，这并不是真正的例外。由于上述错误，导致程序在异常处理程序中结束的真正异常被隐藏。

如果要包含InnerException中的消息，请首先检查它是否为null。

修改

这样做：

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

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

实际上，做你正在做的事情毫无意义。除非您打算对此问题采取某些措施，否则不要捕获异常。你在这里做的只是搞乱调用者的StackTrace，因为你正在重新抛出 new 异常。如果你因为某些原因觉得你必须切入并重新抛出，你应该这样做：

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方法已经抛出InvalidOperationException，这比Exception的抛出更具体。如果您打算对此做些什么，请稍后捕获更具体的异常。现在，您将异常作为程序逻辑的一部分，这不是一种好的做法。

未处理的运行时错误c＃

1 个答案: