Problème de comparaison de chaine de caractères

Mais ici je ne peux pas utiliser cette fonction puisque "non" écrit à la main ne peux être pointé à l'aide d'une pointeur.
Je ne suis pas en train de comparer deux chaines de caractères déclarées, seulement la première l'est.

En C le "non" est considéré par le compilateur comme une chaîne littérale, le compilateur allouant un espace mémoire pour cette chaîne, qui est donc désigné par un pointeur.

Par conséquent, cela ne pose strictement aucun problème d'écrire, en C :

#include <stdio.h>
#include <string.h>

int main(void) {
    char mot[] = "non";

    if (strcmp(mot, "non") == 0) {
        printf("c'est non\n");
    }
                                                                                
    return 0;
}

note la déclaration de mot, qui ne nécessite pas que tu mentionnes la taille, le compilateur calculant cela tout seul (de plus, ton

char mot[3]="non";

était erroné, car il manquait un byte. Pour une chaîne de 3 lettres on a besoin d'au moins 4 char, le 4ème étant pour le '\0' terminant la chaîne.

https://www.commentcamarche.net/contents/118-langage-c-les-chaines-de-caracteres

Effectivement, il est rare que l'on demande à l'utilisateur d'indiquer quelle est la taille de la chaîne (qu'on va lui demander de saisir) : en général, le programme fixe cette limite, et devrait en informer l'utilisateur d'une façon ou d'une autre (sur l'interface utilisateur, dans la documentation, etc.)... cela dit, pourquoi pas :-)

Par contre, le programme ne devrait pas faire confiance à l'utilisateur et vérifier, en tout état de cause (que la limite soit fixée d'avance ou non), que la chaîne saisie grâce à un scanf ou un fgets, ne dépasse pas l'espace mémoire alloué.

Pour allouer l'espace, tu devrais utiliser un

malloc()

(et un

free()

quand tu n'en as plus besoin), plutôt qu'un

char mot[n+1];

, si tu veux que ton code reste compatible avec le C89.

quant à

char mot[n+1]="..."

, je ne sais pas trop ce que tu voudrais faire. Ce n'est pas très cohérent de mettre un littéral à droite (qui a donc une taille fixe), alors qu'à gauche tu aurais une déclaration avec un espace mémoire dépendant de "n", qui, si j'ai bien compris serait déterminé par l'utilisateur... il y a un petit problème là. Je ne crois pas que cela donne rien de bon en C.


Dal

Merci je vais étudier ce code, en particulier les nouvelles commandes ^^

je ne suis pas sûr que cela soit ce que tu veuilles faire

en utilisant

strncmp()

, jisisv limite la comparaison à 3 caractères saisis, alors qu'il en stocke jusqu'à 9 en raison de la taille du buffer.

ainsi, si tu saisis "nonne", le test passera quand même (essaye)

il faudrait utiliser

strcmp()

, pour vraiment tester ce qui a été saisi. Mais comme fgets inclue le '

\n'

tapé (le retour à la ligne est inclus avant le

'\0'

à moins que tu aies tapé exactement le nombre de caractères tenant dans le buffer), tu verras que pour que la comparaison fonctionne, il faut que tu le supprimes.

pour t'en convaincre, change la ligne 11 avec le puts et au lieu de cela mets

printf("vous avez tapé : >%s<\n", buffer);

et répond "non" à la question, tu verras que ce qui a été stocké dans buffer contient le retour à la ligne, cela affiche :

$ ./a.out 
buffer ? non
vous avez tapé : >non
<

et la comparaison échoue aussi.

donc, pour n'avoir que ce que l'on veut, avec fgets, il faut supprimer le retour à la ligne s'il existe, et cela se fait comme cela :

    char * p;
    if ((p = strchr(buffer, '\n')) != NULL)
        *p = '\0';

insére cela avant le printf, et désormais, tu auras :

$ ./a.out 
buffer ? non
vous avez tapé : >non<
c'est non

au final, cela donne :

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main(void) {
    const int SIZE = 10;
    char mot[] = "non";
    char buffer[SIZE];

    printf("%s", "buffer ? ");
    fgets(buffer, SIZE, stdin);
    char * p;
    if ((p = strchr(buffer, '\n')) != NULL)
        *p = '\0';
    printf("vous avez tapé : >%s<\n", buffer);

    if (strcmp(mot, buffer) == 0) {
        printf("c'est non\n");
    }

    return 0;
}

avec scanf on a d'autres types de problèmes :-) mais, cela suffit pour ce soir.


Dal

quelle première explication ?