Использование не всем известного сахара усложняет чтение кода и приводит к потенциальным ошибкам в будущем
<code bash>
--- [ "$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; }

</code>


{(rater>id=1|name=Прочитал_правила_разработки:как_не_надо_делать:2018-07-18_использование_не_популярного_сахара|type=vote|trace=user|tracedetails=1)}


~~OWNERAPPROVE~~