probléme avec une réponse oui ou non

Version imprimable

bonjour
je commence tout juste le scripting en shell. j'ai réaliser un script pour ajouter mes utilisateur samba de part un fichier txt du nom persons le script s'execute correctement mais il ne me demande pas si je veux continuer en rentrant la lettre o ou alors n pour non

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
# !/bin/sh	
#
function choisir ()
{
choix=''
#local invite ="$*"
local reponse
 read -p "$reponse" reponse
case "$reponse" in
 [oO] ) choix='o';;
 [nN] ) choix='n';;
 *		) choix="$reponse";;
esac
}
 # lecture du fichier persones
 	 cat persones | while true 
do
read ligne
if [ "$ligne" == "" ]
then
    echo "fin du fichier persones"
     break
fi
 
set -- $ligne
 
login=$1
pass=$2
 
if [ -d "/home/$1" ]
then
    echo "le compte $1 exise deja"
else
	 printf "ete vous sur de vouloir ajouter l'utilisateur $1 \n"
	 echo "pour continuer entre o/N:"	 
   function choisir ()
	 if [ "$reponse" = "o" ]; then	
	 	 echo "creation du compte $login"
     useradd $1 
     echo "creation du compte samba $login"
     (echo $pass ; echo $pass) | smbpasswd -a -s $login
 
 	  else
 
 	  echo "le compte $1 n'a pas etait crée"
	fi
fi
done

quand je viens à executer le script il me dit

Code:

1
2
3
4
# ./fonctionchoisir.sh
ete vous sur de vouloir ajouter l'utilisateur olivier2
pour continuer entre o/N:
fin du fichier persones

il ne me demande pas d'ecrire o ou n est la je commence à sécher

merci à vous pour votre aide:lol:

Bonjour,

Ta fonction est mal définie et mal appelée. La syntaxe est :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
#définition d'une fonction
#1ere possibilité
function choisir {
}
 
#2eme possibilité
choisir() {
}
 
#Appel de la fonction
echo "J'appelle la fonction choisir"
choisir
echo "C'est fait"

Je ne vois pas trop à quoi te sert ta fonction choisir() avec le case puisque tu refais une sélection suivant la réponse par la suite avec un if.

Je te propose donc de supprimer ta fonction qui est inutile :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
# !/bin/sh	
#
# lecture du fichier persones
cat persones | while true 
do
read ligne
if [ "$ligne" == "" ]
then
	echo "fin du fichier persones"
	break
fi
 
set -- $ligne
 
login=$1
pass=$2
 
if [ -d "/home/$1" ]
then
	echo "le compte $1 exise deja"
else
	printf "ete vous sur de vouloir ajouter l'utilisateur $1 \n"
	echo "pour continuer entre o/N:"	 
	read reponse
	case $reponse in
		o|O)
			echo -e "Creation du compte $login"
			useradd $1
			echo -e "Creation du compte samba $login"
			(echo $pass ; echo $pass) | smbpasswd -a -s $login
			;;
		n|N)
			echo "le compte $1 n'a pas etait crée"
			;;
	esac
fi
done

je viens d'appliquer les modifications, et il me reste une petite erreur que voici "je continue à chercher

Code:

1
2
3
4
# ./fonctionchoix.sh
ete vous sur de vouloir ajouter l'utilisateur olivier2
 
pour continuer entre (oui/non) :Erreur, vous deviez répondre par oui ou par non.fin du fichier persones

voici le code modifier

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
# !/bin/sh	
#
# lecture du fichier persones
 	 cat persones | while true 
do
read ligne
if [ "$ligne" == "" ]
then
    echo "fin du fichier persones"
     break
fi
 
set -- $ligne
 
login=$1
pass=$2
 
if [ -d "/home/$1" ]
then
    echo "le compte $1 exise deja"
else
	 echo "ete vous sur de vouloir ajouter l'utilisateur $1"
	 echo ""
	 echo -n "pour continuer entre (oui/non) :"	 
      read reponse
      case $reponse in 
       o|O|oui) 
		 	 echo -e "creation du compte $login"
          useradd $1 
     			echo "creation du compte samba $login"
     			(echo $pass ; echo $pass) | smbpasswd -a -s $login;;
 
 	  n|N|non)
 
 	  		echo -e "le compte $1 n'a pas etait crée";;
 	  		*) echo "Erreur, vous deviez répondre par oui ou par non.";;
      esac
 
fi
done

Attention à l'utilisation de la fonction read
Que lisez vous ?
le contenu du fichier "personnes" ou le l'entrée Std
précisez le

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
# !/bin/sh

Mon_TTY=`tty`
#
# lecture du fichier persones

while read ligne
do
   set -- $ligne
   login=$1
   pass=$2

   if [ -d "/home/$1" ]
   then
       echo "le compte $1 exise deja"
   else
      echo "ete vous sur de vouloir ajouter l'utilisateur $1"
      echo "pour continuer entre (oui/non) : \c"
      read reponse < $Mon_TTY
      case $reponse in
       o|O|oui)
                    echo  "creation du compte $login"  
                    useradd $1
                    echo "creation du compte samba $login"
                    (echo $pass ; echo $pass) | smbpasswd -a -s $login;;

       n|N|non)

               echo -e "le compte $1 n'a pas etait crée";;
               *) echo "Erreur, vous deviez répondre par oui ou par non.";;
      esac

   fi
done < personnes

06/04/2009, 16h21
zyriuse

merci de ta reponse cela fonctionne :king:mais je ne comprend pas se que tu à fais peut tu essayer de m'expliquer, je te remerci à l'avance pour ton aide et ton explication:):)
06/04/2009, 17h34
argoet
Ici on affecte dans la variable "$Mon_TTY" le nom du terminal sur lequel on est connecté
ce nom est obtenu à partire de la commande "tty"
avec les "quotes" inversées : c'est le resultat de la commande "tty" qui est renseignée dans la variable $Mon_TTY
cf : man tty

Code:

Mon_TTY=`tty`

la réponse est attendu à partir du terminal (Lecture de l'entrée Std)

Code:

read reponse < $Mon_TTY

Tant que l'on peu lire une "ligne" du fichier "personne"
affiche le contenu de la ligne
Code:

1 2 3 4 while read ligne do echo $ligne done < personnes

Il est possible d'affecter directement les variables 'login' et 'pass' depuis la ligne while read :
Code:

1 2 3 while read login pass autre; do echo "login: $login - pass: $pass - reste de la ligne : \"$autre\"" done < personnes

15/04/2009, 18h17
mcoolive
Citation:
Envoyé par lavazavio

Bonjour,

Ta fonction est mal définie […]. La syntaxe est :

Code:

1 2 3 4 5 6 7 8 9 #définition d'une fonction #1ere possibilité function choisir { } #2eme possibilité choisir() { } …
Je me permets d'affirmer le contraire, la définition de la méthode était tout à fait correcte. Voici ce que dit le man(uel) de bash:

Citation:

Shell Function Definitions
A shell function is an object that is called like a simple command and
executes a compound command with a new set of positional parameters.
Shell functions are declared as follows:

[ function ] name () compound-command [redirection]

Voici ce que dit le man(uel) de zsh(misc):

Citation:

function word ... [ () ] [ term ] { list }
word ... () [ term ] { list }
word ... () [ term ] command

Donc on voit que la syntaxe dépend du shell. On voit aussi qu'utiliser "function" et "()" est tout à fait autorisé.

Note: La mot clef "function" est optionelle pour des raisons de compatiblité ascendante avec le shell historique avec sh.