Возвращаемое значение в методе геттера, если объект не найден

Question

Мне интересно, какой лучший дизайн для этой проблемы. У меня есть объект , который имеет ArrayList<Episode>, который содержит каждый эпизод в сезоне для данного телешоу. Теперь, допустим, я хочу найти эпизод 25 данного сезона для данного телешоу. Является ли следующий дизайн подходящим (бросая исключение) или было бы лучше создать объект в состоянии ошибки, вернуть его и позволить вызывающему методу определить, что не так?

public class Season { 
    public Episode GetEpisode(int ep_no) throws Exception { 
     for(Episode ep : episodes) 
      if(ep.GetEpisodeNumber() == ep_no) return ep; 
     throw(new Exception("Episode not found.")); 
    } 

    private ArrayList<Episode> episodes; 
} 

Answer 1

Это зависит. Прежде всего, я бы не бросил голый Exception. Я бы создал новый тип исключения. Возможно, EpisodeNotFoundException. Таким образом, это более описательно.

Во-вторых, должен ли код обрабатывать эту ситуацию, когда это происходит, или это то, что не должно сбой. Если это потребует вмешательства программиста, вы должны выбросить исключение, которое вместо этого добавит RuntimeException. Код, который выбрасывает RuntimeException, не должен обрабатываться кодом, который его вызывает.

Кроме того, это ошибка , если эпизод не найден, или это вполне приемлемая вещь? Если это приемлемо, вы должны, скорее всего, вернуть null. Так работает Map.get().