Início / unix / Perguntas / 415315
Accepted
Freedo
Asked: 2018-01-07 20:06:28 +0800 CST

Corrige o script bash esperando a entrada do caminho com / no final quebrando quando o caminho não termina com /

  • 772

Eu tenho este código:

for file in "$@"*png; do
  echo "$file"
done

Funciona apenas se você fornecer um caminho que termine com /like /root/.

Qual seria a forma correta de adicionar/ao input do caminho, em situações como essa, sem quebrar meu script?

Se você der uma entrada de caminho sem / no final, ele fará isso:

File: /root*png

Se eu modificá-lo para ser for file in "$@"/*png; doe inserir /root/test/, funcionará, mas o resultado parecerá feio:

File: /root/test//sample2.png
bash

3 respostas

  1. Best Answer

    ilkkachu apontou uma grande falha na minha resposta e a corrigiu na dele, então, por favor, dê a ele o crédito que ele merece. Eu vim com outra solução embora:

    #!/bin/bash

for dir in "$@"; do
        find "$dir" -type f -name '*png' -exec readlink -f {}  \;
done

    Exemplo :

    $ ll
total 6
-rwxr-xr-x 1 root root 104 Jan  7 14:03 script.sh*
drwxr-xr-x 2 root root   3 Jan  7 04:21 test1/
drwxr-xr-x 2 root root   3 Jan  7 04:21 test2/
drwxr-xr-x 2 root root   3 Jan  7 04:21 test3/

$ for n in {1..3}; do ll "test$n"; done
total 1
-rw-r--r-- 1 root root 0 Jan  7 04:21 testfile.png
total 1
-rw-r--r-- 1 root root 0 Jan  7 04:21 testfile.png
total 1
-rw-r--r-- 1 root root 0 Jan  7 04:21 testfile.png

$ ./script.sh test1 test2/ test3
/root/temp/test1/testfile.png
/root/temp/test2/testfile.png
/root/temp/test3/testfile.png

    Solução Original :

    for file in "${@%/}/"*png; do
  echo "$file"
done

    O ${@%/} cortará qualquer / fora do final do seu parâmetro e, em seguida, o / fora o adicionará de volta - ou o adicionará a qualquer parâmetro que não tenha um.

    • 7

  2. Usar $@com o globo assim parece um tanto estranho. Se você espera que seja capaz de processar vários argumentos, bem, não funcionará. As strings ao redor $@são anexadas apenas ao primeiro e ao item da lista.

    $ mkdir a b; touch a/a.png b/b.png
$ set -- a b 
$ echo x"$@"x
xa bx
$ echo "$@/"*.png
a b/b.png

    Portanto, para lidar com vários argumentos, você precisará fazer um loop "$@"separadamente:

    for arg in "$@"; do
    for file in "${arg%/}"/*.png; do
         echo "$file"
    done 
done

    Em outro caso, você poderia usar a expansão de substituição de string ${//}(em Bash ou zsh) para adicionar um sufixo a cada parâmetro posicional, mas é bastante difícil fazer isso funcionar com um glob.

    • 3

  3. Você pode usar tr também.

    Sempre adicione / no final e...

    file='/root/test//sample2.png';echo "$file" | tr -s '/'
    • 0

relate perguntas