2018-07-18 Использование Не Популярного Сахара

Использование не всем известного сахара усложняет чтение кода и приводит к потенциальным ошибкам в будущем

--- [ "$date_now" -le "$date_min" ] && echo NIGHT && return 0
+++ [ "$date_now" -le "$date_min" ] && { echo NIGHT; return 0; }
 
--- if [ "$method" == "auth" ] || [ "$method" == "web" ] || [ "$method" == "xml" ]; then
+++ if [ "$method" = "auth" -o "$method" = "web" -o "$method" = "xml" ]; then
 
---[ -n $cert_date_new_start ] && [[ "$cert_date_new_start" -gt "$CERT_DATE_CUR" ]] && return 0
+++[ -n $cert_date_new_start -a "$cert_date_new_start" -gt "$CERT_DATE_CUR" ] && return 0
 
---[ "$DAYS_BEFORE_FINISH" == "$DAYS_FATAL" ] && [ "$(time_of_day)" == 'DAY' ] && STATUS="FATAL "
+++[ "$DAYS_BEFORE_FINISH" ="$DAYS_FATAL" -a "$(time_of_day)" = 'DAY' ] && STATUS="FATAL "
 
 
---[ "$prev" = "$new" ] && echo OK && return 0
+++[ "$prev" = "$new" ] && { echo OK; return 0; }
 
 
---(( new < prev )) && echo OK && return 0
+++Это было бы ошибкой [ $new -lt $prev ] && { echo OK; return 0; }
+++ # используем круглые скобки тк здесь bigint
+++(( new < prev )) && echo OK && return 0
 
---tr ':' ' ' < /proc/net/dev | grep -v 'lo' | grep "[0-9]"
+++cat /proc/net/dev | tr ':' ' ' | grep -v 'lo' | grep "[0-9]"
 
---[ "$(check_diff "$err_prv" "$err_new")" == 'FAIL' ] && rc=1 && break
+++[ "$(check_diff "$err_prv" "$err_new")" = 'FAIL' ] && { rc=1 ; break; }
Прочитал правила разработки как не надо делать 2018-07-18 использование не популярного сахара
Yes(9) No(1) Clear

Yes:
admin, Alexander Sobyanin, Сергей Трошин, Anton Klinskih, Олег Стрижеченко, Николай Глазов, Александр Ефим, Денис Кучаев, Дмитрий Пономарь,

No:
,

~~OWNERAPPROVE~~