Question philosophique fonction statique

Bonjour,

J'ai une petite question philosophique et je n'ai pas su trancher :

Code:

---

1 2 3 4 5 6 7 8 9 10

class A { public: ...   private: static inline const char *MonFormat1(void) { return "==> %d"; } inline const char *MonFormat2(void) const { return "==> %d"; } };

---

La classe A dispose d'une fonction retournant un format d'affichage à utiliser. J'ai 2 possibilités :
- la déclarer en statique (comme pour la fonction MonFormat1)
- la déclarer en membre constant (comme pour la fonction MonFormat2)


Cette fonction est privée car elle n'a pas besoin d'être connue à l'extérieur de la classe.

Cette fonction ne manipule aucunes des variables membres de la classe.

J'aurai aussi pu remplacer cette fonction par un #define.

Je n'ai pas trouvé de raisons valables permettant de choisir une option plutôt que l'autre, donc si vous avez des idées, je suis preneur.

Raymond

Citation:

---

Cette fonction ne manipule aucunes des variables membres de la classe.

---

Donc, statique, non ? Imagine qu'un jour tu ajoutes une méthode statique appelant MonFormat2, tu seras bien obligé de la rendre statique.

Salut,

Il y a aussi la possibilité de retirer complètement cette méthode de la classe et d'en faire une fonction locale, ce qui permet de la définir dans le .cpp au lieu du .h :

Code:

---

1 2 3 4

namespace { const char *MonFormat2() { return "==> %d"; } }

---

C'est personnellement ce que je fais de plus en plus, ça permet de cacher plein de choses aux utilisateurs de la classe et d'alléger les fichiers de déclarations.

MAT.

Citation:

---

Envoyé par Mat007

Code:

---

1 2 3 4

namespace { const char *MonFormat2() { return "==> %d"; } }

---

---

Juste pour avertire. Si t'as fonction est codé comme cela. tu fait des erreurs memoire :
1- tu entre dans la fonction
2- une tableau de char est créé avec "==> %d" dedans
3- tu retourne la position memoire du premier caractère
4- le tableau est détruit => ce que tu récupérer n'est plus valide

Faut faire un truc comme ca

Code:

---

1 2 3 4 5 6 7 8

namespace { const char *MonFormat2() { static const char * s = "==> %d"; return s; } } }

---

Je crois que les chaînes en dur sont statiques, mais d'un autre côté ça fait des années que je n'ai pas fait de C donc je me trompe peut-être...

MAT.

Salut,

De manière générale, s'il y a moyen d'éviter une instruction #define en la remplaçant par autre chose, il faut le faire (pour ainsi dire) systématiquement.

La raison est relativement simple: un #define est une instruction pré processeur, c'est à dire qu'elle est gérée avant même la compilation, et n'offre aucune sécurité.

Si tu crées (idéalement) un code du genre de

Code:

---

1 2 3

#ifndef mondef #define mondef "==>%d" #endif

---

mondef sera remplacé par blabla dans tous les endroits du code où il apparaît, sans vérifier qu'il soit "logique" ni même sensé de le faire.

Ainsi un

Code:

---

int p=mondef

---

ne sera pas vérifié avant la compilation, qui sortira sur une erreur... et comme la modification aura été faite, il sera difficile de comprendre l'erreur :P

Le fait de définir une fonction statique permet, par contre, d'effectuer un test de "cohérence" quant au type utililsé (entre le type de retour de la fonction ou le type de la variable statique et le type de la variable à initialiser).

Citation:

---

Envoyé par Mat007

Je crois que les chaînes en dur sont statiques, mais d'un autre côté ça fait des années que je n'ai pas fait de C donc je me trompe peut-être...

---

Non non, tu as bon...

Citation:

---

Envoyé par Le standard, 2.13.4

An ordinary string literal has type “array of n const char” and static storage duration (3.7),

---