ASP.NET MVC: dove vengono generate le eccezioni?

Question

Le migliori pratiche per lanciare l'eccezione se nessuna voce trovata nel 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); 
} 

o

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

Non gettare mai uno HttpException da un repository ... è il livello di astrazione sbagliato. Se non si desidera che il repository per tornare null, fare qualcosa di simile:

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

Il repository non dovrebbe sapere nulla di HTTP, ma il controller può conoscere il repository. Quindi si lancia un'eccezione di repository dal repository e si "traduce" in un'eccezione HTTP nel controller.

Answer 2

vorrei buttare HttpException nel controller, e restituire null dal repository.

Answer 3

Non lanciare una HttpException nel repository, poiché si potrebbe desiderare di riutilizzare quel codice in futuro in un ambiente non Http. Getta la tua eccezione ItemNotFound se richiedi almeno un elemento e gestisci tale eccezione nel Controller, oppure restituisci null e gestiscilo.