[TIL] 코드스쿼드 팀 프로젝트 29일차

오늘의 활동

• PR 후 리뷰
• Label, Milestone API
  • 리팩토링
  • 예외 처리 시도
  • 새로운 프론트 요구사항 적용
• 백준 2470번

느낀 점

• 오늘은 지금까지 피드백 받은 부분을 토대로 리팩토링을 진행하였다. 리팩토링을 하면서 갈아엎고 싶은 부분들도 많이 보이고, 실제로 수정한 부분도 꽤 많았다.
• 어느 정도의 리팩토링을 진행한 후, 그동안 RuntimeError로 미뤄두었던 예외 처리를 시도해 보았다. 동료들의 코드를 참고해서 짜봤는데 계속 내가 원하는 응답이 나오질 않았다. 분명 내가 예외를 발생시킨 부분에서 원하는 응답이 발생해야 하는데 계속 콘솔에서만 출력됐다. 완벽하게 동료들과 똑같은 로직으로 구현했는데 왜 이러는 건지는 잘 모르겠다.
• 프론트 Jay의 요청으로 마일스톤 GET 요청 응답의 stauts를 milestones 배열 안이 아니라 별도의 객체로 빼는 로직을 재작성했다. 어제 깨부한 것이 도움이 돼서 이 부분은 10분 안에 간단하게 할 수 있었다.
• 이번주 코드를 작성하면서 조인문은 하나도 작성을 하지 않았는데, 그동안 조인을 사용하지 않고 따로 호출하는 것도 좋은 방법이라는 것을 이해 자체를 하지 못했는데 이번에 그 방식으로 직접 진행해 보면서 서비스쪽에서는 더 더러워지더라도 훨씬 깔끔하다는 점을 알 수 있었다. 하지만 그 더러워지는 것이 가장 큰 문제이다. 메인 서비스 메서드 안에 3개의 private 메서드가 들어있는 코드라서 테스트 할 때도 그렇고 아주 최악의 코드라고 생각했다. 나중에 리팩토링할 수 있으면 한 번 해보자!

내일 할 일

• 최종 배포
• 전체 공유
• 스쿼드 세션
• 마스터 클래스