While i if w bashu

Question

Zaraz mnie coś rozniesie. Co znowu robię nie tak? 

Chciałbym zrobić menu, ale w pętli. I zamiast wyświetlać się menu i czekać na wybranie opcji przez użytkownika to ta pętla działa przez cały czas. 

 

 

#!/bin/bash

#menu

loop=1
while [ $loop == 1 ] ; do
		echo "********************************************************"
		echo "*  1 - opcja1                         *"
		echo "*  2 - opcja2                          *"
		echo "*  3 - opcja3                  *"
		echo "*  4 - Wyście                                                         *"
		echo "********************************************************"	
		

		read opt

		if [ $opt == 1 ]; then
			echo "opcja1 "
			read limit
			echo $limit
			clear
			
		elif [ $opt == 2 ]; then
			echo "opcja2"
			
		elif [ $opt == 3 ]; then
				echo "opcja 3"
				
		elif [ $opt == 4 ]; then
			echo "Czy na pewno chcesz wyjść Y/N?"
			read exit
			
			if [ $exit == "Y" || "y" ]; then
			$loop=0
			
		else
			echo "Coś poszło nie tak. Spróbuj jeszzcze raz."
		fi
		
 
done 

Zgłasza mi cały czas problem z tym done. 

Comments

Jaki błąd ci zgłasza?

Tak w ogóle Bash nie jest najlepszym językiem do pisania skryptów, ze względu na jego toporność. Lepiej użyć np. Pythona, który jest bezpieczniejszy i łatwiej się go debugguje.

---

Wiem, że lepiej. Z Pythonem nie mam problemów. Ale to jest skrypt na zaliczenie i muszę go napisać w Bashu właśnie

Answer 1

Jeśli nie musisz to nie używaj basha. Python sprawdzi się o wiele lepiej. Wystarczy dodać pythonowy shebang i skrypt uruchamia się tak samo jak bashowy. A chyba ciężko dzisiaj o linuxa bez pythona.

#!/usr/bin/env python3

Ale do rzeczy, co bylo nie tak:

if [ $exit == "Y" || "y" ]; then
            $loop=0
             
        else

brakuje fi po ifie. I w tym samym ifie nie możesz zrobić tak: 

[ $exit == "Y" || "y" ]

są dwie opcje:

[ $exit == "Y" ] || [ $exit == "y" ]
# lub
[ $exit == "Y" -o $exit == "y" ]

lub operator podwójnych nawiasów, który jest troszkę inteligentniejszy:

[[ $exit == "Y" || $exit == "y" ]]

 

Answer 2

Brakuje fi przed done :)

Gdybyś poprawnie sformatował kod, to byś zauważył błąd.