[C++/MS VS 2005] Namespace problemen

[Verwijderd]

Laat mij beginnen met te stellen dat ik normaal gesproken voornamelijk in C programmeer, en derhalve nooit eerder zelf namespaces heb gemaakt.

De situatie is als volgt.
Ik heb een set functies die met hardware communiceren. Deze heb ik ondergebracht in een .cpp file met een bijbehorende .h. Omdat de functienamen nogal generiek zijn, dacht ik ze onder te brengen in een namespace, zodat ik geen conflicten krijg.

MSSIRadio.h:

C++:

#ifndef MSSIRADIO_H
#define MSSIRADIO H

namespace LLMSSIRadio {
    int some_var;

    int fpga_read( /* ... */);
    int fpga_write( /* ... */);
    // etc.
}

#endif

MSSIRadio.cpp:

C++:

#include "radio.h"

int LLMSSIRadio::fpga_read( /* ... */) {
    // ...
}
// etc.

Ik gebruik deze functies elders, door MSSIRadio.h te includen. Ik krijg dan echter meldingen dat some_var al gedefinieerd is.
Ik heb er omheen gewerkt door some_var in een functie onder te brengen, maar ik heb het vermoeden dat dat niet de aangewezen methode is.

Is mijn ontwerp fundamenteel fout?

Voor de volledigheid moet ik vermelden dat het complete ontwerp er meer zo uitziet:

code:

Main.cpp : includes -> Wrapper.h
Wrapper.cpp : includes -> Wrapper.h, MSSIRadio.h
MSSIRadio.cpp : includes -> MSSIRadio.h

Overigens, mijn ClassView in MS VS 2005 ziet er zo uit (een van de redenen waarom ik vermoed dat er iets fout zit):

[.oisyn]

Vrij logisch, als je die namespace niet zou gebruiken zou je toch gewoon dezelfde error krijgen? Zelfs in C . Je definieert een variabele in een header, ipv 'm te declareren

[ Voor 4% gewijzigd door .oisyn op 01-09-2007 00:52 ]

[eek]

C:

#ifndef MSSIRADIO_H
#define MSSIRADIO H

moet dat niet zijn:

C:

#ifndef MSSIRADIO_H
#define MSSIRADIO_H

Of misschien nog beter LLMSSIRADIO_H

[ Voor 10% gewijzigd door eek op 01-09-2007 12:38 ]

[Verwijderd]

Een oplossing is om de variabelen het keyword 'extern' mee te geven en te definieren in je .cpp file. Googlen met 'C++ extern' moet bruikbare resultaten opleveren.

[farlane]

Variabelen definieren in een header file is (bijna?) nooit wat je wilt. Ieder die de header include krijgt zijn eigen variabele mee, en omdat ie standaard external linkage heeft heb je dus multiple definitions van dat ding.

Dit is trouwens voor C hetzelfde als C++

[ Voor 8% gewijzigd door farlane op 01-09-2007 13:34 ]

[Verwijderd]

Bedankt voor de tips, ik zie in wat ik fout heb gedaan. Blijkbaar heb ik in mijn enthousiasme iets nieuws te gebruiken andere dingen over het hoofd gezien. Of misschien doe ik dit wel niet vaak genoeg om er handigheid in te krijgen en het automatisch goed te doen .

@ eek: dat is een typfout. Ik had e.e.a. geabstraheerd, blijkbaar de _ vergeten.

[RobLemmens]

eek schreef op zaterdag 01 september 2007 @ 12:29:


Of misschien nog beter LLMSSIRADIO_H

Of

#pragma once

en de rest van de defines dumpen.

[ Voor 6% gewijzigd door RobLemmens op 01-09-2007 16:35 ]

[.oisyn]

Zodat je code lekker niet meer compiler-onafhankelijk is, joepie . Als je al #pragma once gebruikt, laat de header guards er dan iig nog omheen staan, zodat je het altijd nog kunt compilen met een compiler zonder #pragma once support.