2020-01-28 Не Плодить Переменные В Баше

Создал лишнюю переменную в баше, чего делать категорически нельзя:

local hostname="${ARG_HOSTNAME:-}"

Вместо этого нужно было использовать

"${ARG_HOSTNAME:-}"

~~OWNERAPPROVE~~

Прочитал правила разработки как не надо делать 2020-01-28 не плодить переменные в баше
Yes(3) No(0) Clear

Yes:
Vladimir Obukhov, Сергей Трошин, ,

No:

ОлегОлег, 21.07.2020 04:14

А при чём тут bash?

Nikolay CarbonsoftNikolay Carbonsoft, 08.07.2021 08:38

Это специфика парсинга агрументов в bash с помощью crab_sys.

Первый вариант - явное определение, что мы принимаем какой-то аргумент. Но по методике crab_sys и bash-ООП мы считаем что эти аргументы мы уже разобрали и явно этого в коде делать не нужно.

Если бы мы вычисляли или парсили аргумент, либо проводили проверку агрументов - другой разговор.

МимопроходящийМимопроходящий, 30.03.2021 10:06

удивляюсь этой помойке из правил)

Nikolay CarbonsoftNikolay Carbonsoft, 08.07.2021 08:35

Соглашусь, что для внешнего пользователя, не владеющего контекстом, это выглядит непонятно. Даже люди из разных функциональных команд иногда путаются. Но над внешней формой правил работаем 👍

По сути, это формат обсуждения и выработки правил разработки. Только не со стороны написания и внедрения практик «сверху», а от конкретных свежих кусков кода из merge-реквестов: обсуждаем, формулируем правило и разработчик своими словами описывает это в виде небольшой заметки.

Ваш комментарий. Вики-синтаксис разрешён: