Discussion :

[Masmeta]

re-Bonjour

Je suis entrain de wrapper une dll c++ pour l'utiliser sur une application C#.
J'utilise du c++/cli pour faire le passage.

J'ai suivi les conseils de la FAQ sur la conversion de chaine de caractères. Cependant j'ai des messages d'erreur à la compilation que je ne comprend pas

J'ai déclaré en private : Class1 *instance;
et dans le constructeur de mon wrapper

Code :

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

1
2
3
4
5
 
WrapperClass1()
{
    instance = gcnew Class1();
}

Cas 1 : prototype de la fonction C++ dans le h:

Code :

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

1
2
 
__declspec(dllexport) string getLabel(int ID);

ci joint ma fonction :

Code :

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

1
2
3
4
5
6
7
8
9
 
String^ getLabel(int ID) 
		{ 
			std::string result("test");
 
			result = instance->getnetCatLabel(catID);
			System::String ^s = gcnew String(result.c_str());
			return s;
		}

L'erreur sur "result = instance->getnetCatLabel(catID);" est la suivante :

error C2679: binary '=' : no operator found which takes a right-hand operand of type 'System::String ^' (or there is no acceptable conversion)

J'ai du mal à saisir mon erreur, Avez vous une idée pour résoudre ce problème.

Cas 2 :
prototype de la fonction C++ dans le h:

Code :

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

1
2
 
__declspec(dllexport) int saveData(char* filename);

ci joint ma fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
int saveData(System::String ^filename){
 
		char * chaineChar;
		pin_ptr<const wchar_t> wch = PtrToStringChars(filename);
		int taille = (filename->Length+1) * 2;
		chaineChar = new char[taille];
		int t = WideCharToMultiByte(CP_ACP, 0, wch, taille, NULL, 0, NULL, NULL); 
		WideCharToMultiByte(CP_ACP, 0, wch, taille, chaineChar, t, NULL, NULL); 
		 int result = 0;
		 result = instance->saveData(chaineChar);
		 chaineChar=NULL;
		 return result;
		}

J'ai donc cette erreur sur " result = instance->saveData(chaineChar);"

error C2664: 'WrapperClass1::Class1::saveData' : cannot convert parameter 1 from 'char *' to 'System::String ^'"

Je ne comprends pas pourquoi cette erreur. Quelqu'un peut m'expliquer?

Merci de votre aide...

[Médinoc]

'WrapperClass1::Class1::saveData'

Attends, tu as recopier Class1 dans WrapperClass1 ?

Aussi, ta class C++/CLI n'est pas censée avoir de __dllexport sur ses fonctions; le seul fait d'être déclarée en public ref class suffit.

Ensuite, pour chaque classe de la DLL C++, tu dois décider si tu la wrappes ou si tu la remplaces. Une classe POD devra typiquement être remplacée (et convertie) plutôt que wrappée.

[Masmeta]

Attends, tu as recopier Class1 dans WrapperClass1 ?

Non, en faite j'ai marqué le même nom que la classe C++. J'ai donc changé le nom de ma classe C++/CLI

Aussi, ta class C++/CLI n'est pas censée avoir de __dllexport sur ses fonctions; le seul fait d'être déclarée en public ref class suffit.

Oui cela est ok. Les __dllexport correspondent uniquement sur les prototypes de fonction de la classe C++. J'ai remarqué dans l'exemple de nico-pyright(c) qu'il ajoutait extern "c" => Est ce que c'est important?

Ensuite, pour chaque classe de la DLL C++, tu dois décider si tu la wrappes ou si tu la remplaces. Une classe POD devra typiquement être remplacée (et convertie) plutôt que wrappée.

Peux tu m'expliqué ce qu'est une classe POD? Cela correspond à des classe dite de data?

[Médinoc]

extern "C" n'a de sens que pour des fonctions libres exportées ou importées par du code C++. Normalement, ta classe managée ne doit pas en avoir non plus.

Et oui, POD = "Plain Ol' Data", une classe contenant juste des données.

Personnellement, je wrapperais un truc pareil:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#ifdef BUILDING_CPPLIB
#define CPPLIB_API __declspec(dllexport)
#else
#define CPPLIB_API __declspec(dllimport)
#endif
 
namespace cpplib {
	class CPPLIB_API class1
	{
	public:
		std::string getLabel(int ID) const;
	};
}

Ainsi:

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
namespace CppCliLayer {
 
	public ref class Class1
	{
	private:
		cpplib::class1 * pObj;
	public:
		Class1() { pObj = new cpplib::class1(); } /*Constructor*/
		!Class1() { delete pObj; }                /*Finalizer*/
		~Class1() { this->!Class1(); }            /*Disposer: calls finalizer*/
 
		System::String^ GetLabel(int ID) { 
			std::string cppRet = pObj->getLabel(ID);
			return gcnew System::String(cppRet.c_str());
		}
	};
}

[Masmeta]

Merci pour ton aide. J'ai pu me rendre compte de mes erreurs de débutant.


J'ai résolu mon souci, qui était plusieurs erreurs à la con.
La principale était l'oubli d'utilisation du namespace et surtout que ma classe en C++ s'écrit class1 et non Class1. J'ai honte...

A quoi correspond la déclaration "CPPLIB_API " ?

[Médinoc]

C'est un type de macros utilisé pour la plupart des DLLs C et C++ pour déclarer leurs fonctions exportées.

Dans le projet de la DLL tu définis BUILDING_NOMDETALIB, et toutes les fonctions déclarées NOMDETALIB_API sont en dllexport.
Dans les projets qui utilisent la DLL tu ne définis pas la macro, et toutes les fonctions déclarées NOMDETALIB_API sont en dllimport.