[Bash] problema con output programma

[ilpinguino89]

Salve a tutti per esercitarmi sto creando un semplicissimo sistema di login.

Sono riuscito quasi in tutto tranne che nel giusto funzionamento del ciclo "while" infatti continua all'infinito.

Io vorrei che questo ciclo si interrompesse dopo tre volte che è stato tentato di "eseguire l'accesso" e sempre se l'accesso non è stato eseguito lo script dovrebbe eseguire un comando in background.

Credo di essere riuscito ad inserire un contatore che però non funziona qualcuno potrebbe suggerirmi qualcosa?

Questo è il codice:

Codice: Seleziona tutto

#! /bin/bash

pinguino="pinguino"
password="pinguino89"
stop="3"
contatore="0"

#marco1=input
#password1=input

EXIT_SUCCESS=0
#EXIT_FAILURE=1

while true [ $contatore -ne $stop ] ; do
   echo "Inserisci nome e password:"
   read pinguino1
   read password1
   if [[ "$pinguino1" = "$pinguino" ]] && [[ "$password1" = "$password" ]]; then
      break
   else
      echo "Input errato"
   fi
let "contatore=$contatore+1"
done


[gigicookie]

Prova cosi per il while:

Codice: Seleziona tutto

while [ contatore <= 3 ]; do
read pinguino1
read password1
if [ pinguino1 == pinguino ]; then
if [ password1 == password ]; then
break
else
echo "Password errata"
fi
else
echo "Nome utente errato"
fi


[M@ttia]

Prova così...

Uhm, però a ogni tentativo errato devi incrementarlo il contatore

Credo di essere riuscito ad inserire un contatore che però non funziona qualcuno potrebbe suggerirmi qualcosa?

• = assegna un valore, == li confronta fra loro, quindi a te serve testare if [[ "$pinguino1" == "$pinguino" ]], ecc.
• Non sono per nulla esperto di bash e quindi non voglio confondere con C++ e simili, ma per quale motivo il tuo ciclo while inizia con un TRUE e poi una condizione? Non ti basta fare, come suggerito, un
  while [ $contatore < $stop ] ; do ?
• Così com'è il codice non riesci comunque a distinguere un login effettuato da un fallimento: ti serve una flag (tipo "accessoeffettuato=true") nell'IF.

[gigicookie]

Prova così...

Uhm, però a ogni tentativo errato devi incrementarlo il contatore

Ops. Me n' ero dimenticato. Basta aggiungere un

Codice: Seleziona tutto

contatore=$(($contatore + 1))

nell' else
Quindi si ottiene:

Codice: Seleziona tutto

while [ $contatore <= 3 ]; do
if [ $pinguino1 == $pinguino]; then
if [ $password1 == $password]; then
break
else
contatore=$(($contatore + 1))
echo "Password errata"
fi
else
contatore=$(($contatore + 1))
echo "User errato"
fi


a te serve testare if [[ "$pinguino1" == "$pinguino" ]]

La sintassi di if in bash è if [ condizione ], quindi le doppie parentesi quadre non sono necessarie.

Così com'è il codice non riesci comunque a distinguere un login effettuato da un fallimento: ti serve una flag (tipo "accessoeffettuato=true") nell'IF

Non è detto, poichè si potrebbe semplicemente controllare se il contatore è a 1 alla fine del codice, infatti se si sbaglia il contatore deve per forza avere un valore superiore

inizia con un TRUE e poi una condizione?

while se non erro accetta le stesse condizioni accettate da if (quindi ad es l' opzione "-e" che indica se il file è presente) e quindi il true non credo che vada messo.

[ilpinguino89]

Ringrazio tutti per le numerose risposte.

questo è il codice modificato

Codice: Seleziona tutto

#! /bin/bash

marco="pinguino" #utente
password="pinguino98" #password

contatore=0
stop=3

#marco1=input
#password1=input

EXIT_SUCCESS=0
#EXIT_FAILURE=1

while [ $contatore != "$stop" ]; do
   echo "Inserisci nome e password:"
   read marco1
   read password1
   if [[ "$marco1" = "$marco" ]] && [[ "$password1" = "$password" ]]; then
      break
   else
      echo "Input errato"
   fi
         let contatore=$contatore+1
done

if [ $contatore = "$stop" ] ; then #se non si accede cosa fare:
   echo "time out accesso" #scrivere questo
   echo `ls`      #eseguire questo comando
      else #se si accede invence
         echo "accesso effettuato" #scrivere questo
fi

exit $EXIT_SUCCESS


Io vorrei dividere l' "if" in due parti una per l'utente e una per la password ma tutti i miei tentativi sono stati inutili; avete qualche suggerimento?

[gigicookie]

Continuo a non capire perche ci sono le doppie quadre, comunque basta mettere gli if uno sotto l' altro:

Codice: Seleziona tutto

if [$pinguino1 == $pinguino];  then
if [$password1 == $password]; then
# Se è giusto
else
# Se è sbagliata
fi
# Codice solito
else
# Se è errato
fi


Inviato dal mio Nexus 7 con Tapatalk 2

[ilpinguino89]

cosa intendi per "codice solito"?

[gigicookie]

Nulla, errore mio. Ma qui:

Codice: Seleziona tutto

echo `ls`      #eseguire questo comando


Per eseguire il comando devi scriverlo non fare echo comando

[ilpinguino89]

Mi sono sbagliato, ho già corretto

[gigicookie]

Ma le doppie quadre perché le metti?