todolist 프로젝트

sally·2022년 4월 9일
0

1주차 3th PR

[] 테스트 코드 - 의도 이해 돕기

가독성을 위한 개행

  • [x]메서드 10줄 이상시 분리/개행
  • given when then

핵심기능 먼저, 테스트 케이스 추가로 예외 처리

  • CardDto.RedirectInfo -> CardDto.Redirection

Card nextOrder(Long maxTodoOrder)

  • nextOrder()

    • 문맥상 파라미터가 없어도되는
  • setOrder( Long todoOrder)
    : set은 어디에나 들어갈 수 있다 의미 있는
    - 매개변수가 꼭 max 일 이유 없다.

    CardService readOf()

  • String.format() 에대해,

  • 메서드 호출마다 String 만들지 않을 방법

  • 상수화

CardDao

  • 상수개수 증가
  • 정리 방법 생각해보거나
  • 비슷한 성격끼리 구분하도록 개행이라도 가독성 돕도록

CardDao

  • 주석은 javadoc 형태로 하면 quick document에 보여서
    가독성에 좋다

CardDao

  • 변수명이 어려운 경우엔 한글 주석정도라도 붙여보자
    ADDED_NEXT_ORDER
profile
sally의 법칙을 따르는 bug Duck

0개의 댓글