局部变量 char数组,不是每次都重新分配的吗??

[liang8305]

写程序的时候发现一个不太理解的现象,本来的目的是做一个监听,按时间把在线玩家的名字保存下来
结果类似:

时间段1,序号1,名字1
时间段1,序号2,名字2
时间段1,序号3,名字3
时间段2,序号1,名字1
时间段2,序号2,名字2
时间段2,序号3,名字3
时间段3,序号1,名字1
时间段3,序号2,名字2
时间段3,序号3,名字3


程序为:

//info:结果信息结构体
void RfC51GaugePlugin::UpdateScoring(const ScoringInfoV2 &info)
{

    DWORD now=GetTickCount();
    //保存间隔
    if(now-lastSaveTime>saveDelay && info.mNumVehicles>0)
    {

        const unsigned int cusBufferSize = 50*1024;
        char cpTemp[cusBufferSize];

        
        //历遍在线玩家
        for( long i = 0; i < info.mNumVehicles; ++i )
        {
           //玩家信息结构体
            VehicleScoringInfoV2 &vinfo = info.mVehicle[ i ];

           //字符串连接 : 时间段1,序号1,名字1 + "\n"+ 时间段1,序号2,名字2 ....
            sprintf(cpTemp, "%s%d,%d,%s\n"
                    ,cpTemp,GetTickCount()/1000,i
                    , vinfo.mDriverName);

        }
        saveLog(cpTemp);
    }


}

多次调用这个方法,得到的结果是类似:
时间段1,序号1,名字1
时间段1,序号2,名字2
时间段1,序号3,名字3
时间段1,序号1,名字1 //时间段2里含有时间段1的"字符串"
时间段1,序号2,名字2
时间段1,序号3,名字3
时间段2,序号1,名字1
时间段2,序号2,名字2
时间段2,序号3,名字3
时间段1,序号1,名字1 //时间段3里含有时间段1和2的"字符串"
时间段1,序号2,名字2
时间段1,序号3,名字3
时间段2,序号1,名字1
时间段2,序号2,名字2
时间段2,序号3,名字3
时间段3,序号1,名字1
时间段3,序号2,名字2
时间段3,序号3,名字3


后来修改程序,每次先将cpTemp 清空,则程序正常
程序为 :

//info:结果信息结构体
void RfC51GaugePlugin::UpdateScoring(const ScoringInfoV2 &info)
{

    DWORD now=GetTickCount();
    //保存间隔
    if(now-lastSaveTime>saveDelay && info.mNumVehicles>0)
    {

        const unsigned int cusBufferSize = 50*1024;
        char cpTemp[cusBufferSize];

        sprintf(cpTemp,"");  //清空
        //历遍在线玩家
        for( long i = 0; i < info.mNumVehicles; ++i )
        {
           //玩家信息结构体
            VehicleScoringInfoV2 &vinfo = info.mVehicle[ i ];

           //字符串连接 : 时间段1,序号1,名字1 + "\n"+ 时间段1,序号2,名字2 ....
            sprintf(cpTemp, "%s%d,%d,%s\n"
                    ,cpTemp,GetTickCount()/1000,i
                    , vinfo.mDriverName);

        }
        saveLog(cpTemp);
    }


}

我的问题是,难道每次 char cpTemp[cusBufferSize]; 的时候,cpTemp不都是新的吗?
怎么会还保存着上次的信息,切让本次 连接在上次的后面呢?

我试过
char *cpTemp=new char[cusBufferSize];
char *cpTemp=(char *)malloc(cusBufferSize);

也都不行,请c++朋友解答一下;

[justkk]

局部变量在使用之前，需要初始化一下，否则它的初始值将是随机的..
另外，也不推荐sprintf(cpTemp, "...", cpTemp)的方式，也就是源和目标都是同一个变量

[bdmh]

你只是分配了空间，但是没有处理所分配空间中原有的数据，可能是以前一些程序存取这个地址留下的垃圾数据，所以你最好在分配好，对立面的数据清零

[高性能架构探索]

可能每次在调用的时候，分配的空间恰好会跟上次在栈上的在同一块，所以这样可能会出现问题

建议楼主

const unsigned int cusBufferSize = 50*1024;
char cpTemp[cusBufferSize];

memset(cpTemp, 0, cusbufferSize);
这样的话，应该不会出现上述问题，可以试试

[pur_e]

新变量都要初始化的啊

你如果int a，

a的值初始也是随即的

char a[10];
这个数组不也一样是随机的吗

[jackyjkchen]

memset养成习惯

[liang8305]

咔~~~懒人语言脚本用多了

原来是字符串没有初始化的 结果

char cpTemp[cusBufferSize]={'\0'};

其实就可以了;

[就想叫yoko]

+1[Quote=引用 6 楼 liang8305 的回复:]
咔~~~懒人语言脚本用多了

原来是字符串没有初始化的 结果

char cpTemp[cusBufferSize]={'\0'};

其实就可以了;
[/Quote]