2010-11-03 34 views
0

我从compilator收到此错误的问题:C++/SDL双列入

1>Linking... 
1>main.obj : error LNK2005: "int g_win_flags" ([email protected]@3HA) already defined in init.obj 
1>main.obj : error LNK2005: "struct SDL_Surface * g_screen" ([email protected]@[email protected]@A) already defined in init.obj 
1>MSVCRTD.lib(cinitexe.obj) : warning LNK4098: defaultlib 'msvcrt.lib' conflicts with use of other libs; use /NODEFAULTLIB:library 
1>.\Debug\Heroes are back!.exe : fatal error LNK1169: one or more multiply defined symbols found 

它看起来像g_win_flags和g_screen两次都包括在内,但我不明白为什么。 这里是源:

的main.cpp

#include <iostream> 
#include "dec.h" 
#include "init.h" 

int main(int argc, char *argv[]){ 

    init(); 
    return 0; 
} 

dec.h

#ifndef DEC_H 
#define DEC_H 

#include <SDL.h> 
#include <iostream> 

#pragma comment(lib, "SDL.lib") 
#pragma comment(lib, "SDLmain.lib") 

using namespace std; 

int g_win_flags = SDL_HWSURFACE|SDL_DOUBLEBUF; 

SDL_Surface *g_screen = NULL; 

#endif 

init.h里

#ifndef INIT_H 
#define INIT_H 

bool init(); 

#endif 

init.cpp

#include "dec.h" 

bool init(){ 
    if (SDL_Init(SDL_INIT_VIDEO | SDL_INIT_AUDIO | SDL_INIT_TIMER) == -1){ 
     cerr << "Unable to initialize SDL" << endl; 
     return false; 
    } 
    g_screen = SDL_SetVideoMode(640, 480, 0, g_win_flags); 

    return true; 
} 

有人可以帮忙吗?在此先感谢,并有一个愉快的一天:)

+1

你不应该在除extern声明以外的头文件中定义变量。 – codymanix 2010-11-03 18:26:27

+0

仅供参考,不是编译器错误,它是链接器错误。 – 2010-11-03 18:48:38

回答

3

您可以定义并初始化标题中的变量。

你应该刚刚宣布他们在头(dec.h)无需任何初始化:

extern int g_win_flags; 
extern SDL_Surface *g_screen; 

然后在一个文件中定义一次 - 大概dec.cpp - 与初始化。

就这样,你在每个包含'dec.h'的源文件中定义它们,然后运行ODR - One Definition Rule。

2

在dec.h你想

extern int g_win_flags; 

extern SDL_Surface *g_screen; 

,然后定义和initalise他们只是dec.cpp

更新:

#include "dec.h" 
int g_win_flags = SDL_HWSURFACE|SDL_DOUBLEBUF; 

SDL_Surface *g_screen = NULL; 

一般的经验法则是“头文件中的任何内容都不应占用编译器输出中的任何空间”。 (显然有一些例外)

实际上,这意味着extern变量声明很好,函数声明也是如此,但不是定义。

0

您已将文件包含到定义实例化变量的两个不同源文件(init.cpp和main.cpp)中。

您需要一种方法来确保它们在除一个源文件之外的其他所有文件中都是“图案”的。

+0

看到乔纳森的答案在下面 - 这就是我的意思(但没有) – KevinDTimm 2010-11-03 18:28:37

0

嗯,我试图做什么你告诉我的家伙,但compilator抱怨:

1>.\heroes are back!\dec.cpp(2) : error C4430: missing type specifier - int assumed. Note: C++ does not support default-int 
1>.\heroes are back!\dec.cpp(4) : error C4430: missing type specifier - int assumed. Note: C++ does not support default-int 
1>.\heroes are back!\dec.cpp(4) : error C2040: 'g_screen' : 'int' differs in levels of indirection from 'SDL_Surface *' 

这里是dec.h

#ifndef DEC_H 
#define DEC_H 

#include <SDL.h> 
#include <iostream> 

#pragma comment(lib, "SDL.lib") 
#pragma comment(lib, "SDLmain.lib") 

using namespace std; 

extern int g_win_flags; 

extern SDL_Surface *g_screen; 

#endif 

(分解)。cpp

#include "dec.h" 
g_win_flags = SDL_HWSURFACE|SDL_DOUBLEBUF; 

g_screen = NULL; 

这段代码有什么问题?对不起,问愚蠢的问题,但我只是在学习C++ :)

+0

你没有在dec.cpp文件中指定类型,这是它告诉你的。你需要在头文件中有'extern(type)(name);'和代码中需要'(type)(name)=(value);'。 – ssube 2010-11-03 18:49:46