Retourner une structure ou -1 si erreur

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:

---

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.

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:

---

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:

---

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:

---

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

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.

Citation:

---

Envoyé par 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?

---

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

Citation:

---

Envoyé par micaz

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

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

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

Citation:

---

Envoyé par superjaja

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

---

Juste un petit bémol sur ce point, la solution 2 est aussi envisageable et largement utilisée (au hasard : fopen, malloc). Le choix entre les deux va fortement dépendre de la sémantique associée à la fonction :

• Dans le cas d'une création ou acquisition de ressource, la solution 2 (retour d'un pointeur) est courante et, à mon avis, plus pertinente.
• Dans le cas d'un traitement ou de la libération de ressource, la solution 1 (passage d'un pointeur en paramètre et code d'erreur en retour) est généralement préférée.

Oui tout a fait d'accord avec toi mais pour moi d'apres son exemple je pense que ca fonction correspond plus à la methode 1 apres je peux me tromper .