HomeAboutMeBlogGuest
© 2025 Sejin Cha. All rights reserved.
Built with Next.js, deployed on Vercel
[팀]조규현 공간(1-2)
[팀]조규현 공간(1-2)
/
🎫
인터파뀨
/
🏷️
5차 회의 - 리팩토링 목록 리스트업
🏷️

5차 회의 - 리팩토링 목록 리스트업

  • 작성자 : 김다희
  • 참여자 : 김다희, 김휘년, 이상민
  • 날짜 : 21.11.02.화
  • 방식 : 코드윗미

리팩토링 대상
  • 필드 사이에 개행 (레코드 포함)
  • 롬복 안쓰여진 코드 수정
  • 각 레이어 메서드명
    • 레이어 마다 꼭 이름이 동일할 필요는 없다.
    • 서비스에서 by~ 뒤는 생략 가능하다.
    • 풀네임으로 간다.
  • dto 네이밍
    • ~Response
    • ~Request
  • final Long id
    • 붙이는걸로
  • private final 로 잘 되어있는지
    • 확인만
  • 라인브레이크 어디까지 허용할것인지
  • 엔드포인드 public static final String으로 관리해야할까?
    • 각 controller에서 관리
  • var 사용 여부
    • 사용 안함
  • PerformanceRepository랑 UserPerformanceRepository가 통일
  • Message Format vs String + 연산자
    • Message Format
  • 빈 중괄호 {} 할건지 { enter } 할건지
  • 주석 작성을 어디까지 허용할것인지
    • 미래에 해야할 일에 대해서 작성
  • 회차 좌석 도메인 확정
  • 에러 코드 영어 vs 한글
    • 영어
  • 패키지 구조
    • 변경 전
      notion image
  • enum에서 예외를 던지면 추가 정보는 어떻게 담을까 → 추후 고민
  • this.
    • 메서드만 사용한다.
  • {} 없는 코드
  • 서비스 메서드 명 ex) getRoundSeatById vs getRoundSeat
    • getRoundSeat
  • final @Valid @RequestBody 순서로 ~