shellcheck建议不要使用basename：为什么？

我正在尝试shellcheck。

我有这样的东西

basename "${OPENSSL}" 

我得到以下建议

Use parameter expansion instead, such as ${var##*/}.

从实际的角度来看，我认为没有区别

$ export OPENSSL=/opt/local/bin/openssl
$ basename ${OPENSSL}
openssl
$ echo ${OPENSSL##*/}
openssl

由于basename是POSIX规范中的内容，因此我不认为这是最佳实践。有什么提示吗？

这不关乎效率，而是关乎正确性。basename使用换行符来分隔输出的文件名。在通常情况下，当您仅传递一个文件名时，它将在其输出中添加尾随换行符。由于文件名本身可能包含换行符，因此很难正确处理这些文件名。

人们通常basename像这样使用，这使情况变得更加复杂"$(basename "$file")"。这使事情变得更加困难，因为从中$(command)剥离了所有尾随的换行符command。考虑$file以换行符结尾的不太可能的情况。然后，basename将添加一个额外的换行符，但"$(basename "$file")"将同时删除两个换行符，从而使文件名不正确。

另一个问题basename是，如果$file以-（破折号，也称为减号）开头，它将被解释为一个选项。这个很容易解决：$(basename -- "$file")

健壮的使用方法basename是：

# A file with three trailing newlines.
file=$'/tmp/evil\n\n\n'

# Add an 'x' so we can tell where $file's newlines end and basename's begin.
file_x="$(basename -- "$file"; printf x)"

# Strip off two trailing characters: the 'x' added by us and the newline added by basename. 
base="${file_x%??}"

一种替代方法是使用${file##*/}，这比较容易，但是有其自身的错误。特别是在或的情况下$file是错误的。/foo/

shellcheck的源代码中的相关行是：

checkNeedlessCommands (T_SimpleCommand id _ (w:_)) | w `isCommand` "dirname" =
    style id "Use parameter expansion instead, such as ${var%/*}."
checkNeedlessCommands (T_SimpleCommand id _ (w:_)) | w `isCommand` "basename" =
    style id "Use parameter expansion instead, such as ${var##*/}."
checkNeedlessCommands _ = return ()

没有明确给出解释，但基于函数（checkNeedlessCommands）的名称，@ jordanm看起来很正确，这建议您避免分叉新进程。

dirname，basename，readlink等（感谢@Marco -这是校正）可以创建可移植性问题时，安全性就变得很重要（需要路径的安全）。许多系统（例如Fedora Linux）将其放置在，/bin而其他系统（例如Mac OSX）将其放置在/usr/bin。然后是Windows上的Bash，例如cygwin，msys和其他。 尽可能保持纯正Bash总是更好。 （根据@Marco的评论）

顺便说一句，感谢您指向shellcheck的指针，我之前从未见过。