Início / unix / Perguntas / 564760
Accepted
Quasímodo
Asked: 2020-01-30 08:06:58 +0800 CST

A exportação serve a algum propósito neste código?

  • 772

Na página Analisando Scripts do TLDP , o seguinte código é fornecido para análise:

export SUM=0
for f in $(find src -name "*.java"); do
    export SUM=$(($SUM + $(wc -l $f | awk '{ print $1 }')))
done
echo $SUM

Eu entendo que ele calcula a soma do número de linhas de todos os arquivos *.java no diretório src. O que eu não entendo é o motivo do uso da palavra- exportchave, que é descrita assim :

O comando export disponibiliza variáveis ​​para todos os processos filhos do script ou shell em execução.

Como SUMnunca é acessado por um processo filho, há algum motivo para exportá-lo?

shell-script

2 respostas

  1. Best Answer

    Eu entendo que ele calcula a soma do número de linhas de todos os arquivos *.java no diretório src.

    Isso não é necessariamente completamente verdade. Ele calcula a soma do número de linhas de todos os *.javaarquivos na árvore de diretórios com raiz src(ou seja, srce todos os seus diretórios filhos). Mas falhará para qualquer caminho de arquivo que contenha espaços em branco ou quando houver nomes de diretórios que terminem com .java.

    Como o SUM nunca é acessado por um processo filho, há algum motivo para exportá-lo?

    Não.

    Eu provavelmente escreveria o trecho de código assim, tornando-o seguro para o nome do arquivo no processo:

    find src -type f -name '*.java' -exec wc -l {} \; | awk '{ s += $1 } END { print s }'

    Uma solução melhor provavelmente seria esta:

    find src -type f -name '*.java' -exec cat {} + | wc -l
    • 11

  2. Você está certo, não há necessidade de usar exportaqui. Há mais problemas com este código embora:

    $  ~/.cabal/bin/shellcheck e.sh

In e.sh line 4:
for f in $(find src -name "*.java"); do
         ^------------------------^ SC2044: For loops over find output are fragile. Use find -exec or a while read loop.


In e.sh line 5:
    export SUM=$(($SUM + $(wc -l $f | awk '{ print $1 }')))
                  ^--^ SC2004: $/${} is unnecessary on arithmetic variables.
                                 ^-- SC2086: Double quote to prevent globbing and word splitting.

Did you mean:
    export SUM=$(($SUM + $(wc -l "$f" | awk '{ print $1 }')))

For more information:
  https://www.shellcheck.net/wiki/SC2044 -- For loops over find output are fr...
  https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...
  https://www.shellcheck.net/wiki/SC2004 -- $/${} is unnecessary on arithmeti...

    E é uma boa prática usar nomes de variáveis ​​em letras minúsculas nos scripts, veja isto para uma explicação.

    Em geral, tldp.org não é recomendado, veja isto ou isto . Também vi pessoas não recomendando o tldp.org aqui no StackExchange.

    • 2

relate perguntas