[BASH] while loop, testen of input numeriek is

[ZodiacQ]

Hoi,

Ik ben een script aan het schrijven zodat gebruikers hun alarm zelf kunnen instellen ( 1 maal per uur). Dit alarm werkt via de cron.

Ze mogen dus enkel een getal invullen tussen 0 en 59, als ze iets anders invullen, moet het script blijven vragen achter de waarde.

Ik dacht een while-loop te gebruiken, maar ik zit hier een beetje vast wanneer er een letter ipv een cijfer gebruikt wordt.

Ik heb al wat opzoekwerk verricht, en heb al het juiste statement gevonden voor in mijn if-statement. Ik heb dit eerst proberen te kopiëren naar de while-statement, maar dit geeft blijkbaar fouten

code:

#!/bin/bash



         while [ "$B" -lt "00" ] || [ "$B" -gt "59" ]
         do
            echo "please give the alarm-threshold (a number between 0 and 59)"
            echo "example: 15 \n"

            read B

            if echo "$B" | grep -q -v "[^0-9]"
            then
               if [ "$B" -ge "00" ] && [ "$B" -le "59" ]
               then
                  echo ""
                  echo "threshold set to $B \n"
               else
                  echo ""
                  echo "number is not between 00 and 59 : $B"
                  echo "threshold is not set \n"
               fi
            else
               echo "$B is not a number"
            fi
         done

Alle hulp is welkom!

[ Voor 1% gewijzigd door ZodiacQ op 07-09-2010 11:32 . Reden: schrijffouten :) ]

[NMe]

Ik heb weinig kennis van bash, maar volgens mijn wil je je or ( || ) veranderen in een and ( && ). Je if wilde je waarschijnlijk uitvoeren als het wel een getal is, je while wil je uitvoeren zolang het geen getal is, waardoor je ofwel de hele expressie moet ontkennen met een not-operator, ofwel alle boolean operators omwisselen.

Verder: Waar hoort mijn topic?

PRG >> NOS

[Wirf]

Altijd leuk, stoeien met Bash :-)

Probeer dit eens

Bash:

#!/bin/bash
B=-1
while [ "$B" -lt 59 ] || [ "$B" -gt 00 ]
do
        echo "please give the alarm-threshold (a number between 0 and 59)"
        echo "example: 15"

        read B

        if echo "$B" | grep -q "^[0-9]*$"; then
                if [ "$B" -ge "00" ] && [ "$B" -le "59" ]; then
                        echo ""
                        echo "threshold set to $B"
                        break
                else
                        echo
                        echo "number is not between 00 and 59 : $B"
                        echo "threshold is not set"
                fi
        else
                echo "$B is not a number"
               B=-1
        fi
done

[Bucket]

Je zou dit kunnen proberen:

a=100

if [ `expr $a + 1 2> /dev/null` ] ; then
echo $a is numeric ;
else
echo $a is not numeric ;
fi ;

En binnen bash kan je ook typeset -i gebruiken, zodat je variable altijd een integer is.

[Aetos]

Als je een letter gebruikt ipv een cijfer gaat het in eerste instantie goed. Je detecteert dat het een niet nummerieke invoer is. Maar in de while lus staat dan ineens

code:

"a" -lt 00 || "a" -gt 59

Je moet dus voorkomen dat de while lus ooit gebruikt kan worden terwijl B een nonnumerieke inhoud heeft.

Hint: De eerste aanpassing van wirf is handig. Maar ik zou hem een paar keer gebruiken.


Ik snap trouwens niet waarom je \n gebruikt.

code:

threshold is not set \n
please give the alarm-threshold (a number between 0 and 59)
example: 15 \n
a
a is not a number
uur.sh: line 4: [: a: integer expression expected
uur.sh: line 4: [: a: integer expression expected

[H!GHGuY]

Deze functie gebruik ik om numeric checks te doen binnen bash.
Het voordeel is hier dat je geen grep of andere subprocessen moet starten:

Bash:

function isnum() {
  local VAR="$1"
  case "$VAR" in
     *[^0-9]*)
        return 1
        ;;
  esac
  return 0
}

[NMe]

H!GHGuY schreef op dinsdag 07 september 2010 @ 12:49:
Deze functie gebruik ik om numeric integer checks te doen binnen bash.

[ZodiacQ]

Tnx voor alle reacties.

@wirf: Dit werkt, maar als ik bvb niets ingeef en gewoon op Enter druk, dan is het spelletje uit.

kan ik soms werken met null?

code:

while [ "$B" -lt 59 ] || [ "$B" -gt 00 ] || [ "$B" != NULL ]

@Aetos: Ja, de \n is een foutje van mij

@H!GHGuY: hier heb ik ook mee zitten knoeien (had dit ook al ergens op internet gezien), maar hoe implementeer ik zo een functie in een while lus, samen met de andere voorwaarden?
Iets in de volgende zin:

code:

while [ isnum $B ] || [ "$B" -lt 59 ] || [ "$B" -gt 00 ]
do
...
done

[ Voor 13% gewijzigd door ZodiacQ op 07-09-2010 13:33 ]

[NMe]

Een lege string is geen null.

[ZodiacQ]

NMe schreef op dinsdag 07 september 2010 @ 13:39:
Een lege string is geen null.

tnx voor de hint.

[sam.vimes]

Bash:

#!/bin/bash
while read -p"number of minutes: " B
do
  case "$B" in
    [0-9]|[0-9][0-9]) test $B -lt 60 && break; echo from 0 to 59 please ;;
    *) echo enter a number of 1 or 2 digits ;;
  esac
done
test -z "$B" && {
  echo emergency exit
  exit 2
}
echo now do something with $B

[H!GHGuY]

ZodiacQ schreef op dinsdag 07 september 2010 @ 13:30:
@H!GHGuY: hier heb ik ook mee zitten knoeien (had dit ook al ergens op internet gezien), maar hoe implementeer ik zo een functie in een while lus, samen met de andere voorwaarden?

code:

while isnum $B && [ "$B" -lt 59 -o "$B" -gt 00 ]
do
...
done

zoiets?

[jayvol09]

NMe schreef op dinsdag 07 september 2010 @ 13:39:
Een lege string is geen null.

Als je die door pipe't naar wc -m zal ie denk wel 0 geven.

[Sprite_tm]

Regexps to the rescue!

code:

#!/bin/bash

echo "Nummer, 0-59, jeweetwel."
read input
while ! [[ $input =~ ^[0-5]?[0-9]$ ]]; do
    echo "$input is nie goe nie :/"
    read input
done

echo "Woei, $input!"

Edit: Laat Nja, een extra oplossing is nooit weg.

[ Voor 10% gewijzigd door Sprite_tm op 10-09-2010 15:36 ]