diff --git "a/15\354\236\245/\354\240\204\355\230\270\354\230\201.md" "b/15\354\236\245/\354\240\204\355\230\270\354\230\201.md" new file mode 100644 index 0000000..da55a39 --- /dev/null +++ "b/15\354\236\245/\354\240\204\355\230\270\354\230\201.md" @@ -0,0 +1,7 @@ +# 느낀점 + +14장까지 나왔던 클린코드 규칙을 적용해 JUnit 소스코드를 분석했다. +- fPrefix 등 모호한 변수명을 prefixLength 로 수정 +- 큰 함수를 작은 함수로 분해 +- 조건문 + - 부정문을 긍정문으로 수정 (가독성 향상) \ No newline at end of file diff --git "a/16\354\236\245/\354\240\204\355\230\270\354\230\201.md" "b/16\354\236\245/\354\240\204\355\230\270\354\230\201.md" new file mode 100644 index 0000000..dd865ba --- /dev/null +++ "b/16\354\236\245/\354\240\204\355\230\270\354\230\201.md" @@ -0,0 +1,11 @@ +# 느낀점 + +15장과 비슷하게, 리팩토링 + +- 안쓰는 주석 정리 +- 상수 및 열거형 수정 + - 불필요하게 상속받은 클래스를 Enum으로 수정 +- 메서드명을 더 서술적이게 수정 +- SerialDate를 추상 클래스로 변경 + - 이름 역시 추상클래스에 맞도록 추상화(DayDate) + diff --git "a/17\354\236\245/\354\240\204\355\230\270\354\230\201.md" "b/17\354\236\245/\354\240\204\355\230\270\354\230\201.md" new file mode 100644 index 0000000..d347f35 --- /dev/null +++ "b/17\354\236\245/\354\240\204\355\230\270\354\230\201.md" @@ -0,0 +1,31 @@ +# 공감하는 안좋은 습관 + +C2: 쓸모없는 주석 +- 코드와 같은 내용을 반복하는 주석 + +C4: 성의없는 주석 +- 대충 작성한 불친절한 주석 +- 작성자만 이해할 수 있는 애매한 설명 + +E1: 여러 단계를 거쳐야 하는 빌드 +- 소스코드 관리 시스템에서 간단한 명령 하나로 빌드가 안 되는 경우 + +F1: 너무 많은 인수 +- 함수 인수는 적을수록 좋음 + +F3: 플래그 인수 +- boolean 인수로 함수 동작을 제어하는 것 +- 함수가 한 가지 이상의 일을 한다는 증거 + +G2: 당연한 동작을 구현하지 않음 +- 함수명으로 예상되는 동작을 구현하지 않는 경우 + +G5: 중복 +- 똑같은 코드가 여러 곳에 반복 + +G11: 일관성 부족 +- 어떤 방식으로 뭔가를 구현했다면 유사한 기능도 같은 방식으로 구현해야 함 + +G25: 매직 숫자를 상수로 교체하지 않음 +- 의미있는 숫자를 그대로 사용하는 경우 +