Discussion :

[kass28]

Bonjour,

J'ai creer in fichier (.h) pour mettre tous mes constantes.

Tout va bien , jusqu'a l'ajout du ligne ci dessous :

Code :

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

static const  std::string pattern("</message>") ;

alors la le compilateur m'insule avec :
error: variable `std::string MyNameSpace::pattern' has initializer but incomplete type

Quelqu'un a une idée de koi s'agit il ?

Merci en avance

[Mat007]

Salut,

Il manque sans doute :

Code :

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

#include <string>

Ou bien ?

MAT.

[koala01]

Salut,

De fait, il est plus que fort vraissemblable que tu aies oublié d'inclure le fichier <string>, car c'est le message typiquement obtenu avec ce genre d'oubli...

Par contre, faut il rappeler que:

1. Les variables globales, de manière générale, c'est mal... fussent-elles statiques
2. L'initialisation d'une variable statique ne devrait être présente que dans un seul fichier d'implémentation, et donc que tu devrais diviser ton code en
   fichier.h

   Code :

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

   static const std::string pattern;

   fichier.cpp
   

   Code :

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

   1
   2
   #include "fichier.h"
   const std::string pattern("</message>") ;

[zais_ethael]

Ce serait pas plutot:
.hpp:

Code :

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

1
2
3
maclasse{
static const std::string pattern;
};

.cpp:

Code :

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

const std::string maclasse::pattern("message");

Je suis presque sur que déclarer static une variable globale n'est pas permis (c'est de toutes façons illogique).

[koala01]

Ce serait pas plutot:
.hpp:

Code :

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

1
2
3
maclasse{
static const std::string pattern;
};

.cpp:

Code :

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

const std::string maclasse::pattern("message");

Je suis presque sur que déclarer static une variable globale n'est pas permis (c'est de toutes façons illogique).

Je ne vois absolument pas pourquoi il ne serait pas permis de déclarer une variable statique (même constante) en dehors d'une classe...

Je suis d'accord avec toi que c'est la pire des manière qui puisse être envisagée, et que, dans l'idéal, ce devrait au moins être ailleurs que dans l'espace de nommage global, mais il n'y a strictement rien qui l'interdise

Maintenant, que tu la déclare dans l'espace de nommage global, dans un espace de nommage particulier, ou au sein d'une classe, cela ne changera strictement rien au problème posé : cela n'influera que sur la portée dans laquelle cette variable sera accessible

[screetch]

hmmm, variable, dites vous, mais il y a ecrit "const" dedans.
c'est loin d'etre une variabe globale ici, il s'agit bien d'une constante.

Le seul defaut c'est qu'elle va etre dupliquée dans toute les unités qui incluent, pas de quoi fouetter un chat.

[screetch]

• L'initialisation d'une variable statique ne devrait être présente que dans un seul fichier d'implémentation, et donc que tu devrais diviser ton code en
  fichier.h

  Code :

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

  static const std::string pattern;

  fichier.cpp
  

  Code :

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

  1
  2
  #include "fichier.h"
  const std::string pattern("</message>") ;

ca c'est faux. le mot clé static devrait alors etre remplacé par extern.

[JolyLoic]

• Les variables globales, de manière générale, c'est mal... fussent-elles statiques

A mon sens, les variables globales posent deux problèmes :

• Elles polluent l'espace de nom avec une variable dont on ne sait pas trop que faire, sauf le bout de code considéré. Si la variable est dans un espace de nom non global, ce défaut est grandement réduit.
• Elles peuvent être accédée en lecture et écriture de n'importe où, la responsabilité de cette variable est totalement diffuse, et elles rendent par là même le code difficile à suivre.

Dans le cas de constantes, le second problème n'existe pas, aussi je n'ai pas de réticences à utiliser des constantes globales.