True, however we do not want to introduce commands that the OP has not learned yet so as not to add to the confusion nor just write the program.

no no i understand, its just it becomes confusing with variables, I mean I'm trying to figure out how to cut the time and make the loop work, its good I dont have answers, but somehow I will find them... :S

Advice using cut & echo combination commands



good link for loops


do you have to use cut to get the data you need?

plus d'une façon à la peau d'un chat

use these google key words for working with string data which is what you are doing

"string manipulation bash linux"

#!/bin/bash


while [ sortie="true" ]
do

echo -n "Entrez l'heure sous forme hh:mm ou fin terminer :"

read temps

heure= "$temps" |cut -d: f1
minute= "$temps" |cut -d: f2

if [ $heure = "fin" ]

then
let sortie = "false"

fi

if [[ "$heure" -ge 0 ]] && [[ "$heure" -le 23 ]] && [[ "$minute" -ge 0 ]] && [[ "$minute" -le 59 ]]
then
if [[ "$heure" -ge 6 ]] && [[ "$heure" -le 18 ]]
then
echo "Bonjour"

else

echo "Bonsoir"
fi


fi

sortie = "false"
done

Looks like you are making some progress.

Did you look at my hint on the proper syntax for cut in my previous post #14? You can substitute $temps for "12:59".

Improper if statments

use echo to see what's going on in your code

ie:

review @michaelk hints

hopefully you know the word temps means time in french

not trying to step on your toes, or a I miss interpreting your english? lol

pay attention to detail

prêter attention aux détails


Variable Assignment

learn your loop control very important!!!

• link1
• link2

faire preuve d'innovation

Nope.

Just trying to explain to the op that

echo "12:59" | cut -d ":" -f1

would become

echo "$temps" | cut -d ":" -f1

Now just assign that to the variable heure.

yeah I got what you meant after I posted ... lol

thanks anyways, why does this seem complicated... well back to study on this hopeless assignment

you're NOT paying attention to detail?

"If you begin with the notion that something is impossible, then it obviously will be for you." - Unknown

I agree with BW-userx. If you break down the assignment into smaller parts it will seem less complicated. Ignore the if [hour=end] for now and work on the hour and minute variables working.

#!/bin/bash
#
#
#
sortie="false"
while [ $sortie = "false" ]
do
read -p "temps : " temps
heure=$(echo "$temps" | cut -d: -f1)
minute=$(echo "$temps" | cut -d: -f2)
if [ "$temps" = "fin" ]
then
sortie="true"
# let sortie="true"
else
if [[ "$heure" -ge "00" ]] && [[ "$heure" -lt "24" ]] && [[ "$minute" -ge "00" ]] && [[ "$minute" -le "60" ]]

then
if [[ "$heure" -gt "5" ]] && [[ "$heure" -lt "19" ]]
then
printf "%s\nBonjour\n"
else
printf "%s\nBonsoir\n"
fi
else
printf "%s\nHeure incorrecte\n"
fi
fi

done
echo

here no more bashing me please.

Apparently your English is poor ... let us try this:

Vous devez utiliser des balises de code lors de l'affichage de votre script