roving

코드 리뷰좀 부탁드릴게요

일단 정답은 맞긴 했는데 뭔가 코드가 지저분한 느낌이 들어서요..
좀더 간결하게 짤 수 없을까요? 답변 부탁드리겠습니다 (_ _)

작성중인 코드
function daysInMonth( month ){
    // 여기에 switch를 이용해 각 달의 날짜수를 반환하는 프로그램을 작성하세요.
  switch(month) {
    case 2:
            return 28;
        break;
    case 4:
            return 30;
        break;
    case 6:
            return 30;
        break;
    case 9:
            return 30;
        break;
    case 11:
            return 30;
        break;
    default :
            return 31;
  }
}
1 개의 답변
정두식

우선 break는 필요 없습니다. break는 switch문을 빠져나가기 위해 사용하는데 return하면 그 함수 자체를 빠져나가기 때문입니다.

    case 2:
      return 28;

그리고 여러개의 case를 하나로 묶어서 다음과 같이 사용할 수 있습니다.

    case 4:
    case 6:      
    case 9:      
    case 11:            
      return 30;

이렇게 하면 더 깔끔해 지겠네요. 31일이 있는 달은 default로 처리하신 부분은 좋은것 같아요.

  • roving
    감사합니다!! ― roving 2017.8.18 11:24
답변 쓰기
이 입력폼은 마크다운 문법을 지원합니다. 마크다운 가이드 를 참고하세요.