Как сделать короче?

0

1

Ребята, подскажите плиз как сделать короче, мне для красоты нужно.

if [ $tunnel_last_state -eq 1 ] && [ $internet_last_state -eq 1  ]; 
then 
   echo none > /sys/class/leds/red/trigger
   echo default-on > /sys/class/leds/blue/trigger
fi
if [ $tunnel_last_state -eq 1 ] && [ $internet_last_state -eq 0  ]; 
then 
   echo default-on > /sys/class/leds/red/trigger
   echo none > /sys/class/leds/blue/trigger
fi
if [ $tunnel_last_state -eq 0 ] && [ $internet_last_state -eq 1  ]; 
then 
   echo none > /sys/class/leds/red/trigger
   echo timer > /sys/class/leds/blue/trigger
fi
if [ $tunnel_last_state -eq 0 ] && [ $internet_last_state -eq 0  ]; 
then
   echo timer > /sys/class/leds/red/trigger
   echo none > /sys/class/leds/blue/trigger
fi

Спасибо!

Ссылка

←	Запросы отправляются дважды

Какие игры под linux умеют динамическое разрешение? ^.^

→

Первое, что приходит в голову: [code]set_leds() { echo «$1» > /sys/class/leds/red/trigger echo «$2» > /sys/class/leds/blue/trigger }

if [ $tunnel_last_state -eq 1 ]; then if [ $internet_last_state -eq 1 ]; then set_leds none default-on else set_leds default-on none fi else if [ $internet_last_state -eq 1 ]; then set_leds none timer else set_leds timer none fi [/code]

anonymous
(10.11.19 18:56:16 MSK)

Ссылка

Первое, что приходит в голову:

set_leds() {
    echo "$1" > /sys/class/leds/red/trigger
    echo "$2" > /sys/class/leds/blue/trigger
}

if [ $tunnel_last_state -eq 1 ]; then
    if [ $internet_last_state -eq 1  ]; then
        set_leds none default-on
    else
        set_leds default-on none
    fi
else
    if [ $internet_last_state -eq 1  ]; then
        set_leds none timer
    else
        set_leds timer none
fi

anonymous
(10.11.19 18:56:56 MSK)

Ну, например, можно перечислить состояния в таблице:

declare -A states
states=(
[0,0]="none default-on"
[1,0]="default-on none"
[0,1]="none timer"
[1,1]="timer none"
)
read red blue <<<${states["$tunnel_last_state,$internet_last_state"]}
echo $red > /sys/class/leds/red/trigger
echo $blue > /sys/class/leds/blue/trigger

legolegs ★★★★★
(10.11.19 19:13:53 MSK)

Ответ на: комментарий от anonymous 10.11.19 18:56:56 MSK

Это красивше чем у меня было и работает, спасибо Вам товарищ!

adminlinwin ★
(10.11.19 19:16:57 MSK) автор топика
Последнее исправление: adminlinwin 10.11.19 19:19:21 MSK (всего исправлений: 1)

как сделать короче, мне для красоты нужно

И так достаточно коротко. Вот если бы вариантов было больше, то можно было бы заморочиться на массив с вариантами.

Разве что вот это

   echo timer > /sys/class/leds/red/trigger
   echo none > /sys/class/leds/blue/trigger

вынеси в функцию (как советует анонимус)

setleds() {
    echo "$1" > /sys/class/leds/red/trigger
    echo "$2" > /sys/class/leds/blue/trigger
}

и тогда

[ $tunnel_last_state -eq 1 ] && [ $internet_last_state -eq 1  ] && setleds none default-on
[ $tunnel_last_state -eq 1 ] && [ $internet_last_state -eq 0  ] && setleds default-on none
[ $tunnel_last_state -eq 0 ] && [ $internet_last_state -eq 1  ] && setleds none timer
[ $tunnel_last_state -eq 0 ] && [ $internet_last_state -eq 0  ] && setleds timer none

no-such-file ★★★★★
(10.11.19 19:20:22 MSK)

Ответ на: комментарий от legolegs 10.11.19 19:13:53 MSK

Спасибо, выглядит - люкс, проверить теперь смогу только завтра вечерком.

adminlinwin ★
(10.11.19 19:23:02 MSK) автор топика

Ссылка

Ответ на: комментарий от adminlinwin 10.11.19 19:16:57 MSK

и работает

Работает не совсем так, как было. Что если tunnel_last_state или internet_last_state не равны 0 или 1? Нужно дополнительно проверять и получаться уже не короче.

no-such-file ★★★★★
(10.11.19 19:24:38 MSK)

Ответ на: комментарий от no-such-file 10.11.19 19:24:38 MSK

Вы правы, но у меня там заведомо либо 0 либо 1 )

adminlinwin ★
(10.11.19 19:28:37 MSK) автор топика

Ответ на: комментарий от adminlinwin 10.11.19 19:28:37 MSK

Ну ты всё равно добавь проверку

[ $red ] && [ $blue ] && (echo $red > /sys/class/leds/red/trigger; echo $blue > /sys/class/leds/blue/trigger)

заведомо либо 0 либо 1

Все так говорят, а потом херак, и ракета на старте взрывается.

no-such-file ★★★★★
(10.11.19 19:32:48 MSK)

Ответ на: комментарий от no-such-file 10.11.19 19:32:48 MSK

Не могу не согласиться )

adminlinwin ★
(10.11.19 19:43:36 MSK) автор топика
Последнее исправление: adminlinwin 10.11.19 19:48:25 MSK (всего исправлений: 1)

Ссылка

Ответ на: комментарий от legolegs 10.11.19 19:13:53 MSK

доработка на случай если в tunnel_last_state и internet_last_state чт-то кроме 0 и 1:

declare -A states
states=(
[0,0]="none default-on"
[1,0]="default-on none"
[0,1]="none timer"
[1,1]="timer none"
)
read red blue <<<"${states["$tunnel_last_state,$internet_last_state"]:-none none}"
echo $red > /sys/class/leds/red/trigger
echo $blue > /sys/class/leds/blue/trigger

legolegs ★★★★★
(10.11.19 19:49:00 MSK)

Ответ на: комментарий от legolegs 10.11.19 19:49:00 MSK

Как же хорошо, что Вы есть!

adminlinwin ★
(10.11.19 19:56:25 MSK) автор топика

Ссылка

Ответ на: комментарий от legolegs 10.11.19 19:49:00 MSK

Как бы не оказалось, что это OpenWrt, судя по лампочкам и одиночным [ ], то есть dash.

leds_tun_0=timer
leds_tun_1=default-on
leds_inet_0=red
leds_inet_1=blue

if [ $tunnel_last_state -le 1 -a $internet_last_state -le 1  ]; then
        eval echo \$leds_tun_$tunnel_last_state \> /sys/class/leds/\$leds_inet_$internet_last_state/trigger
        eval echo none \> /sys/class/leds/\$leds_inet_$((internet_last_state^1))/trigger
fi

vodz ★★★★★
(10.11.19 21:55:44 MSK)
Последнее исправление: vodz 10.11.19 21:56:09 MSK (всего исправлений: 1)

Ссылка

Ответ на: комментарий от no-such-file 10.11.19 19:20:22 MSK

На 16 букв меньше )

[[ $tunnel_last_state != 0 && $internet_last_state != 0 ]] && setleds none default-on
[[ $tunnel_last_state != 0 && $internet_last_state == 0 ]] && setleds default-on none
[[ $tunnel_last_state == 0 && $internet_last_state != 0 ]] && setleds none timer
[[ $tunnel_last_state == 0 && $internet_last_state == 0 ]] && setleds timer none

Deleted
(10.11.19 22:47:40 MSK)

Ответ на: комментарий от Deleted 10.11.19 22:47:40 MSK

Вот ещё проще:

case $tunnel_last_state.$internet_last_state in
 1.1) setleds none default-on;;
 1.0) setleds default-on none;;
 0.1) setleds none timer;;
 0.0) setleds timer none;;
esac

NeXTSTEP ★★
(11.11.19 07:24:21 MSK)

Ссылка

Ответ на: комментарий от Deleted 10.11.19 22:47:40 MSK

На 16 букв меньше )

Если сравниваете строки, то вместо $internet_last_state != 0 тут привильнее $internet_last_state = 1, это точно по ТЗ ТС-а, когда не [0,1] - ничего не делать, заодно даже круче, так как не сломается для internet_last_state=*

vodz ★★★★★
(11.11.19 10:12:21 MSK)
Последнее исправление: vodz 11.11.19 10:13:01 MSK (всего исправлений: 2)

Ответ на: комментарий от vodz 11.11.19 10:12:21 MSK

Да, это OpenWrt. С учётом замечаний, взял такой вариант :

set_leds() {
    echo "$1" > /sys/class/leds/red/trigger
    echo "$2" > /sys/class/leds/blue/trigger
}
[[ $tunnel_last_state == 1 && $internet_last_state == 1 ]] && set_leds none default-on
[[ $tunnel_last_state == 1 && $internet_last_state == 0 ]] && set_leds default-on none
[[ $tunnel_last_state == 0 && $internet_last_state == 1 ]] && set_leds none timer
[[ $tunnel_last_state == 0 && $internet_last_state == 0 ]] && set_leds timer none