01.07.2012 13:02, Igor Chumak пишет: > > 30.06.2012 11:39 пользователь Артём Н. <artio...@yandex.ru > <mailto:artio...@yandex.ru>> написал: >> >> 30.06.2012 12:00, Igor Chumak пишет: >> > Сам код, как по мне, не ужасен;) Слегка удивляют конструкции вида >> > if test $UDCS_PATH >> Отсутствие пустых строк между if, > Придираетесь ;) на таких размерах блоков if..fi границу и так видно Мне не видно. Это не придирки. Дело не в границе. Там много блоков. Всё "сливается".
>> конструкция, типа then : else, вместо [ ! -n >> $VAR ] && do . > Тогда уж есть [ -z $var Ну да, тоже верно. >> Прямые условия вместо VAR=${def:-VAR} > Зато наглядно. Нет башизмам;) Это не башизм. Такая конструкция работает в большом числе оболочек с bash-подобным синтаксисом: ksh (на который рассчитано), dash, варианты busybox (насколько я знаю)... >> then, стоящий хрен знает где. >> Цикл с case по по опциям ради одной опции -D. > А если бы понадобилось обрабатывать и другие опции? Так что Но там их нет. >> Смесь конструкций test и [] (подозреваю, что это писали разные люди). >> Разные стили для переменных: капс и строчные. > Дык переписывали же. Какие -то части тупо скопипастили. Угу. Я заметил. "Китайский код". >> Отсутствие кавычек в переменных, которые, по идее, содержат пути (а значит >> могут >> содержать пробелы). >> Передача пользовательских опций для fiderck _перед_ задаваемыми в конфиге. > Это плохо, ага А, нет? >> Этого мало для скрипта на 50 строчек? > Раз одни люди готовы за скрип из 50 строк платить деньги - найдется кому > написать. Там большая система, состоящая из независимых частей (подсистем), объединённых в одной точке (в идеале). Как человек недавно её узнающий, я могу сказать: у них получилась каша. Видно со стороны. Есть большие системы (например, тот же Debian) в которых всё структурировано и достаточно понятно (хотя бы в общем, согласен, Debian тут далеко не идеален, но у него есть "исторический багаж", от которого никуда не деться). Эта система к ним не относится. Бывает, она падает (скорее, правило, чем исключение, просто её редко трогают). Последний пример: почти сдох ЖД. Был второй, настроенный под станцию. Поменяли. Не взлетела. Товарисч долго парился. В итоге, скопировал всю конфигурацию станции со старого диска. Настроить простыми методами не вышло. -- To UNSUBSCRIBE, email to debian-russian-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/4ff01e49.2080...@yandex.ru