记C++坑：1.被自己坑了一次的全局变量使用

最近遇到一个问题，打算写一个小的测试程序来测试一下自己设计的算法方案。

当时也没考虑就用了全局变量。当时的代码是这样写的：

int intarray[] = {1,10,100,1000};

int CreateArray(int* arrayname)
{
    arrayname = new int[4];
    memccpy(arrayname,intarray,sizeof(int)*4);
    return 1;
}

int DestroyArray(int* const arrayname)
{
    delete[] arrayname;
    return 1;
}

#define CREATE_ARRAY(name)     int* name; int n_##name##_c = CreateArray(name);
#define DESTROY_ARRAY(name)    int n_##name##_d = DestroyArray(name);

CREATE_ARRAY(int_4);

int main()
{
    int int_idx_0 = int_4[0];
    
    return 1;
}

DESTROY_ARRAY(int_4);

乍一看，没什么问题，但是一执行到main函数第一行就崩溃，发现int_4中已经全部变成了无效值。

当时也没看出来到底是什么原因，反正到了饭点，先去吃饭吧。

回来继续看，擦，发现原因了，CREATE_ARRAY和DESTROY_ARRAY都是通过定义一个全局变量来接收返回值的形式得以调用的，这也是全局情况下想要执行一个函数内容的基本方法。但是问题就出在这里。因为全局变量初始化会在main函数之前，所以其实代码的执行逻辑是先初始化的n_int_4_c和n_int_4_d，最后才进入main函数中，也就是说int_4数组先new并初始化了，接着就delete了，最后在main中使用的时候自然也就无效了。

本来是个很简单的问题，却因为自己码代码的方式蒙蔽了，尽然一时没有看出来。。。

所以修改代码如下：

int intarray[] = {1,10,100,1000};

int CreateArray(int* arrayname)
{
    arrayname = new int[4];
    memccpy(arrayname,intarray,sizeof(int)*4);
    return 1;
}

int DestroyArray(int* const arrayname)
{
    delete[] arrayname;
    return 1;
}

#define CREATE_ARRAY(name)     int* name; int n_##name##_c = CreateArray(name);
#define DESTROY_ARRAY(name)    int n_##name##_d = DestroyArray(name);

void callonexit();

CREATE_ARRAY(int_4);

int main()
{
    atexit(callonexit);

    int int_idx_0 = int_4[0];
    
    return 1;
}
void callonexit()
{
DESTROY_ARRAY(int_4);
}

或者将全局变量改成main函数中的局部变量。