Linker can't find function definition in a namespace

Question

I get this /tmp/ccnL7Yz1.o: In function 'main': main.cpp:(.text+0x70): undefined reference to 'dng::genDungeon()' main.cpp:(.text+0xf0): undefined reference to 'dng::clrDungeon(char**)' collect2: error: ld returned 1 exit status error when I'm trying to compile my program. It worked great before I added namespace functions. I'm compiling it like this: g++ -std=c++11 main.cpp Dungeon.cpp

Dungeon.h

namespace dng {
    char** genDungeon();
    void clrDungeon(char**);

    class Dungeon {
    //Methods and variables
    }
}

Dungeon.cpp

#include "Dungeon.h"

using namespace dng;
char** genDungeon() 
{
    //Stuff
}
void clrDungeon(char** dungeon) 
{
    //Another Stuff
}
/*Implementation of class methods
void Dungeon::genStart(){} -> like this */

main.cpp

#include "Dungeon.h"

int main () 
{
    //Stuff
    auto dungeon = dng::genDungeon();
    //Stuff
    dng::clrDungeon(dungeon);
    return 0;
}

I also tried to make .o files by myself g++ -std=c++11 -c main.cpp g++ -std=c++11 -c Dungeon.cpp and then link them, but got the same error. What can be the problem?

You could have searched a bit first: [What is an undefined reference/unresolved external symbol error and how do I fix it?](http://stackoverflow.com/questions/12573816/what-is-an-undefined-reference-unresolved-external-symbol-error-and-how-do-i-fix) — Emil Laine, Aug 07 '15 at 18:44
I'm voting to reopen this. The original title smells of *duplicate*, but the question is really about a misunderstanding of `using namespace`. — Drew Dormann, Aug 07 '15 at 18:50
@Борис Кот Такие вопросы лучше задавать на ru.stackoverflow.:) — Vlad from Moscow, Aug 07 '15 at 18:52
@zenith I searched a lot but didn't find anything similar. And it seems like your link doesn't provide a solution for my problem. — Борис Кот, Aug 07 '15 at 18:59

score 12 · Accepted Answer · edited Aug 07 '15 at 22:05

12

Enclose the function definitions in the namespace dng where they are declared.

#include "Dungeon.h"

namespace dng
{
char** genDungeon() 
{
    //Stuff
}
void clrDungeon(char** dungeon) 
{
    //Another Stuff
}
//...
}

Or use qualified names.

include "Dungeon.h"

using namespace dng;

//...

char** dng::genDungeon() 
{
    //Stuff
}
void dng::clrDungeon(char** dungeon) 
{
    //Another Stuff
}

Otherwise, the functions are defined in the global namespace, and as a result, you declared four functions: two in the namespace dng and another two in the global namespace.

edited Aug 07 '15 at 22:05

Wandering Fool

2,170
3
18
48

answered Aug 07 '15 at 18:44

Vlad from Moscow

301,070
26
186
335

1

Answering obvious duplicates encourages to creating them. Do not do this. – Mateusz Grzejek Aug 07 '15 at 18:45
1

@Mateusz Grzejek I am sorry. I have not understood what you said. – Vlad from Moscow Aug 07 '15 at 18:48
@Mateusz Grzejek Oh, I have understood now.:) – Vlad from Moscow Aug 07 '15 at 18:49
2

You don't need `using namespace dng;` in the second case. – Emil Laine Aug 07 '15 at 19:02
3

@MateuszGrzejek I don't think it is a duplicate. – Борис Кот Aug 07 '15 at 19:03
@zenith It is unknown what this module contain else. – Vlad from Moscow Aug 07 '15 at 19:04
[This](http://stackoverflow.com/q/8681714/3425536) might be helpful for deciding which convention to use. My vote goes for using qualified names. – Emil Laine Aug 07 '15 at 19:05
@VladfromMoscow The functions themselves are inside `dng::` so everything from `dng::` will be automatically available inside the functions. – Emil Laine Aug 07 '15 at 19:06
@zenith You have not understood. It is unknown what else follows the function definitions in the module.:) – Vlad from Moscow Aug 07 '15 at 19:07
1

@VladfromMoscow Oh you mean that. Instead of "module" you usually call them source files in C++. – Emil Laine Aug 07 '15 at 19:09

Linker can't find function definition in a namespace

Dungeon.h

Dungeon.cpp

main.cpp

1 Answers1