[5팀 김나희] [Chapter 2-1] 클린코드와 리팩토링

[imfineimgood]

과제 체크포인트

기본과제

• 코드가 Prettier를 통해 일관된 포맷팅이 적용되어 있는가?
• 적절한 줄바꿈과 주석을 사용하여 코드의 논리적 단위를 명확히 구분했는가?
• 변수명과 함수명이 그 역할을 명확히 나타내며, 일관된 네이밍 규칙을 따르는가?
• 매직 넘버와 문자열을 의미 있는 상수로 추출했는가?
• 중복 코드를 제거하고 재사용 가능한 형태로 리팩토링했는가?
• 함수가 단일 책임 원칙을 따르며, 한 가지 작업만 수행하는가?
• 조건문과 반복문이 간결하고 명확한가? 복잡한 조건을 함수로 추출했는가?
• 코드의 배치가 의존성과 실행 흐름에 따라 논리적으로 구성되어 있는가?
• 연관된 코드를 의미 있는 함수나 모듈로 그룹화했는가?
• ES6+ 문법을 활용하여 코드를 더 간결하고 명확하게 작성했는가?
• 전역 상태와 부수 효과(side effects)를 최소화했는가?
• 에러 처리와 예외 상황을 명확히 고려하고 처리했는가?
• 코드 자체가 자기 문서화되어 있어, 주석 없이도 의도를 파악할 수 있는가?
• 비즈니스 로직과 UI 로직이 적절히 분리되어 있는가?
• 코드의 각 부분이 테스트 가능하도록 구조화되어 있는가?
• 성능 개선을 위해 불필요한 연산이나 렌더링을 제거했는가?
• 새로운 기능 추가나 변경이 기존 코드에 미치는 영향을 최소화했는가?
• 리팩토링 시 기존 기능을 그대로 유지하면서 점진적으로 개선했는가?
• 코드 리뷰를 통해 다른 개발자들의 피드백을 반영하고 개선했는가?

심화과제

• 변경한 구조와 코드가 기존의 코드보다 가독성이 높고 이해하기 쉬운가?
• 변경한 구조와 코드가 기존의 코드보다 기능을 수정하거나 확장하기에 용이한가?
• 변경한 구조와 코드가 기존의 코드보다 테스트를 하기에 더 용이한가?
• 변경한 구조와 코드가 기존의 모든 기능은 그대로 유지했는가?
• 변경한 구조와 코드를 새로운 한번에 새로만들지 않고 점진적으로 개선했는가?

과제 셀프회고

과제에서 좋았던 부분

더티한 코드가 얼마나 가독성을 떨어뜨리고 생산성을 저하시키는지 느꼈습니다.
리팩토링을 진행하면서 이러한 코드에서 기능을 파악하고 분리시키면서 코드를 읽는 법을 알게 되어 좋았습니다.

과제를 하면서 새롭게 알게된 점

주석을 잘 활용하면 엄청난 파워

과제를 진행하면서 아직 애매하게 잘 모르겠다 하는 점, 혹은 뭔가 잘 안되서 아쉬운 것들

클린코드가 몸에 익혀진 상태가 아니라서 어떤 것이 좋은 코드인지 확신이 서지 않고 클린 코드를 염두에 두고 짜는 것이 쉽지 않습니다..
뭐부터 해야할지 모르겠는 느낌? 방향성 자체를 잡기 힘든 느낌? 입니다.

리뷰 받고 싶은 내용이나 궁금한 것에 대한 질문

기존 코드를 개선하고자 할 때 효율적으로 할 수 있는 순서? 가 있을까요?
ex) 동일한 기능을 하는 코드끼리 모음 -> 함수 단일 원칙 확인->비즈니스 로직, UI 로직 분리 -> 변수 관리 -> 네이밍 변경

[confidential-nt]

전체적으로 논리적 단위로 잘 나눠진 것 같아요! 각 논리적 단위 별로 파일로 분리하면 더 완벽해질 것 같은 개인적인 의견입니다!