한달 간 피드백 받은 내용 정리

1. 강제 언레핑 사용하지 마세요. 2.

커밋은 변경하는 최소 작업단위로 작게 나눠야합니다. 3. 특정한 함수 내부에서 exit() 처리하는 것은 좋지 않습니다. 4.

메서드 이름은 동사로 정하세요. 5. enum을 포함해서 타입을 만들때는 대문자로 시작하도록 만드는게 규칙입니다. 6. 서로 상관없는 값이니까 중첩해서 if 구문을 넣을 필요는 없습니다. 7.

변수 이름과 함수 이름이 동일한 것은 어색하네요. 8. inputFromUser()는 (Int, Int) 튜플을 리턴하는데, ladderMake(humanNumber, heightMax)는 매개변수를 따로따로 전달하는 이유가 궁금합니다. 튜플을 그대로 넘기는 건 왜 선택하지 않았나요?

9. 코드리뷰는 제가 일방적으로 고치라고 지적하는 방식이 아닙니다.....

원문링크 : 한달 간 피드백 받은 내용 정리