[StackOverFlowClone Refactoring] Sprint 2 회고

• Github Repository
• 23.02.17 ~ 03.05 진행

◎ 학습 하면서 느꼈던 여러 감정들과 생각

• 다른 것들을 배우면서 "이걸 적용하면 좋겠는데?"라고 생각이 드는 것이 많았다.

  • Test에서 Object Mother
  • Service Layer에서 한 class의 역할을 작게 하는 것

• 앞에 할 일도 충분히 많았는데 지금 배운 것을 보다보니 내 코드가 많이 허접하게 느껴졌다.

  • 그래서 조급함이 많이 느꼈다.
  • 당장 앞에도 배울것이 많은데 다른 것들을 쳐다보면서 시간 낭비를 했다.

◎ 내가 무엇을 잘했고 무엇을 못했는지

• 잘한 점

  • Service Layer마다 Refactoring Convention에 맞춰 길이를 줄임
  • MemberService에 이메일 전송 관련 정보를 분리함
    • 너무 많은 역할을 쥐어져 있었다고 생각함

• 못한 점

  • 초기에는 모든 Service Layer의 Refactoring을 진행하려 했으나, 조금씩 미루다보니 결국 양을 줄임
  • 일부 Test는 Test Code가 많이 긺 (추후 Object Mother를 적용하여 줄이면 될 것 같음)

◎ 수정할 점

• 매 업무마다 하루 진행양을 정하고 지키기
• 배우고 적용할 내용들은 한쪽에 정리해두고 나중에 적용해보기 (일단 앞에 정해진 업무부터 처리하기)

◎ Sprint 3 해야할 일들

• Follow, Vote 관련 Service Layer Refactoring

  • Service 관련 code의 Unit Test 작성
  • Service 관련 code를 Code Convention에 맞추어 리팩토링 실시
  • Service Layer 분리

• 한가지 Service에 많은 책임이 할당되어 있어 분리 실시

  • interface 제거 예정 (다른 방식으로 구현될 일이 없다고 생각함)

• helper Refactoring

  • upload : Unit test가 가능하도록 구조 변경
  • password : interface 구현 (다른 방법으로도 구현 가능하다고 생각)
  • email : 필요없는 파일 제거, TemporaryEmailSender interface 구현, Unit test가 가능하도록 구조 변경

◎ 추후 해야 할 일

• Test Code 정리
  • 필요없는 파일 및 코드 제거
  • Object Mother 적용하여 코드 줄이기
• Controller Layer, Repository Layer 테스트 코드 작성
• 효율 테스트하는 도구 사용해보기
• DB index 설정으로 효율 좋게 만들기 (많이 공부하기)