<< 위임에 관한 70%룰 | Home | 박사란? >>

리팩토링이나 코드 리뷰에 사용할 체크리스트


리팩토링이나 코드 리뷰에 사용할 체크리스트로 사용해도 좋을법한 약어들을 공개합니다. 아직 부족하지만 같이 채워나가요. ^^
  • DC(Duplicate Code) - 중복된 코드(중복된 코드나 조건들).
  • BPC(Bad Performance Code) - 성능에 좋지 않는 코드.
  • BN(Bad Name) - 변수나 클래스, 함수 등에 너무 길거나 가독성 떨어지고 일관되지 않은 이름.
  • VS(Violates Specifications) - 가이드나 지침을 따르지 않음.
  • TCL(Too Complicated/Larged) - 불필요하거나 너무 크고 복잡한 함수나 클래스.
  • LP(Long Parameter) - 불필요하거나 너무 긴 인자값들.
  • II(Inconsistent Indentation) - 비일관적인 들여쓰기.
  • NEC(Not Enough Test Cases) - 테스트 케이스가 너무 적음.
  • TMIS(Too Much If/Switch) - 조건 분기가 많음.
  • NC(No Comments) - 주석이 너무 없음.
  • SS(Shotgun Surgery) - 하나의 함수나 클래스를 변경했는데 많은 것들이 영향을 받는다.
  • FE(Feature Envy) - 다른 클래스의 속성과 메소드들을 사용해 다른 클래스와 강한 관계가 성립한다. 모듈(클래스) 독립적이지 못하다.
  • BC(Boilerplate Code) - Getter/Setter 등 맹복적으로 추가되는 반복적인 코드들.
  • TE(Throws Exception) - 예외 처리.
  • NG(Not General) - 너무 범용적이지 않음.
  • DD(Dead Code) - 사용되지 않거나 미리 만들어진, 불필요한 코드들.
  • MN(Magic Number) - 매직 넘버.
  • PD(Platform Dependency) - 플랫폼(언어, 환경 등) 종속성이 있다.
  • AOS(Abuse Open Source) - 오픈 소스 남용.
추가적으로 생각나는 것은 지속적으로 업데이트 하겠습니다. 그리고 여기에 추가할 좋은 약어들이 있으면 코멘트(댓글)나 이메일 남겨주시면 고맙겠습니다. ^^


Avatar: 룬룬

Re: 리팩토링이나 코드 리뷰에 사용할 체크리스트

 어디선가 많이 보던 약어들입니다. :)

저도 얼마전에 Clean Code를 다 보았는데, 책 마지막에 체크리스트? 저자는 코드에서 냄새가 난다고 표현하는 것들을 잘 정리해놓았길래 블로그에 써두었습니다.

혹시 참고자료가 될런지 모르겠습니다.

http://www.navisphere.net/?p=2013

 

 

 

 

Avatar: 미물

Re: 리팩토링이나 코드 리뷰에 사용할 체크리스트

 네 <span style="font-family: 'Trebuchet MS', 'Lucida Grande', Tahoma, Verdana, Arial, sans-serif; line-height: 18px;">Clean Code에서도 참조를 했고 그중에서 자주 회자되는 것들을 정리하고 제가 생각했던것들을 첨언해서 만들었습니다. ^^</span>


Add a comment Send a TrackBack