bash - 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条件，因为forwithnullglob有效地处理了这一点：

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

问候！