Discussion :

[olive_le_malin]

Bonjour,

je ne comprends pas pourquoi avec le code suivant :

Code :

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

1
2
3
4
5
6
7
8
9
10
	struct Test
	{
		int _a;
		int _b;
		Test(int a, int b) : _a(a), _b(b) {}
	};
 
	Test T(1, 2);
	std::map<int, Test> M;
	M[0] = T;

j'ai l'erreur :

error C2512: 'Test::Test' : no appropriate default constructor available
d:\program files\microsoft visual studio\vc98\include\map(93) : while compiling class-template member function 'struct Test &__thiscall std::map<int,struct Test,struct std::less<int>,class std::allocator<struct Test> >::operator [](const int

Alors que si je redéfinis mon constructeur de struct ainsi :

Code :

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

1
2
3
4
5
6
	struct Test
	{
		int _a;
		int _b;
		Test(int a=0, int b=0) : _a(a), _b(b) {}
	};

Je n'ai plus de problème.

Pourquoi faut-il absolument un constructeur par défaut ?

Merci par avance.

[Laurent Gomila]

L'appel à l'opérateur [] de std::map construit un objet par défaut, puis renvoie celui-ci par référence ; c'est pour ça qu'il te demande un constructeur par défaut. Ceci-dit tu n'est pas obligé d'en fournir un, si tu insères ton objet autrement :

Code :

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

M.insert(std::make_pair(0, T));

[olive_le_malin]

OK, alors il vaudrait peut être mieux que j'utilise map::insert ...
Le seul truc qui m'embete c'est que j'aurais voulu l'insérer même s'il existait déjà.

Merci beaucoup
@+

[deubelte]

pour olive:
j'essaye de comprendre:
Test(int a=0, int b=0) : _a(a), _b(b) {}

que signifient : _a(a) ou _(b) ?

[Laurent Gomila]

que signifient : _a(a) ou _(b) ?

C'est la liste d'initialisation, voir la FAQ pour plus de détails.

Le seul truc qui m'embete c'est que j'aurais voulu l'insérer même s'il existait déjà.

Dans ce cas, si la classe réelle le permet, définis un constructeur par défaut.
Sinon il faudra insérer de cette manière :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
std::map<int, Test>::iterator It = M.find(0);
 
if (It != M.end())
{
    // Clé déjà présente : on réaffecte
    It->second = T;
}
else
{
    // Clé inexistante : on insère
    M.insert(std::make_pair(0, T));
}

[olive_le_malin]

Salut,

En supposant que je définisse un constructeur par défaut.
Si l'opérateur [] de map crée un objet par défaut, alors il va falloir aussi qu'il l'affecte avec celui passé en paramètre, et donc il faudrait aussi pour + de propreté définir le constructeur de copie et l'opérateur d'affectation, non ?

@+

[Laurent Gomila]

et donc il faudrait aussi pour + de propreté définir le constructeur de copie et l'opérateur d'affectation, non ?

Bien sûr
Ici ceux générés par défaut sont corrects, mais si tu as une classe plus compliquée (genre qui gère une ressource brute) alors il faut définir tout ça.