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(31) No(0) Clear Yes: No: |
~~OWNERAPPROVE~~