사내 Review 시스템 상에서 if 문 관련된 코딩 스타일에 대해서 의견을 확인하는 기회가 있었다. "처리할 것이 없는 데, 불필요하게 코드가 추가 되는 것이 아니냐?"라는 의견이 있어 다른 분들의 의견을 묻는 것이었다.
if ( a == b ) { a++; } else { /* do nothing */ }
이런 스타일에 부정적인 의견들은 다음처럼 나왔습니다.
- 필요성을 못 느낀다. 따라서, 불필요한 코드가 많이 차지한다.
- 신입 개발자들이 엄격한 코딩 스타일에 대한 거부 반응이 있다.
- 엄격한 코딩 스타일때문에 자유로운 생각에 방해가 된다.
- 깊은 수준의 indentation을 유발한다.
다음과 같은 긍정적인 의견들이 나왔습니다.
- 실수를 줄이는 데 도움이 된다.
- 가독성을 높여준다.
- else { /* do nothing */ }을 보면 다시 코드를 의심하고 본다.
인원 수로 보면 저를 포함해서 긍정적으로 생각하시는 분들이 많았습니다.
어떻게들 생각하시나요?
