使用 makefile 进行 gcc 编译期间的链接错误

Question

（在继续之前，
没有释放和编写代码中从未使用过的变量，
用于测试工具）

我写了这样的代码和这样的 Makefile：

unread_two.h

#ifndef __UNREAD_TWO_H
#define __UNREAD_TWO_H

const int SIZEOF_INT = sizeof(int);
int addTwo();

unread_twomain.c

#include <stdio.h> 
#include <stdlib.h>
#include "unread_two.h"

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

    x = (int *)malloc(SIZEOF_INT);
    x = addTwo();
    free(x);

    return 0;
}

unread_two.c

#include <stdio.h>
#include <stdlib.h>
#include <unread_two.h>

int addTwo()
{
    int *y, *z, sum;
    y = (int *)malloc(SIZEOF_INT);
    z = (int *)malloc(SIZEOF_INT);

    *y = 3;
    sum = *y + *y;
    return sum;
}

生成文件

CC=gcc
CCFLAGS=-g

%.o: %.c
    $(CC) -c $< $(CCFLAGS)

all: unread_two
clobber: clean
    rm -f *~ \#`\# core

clean:
    rm -f unread_two *.o

unread_two: unread_twomain.o unread_two.o

unread_twomain.o: unread_two.h

unread_two.o: unread_two.h

当我把 make all ，这个消息出现：

unread_twomain.o:(.rodata+0x0): multiple definition of `SIZEOF_INT'
unread_two.o:(.rodata+0x0): first defined here
collect2: error: ld returned 1 exit status

我应该解决什么问题？

Answer 1

您不应该定义 SIZEOF_INT 在标头中，否则当您在多个编译单元中包含此标头时，您将获得多个定义，正如您所见。而是在标题中声明它并在源文件中定义它：

// unread_two.h

extern const int SIZEOF_INT;         // *declare* SIZEOF_INT

// unread_two.c

const int SIZEOF_INT = sizeof(int);  // *define* SIZEOF_INT

或者，在这种特殊情况下，您可能有理由使用“old skool”方式使用宏：

// unread_two.h

#define SIZEOF_INT sizeof(int)

Answer 2

您实际上有 两个 错误，一个是您报告的错误，另一个是更狡猾的错误。

您遇到错误的问题是常量SIZEOF_INT 是在您包含头文件的所有源文件中定义的。包含防护仅防止在同一源文件中多次包含（或从技术上讲是翻译单元） ，但不反对您在多个来源中包含相同的文件。这意味着编译器将在unread_twomain.o 和unread_two.o 中创建SIZEOF_INT 的定义，然后链接器会报错。

对此的解决方案是仅在头文件中声明常量，然后在单个源文件中定义它。

---

另一个问题是，在main 中，您将x 创建为指针，并为它分配内存（顺便说一下，您不应该对malloc 的返回进行类型转换），然后从@ 获取结果987654328@ 指向此指针。但是addTwo 不返回指针，它返回一个直接值，所以你让指针x 指向地址6，我猜这不是你打算做的。当您尝试释放x 指向的内存时，这将导致未定义的行为，很可能会导致崩溃。

在您的程序中，您根本不必使用指针。只需使用普通的非指针变量：

int addTwo()
{
    int y = 3;
    int sum = y + y;

    return sum;
}

和

int main(int argc, char *argv[])
{
    int x = addTwo();

    return 0;
}