Объективные критерии качества Perl кода

[@Sasha_Pitenin]

Что то статья короткая, можно же было привести как нужно писать, а как нет.

[@worldmind]

Да это на многотомник тянет, одни только PBP это уже книга.

[@pavelodintsov]

Как выжимку из PBP можно использовать вот это: www.reg.ru/coding_standards Довольно грамотные рекомендации.

Но главная рекомендация для поддержки крупного проекта на Perl — никогда в жизни не писать на Perl что-либо длиннее сотни строк.

[@dmrt]

В одном модуле не больше сотни строк?

[@pavelodintsov]

Там написано «в одной функции».

[@igorsd]

8.2. Рефакторинг

Там.

[@pavelodintsov]

Там — это «тут»: www.reg.ru/coding_standards

[@zdm001]

Лучше использовать Perl::Critic + Perl::Tidy, например, как плагин для vim.

[@pavelodintsov]

В перл критике много упоротых вещей — его нужно употреблять ограничено :)

[@worldmind]

он конфигурится, как минимум severity, да и отдельные правила можно отключать, хотя по мне дефолтные настройки нормальные

[@worldmind]

чем лучше?

upd, а речь о том что лучше чем стандарты reg.ru, тогда да

[@zdm001]

Лучше тем, например, что Perl::Tidy автоматически форматирует код по правилам PBP, со всеми отступами и пробелами. Руками это делать не реально.

[@knutov]

Никак не могу согласиться. Имея несколько больших проектов годами делаю хорошее форматирование руками.

Сложности нет, после выбора определенного стиля — период привыкания маленький, дальше всё делается автоматически на уровне рефлексов.

[@zdm001]

С Perl::tidy можно любой кусок кода, не ваш, например, привести к стандарту.
И незаменимо для команд разработчков.

[@igorsd]

sudo make install

А отсутствие Makefile, никому не помешает?

[@worldmind]

Да, косяк, поправил, это потому что Makefile обычно генерится и он в .gitignore, а тут пока руками сделал, планирую скоро на стандартную схему перевести

[@worldmind]

Перевёл на стандартную для перловых модулей схему с Makefile.PL
Если нет make это может означать только что модули не ставятся с CPAN, а только из репозиториев, в таком случаем несложно сделать пакет с помощью checkinstall на другой машине

[@worldmind]

) про такое я даже и не подумал, как там с перлом на винде даже не представляю, юзал в студенчестве что-то (вероятно active perl), но без установки CPAN модулей.

[@worldmind]

Да, сапожник без сапог, качество самого скрипта пока хромает, я тестил на реальном проекте, там почти все тесты проходят нормально.
А сам скрипт сейчас перерабатываю, по результатам переработки тестирование самого себя заработает как надо.

[@worldmind]

Теперь сам себя протестит (может только make manifest потребоваться), хотя я для него часть тестов отключил — покрытие тестами тут почти не нужно, ибо модуль это обёртка над кучей модулей у которых свои тесты, а нормальный POD пока лень писать.

P.S. установка изменилась, теперь надо традиционный perl Makefile.PL делать