Closed okhee closed 1 year ago
Bad
public void doSomthing() { Integer min; Integer max; Integer i; Integer n; Integer median; // 무엇의 중간 값일까? min = 10; max = 90; median = (min + max) / 2; i = 0; n = 10; // 0 ~ 10 for문.. }
Good
public void doSomthing() { Integer min = 10; Integer max = 90; Integer median = (min + max) / 2; i = 0; n = 10; // 0 ~ 10 for문..
public void doSomthing() { Article article = new Article(); ... article.setTitle("제목"); article.setContent("내용"); }
public void doSomthing() { Article article = new Article("제목", "내용"); }
반복 변수의 값을 반복문이 종료된 뒤에도 써야 하는 상황이 아니라면 while 보다 for문을 쓰자
Iterator<Element> i = c.iterator(); while (i.hasNext()) { doSomething(i.next()); }
Iterator i2 = c2.iterator(); while (i.hasNext()) { // 버그 발생 (컴파일 오류 x) doSomething(i2.next()); }
- Good ```java for (Iterator<Element> i = c.iterator(); i.hasNext()) { doSomething(i.next()); } for (Iterator<Element> i2 = c2.iterator(); i.hasNext()) { // i가 없다는 컴파일 오류 doSomething(i2.next()); }
지역변수 범위 최소화
지역변수를 사용할 때 선언과 동시에 초기화 하자
지역변수를 미리 선언 해두면 가독성이 떨어진다.
Bad
Good
지역변수 초기화에 필요한 정보가 충분치 않다면, 충분해질 때 까지 선언을 미루자.
Bad
Good
반복문은 while 보다 for문을 사용하자
반복 변수의 값을 반복문이 종료된 뒤에도 써야 하는 상황이 아니라면 while 보다 for문을 쓰자
Bad
Iterator i2 = c2.iterator();
while (i.hasNext()) { // 버그 발생 (컴파일 오류 x)
doSomething(i2.next());
}