Discussion :

[yolpo]

Bonjour,

en suivant quelques tuto je suis tombé sur un cas pratique qui consiste à écrire un convertisseur de degrés (fahrenheit->Celsisus) à l'aide d'un fonction stockée dans un fichier source indépendant. Je l'ai résolu de la sorte :

La fonction

Code :

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

1
2
3
4
 
float farhenheitToCelsius(float fFahr){
	return (5.0f/9.0f)*(fFahr-32.0f);
}

Son utilisation

Code :

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

1
2
3
4
5
6
7
8
9
 
int main() {
	float fCelcius;
	float fFahrenheit = 100.0f;
 
	fCelcius = farhenheitToCelsius(fFahrenheit);
 
	printf("%f F = %f C", fFahrenheit, fCelcius);
}

Tout fonctionne pas de problèmes. En revanche la "correction" utilise une fonction de cette forme

Code :

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

1
2
3
4
5
 
int farhenheitToCelsius(float fFahr, float *fCel){
	*fCel = (5.0f/9.0f)*(fFahr-32.0f);
	return 0;
}

et l'utilise ainsi

Code :

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

1
2
3
4
5
6
7
8
9
 
int main(int argc, char *argv[]) {
	float fCelcius;
	float fFahrenheit = 100.0f;
 
	farhenheitToCelsius(fFahrenheit, &fCelcius);
 
	printf("%f F = %f C", fFahrenheit, fCelcius);
}

D'où ma question, pourquoi ne pas utiliser la valeur de retour de la fonction ? Y a t il un intérêt évident à utiliser un pointeur en paramètre pour la valeur de retour qui m'a échappé ? Y a t il des limites de quelques sortes que ce soit avec les valeur de retour de fonction ?

[NiamorH]

Certainnement que le programmeur utilise une méthode de programmation qui réserve les retours de fonction pour les codes d'erreur (0 == tout va bien).

C'est souvent une technique qu'on retrouve dans les anciennes bibliothèques écrites en C.

Aujourd'hui on peut lever des exceptions si vraiment un problème survient, donc la méthode est un peu désuette si les erreurs possibles ne sont pas dans le flot de traitement de l'application et dans ton cas, totalement inutile. Je préfère de loin ta version.

[yolpo]

Pas de contre-indications pour la valeur de retour donc. Merci pour la réponse.

[Vilmir]

C'est un peu dommage de designer une fonction si simple comme ça. Le code de retour dans ce cas très précis n'a pour moi d'intérêt que si on peut te rentrer des ranges de valeurs de paramètres abbérantes.

Par contre ce type de fonction avec signature void permet d'éviter une création d'object temporaire, celui de ta valeur de retour, en stockant directement la réponse dans l'espace mémoire dans lequel tu vas la ranger par la suite.
Mais tu gagnes peu pour des valeurs de base comme des int pour un confort d'utilisation franchement plus chiant.

De plus, tu prends le risque de rendre ta fonction plus dure à comprendre pour quelqu'un qui voudra la réutiliser, surtout si la doc laisse à désirer.

[NiamorH]

Par contre ce type de fonction avec signature void permet d'éviter une création d'object temporaire, celui de ta valeur de retour, en stockant directement la réponse dans l'espace mémoire dans lequel tu vas la ranger par la suite.

Ca revient au même, si tu passes un pointeur en parametre, il faut bien copier l'adresse car elle est passée par valeur... donc sur un type primitif tu ne gagnes rien.

Par contre, si tu mets ta fonction inline, là tu pourras espérer des gains de perfs si tu renvoies le résultat, car il n'y aura pas de temporaire.

[yolpo]

si j'ai bien compris ça peut être intéressant, d'un point de vu performance, si la valeur de retour est plus complexe qu'un simple type primitif : une copie d'adresse contre un création et une copie d'objet plus complexe.

Et pour une fonction simple, la déclarer inline améliore aussi les choses.

Merci pour toutes ces précisions.