국비때 코드리뷰 한 썰

[익명(117.111)]

조건 검사하는 로직, 반복 처리하는 로직, 중복코드들 메서드 분리하고 코드만 봐선 모르는 로직은 주석도 달아놈

나름 완벽하게 짜왔다고 생각해서 싱글벙글했는데

들은 말은 "왜 보기 힘들게 함수 분리해 놓으셨어요??" 이였음 ㅋㅋㅋ

남들이 짠거 보니깐 두명은 완전 똑같길래 물어보니 서로 도와주다가 같은 코드를 짰나 봐요 ㅎㅎ 이지랄 ㅋㅋ

[익명(125.189)]

리뷰는 누가 해주는데 서로서로? ㅋㅋㅋㅋ

[익명(117.111)]

ㅇㅇ 나름 스터디 한답시고 했는데 지금 생각하면 웃음벨임 ㅋㅋ

[익명(117.17)]

반복되는 로직은 다른 메서드로 분류해서 사용한다는 개념이 없나?

[익명(117.17)]

진짜 스파게티만 인정한다는 노예정신 아주 좋습니다

[익명(117.111)]

중복코드 분리는 알던데 기능별로 나눈다는 개념은 없음

[익명(223.39)]

1메서드 1기능이 기본아니노 ㄷㄷ

[익명(223.39)]

하긴 내가 듣는 국비도 다들 안나누고 클래스하나, 메서드하나에 몰아서 처리하더라

[익명(117.111)]

전체적인 흐름부터 생각 -> 코딩을 해야 기능별로 딱 끊어서 나오는데 국비다니는 뉴비가 뭘 알겠음 코딩부터 하면서 땜질처리하느라 스파게티가 되지

[익명(39.7)]

근데 길지않은 메소드도 딸랑 2번쓸꺼면 걍 안빼놓는게 나을때도 있더라 - dc App