ShellCheck警告：“迭代ls输出很脆弱。 使用glob。 [SC2045]”

Question

我在以下代码的第二行收到ShellCheck警告[SC2045]。 在我尝试最后一次ls之前确保目录不为空时可以忽略它吗？

 if [ "$(ls -A "$retryDir")" ]  ; then
    for thisRetryFile in $(ls "$retryDir"/*.tar.gz) ; do
        scp -o ConnectTimeout=30  "$thisRetryFile"  \             
              "$remoteUser@$remoteHost:$remotePath" >> "$BACKUPLOG"
    done
 fi

更新：阅读帖子评论后。 我将行更改为：

for thisRetryFile in "$retryDir"/*.tar.gz ; do

这已删除警告。

Answer 1

将循环与glob一起使用，并设置nullglob以避免在模式不匹配的情况下执行scp 。 而且您也不需要外部的if条件，因为nullglob的for有效地解决了这一问题：

shopt -s nullglob

for thisRetryFile in "$retryDir"/*.tar.gz; do
    scp -o ConnectTimeout=30  "$thisRetryFile" \
          "$remoteUser@$remoteHost:$remotePath" >> "$BACKUPLOG"
done

如果您想在没有文件与模式匹配时遇到这种情况，可以这样编写，而无需使用shopt -s nullglob ：

for thisRetryFile in "$retryDir"/*.tar.gz; do
    if [ -f "$thisRetryFile" ]; then
        scp -o ConnectTimeout=30  "$thisRetryFile" \
            "$remoteUser@$remoteHost:$remotePath" >> "$BACKUPLOG"
        break
    else
        echo "warn: no tar.gz file in dir: $retryDir"
    fi
done

Answer 2

这样比较安全。 试试吧。

 if [ "$(ls -A "$retryDir")" ]  ; then
    for thisRetryFile in ${retryDir}'/*.tar.gz' ; do
        scp -o ConnectTimeout=30  "$thisRetryFile"  "$remoteUser@$remoteHost:$remotePath" >> "$BACKUPLOG"
    done
 fi

问候！