[Sparta] 내일배움캠프 TIL 13일차

 

오늘의 핵심 키워드

• 과제 피드백

public static boolean validationTestCnt(int inputTestCnt) {
        if (0 >= inputTestCnt || inputTestCnt > 10){
            return false;
        }
        return true;

피드백 내용

>> 1. 안티 패턴사용 :  숫자 리터럴 상수의 경우 사용시 enum 등 상수화로 사용하는게 좋다(의미부여!)

(이유: 처음보는사람한테 이해가 안감)

>> 2. 메서드 리팩토링 필요 : 메서드 하나하나의 길이가 너무 긴것 역시 좋지않다, 고민 후 가능하면 작게 작성하는 습관!

>> 3. 클래스 역할 분할 : application 클래스의 경우 너무 메서드 및 역할이 많고 그외 manage등 클래스 역시

조금더 역할을 작게 만들수 있을것 같음

>> 4. 문자열 비교시 기준이 되는 값은 정답값으로 비교해야 더 안전

<< status는 사용자 입력값 >>
ex)
status.equals("yellow") 보단
"yellow".equals(status)