Jakiego wyjątku powinienem użyć w przypadku "No Record Found"? (C#)

Question

Mam następujący kod, który pobiera rekordy Szczegóły po kliknięciu na siatce tabeli:

public ActionResult City(string rk) 
{ 
    try 
    { 
     var city = _cityService.Get("0001I", rk); 
     if (city == null) 
     { 
      throw new ServiceException("", "Error when fetching city " + rk); 
     } 
    } 
} 

Jakie wyjątku należy używać do tego „Nie znaleziono rekordu” problem? Rozumiem, że istnieją różne rodzaje wyjątków, ale nie jestem pewien, który z nich byłby odpowiedni, a nawet, czy poprawnie to koduję.

Answer 1

KeyNotFoundException byłby to rozsądny wybór, i byłoby zgodne the Microsoft guideline do:

Rozważmy rzucanie istniejących wyjątków znajdujących się w przestrzeni nazw systemowych zamiast tworzenia własnych typów wyjątków.

Jednak można rozważyć stworzenie własnej Exception typu if (ponownie z wytycznymi firmy Microsoft):

... masz stan błędu, który może być obsługiwany programowo w inny sposób niż jakakolwiek inna istniejące wyjątki.

Jeśli utworzysz własny Exception, powinieneś postępować zgodnie z guidelines for designing custom exceptions, np. powinieneś sprawić, by Twój numer seryjny był Exception.

Answer 2

Powinieneś stworzyć własny wyjątek i może w tym przypadku nazwać go RecordNotFoundException.

Answer 3

Tworzenie własnego wyjątku jest dość łatwe. Wystarczy utworzyć klasę, nadać jej nazwę, rozszerzyć Exception lub inny typ wyjątku i dostarczyć konstruktorów, których potrzebujesz (po prostu wywołując konstruktory bazowe Exception).

Jeśli chcesz dodać więcej, możesz, ale często nie musisz.

Jeśli tworzysz kilka wyjątków dla swojego projektu, możesz utworzyć podstawowy wyjątek (który rozszerza wyjątek), który obejmuje wszystkie wyjątki. To jest coś, co możesz zrobić, pisząc bibliotekę. Umożliwiłoby to komuś przechwycenie określonego wyjątku lub wyjątku wyrzuconego z biblioteki lub dowolnego wyjątku.

public class MySuperAwesomeException : Exception 
{ 
    public MySuperAwesomeException() : base() { } 
    public MySuperAwesomeException(string message) : base(message) { } 
    public MySuperAwesomeException(string message, Exception innerException) 
     : base(message, innerException) { } 
}