Discussion :

[Vooid]

Bonjour,
J'ai été surpris par mon compilateur lorsqu'en initialisant une classe Mesh mesh();
je reçois une erreur un peu plus loin en passant mon object en argument dans une fonction draw(Mesh& mesh),
ou que j'essaye d'accéder à une méthode de mesh :

Code :

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

1
2
error C2228: left of '.attachBuffer' must have class/struct/union
error C2664: 'void Renderer::draw(Mesh &,Program &)': cannot convert argument 1 from 'Mesh (__cdecl *)(void)' to 'Mesh &'

Lorsque je déclare mon object comme ça : Mesh mesh; Ou que je l'initialise Mesh mesh = Mesh(), je ne reçois plus aucune erreur...
Est-ce qu'il y a une différence entre les deux syntaxes, ou j'ai un problème plus grave que ça ? :p

[foetus]

Il faut attendre les experts, mais en théorie, il n'y a aucune différence : c'est l'appel du constructeur par défaut.
Tu as aussi Object obj;.


Mais en C++, il y a une ambiguité qui s'appelle "Most vexing parse" (<- lien wiki anglais)
En gros, lorsque tu codes Object obj(); le compilateur hésite parce qu'il voit la déclaration d'une fonction mais aussi l'appel du constructeur par défaut.

[Maghin]

En fait lorsque tu écris "Mesh mesh();" tu ne déclares pas un objet que tu pense initialiser mais en fait tu déclares une fonction sans argument qui retourne un objet du type "Mesh".

Le point clé dans l'erreur c'est le (__cdecl *) qui est un marqueur du compilateur pour indiquer que c'est une fonction. Et donc le problème est bien là, il n'arrive pas à convertir une fonction en référence...

Personnellement la méthode que je préfère serait en c++ 11: "auto mesh = Mesh()".

[Luc Hermitte]

Je rajouterai que je trouve `T o = T(whatever);` très maladroit. On dirait un développeur fraîchement débarqué en C++ sans adapter ses pratiques.

Au choix (sans les trucs non-idiomatiques):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
// Version avec paramètres
T o(whatever);
T o{whatever}; // quand il n'y a pas de constructeur avec initializer-list qui change des choses.
auto o = T(whatever);
auto o = T{whatever};
 
// Version sans paramètres
T o;
T o{};
auto o = T();
auto o = T{};

[Médinoc]

La plupart de ces solutions étant disponibles à partir de C++11 (qui est supporté pratiquement partout aujourd'hui).

Au moyen-âge du C++, si on voulait déclarer un objet d'un type POD (Plain Old Data, donc sans constructeur) et l'initialiser à des valeurs nulles, alors Object obj; ne suffisait pas, et Object obj = Object(); était la seule solution disponible (puisque Object obj {}; et auto n'existaient pas encore).