Discussion :

[François L.]

Bonjour,

je ne connais que peu le langage C, et suis actuellement embourbé dans le problème suivant. J'ai cette fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
void*
Create()
{
  struct TModel* pModel;
/* ... */
  pModel = (struct TModel*)malloc(sizeof(struct TModel));
  memset(pModel, 0x0, sizeof(struct TModel));
 
/* remplit les champs de la structure */
  return (void*)pModel;
}

TModel est défini ailleurs, et jusque là tout va bien.

Je souhaite récupérer (dans une autre fonction) la structure ainsi fabriquée (pour lire certains de ses champs). J'avais cru que le code suivant:

Code :

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

1
2
3
4
 
struct TModel* pModel;
 
 pModel = (struct TModel*) Create;

fonctionnerait, mais... Sans générer d'erreur ou crasher le programme, il ne renvoit pas la bonne valeur des champs (je fais un test sur un champ entier initialisé dans Create() )
Il est possible que ça joue: j'utilise un compilateur C++ (le Visual Studio Express de Microsoft). Mon code est un code C (fichier .c en tout cas), mais il est dans un "projet" C++.

D'avance merci

[Bktero]

Quelques remarques :

• Utiliser un compilateur C++ pour du code purement C n'est pas une bonne idée, d'ailleurs, tu arriveras sûrement à des erreurs compilation.
• Inutile de caster le retour de malloc().
• Pourquoi la fonction Create renvoit-elle un void* et non un struct TModel* ?
• Vue comme tu appelles (mal d'ailleurs) la fonction Create, ce serait logique.

Pour ton problème, la ligne :

Code :

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

pModel = (struct TModel*) Create;

n'est pas un appel à la fonction Create. Il faudrait écrire :

Code :

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

pModel = (struct TModel*) Create();

Je pense que ta ligne est une déclaration d'un pointeur sur fonction, mais en changeant le type de retour. A confirmer ^^

N'as tu pas, à la compilation, un warning lié à cette ligne et qui ressemblerait à ça :

d:\...\test.c||In function 'main'
d:\...\test.c|47|warning: cast from pointer to integer of different size|
||=== Build finished: 0 errors, 1 warnings ===|

[François L.]

En fait, le retour de malloc est fait dans la fonction Create(), dont je ne suis pas maître. Du coup, je ne sais pas non plus pourquoi elle renvoit un void*; c'est sans doute inhérent à son usage par le logiciel de simulation qui génère le fichier C qui la contient.

Merci infiniment de ton aide, utiliser Create() marche effectivement beaucoup mieux