您是否在您的 asp.net mvc 应用程序的控制器或业务服务中捕获了预期的异常

Question

我正在开发一个 asp.net mvc 应用程序，其中 user1 可以删除 user2 之前刚刚加载的数据记录。 User2 要么更改这个不存在的数据记录（更新），要么在另一个违反外键约束的表中使用此数据进行插入。

你在哪里捕捉到这样的预期异常？

在您的 asp.net mvc 应用程序的 Controller 中还是在业务服务中？

只是一个旁注：我只在这里捕获 SqlException，如果它是一个 ForeignKey 约束异常，告诉用户另一个用户已经删除了某个父记录，因此他不能创建测试计划。但是这段代码还没有完全实现！

控制器：

  public JsonResult CreateTestplan(Testplan testplan)
  {
   bool success = false;
   string error = string.Empty;

   try
  {
   success = testplanService.CreateTestplan(testplan);
   }
  catch (SqlException ex)
   {
   error = ex.Message;
   }
   return Json(new { success = success, error = error }, JsonRequestBehavior.AllowGet);
  }

或

商业服务：

public Result CreateTestplan(Testplan testplan)
        {
            Result result = new Result();
            try
            {
                using (var con = new SqlConnection(_connectionString))
                using (var trans = new TransactionScope())
                {
                    con.Open();

                    _testplanDataProvider.AddTestplan(testplan);
                    _testplanDataProvider.CreateTeststepsForTestplan(testplan.Id, testplan.TemplateId);
                    trans.Complete();
                    result.Success = true;
                }
            }
            catch (SqlException e)
            {
                result.Error = e.Message;
            }
            return result;
        }

然后在控制器中：

public JsonResult CreateTestplan(Testplan testplan)
      {
       Result result = testplanService.CreateTestplan(testplan);      
       return Json(new { success = result.success, error = result.error }, JsonRequestBehavior.AllowGet);
      }

Answer 1

应检查并正确显示外键约束违规。您可以轻松检查相关表中的行是否存在并显示正确的消息。行更新也可以这样做。服务器返回受影响的行数，因此您知道会发生什么。

即使您不进行这些检查，您也应该捕获 SQL 异常。对于普通应用程序用户来说，关于违反约束的消息毫无意义。此消息适用于开发人员，您应该使用 ELMAH 或 Log4Net 库记录它。用户应该会看到类似“我们很抱歉。此行可能已被其他用户修改，您的操作无效”的消息。如果他向开发者询问这件事，开发者应该检查日志并查看原因。

编辑

我认为您应该检查服务中的错误。控制器不应该知道数据访问层。对于控制器，将数据存储在 SQL 数据库还是文件中都没有关系。文件可以抛出文件访问异常，SQL 有其他的。控制器不应该担心它。您可以在服务中捕获数据访问层异常并抛出服务层专用类型的异常。控制器可以捕获它并显示正确的消息。所以答案是：

public class BusinessService 
{
    public Result CreateTestplan(Testplan testplan)
    {
        Result result = new Result();
        try
        {
            using (var con = new SqlConnection(_connectionString))
            using (var trans = new TransactionScope())
            {
                con.Open();

                _testplanDataProvider.AddTestplan(testplan);
                _testplanDataProvider.CreateTeststepsForTestplan(testplan.Id, testplan.TemplateId);
                trans.Complete();
                result.Success = true;
            }
        }
        catch (SqlException e)
        {
            ....log in ELMAH or Log4Net using other logging framework...
            throw new ServiceException("We are sorry. Your operation conflicted with another operation in database. It has been cancelled.");
        }
        return result;
    }
}