2020-01-28 Не Плодить Переменные В Баше
Создал лишнюю переменную в баше, чего делать категорически нельзя:
local hostname="${ARG_HOSTNAME:-}"
Вместо этого нужно было использовать
"${ARG_HOSTNAME:-}"
~~OWNERAPPROVE~~
| Прочитал правила разработки как не надо делать 2020-01-28 не плодить переменные в баше |
v.obukhov
Обсуждение
А при чём тут bash?
Это специфика парсинга агрументов в bash с помощью crab_sys.
Первый вариант - явное определение, что мы принимаем какой-то аргумент. Но по методике crab_sys и bash-ООП мы считаем что эти аргументы мы уже разобрали и явно этого в коде делать не нужно.
Если бы мы вычисляли или парсили аргумент, либо проводили проверку агрументов - другой разговор.
удивляюсь этой помойке из правил)
Соглашусь, что для внешнего пользователя, не владеющего контекстом, это выглядит непонятно. Даже люди из разных функциональных команд иногда путаются. Но над внешней формой правил работаем 👍
По сути, это формат обсуждения и выработки правил разработки. Только не со стороны написания и внедрения практик «сверху», а от конкретных свежих кусков кода из merge-реквестов: обсуждаем, формулируем правило и разработчик своими словами описывает это в виде небольшой заметки.