Question

如果在db中找不到任何条目,则放弃例外的最佳做法?

// CONTROLLER
public ActionResult Edit(int categ或yId, int id)
{
    Product target = Products.GetById(id);
    if (target == null) throw new HttpException(404, "Product not found");

    return View("Edit", target);
}

// REPOSITORY
public Product GetById(int id)
{
    return context.Products.FirstOrDefault(x => x.productId == id);
}

或

// CONTROLLER
public ActionResult Edit(int categ或yId, int id)
{
    return View("Edit", Products.GetById(id));
}

// REPOSITORY
public Product GetById(int id)
{
    Product target = context.Products.FirstOrDefault(x => x.productId == id);
    if (target == null) throw new HttpException(404, "Product not found with given id");

    return target;
}

Answer 1

永远不会从存放处扔下,......该存放处的抽象程度是错误的。如果您不希望您的存放处返回null,那么,情况就是如此:

// CONTROLLER
public ActionResult Edit(int categoryId, int id)
{
    try {
       Product target = Products.GetById(id);
    }
    catch(ProductRepositoryException e) {
       throw new HttpException(404, "Product not found")
    }

    return View("Edit", target);
}

// REPOSITORY
public Product GetById(int id)
{
    Product target = context.Products.FirstOrDefault(x => x.productId == id);
    if (target == null) throw new ProductRepositoryException();

    return target;
}

你的存放处不应知道关于吉大港山区的任何情况,但你的控制者可以知道该存放点。因此,你从存放处提取一个存放处的例外情况,将存放处作为控制器的例外情况处理。

Answer 2

不要在存放处投放HttpException,因为你可能希望今后在非Http环境下重新使用该代码。如果你需要至少一个项目并在主计长中处理这一例外情况,则放弃自己的项目。

Answer 3

我向控制器投掷,并将<>null从储存库取回<>。

友情链接