IF/THEN BASH 最佳实践

Question

我想知道我是否做得对。我正在努力学习BASH，并且很想第一次学习“最佳实践”，所以我不采用马虎/简单的方法。

我想知道的是，我可以像下面那样嵌套一个 IF/THEN 语句吗？为什么或者为什么不？改用 elif 会更好地服务于下面的块吗？

最后，我希望有人能帮我解释一下“${foo}”和“$(bar)”的使用...花括号还是圆括号？我（到目前为止）在定义变量时使用了大括号，“foo='bar'”后来被称为“${foo}”，当我捕获命令“foo=$(find”时使用括号。- type f -name bar)" 将被称为 "$foo" ...或者我可能只是在做同样的事情两次，我不知道...我很想听听你的意见我们都得说！:D

# Downloading the script bundle
echo "Lets get the script bundle and get to work!"
wget http://place.to.get/att.tar

# Logic switch, checking if the TAR bundle exists. If it does
# verify the MD5 Checksum (to prevent corruption).
# If verfied, then un-tar the bundle in our working directory
# otherwise, exit with an error code, otherwise 
if [[ -f att.tar ]]
    then
        echo "Okay, we have the bundle, lets verify the checksum"
        sum=$(md5sum /root/att/att.tar | awk '{print $1}')
            if [[ $sum -eq "xxxxINSERT-CHECKSUM-HERExxxx" ]]
                then
                    tar -xvf att.tar
            else
                clear
                echo "Couldn't verify the MD5 Checksum, something went wrong" | tee /tmp/att.$time.log
                sleep 0.5
                exit 1;
            fi
    else
    clear
    echo "There was a problem getting the TAR bundle, exiting now ..." | tee /tmp/att.$time.log
    sleep 0.5
    exit 1;
fi

Answer 1

整体cmets

• 嵌套的“if”没有问题，但提前退出会更清晰
• cut 比 awk 便宜，但 read 仍然更便宜
• 使用“[”而不是“[[”的简单字符串相等测试稍微便宜
• 将错误消息写入 STDERR
• 使用 read 和
• 使用函数

简化版

bail () {
    clear
    echo "${@}" | tee /tmp/att.${time}.log >&2
    exit 1
}

# Downloading the script bundle
echo "Lets get the script bundle and get to work!" >&2
wget http://place.to.get/att.tar || bail "There was a problem getting the TAR bundle, exiting now ..."

sum=''
read sum rest < <(md5sum /root/att/att.tar)

[ $sum == "xxxxINSERT-CHECKSUM-HERExxxx" ] || bail "Couldn't verify the MD5 Checksum, something went wrong"

tar -xvf att.tar || bail "Extract failed"