코드 리뷰에서 자주 지적받는 실수는 무엇일까

 주니어 자바 개발자로 일하면서 자주 언급하는 실수들을 정리했다. 

1. 변수명과 메서드명이 불명확

변수명을 a, b, c나 temp, data 같은 모호한 이름으로 짓는 실수

// 나쁜 예: String s = getUserData();
// 좋은 예: String userName = getUserData();

2. 매직 넘버 사용

숫자를 하드코딩해서 나중에 유지보수가 힘들어진다.

// 나쁜 예: if (user.getAge() > 19)
// 좋은 예: if (user.getAge() >= ADULT_AGE)

3. 메서드가 너무 김

하나의 메서드에서 너무 많은 일을 처리하려고하는 점. 

100줄 넘는 메서드는 읽기도 어렵고 테스트하기도 어렵다. 

4. Exception 처리를 대충

모든 예외를 Exception으로 잡거나 printStackTrace()만 하고 끝내는 실수 

// 나쁜 예: catch (Exception e) { e.printStackTrace(); }
// 좋은 예: catch (SpecificException e) { logger.error("구체적인 에러 메시지", e); }

5. 불필요한 if문

boolean 값을 그대로 리턴하면 되는데 굳이 if문을 사용하는것

// 나쁜 예: if (isValid == true) return true; else return false;
// 좋은 예: return isValid;

6. Stream API 잘못 사용

Stream의 장점을 제대로 활용하지 못하고 기존 방식대로 코딩 

// 나쁜 예: list.stream().forEach(item -> result.add(item.getName()));
// 좋은 예: list.stream().map(Item::getName).collect(Collectors.toList());

7. 주석을 너무 많이 쓰거나 아예 안 씀

코드 한 줄마다 주석을 달거나, 복잡한 비즈니스 로직에 설명이 전혀없는.. 역시 적당한게 좋다. 

8. 테스트 코드를 안 씀

"일단 돌아가니까 괜찮지 않나요?" 하는 마음으로 테스트 코드 작성을 하지 않기 

9. 코드 스타일이 일관성 없음

들여쓰기, 괄호 위치, 공백 사용이 제각각이다. 의 코딩 컨벤션을 따라야한다. 

10. 성능을 고려하지 않음

작은 최적화라도 쌓이면 큰 차이가 나는데 신경안쓰는점. 

// 나쁜 예: for (int i = 0; i < list.size(); i++)
// 좋은 예: int size = list.size(); for (int i = 0; i < size; i++)

 

 

이런 실수들은 경험이 쌓이면서 자연스럽게 줄어들어요.

코드 리뷰 받을 때마다 배우는 자세로 임하고 있습니다.