Discussion :

[rubisvert]

Je n'arrive pas a utiliser new correctement, j'ai le message:
"new peut uniquement être utilisé avec des types valeur simples" avant la compilation.
Si vous avez une idée, merci.

dans mon .h:

Code :

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

1
2
3
4
5
6
7
8
9
 
public ref class CppWrapperClass
{
	private:
		Delaunay::DelaunayClass* pClass;
	public:
		CppWrapperClass();
...	
}

dans mon .cpp:

Code :

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

1
2
3
4
5
 
CppWrapperClass::CppWrapperClass(void)
{
	pClass = new Delaunay::DelaunayClass;  <- !!
}

[ternel]

Cela requiert l'utilisation d'un constructeur.
J'ajoute par principe le unique_ptr (que tu trouveras dans <memory>)
Il est rarissime d'avoir besoin de stocker un new en dehors d'un "smart pointer" tel que unique_ptr (ou shared_ptr, parfois)

Code :

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

1
2
3
4
5
6
public class CppWrapperClass {
	private:
		std::unique_ptr<Delaunay::DelaunayClass> pClass;
	public:
		CppWrapperClass(): pClass(new Delaunay::DelaunayClass() ) {}
};

voire mieux encore:

Code :

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

1
2
3
4
5
6
public class CppWrapperClass {
	private:
		std::unique_ptr<Delaunay::DelaunayClass> pClass;
	public:
		CppWrapperClass(): pClass( std::make_unique<Delaunay::DelaunayClass>() ) {}
};

[Bousk]

Etant donné le public ref class tu n'es pas en C++ mais C++/CX ou CLI et une classe managée ne peut pas contenir d'allocation dynamique sur un type non managé ?
Pour allouer dynamiquement une classe managée, c'est ref new ou gcnew il me semble.

Je trouve bien ça sur SO https://stackoverflow.com/questions/...-managed-class qui indique que ça devrait marcher mais le message d'erreur en français ne retourne aucun résultat chez moi.

[rubisvert]

Je veux du C++, donc j'ai enlever le "ref".
J'ai la même erreur:
"new peut uniquement être utilisé avec des types valeur simples" avant la compilation.

.h:

Code :

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

1
2
3
4
5
6
7
8
9
 
public class CppWrapperClass
{
	private:
		std::unique_ptr<Delaunay::DelaunayClass> pClass;
	public:
		CppWrapperClass();
		~CppWrapperClass();
}

.cpp:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
CppWrapperClass::CppWrapperClass(void)
{
	std::unique_ptr<Delaunay::DelaunayClass> pClass(new Delaunay::DelaunayClass); <- !!
}
 
CppWrapperClass::~CppWrapperClass(void)
{
	pClass.reset();
}

[ternel]

appelle ton constructeur. Il doit au minimum y avoir les parenthèses de l'appel.

Code :

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

new Delaunay::DelaunayClass /* entre là -> */() /* <- et là */

Cela dit, n'utilise pas new, mais make_unique.

Enfin, l'intérêt premier de unique_ptr, c'est de détruire son pointé quand lui-même est détruit.
reset n'est pas nécessaire dans le destructeur.

(Son autre intérêt, c'est de ne pas être copiable, mais transférable.)

Un peu de documentation:
utilisation de new
le RAII
<memory>

[Bousk]

Je sais pas trop ce que tu trafiques, mais choisir de travailler en managé ou non c'est un peu plus complexe que d'enlever le ref de la déclaration d'une classe.
Pourrait-on avoir un semblant de contexte ou quoi que ce soit pour comprendre ce que tu espères faire ?

[bacelar]

On peut avoir la déclaration de la classe "DelaunayClass" ?
Parce que, comme le dit @Bousk, on ne fait pas "comme on veut", à l'arrache.

[Pyramidev]

appelle ton constructeur. Il doit au minimum y avoir les parenthèses de l'appel.

Code :

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

new Delaunay::DelaunayClass /* entre là -> */() /* <- et là */

[...]

Un peu de documentation:
utilisation de new

Je confirme les dires de Bousk : dans une expression avec new qui appelle le constructeur par défaut, les parenthèses du constructeur sont facultatives. Par exemple, le code suivant compile :

Code :

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

std::string* test = new std::string;

D'ailleurs, le lien que tu as donné en parle. Voir le passage : « If initializer is absent, the object is default-initialized. »

[ternel]

ah…
Merci pour ta question, rubisvert, j'aurai appris quelque chose en essayant d'y répondre.

[jo_link_noir]

Je confirme les dires de Bousk : dans une expression avec new qui appelle le constructeur par défaut, les parenthèses du constructeur sont facultatives. Par exemple, le code suivant compile :

Code :

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

std::string* test = new std::string;

D'ailleurs, le lien que tu as donné en parle. Voir le passage : « If initializer is absent, the object is default-initialized. »

J'ajoute une précision, si le type alloué ne possède pas de constructeur, il n'y aura pas d'initialisation. Par contre, avec les parenthèses, il y aura une zéro-initialisation, l'équivalent de T{}.