[Решено] Осваиваю Bash, помогите упростить скрипт

[Disaster1986]

Имеем такой скрипт:

Код: [Выделить]

#!/bin/bash

s=$(mocp --info | grep State | sed 's/State: //gi')
case $s in
PLAY|PAUSE) mocp --stop ;;
esac
cd /home/zaycakitayca/.moc/
mv playlist.m3u playlist99.m3u
mv playlist01.m3u playlist.m3u
bash -c 'i=1; ls playlist??.m3u | while read ; do mv "$REPLY" "playlist$(printf %02u $i).${REPLY//*.}" ; ((i++)) ; done'
case $s in
PLAY) mocp --play ;;
esac
exit 0Всё работает, однако код кажется слишком громоздким. Особенно напрягает использование case, но if для такого условия правильно написать я не смог. Ещё не нравится строка

Код: [Выделить]

bash -c 'i=1; ls playlist??.m3u | while read ; do mv "$REPLY" "playlist$(printf %02u $i).${REPLY//*.}" ; ((i++)) ; done'не понимаю, почему то же самое без bash -c не работает. Как можно упростить этот скрипт?

(Нажмите, чтобы показать/скрыть)

скрипт используется для выбора плэйлиста в MOC для 3-х и более плейлистов, запуск по хоткею (ибо задолбало открывать терминал для переключения плейлиста)

[aliftin]

Код: [Выделить]

bash -c 'i=1; ls playlist??.m3u | while read ; do mv "$REPLY" "playlist$(printf %02u $i).${REPLY//*.}" ; ((i++)) ; done'
Это идет поиск и переименовывание плейлистов? Мне больше нравится вот такой способ:

Код: [Выделить]

IFS=$'\n'
i=1

for j in `ls ./ | grep ".m3u"`
do
echo ""$j"+"$i"" #тут должно быть переименовывание
((i++))
done
Находятся все пробелы в заданной папке, независимо от наличия в них пробелов. Чем вас case не устраивает?

[ArcFi]

не понимаю, почему то же самое без bash -c не работает

Ставьте echo перед mv и смотрите, что пишет.

[RandomNT]

if будет так

if [ "$s" == "PLAY" ] || [ "$s" == "PAUSE" ]; then
  mocp --stop
fi

if [ "$s" == "PLAY" ]; then
  mocp --play
fi

[Disaster1986]

if будет так

if [ "$s" == "PLAY" ] || [ "$s" == "PAUSE" ]; then
  mocp --stop
fi

if [ "$s" == "PLAY" ]; then
  mocp --play
fi

В ответ пишет такую ерунду:

Код: [Выделить]

[: 6: PLAY: unexpected operator
[: 6: PLAY: unexpected operator
[: 13: PLAY: unexpected operator


Это идет поиск и переименовывание плейлистов?

Не совсем так. В папку кидаются файлы playlist01.m3u, playlist02.m3u и т.д, а также файл playlist.m3u, который использует MOC. При запуске скрипта файл playlist01.m3u встаёт на место файла playlist.m3u, а у остальных уменьшается номер (02 => 01; 03 =>02 и т.д.)

Чем вас case не устраивает?

Просто всегда казалось лишним использовать case там, где вполне справился бы if

[Axa-Ru]

Сдается мне вы между [ и "$s" пробельчик не вставили.
Дайте полный текст скрипта.

[Disaster1986]

Сдается мне вы между [ и "$s" пробельчик не вставили.

Отнюдь, я скопипастил условия, которые написал RandomNT, пробелы при этом остались на месте. Остальная часть скрипта без изменений

(Нажмите, чтобы показать/скрыть)

Код: [Выделить]

#!/bin/bash

s=$(mocp --info | grep State | sed 's/State: //gi')
if [ "$s" == "PLAY" ] || [ "$s" == "PAUSE" ]; then
mocp --stop
fi
cd /home/zaycakitayca/.moc/
mv playlist.m3u playlist99.m3u
mv playlist01.m3u playlist.m3u
bash -c 'i=1; ls playlist??.m3u | while read ; do mv "$REPLY" "playlist$(printf %02u $i).${REPLY//*.}" ; ((i++)) ; done'
if [ "$s" == "PLAY" ]; then
mocp --play
fi
exit 0

[Axa-Ru]

Вставте между строками вывод переменной s

s=$(mocp --info | grep State | sed 's/State: //gi')
echo $s
if [ "$s" == "PLAY" ] || [ "$s" == "PAUSE" ]; then

Я взял ваш текст закоментировал то чего у меня нет и все работает.

(Нажмите, чтобы показать/скрыть)

Код: [Выделить]

#!/bin/bash

#s=$(mocp --info | grep State | sed 's/State: //gi')
s=PLAY
if [ "$s" == "PLAY" ] || [ "$s" == "PAUSE" ]; then
  #mocp --stop
  echo $s
fi
#cd /home/zaycakitayca/.moc/
#mv playlist.m3u playlist99.m3u
#mv playlist01.m3u playlist.m3u
#bash -c 'i=1; ls playlist??.m3u | while read ; do mv "$REPLY" "playlist$(printf %02u $i).${REPLY//*.}" ; ((i++)) ; done'
if [ "$s" == "PLAY" ]; then
  #mocp --stop
  echo $s
fi
exit 0


[Disaster1986]

Заменил s=$(mocp --info | grep State | sed 's/State: //gi') на s=PLAY. Тот же результат. echo $s выводит PLAY, как и положено, а условие не работает. Ничего не понимаю...

[aliftin]

Опишите задачу словами. Что в какой момент должно происходить и какой нужен результат.

[RandomNT]

скрипт используется для выбора плэйлиста в MOC для 3-х и более плейлистов, запуск по хоткею (ибо задолбало открывать терминал для переключения плейлиста)

есть подозрение что скрипт исполняется не в баше

[Disaster1986]

есть подозрение что скрипт исполняется не в баше

Гениально! Дописал bash в запускаторе, всё заработало. Спасибо за помощь.

А в чём, если не секрет, оно могло запускаться до этого?

[ArcFi]

А в чём, если не секрет, оно могло запускаться до этого?

dash, sh

[Axa-Ru]

а вот это для чего тогда?

Код: [Выделить]

#!/bin/bash