Discussion :

[micaz]

Bonjour à tous,

j'ai une fonction mafonction qui retourne une variable de type t_perso.

Mon type t_perso étant une structure encapsulant un entier (int), et une chaine de caractères., c'est à dire :

Code :

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

1
2
3
4
5
 
typedef struct {
  int age;
  char nom[MAX_LEN];
} t_perso;

Mon problème, c'est que dans t_perso mafonction() je dois renvoyer -1 si erreur, et si tout se passe bien je renvoie la structure correctement remplie.

Si je met return -1 gcc me renvoie erreur: incompatible types in return. Normal.
Si j'essaie de caster : return (t_perso) -1 j'obtiens erreur: conversion vers un type non scalaire demandée.

Est-ce que la solution est d'utiliser une union qui encapsulerait t_perso et un entier?

Merci.

[ram-0000]

Une fonction ne peut pas retourner une structure ou un nombre. Elle retourne l'un ou l'autre mais pas les 2 (même si ce n'est pas en même temps).

ce que tu peux faire, c'est passer en paramètre une structure à renseigner par ta fonction. Cette fonction retourne alors -1 en cas d'erreur (et différent de -1 ca marche) :

Code :

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

1
2
3
4
5
6
7
8
9
10
int MaFonction(t_perso * Personnel)
{
   // un test
   if(...)
      return -1;
 
   // c'est tout bon, on peut renseigner la structure
   Personnel.age = 20;
   return 0;
}

ou alors un pointeur de structure allouée par la fonction qui vaut NULL en cas d'erreur (mais il faut penser à la libération de la structure)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
t_perso *MaFonction(void)
{
   // un test
   if(...)
      return NULL;
 
   t_perso * new_perso = malloc(sizeof(*new_perso));
   if(new_perso == NULL)
      return NULL;
 
   // c'est tout bon, on peut renseigner la structure
   new_perso->age = 20;
   return new_perso;    // ne pas oublier de liberer cette structure
}

Une autre possibilité est un mélange des 2, la fonction retourne la structure passée en paramètre si c'est bon ou NULL en cas d'erreur (plus de problème de libération

Code :

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

1
2
3
4
5
6
7
8
9
10
t_perso *MaFonction(t_perso * Personnel)
{
   // un test
   if(...)
      return NULL;
 
   // c'est tout bon, on peut renseigner la structure
   Personnel->age = 20;
   return Personnel;
}

Voilà, tu as le choix, après, c'est fonction de ton contexte

[micaz]

J'avais en effet penché pour les solutions 1 et 3. J'essaye de trouver une solution sans modifier la signature de la fonction (qui m'est fournie) si possible. C'est pour celà que j'ai commencé à me renseigner sur les unions, n'ayant jamais eu à en utiliser jusque là.
Une solution dans cette direction est-elle envisageable?

Sinon une idée sur pourquoi on ne peut pas caster un entier en t_perso?
edit: si je comprends bien, on ne peut pas caster vers une structure ou autre type non scalaire.

[ram-0000]

J'avais en effet penché pour les solutions 1 et 3. J'essaye de trouver une solution sans modifier la signature de la fonction (qui m'est fournie) si possible. C'est pour celà que j'ai commencé à me renseigner sur les unions, n'ayant jamais eu à en utiliser jusque là.
Une solution dans cette direction est-elle envisageable?

Oui, pourquoi pas mais faut vraiment que tu sois obligé, changer la signature si possible me parait plus sain.

Sinon une idée sur pourquoi on ne peut pas caster un entier en t_perso?

Parce que c'est mal (et pourquoi pas caster un char * en double). Je te rassure, cela va marcher mais 1/ c'est pas beau, 2/ penses au mal de tête du suivant qui va prendre ton code 3/ c'est la porte ouverte à des bugs

[micaz]

Oui, je pense qu'après discussion avec le chargé de projet je serais autorisé à changer la signature de la fonction, cà sera plus propre. Je marque le sujet comme étant résolu.

En tous les cas, merci pour ces réponses rapides ram-0000

[superjaja]

C'est meme curieux que dans un projet on te donne ce type de signature obligatoire et qui n'a pas beaucoup de sens en C. Pour moi la meilleur solution est la 1. De plus c'est typiquement comme cela qu'on code en C on a cas voir les API systeme Linux ou Windows...