Discussion :

[julien31320]

Bonjour, j'ai un devoir à effectuer en shell, après plusieurs relectures je ne vois pas mes erreurs et j'ai besoin d'aide pour les identifier, j'ai utilisé un logiciel shellcheck pour voir où elles étaient mais je ne comprends pas pour le coup ^^ (c'est un jeux en shell...)

le code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
#!/bin/bash
cptj1=1
while test $cptj1 -le 4
do
echo "Entrez une couleur (rouge,jaune,vert,bleu,orange,noir)"
read couleur
case $couleur in
rouge) echo "La couleur choisi est le rouge"
cptj1=`expr $cptj1 + 1`
;;
jaune) echo "La couleur choisi est le jaune"
cptj1=`expr $cptj1 + 1`
;;
vert) echo "La couleur choisi est le vert"
cptj1=`expr $cptj1 + 1`
;;
bleu) echo "La couleur choisi est le bleu"
cptj1=`expr $cptj1 + 1`
;;
orange) echo "La couleur choisi est le orange"
cptj1=`expr $cptj1 + 1`
;;
noir) echo "La couleur choisi est le noir"
cptj1=`expr $cptj1 + 1`
;;
*) echo "La couleur est fausse veuillez recommencer"
cptj1 = 0
;;
if <$cptj1 = "1">
then
gauche = $couleur
fi
if <$cptj1 = "2">
then
milgauche = $couleur
fi
if <$cptj1 = "3">
then
mildroite = $couleur
fi
if <$cptj1 = "4">
then
droite = $couleur
fi
done
noir = 0
blanc = 0
cptj2=1
fin = 0
while test $fin -le 10
do
while test $cptj2 -le 4
do
echo "Entrez une couleur (rouge,jaune,vert,bleu,orange,noir)"
read couleur2
case $couleur2 in
rouge) echo b"La couleur choisi est le rouge"
cptj2=`expr $cptj2 + 1`
;;
jaune) echo "La couleur choisi est le jaune"
cptj2=`expr $cptj2 + 1`
;;
vert) echo "La couleur choisi est le vert"
cptj2=`expr $cptj2 + 1`
;;
bleu) echo "La couleur choisi est le bleu"
cptj2=`expr $cptj2 + 1`
;;
orange) echo "La couleur choisi est le orange"
cptj2=`expr $cptj2 + 1`
;;
noir) echo "La couleur choisi est le noir"
cptj2=`expr $cptj2 + 1`
;;
*) echo "La couleur est fausse veuillez recommencer depuis le
début"
cptj2 = 0
;;
case $cptj2 in
1) if [ $gauche = $couleur2 ]
then
noir = expr $noir + 1
else
if [ $milgauche = $couleur2 ]
then
blanc = expr $noir + 1
fi
if [ $mildroite = $couleur2 ]
then
blanc = expr $noir + 1
fi
if [ $droite = $couleur2 ]
then
blanc = expr $noir + 1
fi
fi
;;
2) if [ $gauche = $couleur2 ]
then
blanc = expr $noir + 1
fi
if [ $milgauche = $couleur2 ]
then
noir = expr $noir + 1
fi
if [ $mildroite = $couleur2 ]
then
blanc = expr $noir + 1
fi
if [ $droite = $couleur2 ]
then
blanc = expr $noir + 1
fi
;;
3) if [ $gauche = $couleur2 ]
then
blanc = expr $noir + 1
fi
if [ $milgauche = $couleur2 ]
then
blanc = expr $noir + 1
fi
if [ $mildroite = $couleur2 ]
then
noir = expr $noir + 1
fi
if [ $droite = $couleur2 ]
then
blanc = expr $noir + 1
fi
;;
4) if [ $gauche = $couleur2 ]
then
blanc = expr $noir + 1
fi
if [ $milgauche = $couleur2 ]
then
blanc = expr $noir + 1
fi
if [ $mildroite = $couleur2 ]
then
blanc = expr $noir + 1
fi
if [ $droite = $couleur2 ]
then
noir = expr $noir + 1
fi
;;
*) ;;
done
if [ $noir = `4`]
then
fin = 10
partie = C’est gagné !
else
echo "Vous avez $noir pion noir et $blanc pion blanc"
fin = expr $fin + 1
partie = C’est perdue essayer encore !
fi
done
echo "$partie"

[BufferBob]

salut,

• le code n'est pas indenté, c'est mal, l'indentation permet de déceler rapidement certaines erreurs, comme un esac manquant par exemple (ou deux), par ailleurs l'indentation comme les sauts de lignes permettent de gagner en lisibilité
• l'affectation en bash interdit les espaces autour du signe égal, et les chaines doivent être contenues dans des guillemets, ainsi une expression comme partie = C’est perdue essayer encore ! en plus de piquer les yeux n'a aucune chance de fonctionner (en l'occurrence des guillemets simples à cause du point d'exclamation)
• de même la syntaxe if <$cptj1 = "1"> est complètement farfelue (alors que les autres if dans le code sont réglos, vas comprendre ^^)
• enfin if [ $noir = `4`]; then ne fonctionnera pas non plus, les backticks permettent d'exécuter des commandes, comme par exemple echo "on est le `date +%d/%m/%Y`", donc là tu essayes d'exécuter `4`, ça va lever une erreur du type "4: commande introuvable"

je crois que c'est à peu près tout pour les erreurs de syntaxe... reste que c'est censé être un mastermind, la partie structure/algorithmique du programme devra elle aussi subir des améliorations pour que tout fonctionne correctement

en bonus une petite astuce; tu peux éventuellement remplacer les tournures cptj1=`expr $cptj1 + 1` par ((cptj1++)) et tirer parti du pattern matching effectué/autorisé par case afin d'éviter de réécrire plusieurs fois les mêmes choses :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
case couleur in
   rouge|jaune|vert|bleu|orange|noir)
      echo "la couleur choisie est le $couleur"
      ((cptj1++))
      ;;
   *)
      echo "La couleur est fausse veuillez recommencer"
      cptj1=0
      ;;
esac

[N_BaH]

moi, je dis chapeau !

écrire 160 lignes de code sans avoir fait le moindre test dans la console pour valider quelques commandes élémentaires...

[Jipété]

Yep !

Et zéro ligne de commentaire !

Alors pour les lecteurs qui ne sont pas dans ta tête, décortiquer un code buggé pour comprendre comment ça doit fonctionner, euh...

Et dès le départ il y a un truc qui va pas, avec le compteur initialisé à 1 puis incrémenté de 1 dans le case si la couleur entrée est valide, résultat en fin de case le test sur compteur == 1 sera toujours faux et donc la "boîte" gauche toujours vide...

Bon, la première partie pourrait ressembler à ça, avec une grosse simplification des if et un changement de noms pour avoir de beaux alignements (ça permet de voir des erreurs) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
#!/bin/bash
cptj1=0
while test $cptj1 -le 3
do
  echo -n "Entrez une couleur (rouge, jaune, vert, bleu, orange, noir) : "; read couleur
  case $couleur in
    rouge|jaune|vert|bleu|orange|noir) #echo "La couleur choisie est le rouge" -- cette ligne devient inutile
        ((cptj1++)) #cptj1=`expr $cptj1 + 1`
        ;;
    *) echo "La couleur est fausse veuillez recommencer"
        cptj1=0
        ;;
  esac
  [ $cptj1 == "1" ] && extgauche=$couleur
  [ $cptj1 == "2" ] && milgauche=$couleur
  [ $cptj1 == "3" ] && mildroite=$couleur
  [ $cptj1 == "4" ] && extdroite=$couleur
done
#echo $extgauche $milgauche $mildroite $extdroite; exit # ligne de test, à commenter/décommenter au besoin

Ah, la fin de ce bout pourrait être ainsi, par remplacement des 4 if par un case :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
...
  esac
  case $cptj1 in
  1) extgauche=$couleur ;;
  2) milgauche=$couleur ;;
  3) mildroite=$couleur ;;
  4) extdroite=$couleur ;;
  esac
done
echo $extgauche $milgauche $mildroite $extdroite; exit