Проверка качества кода PHP с помощью Scrutinizer

Ага. Так что это сложность, которая вызывает проблему? Хм, посмотрим «Как исправить».

Теперь, хотя мне очень нравится такой подход, как давать советы по ссылкам, просто ничего нельзя сделать для этого класса. Файл представляет собой одну большую коллекцию получателей, и нет никакого способа извлечь из него класс, подкласс или интерфейс. К сожалению, не существует способа игнорировать только эту инспекцию, как в случае с проблемами.

На экране «Код» есть еще одна ссылка, возможно, менее очевидная: «Горячие точки».

Горячие точки отображают наиболее «оптимизируемые» области кода — те, которые, как предполагает Scrutinizer, дадут наибольший прирост качества при наименьшем объеме работы:

В моем случае это был вышеупомянутый (нефиксированный) класс Product и два метода в абстрактном классе Api . Интересный. Давайте посмотрим на buildUrl метода buildUrl .

Хм. Слишком много условий? Метод buildUrl берет различные параметры из API, который его вызывает, и создает из них URL-адрес, поэтому вполне естественно, что он имеет «много» управляющих структур. Можно извлечь циклы в такие методы, как buildFieldString и buildOptionsString , но зачем? Это единственный метод, который нуждается в построении строк, поэтому мне не нужно вносить дополнительные ненужные издержки, чтобы исправить это.

Другой метод, __construct безусловно, оставляет желать лучшего. Давайте улучшим это немного и изменим это от этого:

 public function __construct($url) { if (!is_string($url)) { throw new \InvalidArgumentException('URL param must be a string.'); } $url = trim($url); if (strlen($url) < 4) { throw new \InvalidArgumentException('URL must be at least four characters in length'); } if ($parts = parse_url($url)) { if (!isset($parts["scheme"])) { $url = "http://$url"; } } $filtered_url = filter_var($url, FILTER_VALIDATE_URL); if (false === $filtered_url) { throw new \InvalidArgumentException('You provided an invalid URL: ' . $url); } $this->url = $filtered_url; } 

к этому:

 public function __construct($url) { $url = trim((string)$url); if (strlen($url) < 4) { throw new \InvalidArgumentException( 'URL must be a string of at least four characters in length' ); } $url = (isset(parse_url($url)['scheme'])) ? $url : "http://$url"; $filtered_url = filter_var($url, FILTER_VALIDATE_URL); if (!$filtered_url) { throw new \InvalidArgumentException( 'You provided an invalid URL: ' . $url ); } $this->url = $filtered_url; } 

Теперь, если мы запустим перестроение путем фиксации и нажатия, мы получим следующее:

Качество повысилось с B до A для метода __construct — успех!

Инспекции и статистика

Два оставшихся экрана — это «Инспекции», в которых перечислены все проведенные проверки и их результаты, и «Статистика и тренды», панель графиков, отображающая визуальные подсказки о том, как качество вашего кода прогрессирует (или ухудшается) с течением времени — я буду оставь это на усмотрение тебя.

Вывод

Scrutinizer — это мощный инструмент, обеспечивающий высочайшее качество вашего PHP-кода. Его очень легко настроить, и как только он запущен и работает, требуется минимальное взаимодействие, чтобы оставаться активным и быть в курсе событий. Хотя уровни ценообразования не слишком доступны для частных лиц и индивидуальных разработчиков с частными проектами (начиная с 50 евро, а не 19 евро, как указано на целевой странице), они очень доступны для компаний, и уровень бесплатного доступа более достаточно для проекта с открытым исходным кодом.

Вы пробовали Scrutinizer? Какие услуги качества кода вы используете? Может быть, их конкуренция, Code Climate? Дайте нам знать.