[Comm] shell logical expressions

[Comm] shell logical expressions

[Eugene Prokopiev]

Здравствуйте!

Нужно выполнить несколько команд, если хотя бы одна вернула не 0, тоже 
вернуть не 0 и выругаться. Делаю примерно так:

#!/bin/sh

[ `drbdadm state all | awk -F'/' '{print $1}'` == "Primary" ]
partition=$?
grep drbd0 /proc/mounts > /dev/null
filesystem=$?

if [ $partition != '0' ] || [ $filesystem != '0' ] ; then
mail -s "HAD Monitor fail" root << EOF
drbd state:
`cat /proc/drbd`

mount state:
`grep drbd0 /proc/mounts`
EOF
exit 1
else
exit 0
fi

Но не оставляет ощущение кривизны куска кода с if. Можно правильнее/проще?

-- 
С уважением,
Прокопьев Евгений

Re: [Comm] shell logical expressions

[Michael Shigorin]

On Fri, Aug 17, 2007 at 02:29:05PM +0400, Eugene Prokopiev wrote:
> Нужно выполнить несколько команд, если хотя бы одна вернула не
> 0, тоже вернуть не 0 и выругаться. Делаю примерно так:
> 
> #!/bin/sh
> [ `drbdadm state all | awk -F'/' '{print $1}'` == "Primary" ]
> partition=$?
> grep drbd0 /proc/mounts > /dev/null
> filesystem=$?

Я бы переписал где-то так:

[ `drbdadm state all | awk -F'/' '{print $1}'` == "Primary" ] \
&& grep drbd0 /proc/mounts > /dev/null \
&& exit 0
mail .....

Почитай скрипты из пакетов (с)авторства ldv@ и legion@ (в т.ч.
control, alterator-*) -- там много интересного можно подметить
;-)

У Лёши за последний год или два стиль очень интересный и сжатый
стал.  При этом читается и работает.

> if [ $partition != '0' ] || [ $filesystem != '0' ] ; then
> mail -s "HAD Monitor fail" root << EOF
> drbd state:
> `cat /proc/drbd`
> 
> mount state:
> `grep drbd0 /proc/mounts`
> EOF
> exit 1
> else
> exit 0
> fi
> 
> Но не оставляет ощущение кривизны куска кода с if. Можно правильнее/проще?
> 

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [Comm] shell logical expressions

[Denis Kirienko]

Eugene Prokopiev пишет:
> Здравствуйте!
> 
> Нужно выполнить несколько команд, если хотя бы одна вернула не 0, тоже 
> вернуть не 0 и выругаться. Делаю примерно так:

cmd1 && cmd2 || { echo "ERROR" ; exit 1 ; }

не подойдет?

--
Денис

Re: [Comm] shell logical expressions

[Eugene Prokopiev]

> Я бы переписал где-то так:
> 
> [ `drbdadm state all | awk -F'/' '{print $1}'` == "Primary" ] \
> && grep drbd0 /proc/mounts > /dev/null \
> && exit 0
> mail .....

Т.е. exit 0 выполнится всегда? Это не то, что нужно

-- 
С уважением,
Прокопьев Евгений

Re: [Comm] shell logical expressions

[Eugene Prokopiev]

> Почитай скрипты из пакетов (с)авторства ldv@ и legion@ (в т.ч.
> control, alterator-*) -- там много интересного можно подметить
> ;-)
> 
> У Лёши за последний год или два стиль очень интересный и сжатый
> стал.  При этом читается и работает.

спасибо, почитаю

-- 
С уважением,
Прокопьев Евгений

Re: [Comm] shell logical expressions

[Eugene Prokopiev]

Denis Kirienko пишет:
> Eugene Prokopiev пишет:
>> Здравствуйте!
>>
>> Нужно выполнить несколько команд, если хотя бы одна вернула не 0, тоже 
>> вернуть не 0 и выругаться. Делаю примерно так:
> 
> cmd1 && cmd2 || { echo "ERROR" ; exit 1 ; }
> 
> не подойдет?

если cmd1 или cmd2 (или обе вместе - и это уже постановке задачи не 
удовлетворяет) вернули 0, выполнится то, что в {}, т.к. результат ИЛИ 
еще не известен, т.е. вернется не 0

если cmd1 и cmd2 вернули не 0, то {} не выполнится, т.к. результат ИЛИ 
уже известен, и ругани не будет, хотя вернется и правда не 0

-- 
С уважением,
Прокопьев Евгений

Re: [Comm] shell logical expressions

[Denis Kirienko]

Eugene Prokopiev пишет:

>>> Нужно выполнить несколько команд, если хотя бы одна вернула не 0, тоже 
>>> вернуть не 0 и выругаться. Делаю примерно так:

>> cmd1 && cmd2 || { echo "ERROR" ; exit 1 ; }
>> не подойдет?

> если cmd1 или cmd2 (или обе вместе - и это уже постановке задачи не 
> удовлетворяет) вернули 0, выполнится то, что в {}, т.к. результат ИЛИ 
> еще не известен, т.е. вернется не 0
> 
> если cmd1 и cmd2 вернули не 0, то {} не выполнится, т.к. результат ИЛИ 
> уже известен, и ругани не будет, хотя вернется и правда не 0

Как раз наоборот. Если cmd1 и cmd2 вернули обе ноль, то {} не
выполняется и вернется 0.

Если хотя бы одна из cmd1 или cmd2 вернули не ноль, то выполняется {}.

Поэкспериментируйте, заменив команды на true (возвращает 0) и false
(возвращает 1).


Но Мишин вариант мне нравится больше.

--
Денис

Re: [Comm] shell logical expressions

[Michael Shigorin]

On Mon, Aug 20, 2007 at 10:17:50AM +0400, Eugene Prokopiev wrote:
> >[ `drbdadm state all | awk -F'/' '{print $1}'` == "Primary" ] \
> >&& grep drbd0 /proc/mounts > /dev/null \
> >&& exit 0
> >mail .....
> Т.е. exit 0 выполнится всегда?

Нет.  Проверь с заменой команд на true и false в комбинациях.
(я проверил дальше по треду, где были cmd1/cmd2)

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [Comm] shell logical expressions

[Eugene Prokopiev]

> Поэкспериментируйте, заменив команды на true (возвращает 0) и false
> (возвращает 1).

Ага! А я то думал, что 0 == false, а все остальное == true :)

В shell это, выходит совсем не так ;)

-- 
С уважением,
Прокопьев Евгений

Re: [Comm] shell logical expressions

[Michael Shigorin]

On Mon, Aug 20, 2007 at 01:35:58PM +0400, Eugene Prokopiev wrote:
> > Поэкспериментируйте, заменив команды на true (возвращает 0) и
> > false (возвращает 1).
> Ага! А я то думал, что 0 == false, а все остальное == true :)
> В shell это, выходит совсем не так ;)

И в C это тоже не так... :)

"Правда всегда одна", а вот ошибок можно насовершать разных.
Соответственно и кодов ошибок надо иметь достаточно.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [Comm] shell logical expressions

[Andrew Borodin]

On Mon, Aug 20, 2007 at 01:35:58PM +0400, Eugene Prokopiev wrote:
> > Поэкспериментируйте, заменив команды на true (возвращает 0) и false
> > (возвращает 1).

> Ага! А я то думал, что 0 == false, а все остальное == true :)

> В shell это, выходит совсем не так ;)

Принято, что если программа завершается нормально, то она 
возвращает 0. Это Для test это true.

-- 

С уважением,
А. Бородин.