php code детектор запаха - PullRequest
4 голосов
/ 14 мая 2009

Относительно php-кода, детектор запаха на http://c2.com/cgi/wiki?DetectingCodeSmellsInPhp

Когда я открываю его в своем веб-браузере, я получаю сообщение об ошибке, но когда я открываю его в своей IDE, я не делаю.

Как это исправить?

Ответы [ 2 ]

8 голосов
/ 15 мая 2009

Обнаружение запаха окончательного кода с помощью кода станет возможным в тот же день, когда проблема с остановкой будет устранена. *takes off joke hat*

Он просто устанавливает некоторые параметры для того, что он считает плохим кодом. И это неправильно (имо)

При наличии равных или более 4 параметров кодовый запах? У него самого в коде есть детектор запаха!

Более 20 строк для функции - это запах кода? Основная функция в этом коде детектор запаха составляет 49 строк кода!

Запах кода - это не то, что вы можете программировать, в некоторых случаях лучший способ выполнить какую-либо функцию - это нестандартный способ (каким бы он ни был), а в других - нет. Вы просто должны доверять своему инстинкту и поступать так, как сказал Фрэнк Фармер в комментариях выше. Просто возьми хороший вайф.

2 голосов
/ 15 мая 2009

, если вы хотите перехватить код, используйте Pears PHP_CodeSniffer . он позволяет вам определять свои собственные правила, свой собственный набор правил и т. д.

PHP_CodeSniffer - это скрипт PHP5, который маркирует и «нюхает» файлы PHP, JavaScript и CSS для обнаружения нарушений определенного стандарта кодирования. Это важный инструмент разработки, который гарантирует, что ваш код остается чистым и непротиворечивым. Это также может помочь предотвратить некоторые распространенные семантические ошибки, допущенные разработчиками.

Стандарт кодирования в PHP_CodeSniffer представляет собой набор файлов сниффа. Каждый снифф-файл проверяет только одну часть стандарта кодирования.

это интересно использовать (попробуйте написать одну функцию, которая соответствует стандартам PEAR) и обеспечивает хоть какую-то согласованность в кодовой базе.

это не гарантирует качество кода, но может помочь

...