Scanf dans une boucle while

Bonjour à tous,
J'aurai besoin d'un éclairciceemnt à propos d'un petit bout de code.
Je n'arrive décidemment pas à comprendre pourquoi cela ne marche pas :evilred:
Code:

1 2 3 4 5 6 7 8 char * reponse = malloc(TAILLE*sizeof(char)); assert(reponse); while((strcmp(reponse, "yes") != 0) || (strcmp(reponse, "no") != 0)) { printf(" (yes or no) \n"); scanf("%s", reponse); }
Je veux faire une interface interactive, qui lorsque que je lui donne "yes" ou "no", execute la partie du code que je veux. Or ici, lorsque j'execute, il ne sort jamais de la boucle while, et me re balance toujours le printf et scanf.
GDB ne m'a pas aidé à comprendre le pourquoi du comment :(

Aidez moi siouplez :D. Merci d'avance

~~~~~~~EDIT~~~~~~~~~
Ci dessous le code résolu et sans erreur :
Code:

1 2 3 4 5 6 7 8 9 10 char * reponse = malloc(TAILLE*sizeof(char)); assert(reponse); reponse = strcpy(reponse, "huhu"); while((strcmp(reponse, "yes") != 0) && (strcmp(reponse, "no") != 0)) { printf(" (yes or no) "); scanf("%s", reponse); }

le scanf est TRRRRRRRRRRRRREEEEEEEEEEEEEEEEEEEESSSSSSSSSS difficile à utiliser, surtout pour un débutant.
Utilise fgets(buf, sizeof buf, stdin) puis vire le '\n' de fin de saisie.
Fais une recherche dans la FAQ.
De plus tu as un problème de logique dans ton code, réfléchis bien à ce que tu écris avec le || :mouarf:

Code:

1 2 3 4 char * reponse = malloc(TAILLE*sizeof(char)); assert(reponse); while((strcmp(reponse, "yes") != 0) || (strcmp(reponse, "no") != 0)) {...
- Au moment où on entre la première fois dans le while, le contenu du tableau pointé par reponse est non initialisé et la comparaison n'a pas de sens. Utiliser plutôt un do ... while.

- Revoir la logique de la combinaison xxx || yyy :
Si reponse est "yes" alors yyy est vrai
Si reponse est "no" alors xxx est vrai
Si reponse est autre chose alors xxx et yyy sont vrais
Dans tout les cas xxx || yyy est vrai

D'abord merci pour vos réponses.

Citation:

Envoyé par diogene

- Au moment où on entre la première fois dans le while, le contenu du tableau pointé par reponse est non initialisé et la comparaison n'a pas de sens.

J'ai donc initialisé ma variable à "huhu", histoire d'avoir quelquechose dedans.

Citation:

Envoyé par diogene

- Revoir la logique de la combinaison xxx || yyy :
Si reponse est "yes" alors yyy est vrai
Si reponse est "no" alors xxx est vrai
Si reponse est autre chose alors xxx et yyy sont vrais
Dans tout les cas xxx || yyy est vrai

Oups, petite erreur de logique :D. J'ai remplacé mes "||" par "&&".

Et du coup, ca marche du tonnerre :D
Code:

1 2 3 4 5 6 7 8 9 10 char * reponse = malloc(TAILLE*sizeof(char)); assert(reponse); reponse = strcpy(reponse, "huhu"); while((strcmp(reponse, "yes") != 0) && (strcmp(reponse, "no") != 0)) { printf(" (yes or no) "); scanf("%s", reponse); }
J'édite mon premier post.

Merci à tous.

Citation:
Envoyé par DreakY
Code:

1 2 3 4 5 6 7 8 char * reponse = malloc(TAILLE*sizeof(char)); assert(reponse); while((strcmp(reponse, "yes") != 0) || (strcmp(reponse, "no") != 0)) { printf(" (yes or no) \n"); scanf("%s", reponse); }
assert() pour faire le test de la valeur retournée par malloc(), c'est pas terrible...

Thierry

En effet. Un assertion ne sert pas à ça, un échec de malloc() n'est PAS une erreur de logique.

Citation:

Envoyé par Médinoc

En effet. Un assertion ne sert pas à ça, un échec de malloc() n'est PAS une erreur de logique.

Citation:

Envoyé par Thierry Chappuis

assert() pour faire le test de la valeur retournée par malloc(), c'est pas terrible...
Thierry

Pourtant c'est ce qu'on nous a appris en cours =/
On nous a dit que c'est pour confirmer que notre pointeur a bien été alloué dans la mémoire.

Vous proposez quoi à la place ?

Code:

1
2
3
4
5
6
if (reponse == NULL)
{
  // prendre les mesures appropriées à la situation
  // et sans doute
  return <code_erreur>; 
}

Citation:

Envoyé par DreakY

Pourtant c'est ce qu'on nous a appris en cours =/
On nous a dit que c'est pour confirmer que notre pointeur a bien été alloué dans la mémoire.

C'est une grosse erreur. Si la macro NDEBUG est définie, assert() ne fait rien et l'échec de malloc() n'est plus détecté ni traité. La forme canonique pour l'usage de malloc est:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 Type *p = NULL; /*...*/ p = malloc(TAILLE * sizeof *p); if (p != NULL) { /* L'allocation a reussi: on continue */ /* On libere la memoire allouee par malloc() et on affecte une adresse invalide testable (NULL) a la variable p */ free(p), p = NULL; } else { /* Gestion de l'erreur */ }
Thierry