ASP.NET MVC - Trường hợp ném ngoại lệ?

Question

Thực hành tốt nhất để ném ngoại lệ nếu không có mục nhập nào được tìm thấy trong db?

// CONTROLLER 
public ActionResult Edit(int categoryId, 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); 
} 

hoặc

// CONTROLLER 
public ActionResult Edit(int categoryId, 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

Không bao giờ ném một số HttpException từ một kho lưu trữ ... đó là mức trừu tượng sai. Nếu bạn không muốn kho của bạn để trở null, làm một cái gì đó như thế này:

// 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; 
} 

kho của bạn không nên biết gì về HTTP, nhưng điều khiển của bạn có thể biết về kho. Vì vậy, bạn ném một ngoại lệ lưu trữ từ kho lưu trữ, và "dịch" mà thành một ngoại lệ HTTP trong bộ điều khiển.

Answer 2

tôi sẽ ném HttpException trong bộ điều khiển, và trở null từ kho.

Answer 3

Không ném một HttpException trong Kho lưu trữ, vì bạn có thể muốn sử dụng lại mã đó trong tương lai trong môi trường không phải là Http. Ném ngoại lệ ItemNotFound của riêng bạn nếu bạn yêu cầu ít nhất một mục và xử lý ngoại lệ đó trong Controller, hoặc trả về null và xử lý điều đó.