Боль №2: Code Style

Зачем нужен стандарт?

• Облегчает code review

• Упрощает поддержку кода в команде

• Способствует повышению качества продукта

Как будем внедрять?• Для начала нужно выбрать стандарт

• Оценить масштаб бедствия в legacy code

• Подключить мониторинг изменений кода

• Помочь с настройками IDE и инструментария

• Поощрять соблюдение стандарта ^_^

Поощрять…

Доступные стандарты

• PEAR

• Zend, Symfony2, etc.

• PSR

• Drupal, WordPress, etc.

Оценим масштаб бедствия

Первый анализ проекта, который разрабатывался несколько лет и разными разработчиками: - всего нарушений 11 504 - из них 8 373 критичных - и 3 131 с нормальным приоритетом

Подключим мониторингПодружим Jenkins и PHP Code Sniffer

Сегодня это предельно просто!Поможем с настройками

Усложним задачу

https://confluence.jetbrains.com/display/PhpStorm/PHP+Code+Sniffer+in+PhpStorm

Настроим инспектирование кода

Пройдемся автоформатированием и все!*

* не забываем держать новый код под контролем и рефакторить старый

Пожинаем плоды

и никто не заметил подмены

в идеальном мире должно быть примерно так:

• https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md

• http://cs.sensiolabs.org

• https://github.com/clbustos/PHP_Beautifier

• https://github.com/squizlabs/PHP_CodeSniffer

• https://github.com/badoo/phpcf/

Полезные ссылки

придерживайся стандартов

Спасибо за внимание!

Есть вопросы?

Камиль Самигуллин какой-то разработчик

kamil@samigullin.info @ikamilsk github.com/kamilsk

Проект номер раз

Проект номер два

Еще немного ссылок

• https://plugins.jetbrains.com/plugin/7622?pr=phpStorm

• http://www.php-fig.org