2009-05-14 2 views
4

, 오류가 발생합니다,하지만 난 내 IDE에서 열 때, 나는하지 않습니다.PHP 코드 냄새 검출기는

수정 방법은 무엇입니까?

+0

몰라 도움이 될 수는 * 그것은 PHP 코드를 구글에 높은 위를 기록하고 나를 –

+0

에 * 비린내 냄새 보인다. 나는 그것이 정말로 비린내인지 궁금해. –

+6

작동 시키더라도 3 레벨 이상 들여 쓰기 된 보고서 줄, 20 줄 넘는 보고서 기능 및 4 개 이상의 매개 변수가있는 보고서 기능 만 수행합니다. 몇 개의 파일을 수동으로 스캔하여 이러한 메트릭 각각에 대한 느낌을 몇 분 만에 얻을 수 있어야합니다. 이것이 그들이 "코드 냄새"라고 부르는 이유입니다. 철저한 분석은 필요하지 않습니다. 당신은 좋은 냄새가 필요합니다. –

답변

8

코드로 확실한 코드 냄새를 감지하면 당일 중지 문제가 해결 될 수 있습니다. *takes off joke hat*

그가하고있는 일은 그가 나쁜 코드라고 생각하는 것에 몇 가지 매개 변수를 설정하는 것입니다. 그리고 그것은 틀립니다 (imo)

4 개 이상의 매개 변수가 코드 냄새입니까? 그 자신은 코드 냄새 감지기에 있습니다!

함수에 대한 20 줄 이상이 코드 냄새입니까? 이 코드의 주요 기능은 49 라인의 코드입니다!

코드 냄새는 프로그래밍 할 수있는 것이 아니며 일부 기능을 수행하는 가장 좋은 방법은 비표준 방식 (그 방법이 무엇이든)이며 다른 경우에는 그렇지 않습니다. 프랭크 파머 (Frank Farmer)가 위의 의견에서 말한 것처럼 본능을 신뢰하고 행동해야합니다. 그냥 좋은 wiff을 가져 가라.

+0

또한 토큰 구문 분석 대신 문자열 구문 분석을 사용합니다 ... –

2

코드 스니퍼를 원하면 Pears PHP_CodeSniffer을 사용하십시오. 그것은 당신이 등

PHP_CodeSniffer는 PHP5의 tokenises 스크립트와 "킁킁"PHP, 자바 스크립트, CSS 파일에 정의 된 코딩 표준의 위반을 감지 할 수 있습니다 자신의 규칙, 규정의 자신의 세트를 정의 할 수 있습니다. 코드를 깨끗하고 일관성있게 유지하는 데 필수적인 개발 도구입니다. 또한 개발자가 공통적으로 의미하는 일부 오류를 예방할 수 있습니다.

PHP_CodeSniffer의 코딩 표준은 스 니프 파일 모음입니다. 각 스 니프 파일은 코딩 표준의 한 부분 만 검사합니다.

PEAR 표준을 준수하는 단일 기능을 작성하는 것이 좋습니다. 코드베이스에서 최소한의 일관성을 유지해야합니다.

는 코드 품질을 보증하지 않습니다하지만

+0

훌륭한 제안입니다. 나는 1 년 전에 CodeSniffer를 사용하기 시작했고 결코 되돌아 보지 않았습니다. +1 –