탁구 게임 KPT 회고

나는 이전에 Springboot로 탁구 서비스 백엔드 로직을 구현해본 적이 있다.

 

https://github.com/Leehyeonju0219/pingpong-game

GitHub - Leehyeonju0219/pingpong-game: 탁구 게임 서비스의 백엔드 로직을 구현한 코드입니다.

 

나는 이 프로젝트 코드를 리팩토링 해보고자 한다.

이 프로젝트 당시 급하게 코드를 짜느라 클린코드를 생각하지 못했다.. 물론 변명일 뿐... 반성한다....

 

이 당시에 Swagger api를 연동했는데 제대로 작동하지 않았던 치명적 오류도 있었고,

코드가 대체적으로 깔끔하지 못했던 문제도 있었다.

 

전체적으로 클린코드 원칙을 지켜서 리팩토링 해보기 전에!!

이 미니 프로젝트에 대해 KPT 회고를 해보려고 한다.

회고를 먼저하고 리팩토링은 다음 게시물로 올리겠다!

 

KEEP

• README.md 파일을 잘 구성한 것 같다. 깃허브 레포에 들어왔을 때 README.md만 보고도 이 서비스에 대해 충분히 이해할 수 있을 것으로 본다.
• 커밋 컨벤션을 나름 지켜서 코드를 올렸다. 지금은 git-emoji 방식과 유다시티 방식을 쓰고 있지만 이 프로젝트 당시에는 [UPDATE], [ADD] 이러한 컨벤션을 사용했다.(다른 사람이 쓰는 것을 따라썼던 거라 이 컨벤션에 따로 이름이 있는지는 잘 모르겠다..)

PROBLEM

• Swagger API를 사용하지 않았다. 사실 사용했지만 제대로 작동하지 않아서 Swagger API를 확인할 수 없는 상태이다. API 명세서를 잘 작성했다면 더 보기 좋았을 것이라고 생각해서 Swagger API를 연동할 필요가 있다고 느껴진다.
• 예외 처리가 조금 부족하다고 생각된다. 기본적인 예외 처리는 되어 있지만 더 많은 테스트를 통해서 결함을 찾아낼 수 있을 것 같다.

TRY

• Swagger API 연동 시 발생했던 에러를 찾아 해결한다.
• 로직 상에 발생할 수 있는 더 많은 예외 상황을 찾아서 예외 처리를 더 세부적으로 하면 좋을 것 같다.

 

---

회고를 하면서 예외처리가 부족하다는 것을 뼈저리게 느꼈다...

그리고 API 명세서가 없다는 것 또한 치명적인 문제점이라고 생각했다.

이제 이 회고를 바탕으로 리팩토링을 해보려고 한다. 리팩토링을 하고나서 코드와 함께 게시물 작성해보겠다!!