[grzesiek123321]

Widziałem gdzieś, ale nie mogę tego znaleźć. Chcę wylistować pliki w katalogu o podanych rozszerzeniach, ale coś mi nie wychodzi
for file in *.{png,jpg}; do ..., próbowałem również {png|jpg}
Nie chcę bawić się w find, bo ta krótsza składnia może mi się przydać na przyszłość.

#bash #linux

[Kryspin013]

@grzesiek123321: ls *.png *.jpg

[interface]

@grzesiek123321: for i in *.{png,jpg}; do u mnie dziala w bashu, pokaz reszte skryptu albo powiedz jaka powloka go wywolujesz

[Kryspin013]

@grzesiek123321: Ewentualnie np u mnie
for i in *.{zip,png}; do echo $i; done
działa bez problemu

[grzesiek123321]

@interface: @Kryspin013: #!/bin/bash ... for file in *.{png,jpg}; do printf "File $file\n"; done
W katalogu mam obecnie same png, które są poprawnie listowane. Ale na koniec File *.jpg i wysypuje się convert z imagemagick bo dostaje zamiast pliku *.jpg

[Kryspin013]

@grzesiek123321: Chyba nie rozumiem. Pokaż resztę skryptu albo dokładniej opisz co chcesz zrobić i gdzie masz błąd. I z ciekawości czemu robisz tak na około a nie tak jak ci napisałem w pierwszym komentarzu? W sensie to jest jeszcze krótsza składania.

PS z tą pętlą będziesz miał problem jak będziesz miał spacje w nazwach plików png i jpg - skrypt ci się rozjedzie.

Dobra zrozumiałem, gdzie masz błąd.

[no_one]

Staraj się unikać nazywania zmiennych tak jak programy w systemie. 'file' to polecenie w powłoce, użyj np.: plik

[Kryspin013]

@no_one: Dobra rada ale tym razem to nic nie zmienia.

@grzesiek123321: Najlepiej ls dopisał, bo samo *.{png,jpg} nie listuje po plikach tak jakby.. Sprawdź to:
for FILE in "$(ls *.{png,jpg})"; do printf "File $FILE\n"; done

Edit czekaj jeszcze jeden bug jest. xD

[Kryspin013]

@grzesiek123321: soory jeszcze raz. tak to trzeba zrobić dokładnie:

IFS=$'\n'
for FILE in $(ls *.{png,jpg}); do printf "File $FILE\n"; done

# IFS=$'\n' Zmienia separator tylko na znak nowej linii. Naprawia "bug" z plikami, które posiadają spacje w nazwie.

[grzesiek123321]

@Kryspin013: Wielkie dzięki, masz rację, ls będzie tu lepszym rozwiązaniem.

[birus]

@grzesiek123321 ten pierwszy powinien zadziałać

[birus]

@grzesiek123321 nie będzie lepszy, bo raz że trzeba wywołać o jedną komendę więcej, w tym przypadku ls, dwa, że jak ktoś to zobaczy, to pomyśli że nie umiałes napisać bez użycia ls. Poszukaj na sieci useless use of cat, ls

[grzesiek123321]

@birus: Nie boję się, że to ktoś zobaczy. A nawet jakby... Nie przejmuję się tym bo to tylko proste narzędzie do szybkiego przeskalowania obrazków a nie wpis w portfolio.