Discussion :

[ruben06]

Salut a tous,
Je suis en bts informatique de gestion (1ere anne) et je debute avec le c++.
j'ai un repertoire a rendre en c++ et j'aimerai vous donner mon code pour que vous me puissiez me dire ce qui ne vas pas.
(il ne fonctionne pas vraiment pour ne pas dire pas du tout, j'ai l'impression que c'est mon strcmp qui ne fonctionne pas et je code sur visual)

#include <stdafx.h>
#include <iostream>

using namespace std;
void main()
{

int i,rien,choix;
char nom[20];

struct repertoire
{
char Nom[20] ;
char Prenom[20] ;
char NumTelFixe[11] ;
char NumPortable[11];
char Mail[50];
};
repertoire rep[100]= {{"myara","ruben","0101010101","0202020202","rm@hotmail.com"},
{"sejourne","laurent","0303030303","0404040404","ls@hotmail.com"},
{"bustori","christophe","0505050505","0606060606","bc@hotmail.com"},
{"melodie","mafi","0707070707","0808080808","mm@hotmail.com"}};

cout<<"pour afficher l'integralité du repertoire taper 1"<<"\n";
cout<<"pour rechercher a partir du nom taper 2"<<"\n";
cout<<"pour rechercher a partir du tel taper 3"<<"\n";
cout<<"pour inserer un nouveau contact taper 4"<<"\n";
cout<<"pour supprimer un contact taper 5"<<"\n";

cin>>choix;

if (choix=1)
{
for(i=0;i<4;i++)/*une fois l'etape d'ajout de contact réaliser fair un compteur et rendre ce pour dynamique*/

cout<<rep[i].Nom<<" "<<rep[i].Prenom<<" "<<rep[i].NumTelFixe<<" "<<rep[i].NumPortable<<" "<<rep[i].Mail<<" "<<"\n";

}

if (choix=2)
{
cout<<"entrez le nom rechercher";
cin>>nom;

for(i=0;i<4;i++)
{
if (strcmp(nom,rep[i].Nom) == 0)
cout<<rep[i].Nom;
else
cout<<"ce nom n'existe pas";
}

}
cout<<"entrez le nom rechercher";
cin>>nom;



cin>>rien;
}

[Lavock]

Alors, avant de te répondre, pourrais-tu :

1. Mettre les balises [CODE].
2. Nous dire le comportement attendu.
3. Nous décrire le comportement actuel.

[ruben06]

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
#include <stdafx.h>
#include <iostream>
 
using namespace std;
void main()
{
 
int i,rien,choix;
char nom[20];
 
struct repertoire
		{
       char Nom[20] ;
       char Prenom[20] ;
       char NumTelFixe[11] ;
       char NumPortable[11];
       char Mail[50];
       };
repertoire rep[100]= {{"myara","ruben","0101010101","0202020202","rm@hotmail.com"},
                     {"sejourne","laurent","0303030303","0404040404","ls@hotmail.com"},
                     {"bustori","christophe","0505050505","0606060606","bc@hotmail.com"},
                     {"melodie","mafi","0707070707","0808080808","mm@hotmail.com"}};
 
 cout<<"pour afficher l'integralité du repertoire taper 1"<<"\n";
 cout<<"pour rechercher a partir du nom taper 2"<<"\n";
 cout<<"pour rechercher a partir du tel taper 3"<<"\n";
 cout<<"pour inserer un nouveau contact taper 4"<<"\n";
 cout<<"pour supprimer un contact taper 5"<<"\n";
 
 cin>>choix;
 
 if (choix=1)
 {
	 for(i=0;i<4;i++)/*une fois l'etape d'ajout de contact réaliser fair un compteur et rendre ce pour dynamique*/
 
 	cout<<rep[i].Nom<<" "<<rep[i].Prenom<<" "<<rep[i].NumTelFixe<<" "<<rep[i].NumPortable<<" "<<rep[i].Mail<<" "<<"\n";
 
 }
 
if (choix=2)
{
	cout<<"entrez le nom rechercher";
 	cin>>nom;
 
   for(i=0;i<4;i++)
    {
     if (strcmp(nom,rep[i].Nom) == 0)
         cout<<rep[i].Nom;
     else
		   cout<<"ce nom n'existe pas";
    }
 
}
    cout<<"entrez le nom rechercher";
 	cin>>nom;
 
 
 
cin>>rien;
}

le programme est censé m'afficher l'integralite du repertoire lorsque l'utilisateur saisi 1 et proposer d'entrez le nom a rechercher lorsque que 2 est saisi mais le programme ne tient pa conte de la saisie et affiche systematiquement l'integralite du repertoire ainsi que "entrez le nom a rechercher" qui n'est censé s'afficher uniquement si l'on tape 2. de plus le recherche ne fonctionne pas car il ne trouve aucun nom et affiche 4 fois "ce nom n'existe pas" au lieu d'une.

Desole pour ce premier post mais je n'ai pas l'habitude de frequenter les forums.

[Lavock]

Je te conseil un petit tour du côté faq/tuto !

Vite fait, un test c'est if truc==machin. Si tu met un seul =, la condition est toujours vrai.

Utilise des string et non pas des char [];

Pour le dernier problème, c'est algorithmique. Essaye d'exécuter ta fonction "manuellement" pour voir pourquoi cela agis comme ça !

[ruben06]

merci! Tu as deja resolu quelques problemes mais je ne vois pas ce que tu veux dire par executer la fonction manuelement

[Lavock]

Sans machine, dans ton esprit, peut-être à l'aide d'une feuille et d'un stylo.

[ruben06]

ah ok je vais essayer et je vais fair un tour a la faq et dans les tutos!
Merci encor

[koala01]

Salut,

Je te conseil un petit tour du côté faq/tuto !

+1

Vite fait, un test c'est if truc==machin. Si tu met un seul =, la condition est toujours vrai.

Pire, encore:

Si tu ne met qu'un seul =, tu affecte machin à truc, ce qui justifie, bien sur que la condition sera toujours vraie, mais qui est, surtout, rarement ce que tu veux faire

Pour le dernier problème, c'est algorithmique. Essaye d'exécuter ta fonction "manuellement" pour voir pourquoi cela agis comme ça !

Un fait, j'irais même beaucoup plus loin...

Idealement, tu devrais, avant même de te jeter sur ton clavier et de commencer à écrire ton code, savoir précisément la logique que tu dois mettre en oeuvre et, même si certains n'aiment pas le coté "petit dessin" de la chose, disposer de ton algorithme sous une forme graphique, telle que le nassi-schneiderman (par contre, je déconseille fortement le flowchart)...

De cette manière, tu "NAPUKA" traduire étape par étape ce qui doit être fait.

Si ton algorithme est bon, tu aura la quasi certitude que le code fera ce que tu attend de lui une fois corrigées les inévitables fautes d'attention (erreurs de syntaxe ou fautes d'orthographe) qui l'empêchent de compiler...

Au pire, il ne restera que les erreurs proches de celle qui relèvent de la différence entre = et ==