使用 gcc 编译多个依赖 C 的文件

Question

我在编译多个依赖 c 文件时遇到多重声明问题。

我一直在 main.c 中包含源文件并编译它。 它运作良好。 最近我的同事建议我不要那样做。 所以我试图改变我的代码并编译它制作不同的 .o 文件并最终链接它们。 但它没有用。

基本上我想做的与此类似：

要点.h

#ifndef ESSENTIALS
#define ESSENTIALS
//C headers
#include <stdio.h>
#include <conio.h>

//My Assets
#include <new.h>
#include <global.h>
#endif

此 header 包含任何源文件可能需要的所有基本文件。 它还包括声明全局变量的 header 文件。

---

其他header和源文件如：

全局.h

#ifndef GLOBAL
#define GLOBAL

int glob_int=0;

#endif

---

新.h

#ifndef NEW
#define NEW

int thisFunc();

#endif

---

新.c

#include <essentials.h>

int thisFunc(){
    glob_int++;
    printf("This is new Function %d\n",glob_int);
    return 0;
}

---

主要.c

#include <essentials.h>

/*  include other headers*/

int main(){
    printf("Main Function Start %d\n",glob_int);

    thisFunc();
    return 0;
}

然后我继续编译以下内容，

F:\Codes\prog> gcc -c ./src/*.c -I"F:\Codes\prog\include"

<<<< Compiles separate obj files >>>>

F:\Codes\prog> gcc *.o -o ./bin/main.exe
c:/my_assets/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.2.0/../../../../x86_64-w64-
mingw32/bin/ld.exe: 
new.o:new.c:(.bss+0x0): multiple definition of `glob_int'; main.o:main.c:(.bss+0x0): first defined here
collect2.exe: error: ld returned 1 exit status

我必须在两个源文件中都使用 glob_int。 我在链接阶段遗漏了什么？是否有任何合适的方法来编译此代码。

我搜索了各种此类示例，但对我的情况没有任何帮助。 事情是没有 glob_int 变量，程序编译和运行也没有显示 thisFunc() function 的任何多重声明错误。但它不编译变量。

我也试过将 <global.h> 放在 each.c 文件中，而不是放在 essentials header 文件中。 但这也没有用。

请避免给我使用 extern 的提示。 那将是我最后使用的东西。

另外：我想知道为什么在 main.c 中包含带有保护标头的源文件并编译它通常受到许多 C 程序员的谴责。 如果您相反可以对此进行描述，请通过回答来提供帮助。

感谢 Sammy1410

Answer 1

非extern解决方案 - 在glob_int的声明中省略初始值设定项。

初始化程序的存在告诉编译器这是glob_int的定义，并且在整个程序中你只能有其中一个。 关闭初始化程序会创建一个暂定定义，linker 可以对其进行排序。

由于它是在文件 scope 中声明的， glob_int将被隐式初始化为零，因此不需要初始化器。

当然，正确的做法是根本不使用全局变量； 相反，只在一个源文件中定义它并为其提供 getter 和 setter 函数：

Header：

/**
 * glob.h
 */
#ifndef GLOB_H
#define GLOB_H

void glob_reset( void ) // resets glob_int to 0
int glob_val( void )    // returns current value of glob_int
int glob_inc( void )    // increments glob_int and returns its new value

#endif

源文件：

/**
 * glob.c
 */
#include "glob.h"

static int glob_int; // glob_int is only visible to this source file

void glob_reset( void )
{
  glob_int = 0;
}

int glob_val( void )
{
  return glob_int;
}

int glob_inc( void )
{
  return ++glob_int;
}

此外，您的essentials.h是一个坏主意 - 源文件应该只包含它们实际需要的标头，而不是一大堆不必要的或可能冗余的标头。 它会造成维护难题，并可能导致循环依赖。

编辑

我上面的 getter/setter 示例几乎没有睡觉就被扔掉了； 对于 singleton object，这是正确的做法，但不是您所描述的。

来自评论：

这样我就不必在每个 .c 文件中一次又一次地重写所有库标头。

这与您认为的负担相去甚远，我向您保证，对于足够大的项目，它会引起问题。 我去过这部电影好几次。 每次都以同样的方式结束。 最后它只会创造更多的工作，因为你必须解开依赖关系。

仅包含您知道需要为每个 .h 和 .c 文件包含的文件。

这些全局变量包括键盘键状态、鼠标 position、我必须作为全局变量使用的各种缓冲区。 有什么不同的非外部方式吗？

不要使用全局变量。

严重的是，不要使用全局变量——在main中维护 state（或主事件循环所在的任何地方）并通过 function arguments 传达该信息并返回值：

void eventLoop( void )
{
  struct pos { int x, y; } curPos, lastPos; 
  buffer_t myBuffers[NUM_BUFFERS];
  ...

  while ( !done )
  {
    curPos = getReadPosition();
    readFrom( curPos, myBuffers[RD_BUF] );
    ...
  }

让所有东西都全局化会使你的代码与自身紧密耦合，这使得很难孤立地测试片段，也很难升级或维护。

再一次，我参加了这部电影。 我曾编写过使一切都全球化的代码，而且无法修复。 我在一个未能达到其性能要求的系统上工作，并且因为一切都是全局的（并且因为它对所有控制结构使用了goto ）我们无法在不破坏某些东西的情况下进行任何更改。 它非常脆弱。

函数应该只通过参数、返回值和异常进行通信。 他们不应该通过全局变量共享 state。 C 不支持结构化异常处理，但您可以使用signal and raise或setjmp和longjmp伪造它。