C++ Cast void * / struct

j'essaye de mettre en place en C++ (std= c98 ) un typedef a base de structure pour prendre en compte un retour d'erreur et une autre structure de resultat defini comme suit

Code:

---

1 2 3 4 5 6

typedef struct s_res { bool ks; void *res; } t_res;

---

void * devant contenir le pointeur de la structure

Code:

---

1 2 3 4 5 6

typedef struct s_chk_idx { bool ks; size_t idx; } t_chk_idx;

---

puis lors de l'utilisation

Code:

---

1 2 3 4 5 6 7 8 9

t_res PhoneBook::getCmd(void) { t_chk_idx chk; t_res kr;   ...do somethink kr.res = (void *)(&chk); return kr; }

---

et dans le code l'appellant

Code:

---

1 2 3 4 5 6 7 8 9 10 11

int main(void) { ... s_res ret; t_chk_idx chk;   .... ret = Book.getCmd(); chk =(t_chk_idx)(ret.res); ... }

---

le compilateur renvoie une erreur

error: no matching conversion for C-style cast from 'void *' to 't_chk_idx' (aka 's_chk_idx')
chk =(t_chk_idx)(ret.res);

base.hpp:41:16: note: candidate constructor (the implicit copy constructor) not viable: cannot convert argument of incomplete type 'void *' to 'const s_chk_idx' for 1st argument
typedef struct s_chk_idx

j'ai egalement le meme prb en C ( error: conversion to non-scalar type requested) , est-ce qu'un cast void * vers pointeur de structure est possible ?

Peut-être plutôt :

Code:

---

chk = *(t_chk_idx*)ret.res;

---

Et comme on est en C++ :

Code:

---

chk = *static_cast<t_chk_idx*>(ret.res);

---

En réalité, le compilateur te dit le pourquoi de l'erreur :mrgreen:

Déjà, tu es 1 fou :scarymov: des pointeurs de fonction, du void* … donc du bon C des familles :zoubi:
Et les experts ici te diront que le C++98 est dépassé depuis 25 ans, et le C++ moderne (C++11, C++14, C++17, …) a déjà + de 10 ans.

Déjà grosse erreur : :fessee: tu prends le pointeur d'1 variable locale.
Les variables locales sont détruites à la fin du "bloc", donc les variables kr et chk seront détruites à la fin de l'appel de la fonction PhoneBook::getCmd.

https://www.onlinegdb.com/ me dit

Citation:

---

error: no matching function for call to ‘s_chk_idx::s_chk_idx(void*&)’

---

Le problème est l'affectation chk =(t_chk_idx)(ret.res); : le compilateur ne sait pas convertir 1 void* en t_chk_idx et il te demande de faire 1 constructeur de copie.

Donc, il faut faire 1 constructeur de copie dans ta structure struct s_chk_idx/ t_chk_idx.

:weird: bizarre, tu n'as pas utilisé ta variable entre la définition et cette affectation.

1 truc mal codé : la fonction PhoneBook::getCmd ne retourne ni 1 pointeur ni 1 référence (&)
Donc au retour ret = Book.getCmd();, l'affectation va appeler soit le constructeur de copie soit l'opérateur d'affectation.


La différence entre le C et le C++, en C++, 1 structure est 1 classe avec 1 encapsulation publique par défaut (privé pour les classes). En C, tu ne peux pas mettre dans 1 structure des fonctions et des constructeurs (mais des pointeurs de fonction :whistle:)
1 autre différence, en C, tu n'as pas besoin de caster void* (par exemple la fonction malloc).

Je sais que le langage est dépassé mais pas le choix de celui ci c'est un langage imposé dans le cursus de formation , trouver de la doc d'une vieillerie pareille est plus difficile que trouver celle du C ...
après ce langage c'est l'apprentissage du Rust .

je n'ai pas utilisé des variables autre que locale pour gérer des flag de sortie d'erreur et de retour d'une fonction qui me sert de contrôle de flux (aiguillage) dans la fonction appelante.
effectivement l'autre alternative aurait été de faire un passage par référence de la structure et renvoyer un booléen d'erreur

la solution du cast proposé par CGI fonctionne , merci

Citation:

---

effectivement l'autre alternative aurait été de faire un passage par référence de la structure et renvoyer un booléen d'erreur

---

Les exceptions (lancement), ça existe aussi en C++.