제가 다니는 회사는 대기업 서비스 개발 회사가 아니다보니 레거시 코드가 많습니다.
개발을 하다보니 한 serviceImpl 안에서 메서드 추출을 많이 해놓으면 읽는 사람이 계속 왔다 갔다 해서 읽기 불편하다고 피드백을 받았는데요.
이럴 때는 회사의 내려오는 틀을 따르는 게 맞는지 코드의 가독성이나 개인의 성장을 위해 이렇게 다 추출 하여 코드를 작성하는지 고민이 될 때가 있는데 어느 편이 좋다고 생각하시나요 ?
제가 다니는 회사는 대기업 서비스 개발 회사가 아니다보니 레거시 코드가 많습니다.
개발을 하다보니 한 serviceImpl 안에서 메서드 추출을 많이 해놓으면 읽는 사람이 계속 왔다 갔다 해서 읽기 불편하다고 피드백을 받았는데요.
이럴 때는 회사의 내려오는 틀을 따르는 게 맞는지 코드의 가독성이나 개인의 성장을 위해 이렇게 다 추출 하여 코드를 작성하는지 고민이 될 때가 있는데 어느 편이 좋다고 생각하시나요 ?
안녕하세요 ~
대기업 서비스 개발 회사라고 할지라도 레거시 코드가 엄청 많습니다 ~ ㅎㅎ
본론으로 돌아가서 메서드 추출이 많은 경우 왔다갔다해서 읽기 불편한 경우를 저도 겪었는데요.
그러면 너무 과도한 메서드 추출을 하고 있지는 않은가 고민해 볼 필요가 생각해요
메서드 추출이라는 것이 사실 가독성을 높이는 데 큰 역할을 한다고 생각하는데 코드를 읽는 입장에서 오히려 불편하다고 느꼈으니까요. 이 부분에서 저는 변수나 클래스 이름을 적절히 지었다면 이미 메서드 추출에서 얻는 이득을 이미 보고 있는게 아닐까 생각도 들어요. 그래서 굳이 메서드 추출을 안하고 쭉 나열하는 것이 오히려 가독성을 높이는 길일 수 있습니다.
결론적으로 메서드 추출 없이 메서드 안에 긴 코드가 있어도 읽기 좋다면 메서드 추출을 안하는 것이 더 좋다고 생각해요. 보통 이런 부분들은 같이 일하는 분들과 맞춰나가는 부분이라고 생각합니다.
저도 코드 작성한 다음 , 메서드 추출을 고민하는데 그 메서드 추출이 코드를 읽는데 불편함을 준다면 하지 않습니다.
답글
코드의 재사용성을 극대화 하기 위해서 메소드를 최대하 분리 하는 경우도 있습니다.
답글