<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Как по мне, так маниакальностью попахивает и избыток необъективности )) <br class=""><div class="">
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><div class=""><br class="Apple-interchange-newline">С ув. Роман Юсуфханов <a href="mailto:r.yusufkhanov@gmail.com" class="">r.yusufkhanov@gmail.com</a></div><div class="">моб.: +79310075149</div></div><div class="" style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br class=""></div><br class="Apple-interchange-newline">
</div>
<br class=""><div style=""><blockquote type="cite" class=""><div class="">11 февр. 2016 г., в 17:37, Ivan Petrov <<a href="mailto:i.petro.77.00@gmail.com" class="">i.petro.77.00@gmail.com</a>> написал(а):</div><br class="Apple-interchange-newline"><div class=""><div class=""><br class=""><blockquote type="cite" class=""><blockquote type="cite" class="">вот этот маниакальный вынос констант в константы мне не нравится.<br class=""></blockquote></blockquote><br class=""><blockquote type="cite" class="">Непонятно в чём тут видится маниакальность и в чём суть аргумента "мне не<br class="">нравится".<br class=""></blockquote><br class="">маниакальность - это когда выносят константы которые в виде числа<br class="">читаются лучше чем в виде текстового идентификатора.<br class=""><br class=""><br class="">допустим широта и долгота<br class=""><br class="">croak "Неверная долгота" unless $lon >= -180 and $lon <= 180;<br class="">croak "Неверная широта" unless $lat >= -90 and $lat <= 90;<br class=""><br class="">Тут в коде стоит 4 магических числа (90, -90, 180 и -180), но я<br class="">утверждаю что вынос их в константы будет вредить коду.<br class=""><br class="">подобных примеров привести можно массу<br class=""><br class="">int($interval / 3600)<br class="">int($interval / 86400)<br class=""><br class="">константа 3600 например сама за себя говорит что она означает.<br class="">равно как и константы 90, 180, 270, 360, 60, 3600, 86400, 24, 1900,<br class=""><br class=""><br class="">Пи - особый случай. 3.1415926 сразу дает понять что это пи и если в<br class="">коде встречается не более пары раз на экран, то выносить ее в<br class="">константы как минимум не полезно.<br class="">если же код - какая-то голимая математика через слово вспоминающая PI,<br class="">то тут конечно можно и константу определить (экспортировать).<br class=""><br class=""><blockquote type="cite" class="">Однако если говорить о Perl::Critic, то он настраивается под себя конфигом, а<br class="">сейчас я правлю утилиту так чтобы можно было даже на каждый проект отдельный<br class="">конфиг иметь<br class=""></blockquote><br class="">да я не вашу утилиту обсуждаю, а некий "общепринятый паттерн<br class="">хорошести"<br class=""><br class="">:)<br class="">-- <br class="">Moscow.pm mailing list<br class=""><a href="mailto:moscow-pm@pm.org" class="">moscow-pm@pm.org</a> | <a href="http://moscow.pm.org" class="">http://moscow.pm.org</a><br class=""></div></div></blockquote></div><br class=""></body></html>