Loop for, problema em fazê-lo funcionar

[Chernobyl_User]

Olá, fiz um script para converter alguns arquivos mp3 para que tenham um tamanho menor, segue o script:

Código Selecionar Expandir

#!/bin/bash

#Pergunta pelos arquivos a converter
arquivos=$(zenity --file-selection --multiple --title "Selecione os arquivos que quer converter: " | tr '|' '\n')

#Pergunta pela pasta para por os arquivos convertidos
pasta=$(zenity --file-selection --directory --title "Selecione a pasta onde os arquivos deverão ser salvos: ")

#Conta o total de arquivos a converter
numarq=$(echo "$arquivos" | wc -l)

#Seta o numero base para a contagem
base=1

#Inicia o loop e converte os arquivos
for linha in "$arquivos" ; do

#Obtém o nome do arquivo a ser convertido
nome=$(echo "$linha" | rev | cut -d/ -f1 | rev)

echo "Convertendo arquivo "$base" de "$numarq": "$nome""
case $1 in
f) #Usa o ffmpeg para converter
ffmpeg -i "$linha" -acodec libmp3lame -ac 1 -ab 64k -y -ar 44100 "$pasta"/"$nome" &>> /dev/null
;;
l) #Usa o lame para converter
lame -m m -f -b 64 --cbr "$linha" "$pasta"/"$nome" &>> /dev/null
;;
*) echo "Encoder inválido!"
exit
;;
esac

#Muda o número do arquivo a converter
base=$(($base+1))
done

#Finaliza o programa
echo "Conversão completa!"

#Sai
exit

Ele devia funcionar assim:

1° Pelo terminal eu rodo: bash script.sh (l para o lame ou f para ffmpeg)
2º pede para selecionar os mp3 a converter
3° pede para escolher a pasta de saída
4° converte os arquivos selecionados e manda um output no terminal

O problema é que o laço for não está mandando os arquivos linha por linha para o encoder, ele está mandando tudo de uma vez só:

Código Selecionar Expandir

bash Dropbox/Scripts/mp3.sh l
Convertendo arquivo 1 de 2: musicaa 1.mp3 musica 2.mp3
Conversão completa!

Se eu selecionar só um arquivo, tudo funciona. Alguém tem alguma idéia?

[irtigor]

É que você tem uma string. Trabalhe com um vetor

Código Selecionar Expandir


IFS=\| arquivos=($(zenity --file-selection --multiple --title "Selecione os arquivos que quer converter: "))
for i in "${arquivos[@]}"; do
    #faça alguma coisa o $i
done


Você pode traçar o que o script faz passando a opção "-x".

Código Selecionar Expandir


bash -x /caminho/para/o/script.sh


[adri3d]

sabe para que serve o caracter & em linhas como essa

Código Selecionar Expandir

ffmpeg -i "$linha" -acodec libmp3lame -ac 1 -ab 64k -y -ar 44100 "$pasta"/"$nome" &>> /dev/null

caso não saiba, aprenda vai ser util.

[Chernobyl_User]

Olá irtigor, adaptei sua dica para meu script e a saída do "bash -x" foi:

Código Selecionar Expandir

mp3.sh: linha 49: ${$arquivos[@]}: substituição incorreta

sabe para que serve o caracter & em linhas como essa

Código Selecionar Expandir

ffmpeg -i "$linha" -acodec libmp3lame -ac 1 -ab 64k -y -ar 44100 "$pasta"/"$nome" &>> /dev/null

caso não saiba, aprenda vai ser util.

Seria o "&" do "&>> /dev/null? Pelo que eu sei, serve para redirecionar toda e qualquer saída de texto gerada pelo encoder para o lixo. O "&&" é o operador lógico "E".

[irtigor]

Mostra o código.

[zekkerj]

for linha in "$arquivos" ; do

Acredito que vc colocou a variável entre aspas pra poder lidar com os nomes de arquivos com espaços, correto? Mas assim ele vai tratar todo o conteúdo da variável como uma string só.

[fpissarra]

Olá irtigor, adaptei sua dica para meu script e a saída do "bash -x" foi:

Código Selecionar Expandir

mp3.sh: linha 49: ${$arquivos[@]}: substituição incorreta

Talvez isso ajude:

Código Selecionar Expandir


OLDIFS=$IFS
IFS=$'|'
arquivos=$(zenity --file-selection --multiple)

for i in $arquivos; do
 // faz algo com $i aqui.
done

IFS=$OLDIFS

No post do irtigor, só faltou um salto de linha depois do ajuste da variável IFS.

[irtigor]

Não é necessário, do jeito que passei tem que funcionar, por isso pedi pra ver o código, ele errou ao copiar ou coisa do tipo.

[fpissarra]

Não é necessário, do jeito que passei tem que funcionar, por isso pedi pra ver o código, ele errou ao copiar ou coisa do tipo.

Irtigor... Isso aqui:

Código Selecionar Expandir

IFS=\| arquivos=($(zenity --file-selection --multiple --title "Selecione os arquivos que quer converter: "))

Como está, com certeza não deve funcionar. Não, sem, pelo menos, um ponto-e-virgula depois de '|'.

[irtigor]

Funciona e é útil, por exemplo (nada de ponto e vírgula ou nova linha separando a variável)

Código Selecionar Expandir


echo $LANG
LANG=en_US.UTF-8 __aplicativo__ &
echo $LANG

Executa __aplicativo__ com a língua inglesa como padrão, como foi executado em subshell, a variável $LANG permanece com o valor anterior no shell atual.

[fpissarra]

Funciona e é útil, por exemplo (nada de ponto e vírgula ou nova linha separando a variável)

Código Selecionar Expandir


echo $LANG
LANG=en_US.UTF-8 __aplicativo__ &
echo $LANG

Executa __aplicativo__ com a língua inglesa como padrão, como foi executado em subshell, a variável $LANG permanece com o valor anterior no shell atual.

Hummmmm.... interessante!

[fpissarra]

Funciona e é útil, por exemplo (nada de ponto e vírgula ou nova linha separando a variável)

Código Selecionar Expandir


echo $LANG
LANG=en_US.UTF-8 __aplicativo__ &
echo $LANG

Executa __aplicativo__ com a língua inglesa como padrão, como foi executado em subshell, a variável $LANG permanece com o valor anterior no shell atual.

De fato, é um recurso interessante... mas o IFS só será setado na execução do zenity, não? O loop for continuará separando as strings por espaços, tabs e \n...

[irtigor]

Usei um array, tanto faz o IFS depois, mas sim no bash só é alterado pro comando.

IFS=\| arquivos=($(zenity --file-selection --multiple --title "Selecione os arquivos que quer converter: "))

[fpissarra]

Usei um array, tanto faz o IFS depois, mas sim no bash só é alterado pro comando.

IFS=\| arquivos=($(zenity --file-selection --multiple --title "Selecione os arquivos que quer converter: "))

Ops! É mesmo! Sorry... ficou legal!