Je n'arrive pas à aller rechercher un string, j'obtiens des symbôles à la place

Bonjour tout le monde,

Je dois récupérer une chaine de caractère mais tout ce que je récupère s'est en fait ça :

0x00000000 donc NULL

Voici la méthode qui est censé afficher le résultat :

Code:

1
2
3
4
5
6
7
8
void COiseaux::Afficher() const
{
	for(long n=0;n<m_Tailles.Nombre();n++)
	{
		CTaille& T = m_Tailles.Element(n);
		printf("fourchette n° %ld : %s\n",(n+1),T.Taille());
	}
}

Le problème (enfin je pense) s'est que cet accesseur renvoie "" :

Code:

1
2
3
4
const char* CTaille::Taille() const
{
	return m_Taille.Chaine();
}

Je vous laisse le code de mon cpp :

Code:

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
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
#include <BasicConsole.h>
#include "Oiseaux.h"
 
////////////////////
// Classe CTaille //
////////////////////
 
CTaille::CTaille()
:m_TailleMin(0.0),m_TailleMax(0.0)
{
}
 
CTaille::CTaille(const CTaille& Source)
{
}
 
CTaille::CTaille(double UneTailleMin,double UneTailleMax)
:m_TailleMin(0.0),m_TailleMax(0.0)
{
	TailleMin(UneTailleMin);
	TailleMax(UneTailleMax);
}
 
CTaille::CTaille(const char* UneTaille)
:m_Taille("")
{
	Taille(UneTaille);
}
 
CTaille::~CTaille()
{
 
}
 
CGenerique* CTaille::Cloner() const
{
	return new CTaille(*this);
}
 
bool CTaille::EstValide() const
{
	return true;
}
 
bool CTaille::TailleMin(double UneTailleMin)
{
	if(UneTailleMin>0)
	{
		m_TailleMin = UneTailleMin;
		return true;
	}
	return false;
}
 
bool CTaille::TailleMax(double UneTailleMax)
{
	if(UneTailleMax<0)return false;
	m_TailleMax = UneTailleMax;
	return true;
}
 
double CTaille::TailleMin() const
{
	return m_TailleMin;
}
 
double CTaille::TailleMax() const
{
	return m_TailleMax;
}
 
bool CTaille::GereUnicite() const
{
	return true;
}
 
bool CTaille::Taille(const char* UneTaille)
{
	if((strlen(UneTaille)<1) || (strchr(UneTaille,'-')!=0))return false;
	return m_Taille.Chaine(UneTaille);	
}
 
const char* CTaille::Taille() const
{
	return m_Taille.Chaine();
}
 
int CTaille::Comparer(const CGenerique& Autre, long TypeComparaison) const
{
	return m_Taille.Comparer(((const CTaille&)Autre).m_Taille,Comparaison_TriCroissantNonStrict);
}
 
///////////////////////
// Class CTabTailles //
///////////////////////
 
CTabTailles::CTabTailles()
:CTableauGenerique(CTaille())
{
}
 
CTabTailles::CTabTailles(const CTabTailles& Source)
:CTableauGenerique(Source)
{
}
 
CTabTailles::~CTabTailles()
{
}
 
CGenerique* CTabTailles::Cloner() const
{
	return new CTabTailles(*this);
}
 
CTaille& CTabTailles::Element(long Indice) const
{
	return (CTaille&)CTableauGenerique::Element(Indice);
}
 
long CTabTailles::Indice(const char* UneTaille) const
{
	return CTableauGenerique::Indice(CTaille(UneTaille));
}
 
bool CTabTailles::Ajouter(const CTaille& Modele)
{
	return CTableauGenerique::Ajouter(Modele);
}
 
void CTabTailles::Trier()
{
	CTableauGenerique::Trier(ComparaisonPourTri);
}
 
////////////////////
// class COiseaux //
////////////////////
 
COiseaux::COiseaux()
{
}
 
COiseaux::~COiseaux()
{
}
 
bool COiseaux::Charger(const char* NomFichier)
{
	CChargeurFichierTexte Chargeur;
	if(!Chargeur.Charger(NomFichier,'\t',FctnTraiterLigne,this))return false;
	return true;	
}
 
bool COiseaux::TraiterLigne(const CChargeurFichierTexte& Chargeur)
{
 
	if (Chargeur.NombreChamps() == 2)
	{
		if(strcmp(Chargeur.Champs(0),"taille")==0)
		{
			const char* LaTaille = Chargeur.Champs(1);
			if(!m_Tailles.Ajouter(LaTaille))return true;
 
		}
 
	}
 
 
 
	return true;
 
}
 
bool COiseaux::FctnTraiterLigne(const CChargeurFichierTexte& Chargeur, void* Contexte)
{
	return (Contexte != NULL) ? ((COiseaux*)Contexte)->TraiterLigne(Chargeur) : false;
}
 
void COiseaux::Afficher() const
{
	for(long n=0;n<m_Tailles.Nombre();n++)
	{
		CTaille& T = m_Tailles.Element(n);
		printf("fourchette n° %ld : %s\n",(n+1),T.m_Taille);
	}
}

et aussi de Chaine.cpp :

Code:

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
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
#include <BasicConsole.h>
#include "chaine.h"
 
 
////////////////////
// Classe CChaine //
////////////////////
 
CChaine::CChaine()
:m_Chaine(NULL)
{
}
 
CChaine::CChaine(const CChaine& Source)
:m_Chaine(NULL)
{
	Chaine(Source.m_Chaine);
}
 
CChaine::CChaine(const char* UneChaine)
:m_Chaine(NULL)
{
	Chaine(UneChaine);
}
 
CChaine::~CChaine()
{
	Vider();
}
 
bool CChaine::GereUnicite() const
{
	return true;
}
 
long CChaine::NombreClesDeTri() const
{
	return 4;
}
 
CGenerique* CChaine::Cloner() const
{
	return new CChaine(*this);
}
 
bool CChaine::EstValide() const
{
	return true;
}
 
int CChaine::Comparer(const CGenerique& Autre, long TypeComparaison) const
{
	switch (TypeComparaison)
	{
	case ComparaisonPourUnicite:
		return strcmp(Chaine(),((const CChaine&)Autre).Chaine());
	case Comparaison_TriCroissantStrict:
		return strcmp(Chaine(),((const CChaine&)Autre).Chaine());
	case Comparaison_TriDecroissantStrict:
		return -strcmp(Chaine(),((const CChaine&)Autre).Chaine());
	case Comparaison_TriCroissantNonStrict:
		return stricmp(Chaine(),((const CChaine&)Autre).Chaine());
	case Comparaison_TriDecroissantNonStrict:
		return -stricmp(Chaine(),((const CChaine&)Autre).Chaine());
	}
	return 0;
}
 
void CChaine::Vider()
{
	if (m_Chaine != NULL)
	{
		free(m_Chaine);
		m_Chaine = NULL;
	}
}
 
bool CChaine::Chaine(const char* UneChaine)
{
	if (UneChaine == NULL) return false;
	void* Nouveau = realloc(m_Chaine,(strlen(UneChaine)+1)*sizeof(char));
	if (Nouveau == NULL) return false;
	m_Chaine = (char*)Nouveau;
	strcpy(m_Chaine,UneChaine);
	return true;
}
 
bool CChaine::Concatener(const char* UneChaine)
{
	if (UneChaine == NULL) return false;
	long Taille = strlen(Chaine());
	void* Nouveau = realloc(m_Chaine,(Taille+strlen(UneChaine)+1)*sizeof(char));
	if (Nouveau == NULL) return false;
	m_Chaine = (char*)Nouveau;
	strcpy(m_Chaine+Taille,UneChaine);
	return true;
}
 
bool CChaine::Concatener(long NombreFois, char Caractere)
{
	if (NombreFois < 1) return false;
	long Taille = strlen(Chaine());
	void* Nouveau = realloc(m_Chaine,(Taille+NombreFois+1)*sizeof(char));
	if (Nouveau == NULL) return false;
	m_Chaine = (char*)Nouveau;
	while (NombreFois-- > 0) m_Chaine[Taille++] = Caractere;
	m_Chaine[Taille] = 0;
	return true;
}
 
const char* CChaine::Chaine() const
{
	return (m_Chaine != NULL) ? m_Chaine : "";
}
 
bool CChaine::EstNull() const
{
	return (m_Chaine == NULL);
}
 
bool CChaine::EstVide() const
{
	return (m_Chaine == NULL) || (m_Chaine[0] == 0);
}
 
 
////////////////////////////
// Classe CChaineComposee //
////////////////////////////
 
CChaineComposee::CChaineComposee()
:m_NbrChamps(0),m_Champs(NULL)
{
}
 
CChaineComposee::CChaineComposee(const CChaineComposee& Source)
:CChaine(Source),m_NbrChamps(0),m_Champs(NULL)
{
	if (Source.m_NbrChamps > 0)
	{
		m_Champs = (char**)malloc(Source.m_NbrChamps*sizeof(char*));
		if (m_Champs != NULL)
		{
			m_NbrChamps = Source.m_NbrChamps;
			for (long n=0; n < Source.m_NbrChamps; n++)
			{
				long PosRel = Source.m_Champs[n] - Source.m_Champs[0];
				m_Champs[n] = ((char*)Chaine()) + PosRel;
			}
		}
	}
}
 
CChaineComposee::CChaineComposee(const char* UneChaine, char UnSeparateur)
:CChaine(UneChaine),m_NbrChamps(0),m_Champs(NULL)
{
	Decomposer(UnSeparateur);
}
 
CChaineComposee::~CChaineComposee()
{
	if (m_Champs) free(m_Champs);
}
 
bool CChaineComposee::GereUnicite() const
{
	return false;
}
 
long CChaineComposee::NombreClesDeTri() const
{
	return 0;
}
 
CGenerique* CChaineComposee::Cloner() const
{
	return new CChaineComposee(*this);
}
 
int CChaineComposee::Comparer(const CGenerique& Autre, long TypeComparaison) const
{
	return 0;
}
 
long CChaineComposee::NombreChamps() const
{
	return m_NbrChamps;
}
 
const char* CChaineComposee::Champs(long Indice) const
{
	return ( (Indice >= 0) && (Indice < m_NbrChamps) ) ? m_Champs[Indice] : "";
}
 
bool CChaineComposee::Definir(const char* UneChaine, char UnSeparateur)
{
	if (!Chaine(UneChaine)) return false;
	return Decomposer(UnSeparateur);
}
 
bool CChaineComposee::Decomposer(char UnSeparateur)
{
	m_NbrChamps = 0;
	char* Texte = (char*)Chaine();
	if (Texte == NULL) return false;
	void* Nouveau = realloc(m_Champs,1*sizeof(char*));
	if (Nouveau == NULL)
	{
		if (m_Champs)
		{
			free(m_Champs);
			m_Champs = NULL;
		}
		return false;
	}
	m_Champs = (char**)Nouveau;
	m_Champs[m_NbrChamps] = Texte;
	m_NbrChamps++;
	while (true)
	{
		char* Delimiteur = strchr(Texte,UnSeparateur);
		if (Delimiteur == NULL) return true;
		*Delimiteur = 0;
		Texte = Delimiteur + 1;
		void* Nouveau = realloc(m_Champs,(m_NbrChamps+1)*sizeof(char*));
		if (Nouveau == NULL) return false;
		m_Champs = (char**)Nouveau;
		m_Champs[m_NbrChamps] = Texte;
		m_NbrChamps++;
	}
}
 
 
//////////////////////////////////
// Classe CChargeurFichierTexte //
//////////////////////////////////
 
CChargeurFichierTexte::CChargeurFichierTexte()
{
}
 
CChargeurFichierTexte::CChargeurFichierTexte(const CChargeurFichierTexte& Source)
:CChaineComposee(Source)
{
}
 
CChargeurFichierTexte::~CChargeurFichierTexte()
{
}
 
CGenerique* CChargeurFichierTexte::Cloner() const
{
	return (CGenerique*)((void*)new CChargeurFichierTexte(*this));
}
 
bool CChargeurFichierTexte::Charger(const char* NomFichier, char UnSeparateur, bool (* FctnTraiterLigne)(const CChargeurFichierTexte&, void*), void* Contexte)
{
	const long		TailleLigne = 500;
	char			Ligne[TailleLigne+1];
	FILE*			Fichier;
 
	if ( (NomFichier == NULL) || (*NomFichier == 0) ) return false;
	Fichier = fopen(NomFichier,"rt");
	if (Fichier == NULL) return false;
	while (fgets(Ligne,TailleLigne,Fichier) != NULL)
	{
		long i = strlen(Ligne);
		if ( (i > 0) && (Ligne[i-1] == '\n') ) Ligne[i-1] = 0;
		if (Definir(Ligne,UnSeparateur))
		{
			if (!FctnTraiterLigne(*this,Contexte))
			{
				fclose(Fichier);
				return false;
			}
		}
	}
	fclose(Fichier);
	return true;
}

Vous trouverez en attaché le résultat que ça donne.

Merci d'avance pour votre aide.

beegees

Juste une question, es tu sur que ta chaîne se termine bien par '\0' ?

Sinon, la seule chose peut venir d'un mauvais pointeur mémoire .. je regarde un peu plus en détail ton code ..

Citation:

Envoyé par raptor70

Juste une question, es tu sur que ta chaîne se termine bien par '\0' ?

Sinon, la seule chose peut venir d'un mauvais pointeur mémoire .. je regarde un peu plus en détail ton code ..

Salut,

Merci pour ta réponse.

Oui chaque ligne se termine bien par \0

Je cherches depuis des heures mais même si j'avance, je ne trouve pas la solution au problème.

J'ajoute le fichier .h, peut être que ça peut aider :

Code:

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
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
#ifndef OISEAUX_H
#define OISEAUX_H
 
#include "generique.h"
#include "chaine.h"
 
//prédéclaration des classes
class CTaille;
class CTabTailles;
class COiseaux;
 
class CTaille : public CGenerique
{
	friend COiseaux;
public:
	CTaille();
	CTaille(const CTaille& Source);
	CTaille(double TailleMin, double TailleMax);
	CTaille(const char* UneTaille);
	~CTaille();
 
	CGenerique*			Cloner() const;
	bool				EstValide() const;
	bool				TailleMin(double UneTailleMin);
	bool				TailleMax(double UneTailleMax);
	double				TailleMin() const;
	double				TailleMax() const;
	bool				Taille(const char* UneTaille);
	const char*			Taille() const;
	bool				GereUnicite() const;
	int					Comparer(const CGenerique& Autre, long TypeComparaison) const;
 
private:
	double				m_TailleMin;
	double				m_TailleMax;
	CChaine				m_Taille;
 
};
 
class CTabTailles : public CTableauGenerique
{
public:
 
	CTabTailles();
	CTabTailles(const CTabTailles& Source);
	~CTabTailles();
 
	CGenerique*			Cloner() const;
	CTaille&			Element(long Indice) const;
	long				Indice(const char* UneTaille) const;
 
	bool				Ajouter(const CTaille& Modele);
	void				Trier();
 
};
 
 
class COiseaux
{
public:
	COiseaux();
	~COiseaux();
 
	bool				Charger(const char* NomFichier);
	void				Afficher() const;
 
private:
	static bool			FctnTraiterLigne(const CChargeurFichierTexte& Chargeur, void* Contexte);
	bool				TraiterLigne(const CChargeurFichierTexte& Chargeur);
 
private:
	CTabTailles			m_Tailles;
	CChaine				m_TaillesEnChaine;
};
 
#endif//OISEAUX_H

Merci pour ton aide.

beegees

pkoi avoir recoder CChaine alors que std::string me semble fort approprier?

ERREUR: Ce constructeur de copie ne copie rien du tout.
Code:

1 2 3 CTaille::CTaille(const CTaille& Source) { }

Citation:
Envoyé par Médinoc
ERREUR: Ce constructeur de copie ne copie rien du tout.
Code:

1 2 3 CTaille::CTaille(const CTaille& Source) { }
Salut Medinoc,

Tu est mon sauveur :king:

En plus, s'est pas la première fois que je commets cette erreur :sm:

Encore un super grand merci pour ton aide Médinoc, je n'oublierai pas.

beegees

Citation:
Envoyé par Médinoc
ERREUR: Ce constructeur de copie ne copie rien du tout.
Code:

1 2 3 CTaille::CTaille(const CTaille& Source) { }
Jolie coup d'oeil :king: