c# - 在数据库操作(插入、更新、删除)过大之后，检查受影响的行数是否过大？

Question

最近在我开发的应用程序中，我一直在检查受插入、更新、删除数据库影响的行数，如果该数字是意外的，则记录错误。例如，如果从 ExecuteNonQuery() 调用返回了除一行以外的任意数量的行，则在一行的简单插入、更新或删除中，我将认为这是一个错误并将其记录下来。另外，我现在意识到，当我输入这个时，如果发生这种情况，我什至不会尝试回滚事务，这不是最佳实践，绝对应该解决。无论如何，这里的代码可以说明我的意思:

我将有一个调用数据库的数据层函数:

public static int DLInsert(Person person)
{
    Database db = DatabaseFactory.CreateDatabase("dbConnString");

    using (DbCommand dbCommand = db.GetStoredProcCommand("dbo.Insert_Person"))
    {
        db.AddInParameter(dbCommand, "@FirstName", DbType.Byte, person.FirstName);
        db.AddInParameter(dbCommand, "@LastName", DbType.String, person.LastName);
        db.AddInParameter(dbCommand, "@Address", DbType.Boolean, person.Address);

        return db.ExecuteNonQuery(dbCommand);
    }
}

然后业务层调用数据层函数:

public static bool BLInsert(Person person)
{
    if (DLInsert(campusRating) != 1)
    {
        // log exception
        return false;
    }

    return true;
}

在代码隐藏或 View 中(我同时做 webforms 和 mvc 项目):

if (BLInsert(person))
{
    // carry on as normal with whatever other code after successful insert
}
else
{
    // throw an exception that directs the user to one of my custom error pages
}

我越使用这种类型的代码，就越觉得它太过分了。特别是在代码隐藏/ View 中。是否有合理的理由认为简单的插入、更新或删除实际上不会修改数据库中正确的行数？只担心捕获实际的 SqlException 然后处理它，而不是每次都对受影响的行进行单调检查更合理吗？

谢谢。希望大家能帮帮我。

---

更新

感谢大家花时间回答。我仍然没有 100% 决定我将使用什么设置，但这是我从您的所有回复中摘取的内容。

• 信任 DB 和 .Net 库来处理查询并按设计完成工作。
• 在我的存储过程中使用事务来回滚任何错误的查询，并可能使用 raiseerror 将这些异常作为 SqlException 抛回给 .Net 代码，这可以通过 try/处理这些错误捕获。这种方法将取代有问题的返回码检查。

我遗漏的第二个要点会不会有任何问题？

Answer 1

我猜问题变成了“你为什么要检查这个？”如果这只是因为您不信任数据库来执行查询，那么它可能就有点矫枉过正了。但是，可能存在执行此检查的合乎逻辑的理由。

例如，我曾经在一家公司工作，使用这种方法来检查并发错误。当从数据库中获取要在应用程序中编辑的记录时，它会附带一个 LastModified 时间戳。然后，数据访问层中的标准 CRUD 操作将在执行 UPDATE 时包含一个 WHERE LastMotified=@LastModified 子句并检查记录修改计数。如果没有记录被更新，它会假设发生了并发错误。

我觉得并发检查有点草率(尤其是关于假设错误性质的部分)，但它为业务完成了工作。

在您的示例中，我更关心的是如何实现这一目标的结构。从数据访问代码返回的 1 或 0 是一个“魔数(Magic Number)”。应该避免这种情况。它将实现细节从数据访问代码泄漏到业务逻辑代码中。如果您确实想继续使用此检查，我建议将检查移至数据访问代码中，并在失败时抛出异常。通常，应避免返回代码。

编辑:我刚刚注意到您的代码中还有一个潜在的有害错误，与我上面的最后一点有关。如果多个 记录被更改怎么办？它可能不会发生在 INSERT 上，但很容易发生在 UPDATE 上。代码的其他部分可能假设 != 1 表示没有记录被更改。这可能会使调试变得非常有问题:)