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

문자열을 쓰지않아야할 사례

문자열(String)은 텍스트를 표현하도록 설계되었다. 자바에서 문자열을 의도하지 않은 용도로도 쓰이는 경향이 있는데, 문자열을 쓰지 않아야할 사례에 대해 알아보자.

 

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

많은 사람이 파일, 네트워크, 키보드 입력으로부터 데이터를 받을 때 주로 문자열을 사용한다. 입력받을 데이터가 진짜 문자열일 경우에만 사용해야한다. 받은 데이터가 수치형이라면 int, float, BigInteger 등 적당한 수치 타입으로 변환해야한다. 

 

기본 타입이든 참조 타입이든 적절한 값 타입이 있다면 그것을 사용하고, 없다면 새로 하나 작성하라. 

 

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

문자열보다는 열거 타입이 월등히 낫다.

 

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

여러 요소가 혼합된 데이터를 하나의 문자열로 표현하는 것은 대체로 좋지 않은 생각이다. 

 

혼합 타입을 문자열로 처리한 부적절한 예시 

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

혹여라도 두 요소를 구분해주는 문자 #이 두 요소 중 하나에서 쓰였다면 혼란스러운 결과를 초래한다. 이럴때에는 차라리 전용 클래스를 새로 만드는 편이 낫다. 이런 클래스는 보통 private 정적 멤버 클래스로 선언한다.

 

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

권한(capacity)을 문자열로 표현하는 경우가 종종 있다. 예를 들어 스레드 지역변수 기능을 설계한다고 해보자. 그 이름처럼 각 스레드가 자신만의 변수를 갖게해주는 기능이다. 해당 기능을 설계한 방법은, 바로 클라이언트가 제공한 문자열 키로 스레드별 지역변수를 식별한 것이다.

 

문자열을 사용하여 권한을 구분하는 잘못된 예시 

public class ThreadLocal {
    private ThreadLocal() { } // 객체 생성 불가

    // 현재 스레드의 값을 키로 구분해 저장한다.
    public static void set(String key, Object value);

    // (키가 가리키는) 현재 스레드의 값을 반환한다.
    public static Object get(String key);
}

이 방식의 문제는 스레드 구분용 문자열 키가 전역 이름 공간에서 공유된다는 점이다. 이 방식이 의도대로 동작하려면 각 클라이언트가 고유한 키를 제공해야한다. 만약, 두 클라이언트가 같은 키를 쓰기로 결정한다면 의도치 않게 같은 변수를 공유하게 된다. 이는 보안도 취약하다. 이 API는 문자열 대신 위조할 수 없는 키를 사용하면 해결된다. 이 키를 권한(capacity)라고도 한다.

 

Key 클래스로 권한두기 

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은 이제 정적(static) 메서드일 이유가 없다. Key 클래스의 인스턴스 메서드로 바꾸자. 이렇게 하면 Key는 더이상 스레드 지역변수를 구분하기 위한 키가 아니라, 그 자체가 스레드 지역변수가 된다. 

 

리팩토링 - Key를 ThreadLocal로 변경 

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

이 API에서는 get으로 얻은 Object를 형변환해야한다. 따라서 타입이 안전하지 않다. ThreadLocal을 매개변수 타입으로 선언하여 해당 문제를 해결하자.

 

매개변수화하여 타입 안전성 확보 

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