Início / unix / Perguntas / 432052
Accepted
Curious Sam
Asked: 2018-03-20 02:29:51 +0800 CST

shellcheck avisa sobre loops na saída de localização, mesmo quando é fornecido o caminho para iniciar a pesquisa [duplicado]

  • 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 me avisa mesmo se eu definir o diretório inicial, mas o script funciona bem.

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 respostas

  1. O problema é exatamente o que o shellcheck está dizendo a você: forloops que iteram sobre a saída findou comandos semelhantes são frágeis. Por exemplo:

    $ ls
'a file with spaces' 

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

    A maneira segura seria usar o -execde find:

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

    Ou para usar um whileloop:

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

    Usar uma saída forloop over findé, na melhor das hipóteses, um antipadrão. Consulte BashFAQ/001 - Como posso ler um arquivo (fluxo de dados, variável) linha por linha (e/ou campo por campo)? por que razão. Use um whileloop como abaixo com um readcomando. O comando abaixo delimita a saída findcom um byte NULL e reado comando lê dividindo esse byte, para que todos os arquivos com caracteres especiais em seus nomes sejam tratados com segurança (incluindo novas linhas)

    #!/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

    Ou evite totalmente o uso de tubulações e faça a substituição do processo

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

    O ShellCheck da web não relata nenhum problema para nenhum dos dois snippets acima.

    • 7

relate perguntas