C++ 这些函数有什么问题？

Question

这是一个 class 我正在组织我使用 malloc/calloc 分配的每个内存块，这样我以后可以轻松处理它们。下面是这个class的定义：

#include "memmanager.h"

std::vector<void*> MemoryManager::GarbageCollector(0);

void MemoryManager::AddToGC(void* myThing)
{
    if(__DEBUG__MODE__)
        MainDebugger.Log("Adding 1 element to GC: %p", "Info", myThing);
    MemoryManager::GarbageCollector.push_back(myThing);
    return;
}

void MemoryManager::MultiAddToGC(int args, void* myThing, ...)
{
    if(args < 1)
        return;

#ifdef _DEBUG
        MainDebugger.Log("Adding %i element to GC", "Info", args);
#endif

    va_list chrstr;
    va_start(chrstr, args);

    for(int x = 0; x < args; x++)
        MemoryManager::GarbageCollector.push_back(va_arg(chrstr, void*));

    va_end(chrstr);
    return;
}

void MemoryManager::Flush()
{
    int lasterror = 0;

#ifdef _DEBUG
        MainDebugger.Log("Call to MemoryManager::Flush() with %i items in the GC", "Info", MemoryManager::GarbageCollector.size());
#endif

    for(unsigned int x = 0; x < MemoryManager::GarbageCollector.size(); x++)
    {
        errno = lasterror = 0;
        free(GarbageCollector[x]);
        lasterror = errno;
        if(lasterror > 0)
            MainDebugger.Log("MemoryManager::Flush() Error: %s (%i : %p)", "Error", strerror(lasterror), x, GarbageCollector[x]);
    }
    GarbageCollector.clear();
    return;
}

问题似乎出在函数 "MultiAddToGC" 上。当我在主文件中执行此操作时：

MemoryManager::MultiAddToGC(3,tst,mfile,testfile);
MemoryManager::Flush();

如果我处于调试模式，它工作正常（我在 VC++ 2010 中这样做）。但是，如果我更改为 Release 模式，它会在调用 free() 函数时在 MemoryManager::Flush() 内给我一个错误（关于堆栈被破坏的一些事情）。我可以继续，如果我继续，我会在我的日志中得到以下内容：

15:12:26 f 00 (0 fps) | Error > MemoryManager::Flush() Error: Invalid argument (2 : 00D44784)

但是，如果我这样做：

MemoryManager::AddToGC(tst);
MemoryManager::AddToGC(mfile);
MemoryManager::AddToGC(testfile);
MemoryManager::Flush();

它在调试和发布中都有效。没有错误。所以我假设错误在 MultiAddToGC() 中，但我找不到它。下面的代码是表头，"memmanager.h":

#ifndef __MEMMANAGER_H__
#define __MEMMANAGER_H__

#include <vector>
#include <malloc.h>
#include <stdarg.h>
#include "..\core.h"
#include "..\DEBUGGER\debugger.h"

extern bool __DEBUG__MODE__;
extern GameDebugger MainDebugger;

class MemoryManager {

public:
    static void MemoryManager::AddToGC(void*);
    static void MemoryManager::MultiAddToGC(int, void*,...);
    static void MemoryManager::Flush();
private:
    static std::vector<void*> GarbageCollector;

protected:

};

#endif

任何 help/tips/advise 欢迎。

Answer 1

在va_start(chrstr, args);中，第二个参数应该是最后一个命名参数，所以myThing（但实际上你可能想删除myThing）。

但我会写

template <typename...Ts>
void MemoryManager::MultiAddToGC(Ts*... myThings)
{
    if(sizeof...(myThings) < 1)
        return;

#ifdef _DEBUG
    MainDebugger.Log("Adding %i elements to GC", "Info", sizeof...(myThings));
#endif
    int dummy[] = {(MemoryManager::GarbageCollector.push_back(myThings), 0)...};
    (void) dummy; // avoid warning for unused variable
}

或者干脆

void MemoryManager::MultiAddToGC(const std::initializer_list<void*>& myThings)
{
    if(myThings.size() < 1)
        return;

#ifdef _DEBUG
    MainDebugger.Log("Adding %i elements to GC", "Info", myThings.size());
#endif
    MemoryManager::GarbageCollector.insert(MemoryManager::GarbageCollector.begin(),
                                           myThings.begin(), myThings.end());
}

C++ 这些函数有什么问题？

C++ What's wrong with these functions?

c++

stack

corruption