第四节：Bash编程易犯的错误

1次阅读

共计 1826 个字符，预计需要花费 5 分钟才能阅读完成。

上一篇文章参见第三节：Bash 编程易犯的错误。这一篇翻译得不是非常满意，时间比较赶，请见谅，如果有问题可以在本文后方留言，大家一起深入探讨。

36. [-n $foo] or [-z $foo]

这个例子中，$foo 没有用引号引起来，当 $foo 包含空格或者 $foo 为空时都会出问题：

$ foo="some word" && [-n $foo] && echo yes
-bash: [: some: binary operator expected

$ foo="" && [-n $foo] && echo yes
yes

正确的写法是：[-n "$foo"]
[-z "$foo"]
[-n "$(some command with a"$file"in it)" ]

[[-n $foo]]
[[-z $foo]]

37. [[-e “$broken_symlink”]] returns 1 even though $broken_symlink exists

这里 -e 选项是看文件是否存在，当紧跟的文件是一个软链接时，它不看软链接是否存在，而是看实际指向的文件是否存在。所以当软链接损坏时，即实际指向的文件被删除后，-e 的结果返回 1。

所以如果你确实要判断后面的文件是否存在，正确的写法是：

[[-e "$broken_symlink" || -L "$broken_symlink"]]

38. ed file 命令使用的正则语法，不支持 0 次出现次数，下面的就可以正常工作：

ed file 略过，现在很少会有人用 ed 命令吧。 39. expr sub-string fails for "match" 下面的例子多数情况下运行不会有问题：

word=abcde
expr "$word" : ".\(.*\)"
bcde

但是当 $work 不巧刚好是 match 时，就有可能出错了（MAC OSX 下的 expr 命令不支持 match，所以依然能正常工作）：

word=match
expr "$word" : ".\(.*\)"

原因是 match 是 expr 命令里面的一个特殊关键字，针对 GNU 系统，解决方法是在前面加一个 '+'：

word=match
expr + "$word" : ".\(.*\)"
atch

'+' 号可以让 expr 命令忽略后续 token 的特殊含义。

另外一个建议是，不要再使用 expr 命令了，expr 能做的事情都可以用 Bash 原生支持的参数展开（Parameter Expansion）或者字符串展开（Substring Expansion）来完成。并且相同情况下，内置的功能肯定比外部命令的效率要高。

上面的例子，目的是为了删除单词中的首字符，可以这样做：

$ word=match
$ echo "${word#?}"    # PE
atch
$ echo "${word:1}"    # SE
atch

40. On UTF-8 and Byte-Order Marks (BOM)

多数情况下，UNIX 下 UTF-8 类型的文本不需要使用 BOM，文本的编码是根据当前语言环境，MIME 类型或者其它文件元数据信息确定的。人为阅读时，不会因为在文件开始处加 BOM 标记而腚影响，但是当文件要被脚本解释执行时，BOM 标记会像 MS-DOS 下的换行符（^M）一样奇怪。

41. content=$(&1 >>logfile

这是一个很常见的错误，显然你本来是想将标准输出与标准错误输出都重定向到文件 logfile 中，但是你会惊讶地发现，标准错误依然输出到屏幕中。

这种行为的原因是，重定向在命令执行之前解析，并且是从左往右解析。上面的命令可以翻译成，将标准错误输出重定向到标准输出（此刻是终端），然后将标准输出重定向到文件 logfile 中。所以，到最后，标准错误并没有重定向到文件中，而是依然输出到终端：

somecmd >>logfile 2>&1

更加详细的说明见 BashFAQ。

43. cmd; ((! $?)) || die

只有需要捕获上一个命令的执行结果进，才需要记录 $? 的值，否则如果你只需要检查上一个命令是否执行成功，直接检测命令：

if cmd; then
    ...
fi

或者使用 case 语句来检测多个或能的返回码：cmd
status=$?
case $status in
    0)
        echo success >&2
        ;;
    1)
        echo 'Must supply a parameter, exiting.' >&2
        exit 1
        ;;
    *)
        echo 'Unknown error, exiting.' >&2
        exit $status
esac