Discussion :

[julien25]

Bonjour, j'expose mon probleme.

Une classe me crée un vector comme suit, je n'ai mis que la partie de code utile pour la compréhension de mon pb.
Ma map est bien remplie, (je l'ai verifie)

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
 
map<string, float> Maclasse::Mafonction(Configuration* cnf)
{.....
.....
.....
map<string, int> monvecteur
        if (file)
	{
		while(!file.fail())
 
		{
		  file>>nom>>valeur;
		  monvecteur[nom]=valeur;		
		  cout<<nom<<"-"<<valeur<<endl;
		}
 
 
 
	}
	else
	{
		cout<<"ERROR :  can't open file";
	}
		return monvecteur;		
}

Maintenant et c'est la que j'ai un problème, je veux utiliser les valeurs de cette map pour faire des calculs mathématique. Je fais dans une autre classe(un autre .cpp)

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

bool Moncalcul::CalculMath(map<string, double>*& blabla)

{	
	cout<<"bonjour : "<<endl;    Jusque la ca marche
	for (map<string, float>::iterator myiter = monvecteur->begin(); myiter!=monvecteur->end(); myiter++)
	{
		cout<<"nom: "<<myiter->first<<endl;
                mon calcul = puis la formule;
	}
	
	return true;
        blabla = moncalcul;
}

C'est comme si il n'arrivait pas a rentrer dans la boucle for. J'ai l'impression qu'il n'arrive pas a trouver cette map.
Pourtant cette fois ci j'ai bien fais une instance de ma classe en créant l'objet
dans mon main de la manière suivante :

Code :

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

1
2
MaClasse* truc = MaClasse
truc->Mafonction();

puis idem pour ma seconde classe.

Code :

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

1
2
MonCalcul* truc2 = MonCalcul
truc2->CalculMath();

J'ai n'ai aucun message d'erreur. Juste un plantage sur le moment en rouge
Merci.

[Kalith]

Je ne vois aucun appel à "Maclasse::Moncalcul" dans le code que tu as posté, et c'est là que je pense qu'il y a une erreur.

Par contre fait attention au code que tu poste, ça ne compilera pas là

Code :

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

vector<string, int> map

Il manque le ";" et tu as mis "vector" au lieu de "map".

Code :

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

1
2
3
bool Moncalcul::Moncalcul(map<string, double>*& blabla)
float moncalcul;
{

J'ai mis du temps à comprendre Tu as inversé les 2 dernière lignes je suppose ?

Ensuite, le nom de ta map passée en argument est "blabla", alors que dans le code de "Moncalcul" tu utilises "map".

Sinon, pourquoi utiliser :

Code :

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

map<string, double>*& blabla

... comme argument de Moncalcul ? Un référence ou un pointeur aurait suffi, non ?

[julien25]

Par contre fait attention au code que tu poste, ça ne compilera pas là

Oui, c'est clair, c'est juste pour l'exemple.

Tu as inversé les 2 dernière lignes je suppose ?

Effectivement, mais tout ceci est bien fait dans mon code original. Je fais tout de même la correction ici.

Je ne vois aucun appel à "Maclasse::Moncalcul" dans le code que tu as posté, et c'est là que je pense qu'il y a une erreur.

J'ai écris "idem pour ma deuxième classe", donc j'ai bien circonstancie cette objet.

Code :

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

Un référence ou un pointeur aurait suffi, non ?

Oui et non,
En fait, je crée de cette façon un pointeur placé en référence pour ne pas a avoir de problème lors de la destruction. Par ailleurs, j'ai beaucoup de fonction et le fais de renvoyer un bool me permet de rester plus clair et de voir si tout c'est passe correctement rapidement.
Cela dit, je peux aussi faire comme tu dis.


Merci d'avoir pris le temps de lire mon code.

Dans la logique tout y est...mais je suis tout de même passé a cote de qq chose .

[Solkien]

Salut,

j'ai du mal a suivre ton code mais ...

Code :

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

for (map<string, float>::iterator myiter = map->begin(); myiter!=map->end(); myiter++)

ne devrait-t'il pas se réécrire

Code :

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

for (map<string, float>::iterator myiter = blabla->begin(); myiter!=blabla->end(); myiter++)

?

[julien25]

Pas tout a fait tu mais par contre :

Code :

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

1
2
 
for (map<string, float>::iterator myiter = monvecteur->begin(); myiter!=monvecteur->end(); myiter++

Mais dans mon vrai code cela est bien fait aussi.
Pour donner plus de précision, je viens de créer un map dans la classe 1
que je cherche a appeler dans la classe 2 en itérant dessus .Ce processus d'itération ne fonctionne pas.

Cette map est crée est vit quelque part (ou pas). Je me demande tout a coup ci la map n'a pas été détruite ce qui expliquerait le pb. Mais alors Comment la recuperer?
Tout en sachant que j'ai instancié mon nouvel objet dans le meme ce qui revient utiliser le mot static.

Encore Merci

[3DArchi]

Salut,
1/ Je ne suis pas convaincu de ton explication sur le pointeur et je sens une mauvaise raison.
2/ Tu as trop simplifié. Fournis un code minimal à partir de ce que tu as fais qui compile et qui reproduit le bug.