Problem z wyborem dni tygodnia w skrypcie

[piotrino]

Witam. Mam taki skrypt zmodyfikowany według mnie:
[code
#!/bin/bash
echo "Podaj cyfrę dnia tygodnia"
read d
case   "$d" in
  "1") echo "Poniedziałek" ;;
  "2") echo "Wtorek" ;;
  "3") echo "Środa" ;;
  "4") echo "Czwartek" ;;
  "5") echo "Piątek" ;;
  "6") echo "Sobota" ;;
  "7") echo "Niedziela";;
   if [ $d -le 7 ]
 then
 *) echo "DZISIAJ JEST: $d"
else
echo "Podales zbyt wysoką liczbe wiekszą od 7"
echo "Podaj liczbę od 1 - 7 tyle ile wynosi tydzien"
essac] [/code]
Chcę doprowadzić żeby jak wstawię cyfre tygodnia ponad "7" to dostaję komunikat " Podałeś zbyt wysoką liczbę większa od7"  i dalej analogicznie do dalszej treści skkrtypu.
Po uruchomieniu skryptu i wpisaniu cyfry tygodnia:

Kod: [Zaznacz]

 [root@centos programowanie]# ./case
Podaj cyfrę dnia tygodnia
1
./case: line 13: błąd składni przy nieoczekiwanym znaczniku `['
./case: line 13: `   if [ $d -le 7 ]'
[root@centos programowanie]#

dostaję powyższy komunikat błedu.... hm... co robię zle, gdzie zrobiłem bład, składnia  wygląda ok na moje...

[ultr]

Popsułeś tag code i połowy skryptu nie widać:)

Kod: [Zaznacz]

#!/bin/bash
echo "Podaj cyfrę dnia tygodnia"
read d
case   "$d" in
  "1") echo "Poniedziałek" ;;
  "2") echo "Wtorek" ;;
  "3") echo "Środa" ;;
  "4") echo "Czwartek" ;;
  "5") echo "Piątek" ;;
  "6") echo "Sobota" ;;
  "7") echo "Niedziela";;
   if [ $d -le 7 ]
 then
 *) echo "DZISIAJ JEST: $d"
else
echo "Podales zbyt wysoką liczbe wiekszą od 7"
echo "Podaj liczbę od 1 - 7 tyle ile wynosi tydzien"
essac

[ultr]

A skrypt nie działa, bo
1) wstawiasz inny kod przed zamknięciem case..esac.
2) esac nie essac
3) nie podałeś fi za if-em
4) *) służy do obsłużenia pozostałych przypadków w case (taki else)

Fixed:

Kod: [Zaznacz]

#!/bin/bash

read -p "Podaj liczbe dnia tygodnia: " d

if [ $d -gt 7 ] || [ $d -lt 1 ]; then
  echo -e "Podales niepoprawna liczbe.\\nPodaj liczbę od 1-7, tyle ile dni ma tydzien."
  exit 1
fi

echo -n "DZISIAJ JEST "
case "$d" in
  "1") echo "PONIEDZIALEK" ;;
  "2") echo "WTOREK" ;;
  "3") echo "SRODA" ;;
  "4") echo "CZWARTEK" ;;
  "5") echo "PIATEK" ;;
  "6") echo "SOBOTA" ;;
  "7") echo "NIEDZIELA";;
esac

exit 0

[piotrino]

Wielkie dzięki, o to mi chodziło. Niby kombinowałem dobrze ale nie w tej kolejności... Dużo nauki przede mną jeszce Dzieki

P.S Czy jest mozliwość kontynuacji wybierania dnia tygodnia po tym komunikacie:

Kod: [Zaznacz]

 echo -e "Podales niepoprawna liczbe.\\nPodaj liczbę od 1-7, tyle ile dni ma tydzien."

bez wpisywania kodu od początku? pOzdrawiam

[darkdancer]

P.S Czy jest mozliwość kontynuacji wybierania dnia tygodnia po tym komunikacie

Jest możliwość, np. za pomocą pętli while:

Kod: bash [Zaznacz]

while : ; do
  read -p "Podaj liczbe dnia tygodnia: " d
  if [ $d -gt 7 ] || [ $d -lt 1 ]; then
    echo -e "Podales niepoprawna liczbe.\\nPodaj liczbę od 1-7, tyle ile dni ma tydzien."
  else
    break
  fi
done

Albo tak:

Kod: bash [Zaznacz]

read -p "Podaj liczbe dnia tygodnia: " d
while [ $d -gt 7 ] || [ $d -lt 1 ]; do
  echo -e "Podales niepoprawna liczbe.\\nPodaj liczbę od 1-7, tyle ile dni ma tydzien."
  read -p "Podaj liczbe dnia tygodnia: " d
done

Oczywiście w obydwu przypadkach zakładamy, że "juser jest mondry i nie wpisze np. szejść".

[piotrino]

"juser jest mondry i nie wpisze np. szejść".

Kod: [Zaznacz]

 user jest mądry i nie wpisze np. sześć

Sorki za poprawkę, ale nie mogłem się powstrzymać
Co do skryptu, trochę pomógł, ale chodzi mi żeby po wpisaniu liczby większej od 7 i mniejszej od 1 dawał mi komunikat o błedzie jak daje w skrypcie a po tym miał dalej możliwość wpisania poprawnej liczby od 1-7 i po  wpisaniu pokazuje się wynik" Dziś jest poniedziałek .... niedziela " itp... Ale pokombinuję jeszcze... W koncu się uczę basha...

Pozdrawiam

[ultr]

chodzi mi żeby po wpisaniu liczby większej od 7 i mniejszej od 1 dawał mi komunikat o błedzie jak daje w skrypcie a po tym miał dalej możliwość wpisania poprawnej liczby od 1-7 [...]

Tak właśnie działają oba fragmenty kodu zaproponowane przez darkdancera. Wystarczy dodać do tego case-a z wyświetlaniem wyniku.

Co do nauki Basha, polecam na początek http://dief.republika.pl/main.html i ewentualnie inne kursy z Google.

A dla wprawki polecam 1) tą instrukcję case zastąpić tablicą nazw dni, oraz 2) odczytywać dzień tygodnia z polecenia "date" zamiast od użytkownika.

[piotrino]

chodzi mi żeby po wpisaniu liczby większej od 7 i mniejszej od 1 dawał mi komunikat o błedzie jak daje w skrypcie a po tym miał dalej możliwość wpisania poprawnej liczby od 1-7 [...]

Tak właśnie działają oba fragmenty kodu zaproponowane przez darkdancera. Wystarczy dodać do tego case-a z wyświetlaniem wyniku.

Co do nauki Basha, polecam na początek http://dief.republika.pl/main.html i ewentualnie inne kursy z Google.

A dla wprawki polecam 1) tą instrukcję case zastąpić tablicą nazw dni, oraz 2) odczytywać dzień tygodnia z polecenia "date" zamiast od użytkownika.

Dzięki. I tu ma pytanie, gdzie mogę dodać tego case w kodzie, w którym miejscu? Kombinuję cały czas ale mam błedy w skrypcie przy uruchomieniu...

[ultr]

Pod tym kodem. Ten kod wczyta zmienną d, a potem case sprawdzi jej wartość i wyświetli wynik. To chyba dość proste

[piotrino]

Pod tym kodem. Ten kod wczyta zmienną d, a potem case sprawdzi jej wartość i wyświetli wynik. To chyba dość proste

Hm....nie takie proste, jak wpisuję case pod tym kodem to błedy się sypią...
Podaję cały skrypt po modyfikacji:

Kod: [Zaznacz]

#!/bin/bash

read -p "Podaj liczbe dnia tygodnia: " d

if [ $d -gt 7 ] || [ $d -lt 1 ]; then
  echo -e "Podales niepoprawna liczbe.\\nPodaj liczbę od 1-7, tyle ile dni ma tydzien."
read -p "Podaj liczbe dnia tygodnia: " d
while [ $d -gt 7 ] || [ $d -lt 1 ]; do
  echo -e "Podales niepoprawna liczbe.\\nPodaj liczbę od 1-7, tyle ile dni ma tydzien."
  read -p "Podaj liczbe dnia tygodnia: " d
done
 exit 1
fi
echo -n "DZISIAJ JEST "
case "$d" in
  "1") echo "PONIEDZIALEK" ;;
  "2") echo "WTOREK" ;;
  "3") echo "SRODA" ;;
  "4") echo "CZWARTEK" ;;
  "5") echo "PIATEK" ;;
  "6") echo "SOBOTA" ;;
  "7") echo "NIEDZIELA";;
esac
exit 0

Jak rozumuję i dobrze odczytałem twóją radę

Kod: [Zaznacz]

 Wystarczy dodać do tego case-a z wyświetlaniem wyniku

to dodaję case pod poleceniem done? Ale sypią się błedy po uruchomieniu skryptu. Jeśli chodzi o naukę basha to preferuję naukę od podstaw ale na przepisywaniu gotowych skryptów, ale probowanie modyfikacji tych skryptów przez dodawanie do tych skryptów dodatkowych parametrów. W ten sposób się uczę...

[ultr]

W którym miejscy tego wątku widzisz tego potworka, którego masz w swoim kodzie przed tym case-m? Czy tak trudno jest poprawnie użyć kopiuj-wklej?

Może przeanalizuj podane w wątku fragmenty linia po linii i pomyśl przez chwilę nad tym (prostym) kodem, zamiast bezmyślnie go sklejać. Każda z linii zamieszczonego w tym wątku kodu mówi praktycznie sama za siebie co robi.

[piotrino]

W którym miejscy tego wątku widzisz tego potworka, którego masz w swoim kodzie przed tym case-m? Czy tak trudno jest poprawnie użyć kopiuj-wklej?

Może przeanalizuj podane w wątku fragmenty linia po linii i pomyśl przez chwilę nad tym (prostym) kodem, zamiast bezmyślnie go sklejać. Każda z linii zamieszczonego w tym wątku kodu mówi praktycznie sama za siebie co robi.

Dzięki, jednak jak trochę nad tym posiedziałem to w koncu zrobiłem.  Opieprz dostałem zasadnie od ciebie... Zamiast samemu kombinować, to czekam na gotowca od was.... Przy okazji tak to zrobiłem:

Kod: [Zaznacz]

  #!/bin/bash

 
read -p "Podaj liczbe dnia tygodnia: " d
while [ $d -gt 7 ] || [ $d -lt 1 ]; do
  echo -e "Podales niepoprawna liczbe.\\nPodaj liczbę od 1-7, tyle ile dni ma tydzien."
  read -p "Podaj liczbe dnia tygodnia: " d
done
if [ $d -gt 7 ] || [ $d -lt 1 ]; then
  echo -e "Podales niepoprawna liczbe.\\nPodaj liczbę od 1-7, tyle ile dni ma tydzien."
  exit 1
fi

echo -n "DZISIAJ JEST "
case "$d" in
  "1") echo "PONIEDZIALEK" ;;
  "2") echo "WTOREK" ;;
  "3") echo "SRODA" ;;
  "4") echo "CZWARTEK" ;;
  "5") echo "PIATEK" ;;
  "6") echo "SOBOTA" ;;
  "7") echo "NIEDZIELA";;

esac

exit 0

Offline

Teraz pobawię się w wywoływanie "date".

[ultr]

Trochę powtórzeń kodu:

read -p "Podaj liczbe dnia tygodnia: " d
while [ $d -gt 7 ] || [ $d -lt 1 ]; do
____echo -e "Podales niepoprawna liczbe.\\nPodaj liczbę od 1-7, tyle ile dni ma tydzien."
____read -p "Podaj liczbe dnia tygodnia: " d
done
if [ $d -gt 7 ] || [ $d -lt 1 ]; then
____echo -e "Podales niepoprawna liczbe.\\nPodaj liczbę od 1-7, tyle ile dni ma tydzien."
____exit 1
fi

Od razu widać, że tego całego tego if-a można wyrzucić, bo wcześniejszy while sprawdza ten sam warunek. Skoro pętla while się zakończy, to znaczy,. że warunek nie jest już spełniony, a więc i ten if sprawdzający ten sam warunek nigdy się nie wykona. I dobrze, że się nie wykonuje, bo "exit 1" zdecydowanie nie jest tym, co chcesz.

Zostaje jeszcze powtórzenie:

Kod: [Zaznacz]

read -p "Podaj liczbe dnia tygodnia: " d

Tutaj lepszym rozwiązaniem jest ten drugi kod zaproponowany przez darkdancera. W nim żaden kod się nie powtarza:

Kod: [Zaznacz]

while : ; do
  read -p "Podaj liczbe dnia tygodnia: " d
  if [ $d -gt 7 ] || [ $d -lt 1 ]; then
    echo -e "Podales niepoprawna liczbe.\\nPodaj liczbę od 1-7, tyle ile dni ma tydzien."
  else
    break
  fi
done

.

Co do polecenia "date", przydadzą Ci się konstrukcje $() lub `` oraz przekazywanie danych za pomocą |, tzw. pipe. Powodzenia.

[piotrino]

OK, zrobiłem tak:

Kod: [Zaznacz]

  #!/bin/bash

 
read -p "Podaj liczbe dnia tygodnia: " d
while [ $d -gt 7 ] || [ $d -lt 1 ]; do
  echo -e "Podales niepoprawna liczbe.\\nPodaj liczbę od 1-7, tyle ile dni ma tydzien."
  read -p "Podaj liczbe dnia tygodnia: " d
done

echo -n "DZISIAJ JEST "
case "$d" in
  "1") echo "PONIEDZIALEK" ;;
  "2") echo "WTOREK" ;;
  "3") echo "SRODA" ;;
  "4") echo "CZWARTEK" ;;
  "5") echo "PIATEK" ;;
  "6") echo "SOBOTA" ;;
  "7") echo "NIEDZIELA";;

esac

Też działa, przy moim ostatnim kodzie z if też działało, tylko tu mniej kodu zostało zastosowane. Ale czaję powoli o co chodzi.  Dzięki za wskazówki co "date" .....