Как по мне, так маниакальностью попахивает и избыток необъективности ))
С ув. Роман Юсуфханов [email protected] <mailto:[email protected]> моб.: +79310075149 > 11 февр. 2016 г., в 17:37, Ivan Petrov <[email protected]> написал(а): > > >>> вот этот маниакальный вынос констант в константы мне не нравится. > >> Непонятно в чём тут видится маниакальность и в чём суть аргумента "мне не >> нравится". > > маниакальность - это когда выносят константы которые в виде числа > читаются лучше чем в виде текстового идентификатора. > > > допустим широта и долгота > > croak "Неверная долгота" unless $lon >= -180 and $lon <= 180; > croak "Неверная широта" unless $lat >= -90 and $lat <= 90; > > Тут в коде стоит 4 магических числа (90, -90, 180 и -180), но я > утверждаю что вынос их в константы будет вредить коду. > > подобных примеров привести можно массу > > int($interval / 3600) > int($interval / 86400) > > константа 3600 например сама за себя говорит что она означает. > равно как и константы 90, 180, 270, 360, 60, 3600, 86400, 24, 1900, > > > Пи - особый случай. 3.1415926 сразу дает понять что это пи и если в > коде встречается не более пары раз на экран, то выносить ее в > константы как минимум не полезно. > если же код - какая-то голимая математика через слово вспоминающая PI, > то тут конечно можно и константу определить (экспортировать). > >> Однако если говорить о Perl::Critic, то он настраивается под себя конфигом, а >> сейчас я правлю утилиту так чтобы можно было даже на каждый проект отдельный >> конфиг иметь > > да я не вашу утилиту обсуждаю, а некий "общепринятый паттерн > хорошести" > > :) > -- > Moscow.pm mailing list > [email protected] | http://moscow.pm.org
-- Moscow.pm mailing list [email protected] | http://moscow.pm.org
