Discussion :

[DreakY]

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

Code :

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

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 . Merci d'avance


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

Code :

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

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

[Trap D]

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

[diogene]

Code :

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

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

[DreakY]

D'abord merci pour vos réponses.

- 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.

- 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 . J'ai remplacé mes "||" par "&&".


Et du coup, ca marche du tonnerre

Code :

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

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.

[Thierry Chappuis]

Code :

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

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

[Médinoc]

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

[DreakY]

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

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 ?

[Trap D]

Code :

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

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

[Thierry Chappuis]

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 :

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

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