代码替换修复总结

写于2015-01-09 17:27 由qq空间中转过来,格式有点乱

   奋斗了近两个月了,下了决心,定制一个可执行的计划,对一个老游戏的服务端进行.net化, 前段时间处理的一直都挺顺利,使用C#比较速度的重写了GateServer,LoginServer,RoleServer以及GuildServer. 对于GameServer的庞大工程,则使用了c++cli进行逐步的重构,抽象.     

   首先对一些常用的接口函数使用.net重新封装,核心逻辑部分的c++直接替换调用.net接口.    

   后来对各个模块的网络通信部分使用C#编译的XKSocket库进行处理,摒弃原来c++的还得依赖于windows消息 机制的select socket模型.    

   与GameServer进行网络通信的GateServer,LoginServer,RoleServer,GuildServer都处理的相当顺利,而灾难来 了,那就是GameServer和Player Client的通信.    

   之所以说灾难,那是工程量相当的大,7-8w的代码中,绝大多数函数都涉及或者依赖于Player的管理,而基于一定要 跨过一步的决定,我的代码也没有进行备份,自己一个人开发,也没有使用svn,git等版本管理工具.     

   当把原来的socket通信拿掉接上XKSocket后,与Player管理相关的代码导致了不知道到多少错误.     

   原来代码中使用的数组对玩家进行管理,数组里的内容即对应一个Player对象,而我这里使用了hashtable维护玩家列表, Key是XKSocket底层生成的ConnectionID,Value即Player对象.    

   代码中出现 PlayerArray[iPlyIndex]的频率非常高,同样遍历PlayerArray时 PlayerArray[i]的频率也非常高,由于为了方便 的让使用.net进行控制,对Player进行了一个简单的包装,使用了一个.net的PlyItem包装对应的Player[iPlyIndex].     

   犯了一个致命的错误,即对所有代码中的 PlayerArray[iPlyIndex]和PlayerArray[i] 都替换成了 PlyItem->CppPly  这里要说的重点也就是,在对工程量比较大的代码进行替换过程中,不能因为意思是同一对象或类型,对不同的文本 替换为自己重新定义的代码!!!!!!    

 

    如果当时自己替换后,发现这个问题,还有挽救的机会,随着代码不断的修复,修改,这个问题才浮出水面,而此时自己只能为此买 单,将错就错下去了.
    比如说如下简单代码(随便写的)

void SyncPlyName(int iPlyIndex)

{        

    int i=0;      

    for(i=0;i<MaxPlyCount;i++)        

   {            

     PlyArray[i]->send(PlyArray[iPlyIndex]->Name);        

   }

} 

    替换后的代码,即为 

void SyncPlyName(int iPlyIndex)

{

         int i=0;

       PlyItem^ plyItem=(PlyItem^)GameSrvMng::Instance->htPlyItem[i];

       if(plyItem==nullptr || plyItem->CppPly==NULL) return ;

       for(i=0;i<MaxPlyCount;i++)

       {

             plyItem->CppPly->send( plyItem->CppPly->Name);

       }

}

     显然上在的代码是可以正常编译的, 在这几句简单的代码中自然可以看出问题,但是代码量非常大的情况下,这种大量的错误就会被 隐藏起来..那么各种奇怪的问题,也就出来了.

     虽然发现了这种问题,解决起来不难,但是非常麻烦,因为在改回去的时候,就得注意同样的plyItem->CppPly到底应该改成哪一个对象变量了.       过程中还遇到了其它一些问题,修复过程中的错误方法,导出后来解决难度比较大,但是好在量少,已基本解决完成,而上面的错误,还没有 解决完,而每解决一处都得异常的小心..