[일반] 이런것도 매직넘버인가 ?

[익명(211.213)]

list.get(0) + list.get(1); 에서 0, 1 이런거

list에는 문자열이 쪼개져 있음 ex. "셀렉트 x.a, x.b from x where xxx = '", "'~~~" 

뭐라 변수로 표현하기 애매하지 않음? 

[김대기(waitkim)]

1. 외부에서도 반복적으로 생성되는 로직이라면 해당 숫자들을 변수로 빼야한다 2. 가능하면 해당 로직들을 한 군데로 공통화 시키고 의미를 지니는 dto로 만들어서 리턴하는게 좋다

[김대기(waitkim)]

매직넘버 맞음

[김대기(waitkim)]

2번으로 하면 공통화된 로직에서 변수로 받을테니까, 해당 변수명이 곧 매직넘버에 대한 설명이라서 굳이 숫자를 변수로 뺄 필요가 없어짐