Question philosophique fonction statique

**ram-0000** · 27/09/2007, 15h14

Bonjour,

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

Code :

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

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

**aoyou** · 27/09/2007, 15h51

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.

**Mat007** · 27/09/2007, 16h09

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 :

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

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.

**yan** · 27/09/2007, 16h50

Envoyé par Mat007

Code :

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

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 :

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

1
2
3
4
5
6
7
8
namespace
{
  const char *MonFormat2() 
{ 
static const char  * s =   "==> %d";
return s; }
}
}

**Mat007** · 27/09/2007, 16h58

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.

**koala01** · 27/09/2007, 17h09

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 :

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

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 :

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

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

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