아이템 54. null이 아닌, 빈 컬렉션이나 배열을 반환하라

[YunDaHyee]

null 리턴을 종종 쓰곤 했는데 정리한 걸 토대로 해서 올바르게 리턴해보겠습니다

[YunDaHyee]

빈 값이라는 결과를 위해 null을 반환하는 API의 단점

• 사용하기 어려움
• null 반환 해주는 상황을 추가해주면서
  동시에 반환 받는 쪽에서도 null을 리턴 받는 상황을 대비하기 위해
  if문의 조건에 리턴값 != null과 같은 방어 또는 오류 처리 코드를 넣어줘야함
• 성능이 좋지도 않음

빈 컬렉션 리턴의 올바른 방법

• 빈 컬렉션 할당해서 반환하는 방법
  가능성은 작지만 사용 패턴에 따라 성능 저하 유발 가능

• 빈 불변 컬렉션을 매번 반환하는 방법 (최적화 방법)
  첫 번째 방법의 대안으로 불변 객체는 공유해도 안전하다(아이템 17)는 특징 이용한
  => 필요할 때만 사용하고 수정 전/후의 성능 측정을 통해 성능 개선을 확인해야함.

  • ex) Collections.emptyList()
    집합이 필요하다면 Collections.emptySet, 맵 => Collections.emptyMap

    // 빈 컬렉션이 매번 새로 할당되지 않음
    public List<Cheese> getCheeses() {
        return cheeseslnStock.isEmpty() ? Collections.emptyList() : new ArrayList<> (cheeseslnStock};
    }

빈 배열 리턴의 올바른 방법

• 단순하게 길이가 0인 배열을 반환하는 방법
  성능 저하 유발 가능

  public Cheese[] getCheeses() {
      return cheeseslnStock.toArray(new Cheese[이 );
      }

• 길이가 0인 불변 배열을 미리 선언해두고 매번 반환하는 방법 (최적화 방법)

  private static final Cheese[] EMPTY_CHEE5E_ARRAY = new Cheese[0];
  
  /*
      원소가 하나라도 있을 경우,
      Cheese[] 타입의 배열을 새로 생성해서 반환, 없을 경우면 EMPTY_CHEESE_ARRAY 반환
  */
  public Cheese [] getCheeses() {
      return cheeseslnStock.toArray(EMPTY_CHEESE_ARRAY);
  }

  cf) 성능 개선이 목적이라면 toArray에 넘기는 배열을 미리 할당하는 건 성능 저하의 원인이 된다.
  return cheeseslnStock.toArray(new Cheese[cheeseslnStock.size()]);

요약

• null 리턴 보단 빈 컬렉션/배열을 리턴해라
• 이유
  • 빈 컨테이너 할당이 성능 저하의 뚜렷한 원인이 아닌 이상, 성능 차이는 미미하다.
  • 빈 컬렉션과 배열은 새로 할당하지 않고도 반환할 수 있다.

[jioome]

앞으로 웬만하면 Null 리턴을 하지 않도록 주의해야겠네요