Discussion :

[titoine1978]

Bonjour,

Je travaille sur la création de ma 1ere API pour contrôler un de mes modules. Etant donné que j'ai beaucoup de choses à contrôler, je passe des flags a plusieurs méthodes dont 2 qui peuvent être assimilées à un Set() et un Get().
On pourrait écrire ça comme suit :

Code :

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

1
2
3
 
bool Set(int ControlFlag, void* Value);
bool Get(int ControlFlag, void* Value) const;

void* permet de faire passer des types quelconques. L'utilisateur est sensé connaitre quel type devra être passé avec tel ou tel control flag.

Exemple, un appel comme celui la permet de contrôler un path dans le module :

Code :

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

1
2
3
4
5
6
 
#define MY_PATH    1
...
...
string MyPath="C:\\ToTo";
Set(MY_PATH, const_cast<char *>(MyPath.c_str()));     // Set ne modifie jamais les données du void*

Maintenant pour le Get, j'écris :

Code :

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

1
2
3
4
5
6
7
 
#define MY_PATH    1
...
...
char* pMyPath=NULL;
Get(MY_PATH, &pMyPath);     // Get renvoie un pointeur interne du module
string MyPath=pMyPath;

Dans ce cas il ne faut surtout pas toucher au pointeur pMyPath et en faire une copie immédiatement.

Je pense que la méthode Set est plus ou moins propre : absence de const, passage obligé par le const_cast pour les string stl.

Le méthode Get me semble moche : possibilité de modifier le buffer interne au module si on ne fait pas attention. Si je demande la taille de la chaine au module, je dois faire une allocation de la chaine à passer, le module recopiera sa chaine dedans, et je devrais refaire une copie dans ma string stl puis un delete de ma chaine. Je préférais avoir un pointeur directement.

J'aimerai avoir vos conseils d'experts sur ces mochetés
Merci.

[Laurent Gomila]

Quel est l'avantage de ceci

Code :

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

1
2
Set(MY_PATH, blabla);
Set(MY_NUMBER, pwetpwet);

Par rapport à ceci ?

Code :

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

1
2
SetPath(blabla);
SetNumber(pwetpwet);

Si tu tiens vraiment à avoir des Get et des Set "génériques", fais-en au moins une version pour chaque type. Un peu comme le fait OpenGL par exemple :

SetString(MY_PATH, blabla);
SetInteger(MY_NUMBER, pwetpwet);

Mais honetement, à part perdre du typage et faire un code crade, je ne vois aucun intérêt à utiliser des fonctions aussi génériques.

[titoine1978]

En fait c'est parceque je me retrouve avec des centaines de methodes Set/Get qui je trouve alourdissent le code. Il y en plus une couche entre les appels utilisateurs et les appels au module (gestion de threads) et je suis obligé de reporter toutes les fonctions dans cette interface. Là j'en ai simplement 4.

[titoine1978]

Salut,

Quand je parlais module je pensais DLL. Je ne voudrais pas avoir des centaines de fonctions dans cette DLL.

J'ai vu une API de ce style, maximum 10 fonctions pour contrôler un module et c'est très propre, même dans le code interne (c'est une grosse boite de produits graphique qui fais ça, le code est nickel). Le hic c'est qu'il y a très peu de passage de string, quand on veut en récupérer une, on doit demander sa taille, allouer un buffer et le passer dans un second appel. C'est juste ce point que je trouve lourd.

Je vais surmonter ça en supprimant les const_cast en me passant des string à l'appel. Pour le Get je vais devoir demander la taille et allouer un buffer temporaire.

Merci pour les conseils Laurent.

[loufoque]

Le hic c'est qu'il y a très peux de passage de string, quand on veut en récupérer une, on doit demander sa taille, allouer un buffer et le passer dans un second appel. C'est juste ce point que je trouve lourd.

Retourne tout simplement une std::string.

[titoine1978]

Retourne tout simplement une std::string.

J'y avais pensé mais je voudrais garder juste des types de base pour que ce soit plus simple à interfacer avec d'autre langage comme le C#, VB etc.

[loufoque]

Va donc dans le forum C.

[Médinoc]

Si tu veux interfacer avec les autres langages dont VB, je te conseille de retourner carrément une BSTR (qui s'obtient à partir d'une chaîne de wchar_t avec la fonction SysAllocString()).