Discussion :

[titmael]

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 :

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

[jbrasselet]

Essaye de poster sur le bon forum

essaye avec ça

Code :

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

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

[titmael]

Merci beaucoup !

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

[titmael]

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 !

[jbrasselet]

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!

[Auteur]

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 :

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

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 :

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

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

Code :

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

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.

[titmael]

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

voici la solution que je viens juste de trouver :

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

Merci encore de votre aide !

[Jérémy Lefevre]

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!

[Médinoc]

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

[crocodilex]

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

Code :

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

char continu;

alors le "scanf" devrait être comme cela :

Code :

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

 scanf("%c",&continu);

[titmael]

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

[millie]

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.

[thewho]

Bonjour,

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:

Code :

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

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 ?"

[Emmanuel Delahaye]

Merci beaucoup !

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

Code :

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

10 GOTO 10

[kamouminator]

heu...moi je comprend pas ca:

Code :

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

while (!(sortie=1));

il faudrait pas plutot ca ?:

Code :

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

while (!(sortie==1));

[Auteur]

heu...moi je comprend pas ca:

Code :

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

while (!(sortie=1));

il faudrait pas plutot ca ?:

Code :

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

while (!(sortie==1));

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


Pour ce qui est de ce test :

Code :

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

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 :

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

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

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

[kamouminator]

bon, arretez moi si je dis des betises mais:

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

Code :

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

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

il faudrai plutot

Code :

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

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

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

Code :

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

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 :

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

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 !

[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 :

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

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);

[Emmanuel Delahaye]

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 :

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

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