如何在我的 ASP.NET CORE 5 应用程序中清理我的控制器操作和引用？ [关闭]

Question

我的 Web 应用程序的前端有一个用户可以提交的表单，该表单与一个名为 Requirement 的模型相关联。因为创建和编辑数据只有一个表单，所以我需要一种方法来在我的控制器中处理这个过程。

为了让事情变得更简单，我将模型中传入的 Id 作为参考点并从那里开始工作。

Process_Requirement

这是点击提交时表单调用的操作。它查看传入的 Id，条件是如果它是 0，它是新的，它在 0 之上，它已经存在。

public async Task<JsonResult> Process_Requirement(Requirement model)
{
    //It's new
    if (model.Id == 0) {
        var create = await Create_Requirement(model);
        return Json(new { model = create });        
    }
    //It exists
    else {
        var update = await Update_Requirement(model);
        return Json(new { model = update });
    }
}

此父操作根据满足的条件引用以下两个操作。

public async Task<JsonResult> Create_Requirement(Requirement model) 
{
    try
    {
        var requirement = await _requirement.CreateAsync(model);
        return Json(new { success = true, data = requirement.Id, operation = "create" });                
    }
    catch (Exception ex)
    {
        Console.WriteLine(ex.Message);
    }
    return Json(new { success = false, data = model });
}

public async Task<JsonResult> Update_Requirement(Requirement model)
{
    try
    {
        var requirement = await _requirement.UpdateAsync(model);
        return Json(new { success = true, data = requirement.Id, operation = "update" });
    }
    catch (Exception ex)
    {
        Console.WriteLine(ex.Message);
    }
    return Json(new { success = false, data = model });
}

那么，这里有什么问题？好吧，我只是认为我在这里的方法并不像它应该的那样干净，笨拙，而且也许不太正确。是的，它有效！但我想确保我正在生成运行良好的流畅代码，并认为我会在这里发布它，看看人们的想法是什么以及我怎样才能做得更好？

Answer 1

你违反了SRP原则

SRP：

单一职责原则 (SRP) 是一种计算机编程原则，它指出计算机程序中的每个模块、类或函数都应对该程序功能的单个部分负责，并且应该封装该部分...... .

您应该在后端有两个操作，一个用于创建，一个用于编辑 并且您必须使用相同的表单在前端创建两个状态。如果您的状态处于创建模式，则应调用 create api，如果处于编辑模式，则应调用 update api。

 public async Task<JsonResult> Create_Process_Requirement(Requirement model)
    {
        var create = await Create_Requirement(model);
        return Json(new { model = create });
    }

   public async Task<JsonResult> Update_Process_Requirement(Requirement model)
    {
        var update = await Update_Requirement(model);
        return Json(new { model = update });
    }

Answer 2

您的Update_Requirement 和Create_Requirement 实际上是在做同样的事情，只是调用了两种不同的方法。您可以将这两种方法合并为一个。

    /// <summary>
    /// Create or Update Requirement object.
    /// </summary>
    /// <param name="model"></param>
    /// <returns></returns>
    private async Task<JsonResult> CreateOrUpdate_Requirement(Requirement model)
    {
        try
        {
            string operationValue = string.Empty;
            int requirementId;
            if(model.Id == 0)
            {
                operationValue = "create";
                requirementId = (await _requirement.CreateAsync(model)).Id;
            }
            else
            {
                operationValue = "update";
                requirementId = (await _requirement.UpdateAsync(model)).Id;
            }

            return Json(new { success = true, data = requirementId, operation = operationValue });
        }
        catch (Exception ex)
        {
            Console.WriteLine(ex.Message);
        }
        return Json(new { success = false, data = model });
    }

我不确定您的 await _requirement.CreateAsync(model) 和 _requirement.UpdateAsync(model) 实际做了什么以及它们返回的值类型。你也可以合并它们。 您可以将您的主要方法更改为：

public async Task<JsonResult> Process_Requirement(Requirement model)
{
    return await CreateOrUpdate_Requirement(model);
}