主页 / unix / 问题 / 432052
Accepted
Curious Sam
Asked: 2018-03-20 02:29:51 +0800 CST

即使给出了开始搜索的路径，shellcheck 也会警告关于查找输出的循环[重复]

  • 772

Ubuntu 16.04

#!/bin/bash

site="hello"
wDir="/home/websites/${site}/httpdocs/"

for file in $(find "${wDir}" -name "*.css")
do
   echo "$file";
done
exit 0;

即使我定义了开始目录，shellcheck 也会警告我，但脚本工作得很好。

root@me /scripts/ # shellcheck test.sh

In test.sh line 6:
for file in $(find "${wDir}" -name "*.css")
            ^-- SC2044: For loops over find output are fragile. Use find -exec or a while read loop.
bash find

2 个回答

  1. 问题正是 shellcheck 告诉你的：for迭代find或类似命令的输出的循环是脆弱的。例如：

    $ ls
'a file with spaces' 

$ for file in $(find . ); do    echo "$file"; done
.
./a
file
with
spaces

    安全的方法是使用-execof find：

    $ find . -exec echo  {} \;
.
./a file with spaces

    或者使用while循环：

    $ find . -print0 | while IFS= read -r -d '' file; do echo "$file"; done
.
./a file with spaces
    • 12
  2. Best Answer

    使用for循环find输出充其量是一种反模式。请参阅BashFAQ/001 - 如何逐行（和/或逐字段）读取文件（数据流、变量）？为什么。使用while下面的循环和read命令。以下命令find用 NULL 字节分隔输出，并read通过拆分该字节来读取命令，以便安全处理名称中包含特殊字符的所有文件（包括换行符）

    #!/usr/bin/env bash

site="hello"
wDir="/home/websites/${site}/httpdocs/"

find "${wDir}" -name "*.css" -type f -print0 | while IFS= read -r -d '' file; do
    printf '%s\n' "$file"
done

    或者完全避免使用管道并进行流程替换

    while IFS= read -r -d '' file; do
    printf '%s\n' "$file"
done< <(find "${wDir}" -name "*.css" -type f -print0)

    Web ShellCheck不会报告上述两个片段中的任何一个的任何问题。

    • 7

相关问题