Discussion :

[ooxoo]

Bonjour !

Je ne comprends pas l'erreur que me fait mon compilateur sur ce code simplifié:

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
 
 
#include "stdafx.h"
#include <iostream>
#include <vector>
class A
{
public:
 
	const int x;	
	A(int ival) :x(ival){};
	//const A& operator =(const A & a){return a;};  
};
 
int _tmain(int argc, _TCHAR* argv[])
{
 
	std::vector<A> levector;
	A a(5);
	levector.push_back(a);//C2582
	A obja = a;	
 
	return 0;
}

L'erreur à la compilation avec visual studio étant:

Code :

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

1
2
 
Erreur	1	error C2582: 'operator =' fonction non disponible dans 'A'	c:\program files\microsoft visual studio 8\vc\include\xutility	2726

Si on enlève les // devant const A& operator =(const A & a){return a;};
il n'y a plus d'erreur.
Donc le problème m'échappe, je ne vois pas en quoi j'ai besoin de redéfinir l'opérateur =, qui est normalement défini par défaut.
Sachant que le problème est aussi lié au fait que x soit constant. Si x n'est plus constant, alors je n'ai plus d'erreur.

merci de votre aide

[damess67]

Bonjour

Les vecteurs ont besoins de savoir comment copier les éléments qu'ils contiennent.

[ram-0000]

Bonjour

Les vecteurs ont besoins de savoir comment copier les éléments qu'ils contiennent.

En clair, cela veut dire qu'il faut que tu rajoutes un constructeur de copie à ta classe:

Code :

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

1
2
3
4
5
 
   A(const A & Other)
   {
      ...
   }

[méphistopheles]

le constructeur suffit? pas besoin de l'operateur = justement ?

[ooxoo]

En rajoutant

Code :

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

1
2
 
A(const A & other) : x(other.x){};

ça ne marche pas...

Sachant qu'à la base, je n'ai pas défini de constructeurs de recopie, je devrai en avoir un généré par défaut.

[méphistopheles]

crée aussi un void operator=

[ooxoo]

Bah j'ai déjà essayé de rajouter un opérateur =(c'est ce que semble vouloir le compilo vu le message d'erreur !) ,et ça résout le problème.
Mais c'est pas la résolution du problème qui m'intéresse, c'est plutôt comprendre pourquoi le compilateur a besoin qu'on redéfinisse = pour réussir la génération

[yan]

Mais c'est pas la résolution du problème qui m'intéresse, c'est plutôt comprendre pourquoi le compilateur a besoin qu'on redéfinisse = pour réussir la génération

Surement parce que x est const. Le compilateur ne peut donc modifier cette valeur et te demande quoi faire.
Et là tu as un problème, dans l'operateur = tu ne peut non plus modifier cette valeur.

[JolyLoic]

Pour mettre un objet dans un vector, il doit répondre à certaines contraintes. Entre autre, on doit pouvoir affecter un objet à un autre. Ce qui n'est pas possible avec un objet constant.

[3DArchi]

Surement parce que x est const. Le compilateur ne peut donc modifier cette valeur et te demande quoi faire.
Et là tu as un problème, dans l'operateur = tu ne peut non plus modifier cette valeur.

L'erreur est ici. Un membre const doit être initialisé à la construction et ne peut plus être modifié ensuite sans rupture de la contrainte const. Donc, le compilateur ne sait pas construire un opérateur = par défaut et te demande quoi faire (donc à toi de le redéfinir). Mais, tu ne pourra pas modifier la valeur const (disons, pas proprement).
2 solutions:
1/ Soit tu a besoin de l'objet: donc au lieu d'en faire une copie, tu en déclare une référence:

Code :

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

A &obja = a;

2/ Soit une copie de l'objet: donc tu utilise le constructeur par copie!

Code :

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

A obja(a);

[ooxoo]

ok j'ai compris, merci pour vos explications.