Discussion :

[helww]

Bonjour ,
J'ai ecris ce bout de code , le programme doit tester la valeur nom1 si elle est égale à nomt , mais quand j'entre la valeur de nomt , il me donne le message d'erreur que j'ai mis (meme si la valeur est correct , il ne l'est pas pour le prog )

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
16
 
#include <iostream>
using namespace std ;
 
void main () {
 
	char nom1[9] ;
	char nomt[9]="bonjour+++" ;
	char word[]="code incorrect\n" ;
    char word2[]="correct , c'est le vrai code" ;
cout <<  "welcome\n" ; 
cout << "Entrez le code (!!) :\n" ;
cin >> nom1 ;
if (nom1==nomt)  cout << word2 ;
else cout << word ;
}

Merci

[koala01]

Salut,

Déjà, en C++, il est préférable d'utiliser les std::string plutot que les tableaux de char

Et, avec une std::string, tu pourras effectuer un test du genre de

Code :

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

1
2
3
4
 
if(chaine1==chaine2)
{
}

Par contre, Si tu décide d'utiliser un tableau de char comme chaine de caractères (chaines "C style"), alors, il faut utiliser le système de comparaison "C style"...

Et cela se fait avec un strcmp

[hiko-seijuro]

c'est normal avec cette condition tu testes si le pointeur nom1 est le meme que le pointeur nomt ce qui est evidemment pas le cas

[helww]

pour koala01 :
comment faire pour utiliser std::string , ou comment faire la condition du "c style" ?

pour hiko-seijuro :
Je teste le pointeur nom1 lorcequ'il est remplit par le contenu de cin >> , normalement il doit marcher

[Lucien63]

Code :

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

	char nomt[9]="bonjour+++" ;

Là il y a déjà un gros problème, tu initialises un tableau de 9 char avec 10 char !

[koala01]

L'utilisation de la std::string est simple:
inclusion du fichier <string>
utilisation de std::string ou de string (apres un usign namespace std)

Pour le reste, on peut tres bien faire un

Code :

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

1
2
3
4
string str="machin";
str+=" brol";
(...)
if(str==str2)//ou str2 serait une autre chaine dont la valeur est définie

En "C style", il faut utiliser strcmp:

Code :

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

1
2
3
4
5
6
7
8
9
if(strcmp(tab1,tab2)==0)
{
    //tab1 et tab2 ont des valeurs identiques
}
else
{
    //tab1 et tab2 ont des valeurs différentes
    // (ex: "titi" et "toto")
}

[hiko-seijuro]

rempli ou pas rempli ca change rien ce que tu compares c'est l'adresse de nom1 avec l'adresse de nomp

[loufoque]

cin >> nom1 ;

Faille de sécurité.

[G-OnZa]

En résumé :

Pour comparer 2 chaines de caractère (type char *), utilise strcmp().
Donc au lieu de ton

Code :

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

 if( nom1 == nomt)

qui compare les adresses du premier octet de chaque chaîne de caractère
tu as

Code :

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

1
2
3
4
 
// #include <string.h>
//...
strcmp (nom1, nomt);

[poukill]

En résumé :

Pour comparer 2 chaines de caractère (type char *), utilise strcmp().
Donc au lieu de ton

Code :

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

 if( nom1 == nomt)

qui compare les adresses du premier octet de chaque chaîne de caractère
tu as

Code :

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

1
2
3
4
 
// #include <string.h>
//...
strcmp (nom1, nomt);

La solution de koala est meilleure : std::string, pas de char*

[G-OnZa]

Entièrement d'accord avec toi, mais je réponds juste à sa question.