Discussion :

[ringo_ringo]

Bonsoir
J'ai un petit soucis de débutant avec g++.
Etant donné les quatre fichiers suivants:

1 - Fichier: a.hpp

Code :

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

1
2
3
4
5
6
7
8
9
 
#ifndef a_hpp
#define a_hpp
 
namespace foo
{
    void a();}
 
#endif

2 - Fichier: a.cpp

Code :

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

1
2
3
4
5
6
7
8
9
10
11
#include "a.hpp"
#include <iostream>
 
namespace foo
{
    static const m = 100;
    void a()
    {
        std::cout << m << std::endl;
    }
}

3 - Fichier: b.hpp

Code :

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

1
2
3
4
5
6
7
8
9
10
 
#ifndef b_hpp
#define b_hpp
 
namespace foo
{
    void b();
}
 
#endif

4 - Fichier: b.cpp

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#include "a.hpp"
#include "b.hpp"
#include <iostream>
 
namespace foo
{
    void b()
    {
        std::cout << m << std::endl;
    }
}

La commande g++ -c a.cpp génère un a.o (parfait).
Par contre quand je lance la commande g++ -c b.cpp le compilateur ne reconnait pas la variable m.
De ce fait ma question est double:

1. pourquoi m n'est pas reconnue ?
2. quelle commande g++ lancer pour pouvoir générer un b.o étant donné les fichiers précédent (sans changer le code) ?

Merci d'avance

[cob59]

La compilation échoue parce que le code est vérolé :

Il faut virer le static de static const m = 100;, sinon m ne sera accessible que depuis a.cpp. Au passage, il vaudrait mieux préciser un type pour m, même si des compilos peu scrupuleux en permettent visiblement l'omission et collent implicitement un type int :

Code :

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

const int m = 100;

Il faut déclarer m en extern dans a.hpp, pour que tous les fichiers incluant a.hpp sachent que m existe, même s'il est défini ailleurs :

Code :

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

1
2
3
4
5
namespace foo
{
extern const int m;
void a();
}

[ternel]

La raison pour laquelle m n'est pas connue dans b.cpp, c'est qu'elle n'est déclarée dans aucun en-tête importé, ni dans le corps de b.cpp.

Les corrections proposées par cob59 sont effectivement celles qui s'imposent.

Tu veux utiliser une variable globale m. Pour cela, il faut:
extern const int m; dans a.hpp pour avoir une variable globale déclarée.
const int m = 100; dans a.cpp pour définir m.
Je rappelle au passage qu'une entité ne peut avoir qu'une et une seule définition. Le mot clé static (hors d'une classe) permet d'annoncer au compileur "cette entité est définie pour l'unité de compilation courante uniquement, elle est différente de celles portant le même nom dans d'autres unités."

Par ailleurs, on ne parle pas de vérolé pour un code source erroné, ou parle d'erreur ou de bug...

[ringo_ringo]

Merci pour vos réponses rapides !
Tout fonctionne correctement et je capitalise en connaissances