10.14~10.18 21조 KPT회고

Keep

👍 테스트 코드

• 프로젝트 엔티티 설계 후 테스트 서비스를 별도로 작성하여 테스트용 데이터를 DB에 한번에 입력했다. 개발 과정에서 데이터를 입력하는 시간을 줄일 수 있었고 연관되어있는 데이터 때문에 발생하는 개발 지연을 방지할 수 있었다.

👍 작은 작업 단위로 Merge

• 작업을 작은 단위로 분리하여 작업이 완료될 때마다 자주 merge를 진행한 덕분에, 코드 충돌을 크게 줄일 수 있었다. 작은 범위에서 발생하는 변경사항은 관리가 쉬워지고, 충돌이 발생하더라도 이를 신속하게 해결할 수 있었기에 협업 효율성이 높아졌다.

👍 코드 리뷰

• 코드를 Merge하는 경우 팀원이 모두 모여 충돌을 함께 확인했다. 충돌을 해결하는 과정에서 어떤 방식으로 가는게 맞는지 회의를 진행했고 각자 코드를 개발하는 과정에도 방향성을 적용할 수 있었다.

Problem - Try

⚠️ 사용자 ADMIN 가입 제한

• 사용자가 회원가입시 자신의 권한을 ADMIN으로 직접 적어 가입할 수 있는 방식으로 실제 서비스하기에는 위험이 있다.
• Try
  • 서버에서 발급하는 토큰값을 가진 사용자만 ADMIN으로 가입할 수 있도록 제한하면 무분별한 ADMIN사용자 가입을 막을 수 있다.

⚠️ Exception 메시지 통합

• 예외상황 에서의 에러 메세지를 모두 하드코딩으로 작업하여 동일한 상황에서도 다른 메시지가 나오고 기능 수정 시 일괄적인 에러 메세지 수정이 불가능하다.
• Try
  • 에러 메세지를 하나의 Enum파일에 모아 관리하면 같은 예외 상황이 발생하는 경우 일관성 있는 메세지를 출력할 수 있고, 한번에 에러 메세지를 수정할 수 있다.

⚠️ 슬랙 알림 시스템의 유지보수, 확장성 한계

• 현재 시스템은 특정 메소드 호출 후 슬랙 알림을 보내는 방식으로 각 서비스가 강하게 결합된 구조를 가지고 있어 유지보수와 확장성에 한계가 있다.
• Try
  • EDA를 도입해 이벤트 기반으로 알림을 분리하여 각 기능이 독립적으로 작동할 수 있도록 개선한다. 메소드 실행 후 이벤트가 발생하면 이를 감지하는 이벤트 리스너가 비동기적으로 슬랙 알림을 처리하는 방식을 적용하면 유지보수와 확장성을 확보할 수 있다.

⚠️ README 개선 방향

• 현재 git의 readme가 내용적 면으로 부족한 것 같아, 추가적인 부분을 서술해주는 것이 좋을 것 같다.
• Try
  • 구현한 부분의 성공 여부 추가, 각 기능이 정상적으로 구현되었는지 README에 명확히 표시. 예를 들어 ‘카드 내용이 변경될 때마다 슬랙 알림이 성상적으로 발생함’
  • 가시성 있는 내용 추가, 프로젝트 ERD나 와이어프레임을 링크와 한장요약등을 올리는 게 좋아보인다.