[2팀 김경민] [Chapter 2-1] 클린코드와 리팩토링

[gmkim716]

과제 체크포인트

기본과제

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

심화과제

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

과제 셀프회고

이직 시기를 앞당기면서 과제에 집중하기가 다소 어려웠습니다. 그럼에도 불구하고 기본과제을 어떻게 손대야 하는건지 갈피를 잡지 못하고 많은 시간을 낭비한 것 같았습니다. 좋은 코드를 많이 보지 못했다보니 어떻게 수정해야 하는건지 용기내는데 어려움이 많았습니다. 과제 수행은 못했지만, 제출 이후에 다른 분들의 코드를 보면서 많은 인사이트를 얻어갈 수 있도록 추가적인 노력이 필요하다 생각됩니다.

과제에서 좋았던 부분

코드를 작성하는 입장과 다르게 파악해야 하는 사람의 입장에서, 약어가 오히려 편의성을 저하시킬 수 있다는 점을 느꼈습니다. 책을 읽듯이 동일한 방향으로 읽어나갈 수 있도록 시각적 움직임을 가급적 줄이는 방향으로 작성해야 하고, 문단과 주석을 적절히 구성하는 것이 다음 사람을 위한 배려임을 알게되었습니다.

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

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

머리로는 ui와 기능을 분리해야 한다는 것을 알고 있는데, 실제로 분리해내는 과정에 자신이 없었던 듯 합니다. 더 많이 고민하고 작성하신 분들의 코드를 리뷰하며 많이 얻어갈 수 있도록 하겠습니다.

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