[아이템 62] 다른 타입이 적절하다면 문자열 사용을 피하라

[배경]

문자열은 워낙 흔하고 자바가 또 잘 지원해주어서 원래 의도하지 않은 용도로도 쓰이는 경향이 있다.

 

 

[Why]

문자열은 다른 값 타입을 대신하기에 적합하지 않다.

• 많은 사림이 파일, 네트워크, 키보드 입력으로부터 데이터를 받을 때 주로 문자열을 사용한다.
• 하지만 입력받을 데이터가 진짜 문자열일 때만 그렇게 하는 게 좋다.

문자열은 열거 타입을 대신하기에 적합하지 않다. (아이템34)

 

문자열은 혼합 타입을 대신하기에 적합하지 않다. 

String compoundKey = className + "#" + i.next();

• 이는 단점이 많은 방식이다.
  • 두 요소를 구분해주는 문자 #이 두 요소 중 하나에서 쓰였다면 혼란스러운 결과를 초래한다.
  • 각 요소를 개별로 접근하려면 문자열을 파싱해야 해서 느리고, 귀찮고, 오류 가능성도 커진다.
  • 적절한 equals, toString, compareTo 메서드를 제공할 수 없으며, String이 제공하는 기능에만 의존해야 한다.
• 이런 클래스는 보통 private 정적 멤버 클래스로 선언한다(아이템24).

문자열은 권한을 표기하기에 적합하지 않다.

• 권한(capacity)을 문자열로 표현하는 경우가 종종 있다.
• 예를 들어 스레드 지역변수 기능을 설계한다고 해보자.

public class ThreadLocal {
    private ThreadLocal() { }  // 객체 생성 불가
    
    // 현 스레드의 값을 키로 구분해 저장한다.
    public static void set(String key, Object value);
    
    // (키가 가리키는) 현 스레드의 값을 반환한다.
    public static Object get(String key);
}

• 클라이언트가 제공한 문자열 키로 스레드별 지역변수를 식별한 것이다.
• 이 방식의 문제는 스레드 구분용 문자열 키가 전역 이름공간에서 공유된다는 점이다.
  • 만약 두 클라이언트가 같은 키를 써버리면, 의도치 않게 같은 변수를 공유하게 된다.
• 보안도 취약하다.
  • 악의적인 클라이언트라면 의도적으로 같은 키를 사용하여 다른 클라이언트의 값을 가져올 수도 있다.

 

[How]

위 ThreadLocal 클래스는 문자열 대신 위조할 수 없는 키를 사용하면 해결된다.

public class ThreadLocal {
    private ThreadLocal() { }  // 객체 생성 불가
    
    public static class Key {  // (권한)
        Key() { }
    }
    
    // 위조 불가능한 고유 키를 생성한다.
    public static Key getKey() {
        return new Key();
    }
    
    public static void set(Key key, Object value);
    public static Object get(Key key);
}

• 앞의 두 가지 문제는 해결할 수 있지만, 개선할 여지가 있다.
  • set과 get은 이제 정적 메서드일 이유가 없으니 Key 클래스의 인스턴스 메서드로 바꾸자.
    이렇게 하면 Key는 더 이상 스레드 지역변수를 구분하기 위한 키가 아니라, 그 자체가 스레드 지역변수가 된다.
  • 결과적으로 지금의 톱레벨 클래스인 ThreadLocal은 별달리 하는 일이 없어지므로 치워버리고, 중첩 클래스 Key의 이름을 ThreadLocal로 바꿔버리자.

public final class ThreadLocal {
    public ThreadLocal();
    public void set(Object value);
    public Object get();
}

• get으로 얻은 Object를 실제 타입으로 형변환해 써야 해서 타입 안전하지 않다.
• ThreadLocal을 매개변수화 타입(아이템29)으로 선언하면 간단하게 문제가 해결된다.

public final class ThreadLocal<T> {
    public ThreadLocal();
    public void set(T value);
    public T get();
}