aronalee / swm_14_div_study

</div>팀의 클린코드 스터디
1 stars 0 forks source link

[5장] 형식 맞추기 #6

Closed aronalee closed 1 year ago

aronalee commented 1 year ago

요약

코드를 신문기사 쓰듯이 써라


생각

연산자 우선순위를 공백으로 표현하는 방법은 신기했다. 기존에 ide 코드 포맷팅을 사용하다보니 연산자 우선순위를 공백으로 표현하는 건 신기했다. 우선순위로 바뀐 것을 보니 읽기가 더 쉬워졌다. 이 방법은 연산자를 사용할 때면 써보고 싶다는 생각을 하였다.

Roddyd94 commented 1 year ago

필요성

역시나 가독성

방법

행 길이 맞추기

약 200줄의 파일 길이로도 대규모 시스템 구축 가능 빈 행: 개념 단위 분리 및 유사 개념 밀집 수직 거리: 연관성 높은 개념 클러스터링

  • 변수: 사용하는 위치에 근접한 위치
  • 로컬 변수: 각 함수 시작 부분(함수가 짧기 때문)
  • 루프 제어 변수: 루프문 내부
  • 인스턴스 변수: 언어에 따라 클래스 시작/마지막 부분
  • 종속 함수: 가능한 자신이 호출된 함수 아래

개념 친화도: 높을수록 가까이 배치

열 길이 맞추기

가로 공백: 개념(연산자, 함수, 인수) 구분 가로 정렬: deprecated 들여쓰기: 범위 계층 표현 빈 반복문: 아래 행에 세미콜론

팀 규칙 따르기

생각

형식이래서 전체적인 코딩 컨벤션을 떠올렸는데 모양새에 관한 이야기였다. 모양새는 알게 모르게 적용하던 내용이라 신기했다. 파일의 크기와 파일의 개수는 역함수 관계이므로 둘을 적당히 고려해야 한다고 생각했었다. 하지만 파일 하나가 신문 기사 하나라는 비유를 통해 파일의 개수가 늘어나더라도 파일 개수는 색인에 맡기고 파일의 크기를 적당히 작게 유지하는 것이 좋겠다는 생각을 했다. 팀 규칙을 예시로 때워 아쉬웠다.

chj990707 commented 1 year ago

생각

다른 사람이 짠 코드를 분석할 때 대부분의 조작이 함수나 변수의 선언/용례를 찾기였던 것 같다. 이 장에서 제시된 규칙을 따르면 수고를 확실히 줄일 수 있을 것이다. 더해서, 파일의 길이를 짧게 유지하는 규칙도 개별 파일을 읽는 데에는 크게 도움이 되지만 실제로 코드를 짤 때는 기능이 하나씩 추가되는 일이 많아 자주 상기해야 할 것이다.