Discussion :

[salseropom]

Bonjour, ce post fait suite à un autre où vous m'avez convaincu de déclarer des fonctions static. J'ai fait un 1e essai en mettant tout simplement le mot clé static dans la déclaration de la fonction dans le .h et idem dans la définition de la fonction dans le .c Là, j'ai un warning disant

../Headers/Myheaders/mechanism.h:166: attention : `CheckUnusedSpecies' declared `static' but never defined

ce que je comprends car dans le .h on ne met que des fonctions "publiques" (et si j'ai bien compris les fonctions static sont "privées"). Donc, je pense organiser mes fichiers sources .c de la manière suivante

/* déclaration de toutes les fonctions privées */
/*ici je mets tous les prototypes de mes fonctions static */
static int f1(int,int);
static double f2(void);

/* définition de toutes les fonctions publiques */
ici je mets l'implémentation des fonctions non static

/* définition de toutes les fonctions privées */
ici je mets l'implémentation des fonctions static

et bien sûr, dans les headers, je ne mets que les prototypes des fonctions publiques (non static)
Est-ce une bonne chose de raisonner ainsi ?

Merci encore de votre aide

[diogene]

Est-ce une bonne chose de raisonner ainsi ?

Oui.
- On doit aussi inclure le header comportant les déclarations de type et les prototypes des fonctions "publiques"
- Le prototype des fonctions static n'est utile que si elles sont utilisées avant leur définition.
- L'ordre de la définition des fonctions (static ou non) est libre

[Thierry Chappuis]

Tu trouveras, je pense, la réponse à ta question ici: http://emmanuel-delahaye.developpez....ganiser_source

Thierry

[salseropom]

merci

[Emmanuel Delahaye]

(et si j'ai bien compris les fonctions static sont "privées").

Oui. Ce que tu as fait est une très bonne démarche. C'est ce que j'appelle en cours, la méthode de "l'erreur juste."

On crée, une erreur, on constate la réaction du compilateur, on cherche à l'intepréter, voir si elle est cohérente, et on conclus "l'erreur est juste". Tout va bien. Ne reste plus qu'à la corriger.

Evidemment, cette technique n'est pas assez intellectuelle, alors elle n'est jamais enseignée...

Donc, je pense organiser mes fichiers sources .c de la manière suivante

/* déclaration de toutes les fonctions privées */
/*ici je mets tous les prototypes de mes fonctions static */
static int f1(int,int);
static double f2(void);

/* définition de toutes les fonctions publiques */
ici je mets l'implémentation des fonctions non static

/* définition de toutes les fonctions privées */
ici je mets l'implémentation des fonctions static

et bien sûr, dans les headers, je ne mets que les prototypes des fonctions publiques (non static)
Est-ce une bonne chose de raisonner ainsi ?

Oui.

Mais je suggère plutôt ceci, c'est plus logique et plus simple. (J'aime bien quand c'est simple).

Dans les sources (.c)

/* définition de toutes les fonctions privées */
ici je mets l'implémentation des fonctions static

/* définition de toutes les fonctions publiques */
ici je mets l'implémentation des fonctions non static

Et voilà. En principe, pas besoin de prototypes séparés pour les fonctions privées.