Skip to content

Commit 47a2d24

Browse files
authored
Merge pull request #18 from HD-Malrang/김수빈
docs: Update to item54
2 parents 9a6d066 + 3922fb6 commit 47a2d24

3 files changed

Lines changed: 289 additions & 0 deletions

File tree

김수빈/5장/item32.md

Lines changed: 125 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,125 @@
1+
## 제네릭과 가변인수를 함께 쓸 때는 신중하라
2+
3+
4+
### 가변인수 메서드
5+
6+
하나의 메서드가 가변적인 수의 매개변수를 받을 수 있도록 지원
7+
8+
9+
10+
가변인수(Varargs)를 사용하는 메서드를 호출하면 Java 컴파일러가 자동으로 배열을 생성
11+
12+
```java
13+
public static void varargsExample(String... args) {
14+
// args는 배열로 처리됨
15+
System.out.println("Length: " + args.length);
16+
}
17+
```
18+
호출시 아래처럼 컴파일
19+
20+
```java
21+
varargsExample(new String[]{"A", "B", "C"});
22+
```
23+
24+
### 이 배열은 내부로 감춰져야 하는데, 클라이언트에 공개되면서 문제가 발생할 수 있다.
25+
26+
27+
28+
가변인수 매개변수에 제네릭이나 매개변수화 타입이 포함되면 컴파일 경고 발생
29+
30+
warning: [unchecked] Possible heap pollution from
31+
32+
parameterized vararg type List<String>
33+
34+
<b>매개변수화 타입의 변수가 타입이 다른 객체를 참조하면 힙 오염이 발생한다.</b>
35+
36+
37+
38+
ex)
39+
```java
40+
static void dangerous(List<String>... stringLists) {
41+
List<Integer> intList = List.of(42);
42+
Object[] objects = stringLists;
43+
objects[0] = intList; // 힙 오염 발생
44+
String s = stringLists[0].get(0); // ClassCastException
45+
}
46+
```
47+
48+
마지막 줄에 컴파일로가 생성한 형변환이 숨어 있기 때문에 ClassCastException 발생.
49+
```java
50+
String s = (String)stringLists[0].get(0); // ClassCastException
51+
```
52+
53+
54+
55+
56+
57+
-> 이처럼 타입 안전성이 깨지니 제네릭 가변인수 배열 매개변수에 값을 저장하는 것은 안전하지 않다.
58+
59+
60+
61+
### 제네릭 가변인수 메서드를 선언할 수 있게 한 이유?
62+
63+
실무에서 유용!
64+
65+
자바 라이브러리에서도 이런 메서드를 여럿 제공 (타입 안전)
66+
67+
ex. Arrays.asList(T... a), Collections.addAll(Collection<? super T> c, T... elements)
68+
69+
70+
71+
타입 안전한 메서드의 조건
72+
73+
1. 메서드가 제네릭 배열에 아무것도 저장하지 않는다.
74+
75+
2. 그 배열의 참조가 밖으로 노출되지 않는다.
76+
77+
-> 가변인수(Varargs)를 사용하는 메서드가 전달된 데이터를 단순히 읽고 사용만 할 경우에는 문제가 발생하지 않는다
78+
79+
(가변인수의 목적 : 메서드 호출 시 임의의 개수의 인수를 간편하게 전달)
80+
81+
82+
83+
### @SafeVarargs
84+
85+
- 메서드 작성자가 그 메서드가 타입 안전함을 보장하는 장치
86+
87+
컴파일러는 이 약속을 믿고 그 메서드가 안전하지 않을 수 있다는 경고를 더 이상 하지 않는다.
88+
89+
메서드가 안전한 게 확실하지 않다면 절대 @SafeVarargs 애너테이션을 달아서는 안 된다.
90+
91+
92+
93+
### 제네릭 가변인수 매개변수를 안전하게 사용하는 메서드
94+
```java
95+
@SafeVarargs
96+
static <T> List<T> flatten(List<? extends T>... lists) {
97+
List<T> result = new ArrayList<>();
98+
for (List<? extends T> list : lists)
99+
result.addAll(list);
100+
return result;
101+
}
102+
```
103+
104+
flatten 메서드는 임의 개수의 리스트를 인수로 받아, 받은 순서대로 그 안의 모든 원소를 하나의 리스트로 옮겨 담아 반환한다.
105+
이 메서드에는 @SafeVarargs 어노테이션이 달려있으니 선언하는 쪽과 사용하는 쪽 모두에서 경고를 발생시키지 않는다.
106+
제네릭이나 매개변수화 타입의 varargs 매개변수를 받는 모든 메서드에 @SafeVarargs 어노테이션을 달자
107+
108+
109+
### 제네릭 varargs 매개변수를 List로 대체한 예 - 타입 안전
110+
```java
111+
static <T> List<T> flatten(List<List<? extends T>> lists) {
112+
List<T> result = new ArrayList<>();
113+
for (List<? extends T> list : lists)
114+
result.addAll(list);
115+
return result;
116+
}
117+
```
118+
119+
120+
121+
### 정리
122+
123+
메서드에 제네릭 (혹은 매개변수화된) varargs 매개변수를 사용하고자 한다면, 먼저 그 메서드가 타입 안전한지 확인한 다음
124+
125+
@SafeVarargs 애너테이션을 달아 사용하는 테 불편함이 없게끔 하자.

김수빈/8장/item53.md

Lines changed: 87 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,87 @@
1+
## 가변인수는 신중히 사용하라
2+
3+
- 가변인수 메서드 : 명시한 타입의 인수를 0개 이상 받을 수 있음
4+
5+
```java
6+
static int sum(int...args){
7+
int sum=0;
8+
for(int arg:args)
9+
sum+=arg;
10+
return sum;
11+
}
12+
```
13+
14+
### 인수가 1개 이상이어야 할때
15+
16+
- 잘못 구현한 예
17+
18+
-- 인수를 0개 넣어 호출하면 컴파일이 아닌 런타임에 실패. 코드 지저분
19+
20+
```java
21+
static int min(int...args){
22+
if(args.length=0)
23+
throw new IllegalArgumentException("인수가 1개 이상 필요합니다.");
24+
25+
int min=args[0];
26+
for(int i=1;i<args.length;i++)
27+
if(args[i]<min)
28+
min=args[i];
29+
return min;
30+
}
31+
```
32+
33+
- 제대로 사용하기
34+
35+
: 매개변수를 2개 받기 - 하나는 평범한 매개변수, 두번째는 가변인수
36+
```java
37+
static int min(int firstArg, int... remainingArgs) {
38+
int min = firstArg; for (int arg : remainingArgs)
39+
if (arg < min) min = arg; return min;
40+
}
41+
```
42+
43+
44+
### 성능을 고려해야할때
45+
46+
: 가변 인수 메서드는 호출될 때마다 배열을 새로 하나 할당하고 초기화 -> 성능에 민감하면 걸림돌이 될 수 있음
47+
48+
-> 다중정의를 활용하면 성능 최적화 가능
49+
```java
50+
public void foo() { }
51+
public void foo(int a1) { }
52+
public void food(int a1, int a2) { }
53+
public void foo(int a1, int a2,int a3) { }
54+
public void foo(int a1, int a2, int a3, int... rest) { }
55+
```
56+
57+
58+
- EnumSet의 정적 팩터리도 이 기법을 사용해 열거 타입 집합 생성 비용을 최소화함.
59+
```java
60+
EnumSet<DayOfWeek> weekend = EnumSet.of(DayOfWeek.SATURDAY, DayOfWeek.SUNDAY);
61+
```
62+
63+
```java
64+
public static <E extends Enum<E>> EnumSet<E> of(E e) {
65+
EnumSet<E> result = noneOf(e.getDeclaringClass()); result.add(e);
66+
return result;
67+
}
68+
69+
...
70+
public static <E extends Enum<E>> EnumSet<E> of(E e1, E e2, E e3, E e4, E e5){
71+
EnumSet<E> result = noneOf(e1.getDeclaringClass());
72+
result.add(e1);
73+
result.add(e2);
74+
result.add(e3);
75+
result.add(e4);
76+
result.add(e5);
77+
return result;
78+
}
79+
80+
@SafeVarargs public static <E extends Enum<E>> EnumSet<E> of(E first, E... rest) {
81+
EnumSet<E> result = noneOf(first.getDeclaringClass());
82+
result.add(first);
83+
for (E e : rest) result.add(e);
84+
return result;}
85+
```
86+
87+

김수빈/8장/item54.md

Lines changed: 77 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,77 @@
1+
## null이 아닌, 빈 컬렉션이나 배열을 반환하라
2+
3+
4+
5+
- 따라하면 안 되는 코드 (컬렉션이 비었으면 null을 반환)
6+
```java
7+
private final List<Cheese> cheesesInStock = ...;
8+
/**
9+
* @return 매장 안의 모든 치즈 목록을 반환한다.
10+
* 단, 재고가 하나도 없다면 null을 반환한다.
11+
*/
12+
public List<Cheese> getCheeses() {
13+
return cheesesInStock.isEmpty() ? null
14+
: new ArrayList<>(cheesesInStock);
15+
}
16+
```
17+
18+
19+
20+
-- null을 반환하게 되면 클라이언트는 항상 방어 코드를 넣어줘야함, 방어코드를 빼먹으면 오류가 발생할 수 있
21+
22+
getCheeses()를 사용하는 클라이언트는 null 확인을 추가로 해야한다.
23+
```java
24+
if (cheeses != null && cheeses.contains(Cheese.STILTON)) {
25+
... }
26+
```
27+
28+
29+
30+
### null 대신 빈 컬렉션 할당시 비용.
31+
32+
1. 빈 컬렉션을 반환해도 사실상 신경 쓸 만한 성능 차이가 나지 않는다.
33+
34+
2. 빈 컬렉션과 배열은 새로 할당하지 않고도 반환할 수 있다.
35+
36+
- 빈 불변 컬렉션 반환 : Collections.emptyList(), Collections.emptySet(), Collections.emptyMap()
37+
38+
: 최적화가 필요할 때만 사용.
39+
40+
41+
42+
### 빈 컬렉션을 반환하는 올바른 예
43+
```java
44+
public List<Cheese> getCheeses() {
45+
return new ArrayList<>(cheesesInStock);
46+
}
47+
```
48+
49+
50+
51+
### 빈 배열을 반환하는 올바른 예
52+
```java
53+
public Cheese[] getCheeses() {
54+
return cheesesInStock.toArray(new Cheese[0]);
55+
}
56+
```
57+
58+
59+
60+
- 최적화
61+
```java
62+
private static final Cheese[] EMPTY_CHEESE_ARRAY = new Cheese[0];
63+
64+
public Cheese[] getCheeses() {
65+
return cheesesInStock.toArray(EMPTY_CHEESE_ARRAY);
66+
}
67+
```
68+
69+
70+
71+
72+
73+
### 핵심 정리
74+
75+
null이 아닌, 빈 배열이나 컬렉션을 반환하라. null을 반환하는 API는 사용하기 어렵고 오류 처리 코드도 늘어난다.
76+
77+
그렇다고 성능이 좋은 것도 아니다.

0 commit comments

Comments
 (0)