Discussion :

[10_GOTO_10]

Je suis confronté à un problème qui me laisse perplexe: J'ai une classe MaClasse que je peux évidemment copier très classiquement avec un =

Code :

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

1
2
3
4
5
6
7
  TMaClasse MaClasse1;
  TMaClasse MaClasse2;
 
  MaClasse1.X = 1;
  MaClasse1.Y = 2;
 
  MaClasse2 = MaClasse1;

J'ai surchargé l'opérateur = pour pouvoir l'initialiser avec une structure. Pour moi, le prototype étant différent, ça devrait juste être une méthode polymorphe et ne pas affecter l'opérateur = par défaut:

le .h

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
//---------------------------------------------------------------------------
typedef struct
{
  short x;
  short y;
} MASTRUCTURE;
//---------------------------------------------------------------------------
class TMaClasse
{
private:
  int FX;
  int FY;
public:
  __fastcall TMaClasse(void);
  __property int X = {read = FX, write = FX};
  __property int Y = {read = FY, write = FY};
 
  const TMaClasse &operator = (const MASTRUCTURE &MaStructure);
};
//---------------------------------------------------------------------------

le .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
20
21
22
23
24
25
26
27
28
29
30
//---------------------------------------------------------------------------
__fastcall TMaClasse::TMaClasse()
{
  FX = 0;
  FY = 0;
}
//---------------------------------------------------------------------------
const TMaClasse &TMaClasse::operator = (const MASTRUCTURE &MaStructure)
{
  FX = MaStructure.x;
  FY = MaStructure.y;
 
  return *this;
}
//---------------------------------------------------------------------------
void __fastcall TForm1::Button1Click(TObject *Sender)
{
  TMaClasse MaClasse1;
  TMaClasse MaClasse2;
 
  MaClasse1.X = 1;
  MaClasse1.Y = 2;
 
  MaClasse2 = MaClasse1; // <- Erreur 
 
  MASTRUCTURE MaStructure;
  MaStructure.x = 3;
  MaStructure.y = 4;
  MaClasse2 = MaStructure;
}

Dans ce cas, j'ai une erreur E2328 les classes avec des propriétés ne peuvent pas être copiées par valeur que je ne comprends pas. Faut-il dans ce cas que je redéfinisse aussi l'opérateur = par défaut ? Pourquoi le fait de redéfinir le = avec un prototype différent affecte le = par défaut ?

[diogene]

Tu dois définir explicitement un opérateur = pour ta TMaClasse. L'opérateur par défaut ne peut opérer sur des classes avec propriétés
//---------------------------------------------------------------------------

Code :

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

1
2
3
4
5
6
const TMaClasse &TMaClasse::operator = (const TMaClasse &MaClasse)
{
  FX = MaClasse.FX;
  FY = MaClasse.FY;
  return *this;
}

[10_GOTO_10]

L'opérateur par défaut ne peut opérer sur des classes avec propriétés

Ben non, puisque dans le premier cas, ça marche très bien (propriétés comprises).

Code :

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

1
2
3
4
5
6
const TMaClasse &TMaClasse::operator = (const TMaClasse &MaClasse) 
{ 
  FX = MaClasse.FX; 
  FY = MaClasse.FY; 
  return *this; 
}

Oui, bien sûr, j'ai bien pensé à faire ça. Seul problème, c'est que j'ai un certain nombre de classes avec plusieurs dizaines de propriétés chacune. Si j'avais pu éviter cette solution, ça m'arrangerais.

[Djob]

le probleme est que l'operateur égal fait partie de ceux que j'appelle les "4 operateurs maudits (), [], -> et = "
que tu n'as pas le droit de déclarer comme amis de la classe ,
ce que l'on fait généralement quand il s'agit de type différents...

pourquoi ne pas envisager le probleme d'une manière différente pour l'affection de ta structure :

Au lieu d'autoriser

Code :

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

 MaClasse = MaStructure;

pourquoi ne pas obliger l'utilisateur à utiliser un constructeur par exemple

Code :

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

MaClasse = MaClasse(MaStructure);

[10_GOTO_10]

pourquoi ne pas obliger l'utilisateur à utiliser un constructeur

Parce que c'est pas forcément au moment de sa construction qu'on veut affecter des valeurs à une classe.

Je pourrais également faire une méthode (c'est sans doute ce que je vais faire), mais:

1) Le code sera moins clair, et il n'avait réellement pas besoin de ça pour être déjà incompréhensible.

2) J'aurais pas compris pourquoi ça marche dans un cas et pas dans l'autre, et j'aime pas ne pas comprendre.

[Djob]

j'ai revérifié en c++ standard ,cette façon de faire passe sans probleme donc tu as raison.

Par contre j'ai l'impression que le probleme se situe au niveau de l'emploi de __property

lors de :

Code :

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

MaClasse2 = MaClasse1; // <- Erreur

toutes les propriétés VCL semble ne pas être acceptée par le compilateur
quand l'opérateur = entre TMaClasse et MASTRUCTURE est défini...

si on enleve les __property du code , la compilation se fait normalement.

il faudrait pouvoir savoir comment fonctionnne en détail le mecanisme de __property .