syntax error before '!' token

bonjour à tout le forum,


je suis en TP de développement mais mes profs ne sont pas là alors je ne ne peux pas déboguer seul, car je ne comprend pas l'erreur, j'ai un programme auquel j'ai ajouté à la fin :

Citation:

---

73.do { //ce do{ se trouve en réalité au début du programme !
74.do {
75. printf("Autre foyer ? (O/N)");
76. scanf("%s",&continu);
77. if (continu=='O'||continu=='o') {
78. sortie=0;
79. } else {
80. sortie=1;
81. }
82. }while !(continu=='o' && continu=='O' && continu=='N' && continu=='n');
83. } while !(sortie=1);

---

lors de la compilation avec ggc sous linux voici l'erreur :
82: error: syntax error before '!' token
83: error: syntax error before '!' token

Si vous pouviez me dire d'où vient l'erreur j'en serais surper content !


Merci !


Petithomme
:mouarf:

Essaye de poster sur le bon forum ;)

essaye avec ça

Code:

---

1 2	82. }while (!(continu=='o' && continu=='O' && continu=='N' && continu=='n')); 83. } while (!(sortie=1));

---

Merci beaucoup !

NB : sur quel forum je devais poster, car je vais souvent revenir ^^ :oops:

Heu en fait j'ai encore un ptit blem :

while !(continu=='o' && continu=='O' && continu=='N' && continu=='n');

semble ne pas marcher comme je le souhaite :

je voudrais que si la personne entre 'o' ou 'O' (pour oui) le programme retourne au début avec mon do{, et que s'il met 'n' ou 'N' alors programme s'arrete . . .


Marci !

Poste dans le forum du language que tu utilises (surement du C ou un tru du genre je suppose)

Dans ton while n'oublies pas la parenthèse avant le !

Ensuite il faut que tu testes la condition de ton while.

Si ton utilisateur appuis sur 'O' par exemple, ta condition de boucle devient
!(0&&1&&0&&0) soit 1
Donc tu boucles dans ton while!

Je crois que tu ne fais pas le bon test : que ce passe-t-il si l'utilisateur tape autre chose que O ou N ?

Cas de O et N :
-tu sors du programme si l'utilisateur à taper N
-tu restes dans le programme si l'utilisateur à tapé O

Plus globalement :
Si continu='O' ou si continu='o' alors on reste
Si continu='N' ou si continu='n' alors on part
Si continu!='N' ou continu!='n' ou continu!='O' ou continu!='o' alors on repose la question.

Pour ton test

Code:

---

1 2	while !(continu=='o' && continu=='O' && continu=='N' && continu=='n');

---

commence par faire un test simple et regarde comment le programme réagit :

Code:

---

1 2	while (!continu=='o')

---

Code:

---

1 2	while (continu!='o')

---

et fais une table de vérité qui rescence les cas où tu dois sortir (1) et ou tu dois rester (0).

---------------------------
Pour ce qui est du forum : il aurait fallu poster en C++...
Ce n'est pas grave si un modérateur passe par ici il déplacera le sujet dans le bon forum : il n'est donc pas nécessaire d'en recréer un pour cette discussion.

Ok je posterais au bon endroit, si un modo veut le déplacer, nop !

voici la solution que je viens juste de trouver :

Citation:

---

while ((continu=='o' && continu=='O' && continu=='N' && continu=='n'));

---

Merci encore de votre aide !

ca :
while ((continu=='o' && continu=='O' && continu=='N' && continu=='n'));
et ca :
while (continu=='o' && continu=='O' && continu=='N' && continu=='n');

c la meme chose!

Il sera toujours faux, ce test...
Tu testes si continu est à la fois égal à 'o' et à 'O' (sans compter les N)...

Je rajouterais aussi que si "continu" est déclaré comme ceci :

Code:

---

char continu;

---

alors le "scanf" devrait être comme cela :

Code:

---

scanf("%c",&continu);

---

pourtant ma solution a l'arrache fonctionne . . . LOL

Citation:

---

Envoyé par titmael

pourtant ma solution a l'arrache fonctionne . . . LOL

---

Ne rigole pas, ce n'est pas vraiment marrant, surtout quand tu vas devoir travailler sur des projets a plusieurs. Il faut tout faire sauf du code a l'arrache... (enfin il y a aussi d'autres choses a ne pas faire).

Et ce n'est pas parce que ton code fonctionne sur un jeu de test precis qu'il est bon.

Bonjour,

Citation:

---

Envoyé par titmael

pourtant ma solution a l'arrache fonctionne . . . LOL

---

Si tu utilises le code qu'on a vu, y compris les modifications, je ne pense pas que ça fonctionne.
En particulier, si tu contrôles ta boucle avec ce test:

Citation:

---

Envoyé par titmael

Code:

---

while ((continu=='o' && continu=='O' && continu=='N' && continu=='n'));

---

---

ça ne marchera pas : comme l'a déjà écrit Médinoc, le test est toujours faux.

Et comme te l'a rappelé millie, une solution à "'l'arrache" n'en est pas une, car ça montre tout simplement que tu n'as pas compris ce que tu fais.
Donc, que répondras-tu à ton prof quand il te demandera "comment ça marche ?"

Citation:

---

Envoyé par titmael

Merci beaucoup !

NB : sur quel forum je devais poster, car je vais souvent revenir ^^ :oops:

---

Code:

---

10 GOTO 10

---

heu...moi je comprend pas ca:

Code:

---

while (!(sortie=1));

---

il faudrait pas plutot ca ?:

Code:

---

while (!(sortie==1));

---

Citation:

---

Envoyé par kamouminator

heu...moi je comprend pas ca:

Code:

---

while (!(sortie=1));

---

il faudrait pas plutot ca ?:

Code:

---

while (!(sortie==1));

---

---

en effet..........


Pour ce qui est de ce test :

Code:

---

1 2	while (continu=='o' && continu=='O' && continu=='N' && continu=='n');

---

je rejoins ce qui a été dit au-dessus : il est toujours faux.

ce test serait plus approprié :

Code:

---

1 2	while (continu=='o' || continu=='O' || continu=='N' || continu=='n');

---

et encore... Il faudrait ajouter des négations devant certains tests

bon, arretez moi si je dis des betises mais:

d'abord comme tout le monde l'a fais remarqué:

Code:

---

while !(continu=='o' && continu=='O' && continu=='N' && continu=='n');

---

il faudrai plutot

Code:

---

while (!(continu=='o' && continu=='O' && continu=='N' && continu=='n'));

---

mais ca n'a pas beaucoup de sens,
c'est la meme chose que:

Code:

---

while (continu!='o' || continu!='O' || continu!='N' || continu!='n');

---

ce qui est TOUJOURS le cas...

autre chose: !(sortie==1) c'est pareil que (sortie!=1)....c'est plus clair comme ca...non ?

d'autre par, je ne comprend pas du tout l'utilité des deux boucle...mais ca je pense que c'est parce que ta condition sur le premier while est mauvaise

ton premier while, c'est pour verifier que tu as bien appuié sur 'o' 'O' 'n' ou 'N' non ?

la condition serai donc:

Code:

---

while (continu!='o' && continu!='O' && continu!='N' && continu!='n');

---

ce qui signifie bien: "l'utilisateur n'appuie sur aucune de ces touches"

y'a un autre probleme:
ton if !!!!

tu lui dis si continue == 'O' ou 'o' tu met sortie à 0

sinon sortie = 1

si j'ai bien compris c'est pas ca, parce que si tu appuie sur une autre touche comme 'b' par exemple sortie sera mise a 1....

il faut mettre une autre condition
sur continue quand il vo 'n' ou 'N'

bien sur, je peu avoir mal compris ce que tu attends de ton programme...si c'est le cas, je m'en excuse !

Si j'ai bien compris, tu veux boucler tant qu'on n'a pas saisi 'o', 'O', 'n' ou 'N'.
Alors, plutôt que de s'embêter avec ces tests et ces && qui ne marchent pas, autant utiliser une autre méthode:

Code:

---

1 2 3 4 5 6 7 8 9 10

#include <string.h>   /* ... */   do { /* Saisir un caractère dans continu */ /* ... */ } while( strchr("OoNn", continu) == NULL);

---

Citation:

---

Envoyé par Médinoc

Si j'ai bien compris, tu veux boucler tant qu'on n'a pas saisi 'o', 'O', 'n' ou 'N'.
Alors, plutôt que de s'embêter avec ces tests et ces && qui ne marchent pas, autant utiliser une autre méthode:

Code:

---

1 2	/* ... */ while( strchr("OoNn", continu) == NULL);

---

---

:yaisse: