MVC - 数据计算最佳实践 - 视图模型与控制器

Question

r我需要一些关于在哪里对数据进行计算的建议。

我有一个视图模型，其中包含我的计算所需的所有字段，我为我的一个计算创建了以下内容：

public class CommissionVM
{
public int? LoanAmountLock { get; set; } // from loan table    
public decimal BranchRev { get; set; }   // from revenue table
public decimal BranchProcessFee { get; set; }   // from revenue table

public decimal BranchGrossTotal
    {
        get
        {               

            return Convert.ToDecimal(LoanAmountLock * (BranchRev/ 100) + BranchProcessFee);
        }
    }

}

我尝试在我的视图中使用 Model.BranchGrossTotal，但它返回 0。我认为我有一个操作顺序问题。值 LoanAmountLock、BranchRev 和 BranchProcessFee 作为查询结果返回：

public ActionResult Reconcile(int? id, string RevenueLoanType)
    {
var model = new CommissionVM()
        {

            Loan = db.Loan.FirstOrDefault(a => a.id == id ),                
            BranchRevenues = db.BranchRevenues.FirstOrDefault(a => a.RevenueLoanType == RevenueLoanType),  
        };
return View(model);
}

在我用查询填充视图模型后，我最初能够通过在控制器中进行所有数学运算来使这些计算工作，但将有大约 10 次计算，据我了解，我不应该弄乱我的具有业务逻辑的控制器。

对此最好的解决方案是什么？我需要为计算创建另一个类吗？如果是这样，我如何用我的数据填充该类并在我的控制器中使用它？

编辑：我不确定如何设置业务类并在控制器中使用它们。谁能指出我的教程方向？

Answer 1

您不应在控制器或视图模型中进行计算。您应该在业务层中执行此操作。想想 View Model 是非常简单的类，其中包含要向用户显示的数据，就是这样。

关于计算，您应该将其中一项转换为十进制，而不是计算结果。如果你将整数相除，你会得到一个整数。

例如，您可以创建一个类并将其命名为 CommissionService。该类应该调用您的数据层来获取数据，进行任何额外的计算并将数据（可能是 DTO）返回给控制器。控制器应该基于 DTO 创建视图模型并将它们发送到视图。

阅读这些文章：

1)https://msdn.microsoft.com/en-us/library/hh404093.aspx

2)http://www.asp.net/mvc/overview/older-versions-1/models-%28data%29/validating-with-a-service-layer-cs

3)http://blog.diatomenterprises.com/asp-net-mvc-business-logic-as-a-separate-layer/

4)http://sampathloku.blogspot.com.ar/2012/10/how-to-use-viewmodel-with-aspnet-mvc.html

Answer 2

我不喜欢在我的视图模型上进行计算——你不能在其他地方轻松地重复使用计算，而且更难测试和调试。创建单独的类来执行业务逻辑。

您的业务逻辑类可以返回您的视图模型或返回用于填充这些模型的值。权衡是易用性和可重用性。

我通常更喜欢返回值而不是大对象，因此我的服务更可重用。

控制器

public class BranchController : Controller
{
    private IBusinessService service;

    public BranchController()
    {
        this.service = new BusinessService(db);
    }

    public ActionResult Reconcile(int? id, string RevenueLoanType)
    {
        var model = new CommissionVM
        {
            BranchGrossTotal = this.service.GetBranchGrossTotal(id, RevenueLoanType),
            ...
        };

        return View(model);
    }
}

服务

您可以制作任意数量的这些，并且您的控制器会根据需要使用它们。如果您需要查询，您应该传递 DbContext 实例，否则您可能会遇到不同上下文中的相关实体的问题。

public interface IBusinessService
{
    decimal GetBranchGrossTotal(int id, string revenueLoanType);
}

public class BusinessService : IBusinessService
{
    private DbContext db;

    public BusinessService(DbContext db)
    {
        this.db = db;
    }

    public decimal GetBranchGrossTotal(int id, string revenueLoanType)
    {
        var branch = db.Branch.First(b => b.Id == id);
        // do stuff
        return total;
    }
}

如果您愿意，您可以在 GetBranchGrossTotal() 中完全填充并返回视图模型。

Answer 3

首先，您分配给控制器上的 CommissionVM 的属性与模型上声明的属性不匹配。当您的模型上只有 LoanAmountLock 和 BranchRevs 可用时，您分配 Loan 和 BranchRevenues。 请注意，Loan 属性本身就是一个对象，必须从该对象中检索 LoanAmountLock (Loan.LoanAmountLock)。 BranchRevenues 对象也是如此。您应该根据需要将 BranchRevs 分配给 BranchRevenues 对象的相应属性。如果您不这样做，那么这些值将默认为 0，并且在尝试计算 BranchGrossTotal 时，它显然是 0。

另一个原因是，假设您正确填充模型属性，FirstOrDefault 方法会呈现空值，因为没有这样的实体。这也会导致 BranchGrossTotal 为 0。

你是对的，你不需要用计算和数据库访问来弄乱你的控制器。我将创建一个业务类 ComissionBusiness 并在您的控制器顶部实例化它。此类将有一个执行所有计算的方法。您应该将 Reconcile 方法移至新的业务类方法并在 reconcile 操作中调用它。类似的东西（请原谅缺少语法）

public MyController : Controller {

public ComissionBusiness comissionBusiness;

public MyController(){
comissionBusiness  = new ComissionBusiness();
}

public ActionResult Reconcile(int? id, string RevenueLoanType)
    {
var model = comissionBusiness.Reconcile(id, revenueLoanType);
return View(model);
}
}