我应该将这段代码放在 MVC 中的什么位置？

Question

我的代码运行完美，但是。这种情况下的最佳做法是什么？

这是重要的代码。

这是在控制器中。

    private IProductRepository repository;
    [HttpPost]
    public ActionResult Delete(int productId) {
        Product prod = repository.Products.FirstOrDefault(p => p.ProductID == productId);
        if (prod != null) {
            repository.DeleteProduct(prod);
            TempData["message"] = string.Format("{0} was deleted", prod.Name);
        }
        return RedirectToAction("Index");
    }

这是存储库（包括接口等）

public interface IProductRepository {
    IQueryable<Product> Products { get; }
    void SaveProduct(Product product);
    void DeleteProduct(Product product);
}

，这里是存储库......（重要的部分）但我想指出......这不是一个假类，这一点非常清楚。测试是在 fakeclass 上完成的。

    private EFDbContext context = new EFDbContext();

    public IQueryable<Product> Products {
        get { return context.Products; }
    }

    public void DeleteProduct(Product product) {
        context.Products.Remove(product);
        context.SaveChanges();
    }

那么第一个问题： 在对此进行测试时，我将在“ControllerTest”中的控制器上创建两个测试方法。 “Can_delete_valid_product”和“Cannot_delete_invalid_product”。为存储库提供测试类有什么意义吗？就像“RepositoryTest”一样，毕竟控制器测试删除功能是否有效，不需要测试两次，对吗？

第二个问题： 在此，我在尝试删除产品之前在控制器中测试该产品是否存在。如果存在，我会调用存储库中的删除函数。这意味着永远不应该有例外的可能性。但是如果您发送 null，您仍然可以在存储库中创建异常。 （这在这里不会发生，但如果您忘记检查是否为空，您仍然可以这样做）。问题是产品是否存在的测试是否应该在存储库中完成？

Answer 1

在大多数情况下，我更喜欢将逻辑保留在控制器之外。控制器操作的测试验证存储库是否被调用，但存储库本身在该测试中被模拟。我会让存储库负责处理空检查。

Answer 2

1. 我个人为我的存储库/数据访问创建单独的测试，以确保其正常工作。控制器本身将使用模拟进行测试。

2. 实际上，完全有可能（只是可能性不大）某人可以在其他人试图删除产品时删除该产品。在这种情况下，您可能不关心/需要知道有人这样做了，所以我可能会在存储库中吞下该异常（尽管我会先记录它）。就空检查/防御性编程而言，这完全是个人选择。有些人将这样的检查留在系统的入口点，而其他人则构建分层防御，在整个代码中进行额外的检查。问题是这些检查可能会变得相当难看，这也是我希望 代码合约的重要原因 会获得更多关注。

Answer 3

这意味着永远不应该有发生异常的可能性。但是如果您发送 null，您仍然可以在存储库中创建异常。 （这在这里不会发生，但如果您忘记检查是否为空，您仍然可以这样做）。

或者，如果它在您检查它存在之后但在删除它之前被删除。或者，如果您失去了与存储库的连接（或者在这种情况下该方法永远不会返回？）。你无法通过这种方式避免异常。