코드 가독성, 이제 함수 관계를 파악하세요!
by DD
9개월 전
조회수 2
LY Corporation의 코드 품질 개선 노력의 일환으로, Review Committee 활동을 소개함
중첩 루프를 가진 코드의 가독성 저하 문제를 지적하고, Sequence를 활용한 리팩토링 제안
함수 추출 시 코드의 의미 단위를 고려하여 가독성과 유지보수성을 향상시킴
중첩 루프, 가독성의 적
중첩된 루프는 코드의 흐름을 파악하기 어렵게 만들어 유지보수를 어렵게 한다. 구체적으로, `saveAllItemMetadata` 함수 내 중첩된 `while`과 `for` 루프는 코드 복잡도를 증가시킨다. 따라서, 코드의 의미 단위를 고려한 리팩토링이 필요하다.
Sequence를 활용한 리팩토링
내부 루프를 추출하는 대신, Sequence를 활용하여 코드 구조를 개선한다. requestItemSequence 함수를 통해 Item의 열을 Sequence로 반환하여 중첩된 루프를 제거한다. 결과적으로, 코드 가독성을 높이고 유지보수성을 향상시킨다.
리팩토링, 의미 단위에 집중
리팩토링 시, 코드의 의미 단위를 파악하는 것이 중요하다. 함수 추출 시, 단순히 코드의 재사용성을 높이는 것보다 코드의 역할을 명확히 하는 데 집중해야 한다. 따라서, 코드의 의미를 고려하여 구조를 재구성하는 것이 더 효과적일 수 있다.
