0

考虑以下4个文件：

[核心.h]

#pragma once  
static int MY_MARK = -1;

[计数.h]

#pragma once
#include "core.h"
#include <stdarg.h>
#define Count(...) CountImpl(&MY_MARK, __VA_ARGS__, &MY_MARK)
int CountImpl(void*, ...);

[计数.c] 

#include "count.h"

int CountImpl(void* dummy, ...) {
    int     count = 0;
    va_list lst;

    va_start(lst, dummy);

    for(;;) {
        void* ptr = va_arg(lst, void *);
        if(ptr == &MY_MARK) {break;}
        ++count;
    }

    va_end(lst);
    return count;
}

[奇怪的问题.c] 

#include <tchar.h>
#include "count.h"

int _tmain(int argc, wchar_t* argv[], wchar_t* envp[]) {
    int a = 1;
    int b = 2;
    int c = 3;
    int d = Count(&a, &b, &c);
    return 0;
}

我正在使用 Microsoft Visual Studio Community 2013。
运行上面的代码时，我希望“d”变量的值为 3。问题是执行永远不会中断循环，因为它不比较“ptr” ' 和 '&MY_MARK' 相等。（最终它会在尝试读取受保护的内存或其他内容时引发错误。）
实际上，我在 Watch 窗口中看到了两个不同的地址： 

(int*)ptr       0x000000013f8d9004 {WeirdProblemTest.exe!int MY_MARK} {-1}  int *
&MY_MARK        0x000000013f8d9000 {WeirdProblemTest.exe!int MY_MARK} {-1}  int *

我知道我可以使用 'dummy' 变量而不是引用 '&MY_MARK' 来解决这个问题，但这不是重点。
我真的需要了解发生了什么，因为同样的问题发生在我的代码的不同部分，并且在那里没有很好的解决方法。

很抱歉这篇文章很长，但我发现没有办法让它更短。

4

1 回答 1

4

您已经在.h文件中声明并定义了变量，static这意味着标题中的每个文件#includes都将获得自己的变量私有副本。

您需要拆分core.h并core.h看起来core.c像：

核心.h：

#pragma once  
extern int MY_MARK;

核心.c：

#include "core.h"

int MY_MARK = -1;

所有其他文件都应该#include "core.h"，并且在编译和链接时，您应该链接core.c

于 2015-02-18T20:26:52.743 回答