Discussion :

[nina2007]

Bonjour à tous,

mon problèm est tout simple, pourant je ne vois pas de quoi il s'agit
voilà, j'utilise un méthode qui prend comme paramétre un pointeur su un map<int,int>

dans cette méthode, je voudrai inserer des elements dans le map
la compilation marche bien mais aprés le insert provoque une erreur de segmentation

Code :

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

1
2
3
4
5
6
7
8
 
 
void ma_methode(fstream *f,map<int, int> *m)
{
//exemple
(*m).insert(pair<int, int>(3,2));
 
}

apres dans le main je fais appel à ma methode de la façon suivante

Code :

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

1
2
3
4
5
 
 
map<int,int> *m;
 
ma_methode(&file,m);

meci si vous avez des idées

[salseropom]

Code :

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

1
2
3
4
5
6
7
8
 
 
void ma_methode(fstream *f,map<int, int> *m)
{
//exemple
(*m).insert(pair<int, int>(3,2));
 
}

Code :

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

1
2
3
4
5
 
 
map<int,int> *m;
 
ma_methode(&file,m);

j'aurais plutôt fait ceci

Code :

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

1
2
3
4
5
6
 
void ma_methode(fstream *f,map<int, int> *m)
{
//exemple
m->insert(std::make_pair(3,2));
}

sache que (*m). est identique à m->

Dans ton main, il y a un problème : jamais tu n'as alloué la mémoire pour la variable m. A ta place, j'aurais fait :

Code :

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

1
2
3
4
5
 
 
map<int,int> m;
 
ma_methode(file,m);

avec comme fonction

Code :

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

1
2
3
4
5
6
 
void ma_methode(fstream  & f,map<int, int>  & m)
{
//exemple
m.insert(std::make_pair(3,2));
}

ce code ressemble plus à du C++. Le tiens ressemblait plus à du C

[koala01]

Salut,

Déjà, je ne comprend pas trop pourquoi passer un pointeur sur fstream et sur map au lieu d'une référence ... c'est souvent se faire du mal pour rien

Ensuite, l'idéal est souvent de passer par la fonction make_pair pour obtenir quelque chose d'insérable de manière correcte dans ta map.

Cela se traduirait donc par un code proche de

Code :

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

1
2
3
4
5
6
7
8
9
10
void ma_methode(ifstream & f,map<int, int> &m)
{
    /* je présumes que, si tu transmet un fichier, c'est pour y lire les infos
     * qui t'intéressent  :D 
     */
    int key; 
    int value;
    f>>key>>value;
    m.insert(make_pair(3,2));
}

[nina2007]

merci

le problème c'etait effectivement le passage du map
mais rien avoir avec make_pair (ça marche avec mon code)

sinon je comprends toujours pas pourquoi quand je fais passer un pointeur sur map, ça marche pas!!!

[salseropom]

sinon je comprends toujours pas pourquoi quand je fais passer un pointeur sur map, ça marche pas!!!

tout simplement parce que tu n'as jamais alloué ta mémoire...

[nina2007]

quand je fais

Code :

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

1
2
 
map<int,int> *m

c'est pas une allocation de memoire???
un pointeur sur un map n'implique pas une allocation de memoire?
merci

[koala01]

quand je fais

Code :

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

1
2
 
map<int,int> *m

c'est pas une allocation de memoire???
un pointeur sur un map n'implique pas une allocation de memoire?
merci

Ou la... non pas du tout: un pointeur, ce n'est qu'une variable qui permet de représenter une adresse mémoire, rien de plus.

C'est bien souvent utilisé en conjonction avec l'allocation dynamique de mémoire (new) qui implique, elle-même de veiller à la libérer correctement lorsque l'on n'a plus besoin de l'objet pointé (delete), car l'allocation dynamique de la mémoire implique que l'on prend la responsabilité de la durée de vie de l'objet.

[nina2007]

merci pour votre explication