Erreur sur chaîne de caractères

Version imprimable

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:

1 2 3 4 5 WrapperClass1() { instance = gcnew Class1(); }
Cas 1 : prototype de la fonction C++ dans le h:
Code:

1 2 __declspec(dllexport) string getLabel(int ID);
ci joint ma fonction :
Code:

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 :

Citation:

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:

1 2 __declspec(dllexport) int saveData(char* filename);
ci joint ma fonction :
Code:

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);"

Citation:

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...

23/04/2013, 16h01
Médinoc

Citation:

'WrapperClass1::Class1::saveData'

Attends, tu as recopier Class1 dans WrapperClass1 ? 8O

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.
23/04/2013, 16h21
Masmeta

Citation:

Envoyé par Médinoc

Attends, tu as recopier Class1 dans WrapperClass1 ? 8O

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

Citation:

Envoyé par Médinoc

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?

Citation:

Envoyé par Médinoc

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?

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:

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:

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());
		}
	};
}

23/04/2013, 16h56
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. :oops: J'ai honte...

A quoi correspond la déclaration "CPPLIB_API " ?
23/04/2013, 17h43
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.