item 62

다른 타입이 적절하다면 문자열 사용을 피하라

핵심정리

바로 문자열을 쓰기 보다는 데이터에 더 적합한 타입을 찾아주거나 새로 작성을 하자. 편하기 위해 문자열을 사용하면 결국 더 번거럽고, 덜 유연하고, 느리고 오류 가능성도 커진다. 문자열을 쓰기 전에 더 적합한 타입은 없는지 기본타입, 열거 타입, 혼합 타입을 다시 한 번 생각해보자.

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

문자열은 텍스트를 표현하도록 설계되었고, 그 일을 아주 멋지게 해낸다. 그런데 문자열이 워낙 흔하고 자바가 잘 지원을 해주다보니 원래 의도하지 않은 용도로 쓰이는 경향이 있다. 대표적인 예로 많은 사람들이 파일, 네트워크, 키보드 입력으로 데이터를 주고 받을 때 주로 문자열을 사용하는데 이는 적절하지 않은 방식이다.

받은 데이터가 수치형이라면 int, float 등 적당한 수치 타입으로 변환하는게 좋고, 예/아니오 질문이라면 열거타입이나 boolean으로 변환하는게 좋다. 기본 타입이든 참조 타입이든 적절한 값 타입이 있다면 그것을 사용하고, 없다면 새로 하나 작성하라.

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

(아이템 24) 상수를 열거할 때는 문자열보다는 열거타입이 월등히 낫다.

// 문자 상수 
public static final String CHEESE_PIZZA = "cheese pizza"
public static final String CHEESE_BURGER = "cheese burger" 
public static final String CHEESE_STICK = "cheese stick" 

// 열거형으로 변환
public enum Cheese { PIZZA, BURGER, STICK }

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

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

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

• 구분자인 '#' 문자가 className이나 i.next 혼란스러운 결과를 초래한다.
• 각 요소를 개별로 접근하려면 문자열을 파싱해야 해서 느리고, 오류 가능성까지 커진다.
• 적적한 equals, toString, compareTo 메서드를 제공할 수 없으며, String이 제공하는 기능에만 의존해야 한다.
• 이런 경우 차라리 전용 클래스를 새로 만드는 게 좋으며 private 정적 맴버 클래스 형태로 만든다.

public class TestClass {
	private String dummy; 

	private static class testInnerClass {
		public void printMethod() {}
	}

}

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

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

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

가끔 권한을 문자열로 표현하는 경우가 종종 있다. 위의 코드는 스레드 지역변수 기능을 설계할 때 각 스레드가 자신만의 변수를 갖게 해주는 일을 한다. 이 코드는 key라는 권한을 문자열로 관리하고 있는데 무슨 문제가 있을까?

• 스레드 구분용 문자열 키가 전역 이름공간에서 공유되고 있다.
• 처음의 의도대로 동작하려면 각 클라이언트가 고유한 키를 제공해야 한다.
• 키가 전역공간에서 공유되고 있기 때문에 의도지 않게 같은 키를 쓰면 같은 변수를 공유하게 된다.
• 결국 키를 가지고 있는 클라이언트들이 모두 제 기능을 하지 못할 것이고, 악의적인 클라이언트는 심지어 의도적으로 같은 키를 사용해 다른 클라이언트의 값을 갈취할 수 있다.
• 문자열 대신 위조할 수 없는 Key class를 사용해서 해결할 수 있다.

ThreadLocal class 개선1

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);
}

ThreadLocal class 개선 2

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

• 1에서 공유하지 않은 고유 key를 사용하기 때문에 개선2에서 get과 set을 key의 클래스 인스턴스 메서드로 바꿨다.
• 이렇게 함으로 Key는 더 이상 스레드 지역변수를 구분하기 위한 키가 아니라 Key 자체가 스레드 지역변수가 된다.
• 그래서 톱클래스의 ThreadLocal을 날려버리고 Key를 ThreadLocal로 바꿔버렸다.
• 문제점: Object를 실제타입으로 casting 해야 해서 타입 안전하지 않다.
• 타입 세이프하게 만들기 위해 ThreadLocal을 매개변수화 타입(제네릭)으로 만들어보자.

ThreadLocal class 최종본

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

→ 혼합 타입을 String으로 퉁치는 문제를 해결하고, 키(String 키, public static class Key) 보다 빠르고 우아한 코드가 완성되었다!