Problème chaine de caractère

Bonsoir, cela fait déjà pas mal de temps que je code en C par passion mais là je ne comprend pas ce qu'il se passe, je m'explique. En ce moment je suis sur la gestion des chaines de caractère et donc j'ai un problème avec une condition qui passe 'inaperçut' dans mon programme :
Dans tous les cas le if n'est pas pris en compte :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

#include <stdio.h> #include <stdlib.h> #include <string.h>   int main() { char CHbonjour[] = ""; printf("Bonjour !\n"); scanf("%s", CHbonjour);   printf("vous avez saisi : '%s'\n", CHbonjour);   if(CHbonjour=="bonjour") { printf("comment vas tu ? :)"); }   return 0; }

---

Dans tous les cas le if n'est pas pris en compte...
Je pense que j'ai du pas faire attention à quelque chose, mais quoi ?
Quelqu'un serait d'où vient mon problème ?
Cordialement Jurams.

Bonjour,

Il y a trois problèmes dans ton code :

1. Code:

   ---

   char CHbonjour[] = "";

   ---

   sans taille entre les crochets, celle-ci va être déduite de la partie droite. Ici "" (='\0') donc seulement 1 caractère. Il y aura donc de très fortes chances de dépassement.
2. Ensuite : CHbonjour=="bonjour" on ne compare pas des chaînes de caractères avec l'opérateur == mais avec la fonction strcmp :arrow: FAQ Comment comparer 2 chaînes de caractères ?
3. scanf("%s", CHbonjour); si tenté que tu aies réglé le premier point, tu auras toujours des chances de dépassement si tu saisis plus de caractères que ne peut stocker CHbonjour. Je te conseille la lecture de ce tutotiel.

Bonsoir,

Le problème est que, par définition, lorsqu'une expression du type tableau n'est ni l'opérande de l'opérateur sizeof, ni de _Alignof, ni de l'opérateur unaire &, ni une chaîne littérale utilisée pour initialiser un tableau, elle est convertie en un pointeur sur le premier élément de l'objet tableau, et n'est pas une lvalue [§ 6.3.2]. De ce fait, sur une machine conventionnelle, CHbonjour désigne l'adresse du premier élément de la chaîne, et "bonjour" l'adresse d'une chaîne de caractère anonyme, située généralement quelque part dans une zone mémoire en lecture seule (correspondant parfois au segment .rodata). Tu vas donc comparer ces deux pointeurs, ce qui ne mènera évidemment à rien (et le comportement est indéfini du fait de la comparaison de deux pointeurs issus d'agrégats de données différents [§ 6.5.8]). La solution est d'effectuer une comparaison caractère par caractère, en utilisant la fonction standard strcmp par exemple [§ 7.24.4.2].

Un deuxième problème est que tu ne spécifies pas la taille de ta chaîne lors de sa déclaration ; elle prendra donc la taille de la chaîne littérale "", c'est-à-dire un caractère, ce qui ne sera pas suffisant pour stocker l'entièreté de l'entrée de l'utilisateur. Tu vas donc, à coup sûr, te retrouver avec des accès mémoires invalides qui conduisent à nouveau à un comportement indéfini. Tu noteras de plus que la fonction scanf est davantage adaptée à la lecture de flux formatés [§ 7.21.6.4], ce qui n'est pas le cas des entrées humaines. La fonction standard fgets sera sans doute meilleure [§ 7.21.7.2].

Bonne soirée ! :)

Ok je vais allez lire le tutoriel tout de suite :).
Merci à vous pour vos conseils et bonne soirée.