• <ins id="pjuwb"></ins>
    <blockquote id="pjuwb"><pre id="pjuwb"></pre></blockquote>
    <noscript id="pjuwb"></noscript>
          <sup id="pjuwb"><pre id="pjuwb"></pre></sup>
            <dd id="pjuwb"></dd>
            <abbr id="pjuwb"></abbr>

            socketref,再見!高德

            https://github.com/adoggie

              C++博客 :: 首頁 :: 聯系 :: 聚合  :: 管理
              246 Posts :: 4 Stories :: 312 Comments :: 0 Trackbacks

            常用鏈接

            留言簿(54)

            我參與的團隊

            搜索

            •  

            最新評論

            閱讀排行榜

            評論排行榜


            一友人做游戲服務器,涉及通信模塊和業務模塊,在軟件行業也算個老兵了,也搞c++n多年了,對于軟件開發也有自己的一些想法和心得,將其代碼與我分享,讓我觀贊一下。
            c++是我從業一來一直信奉的開發工具,雖然近些年一直用python來搞定應用需求,但關鍵時刻還必須是c++搞定, 所以c++的程序代碼能hold住的。 

            我知道寫通信部分代碼的關鍵點在哪里,比如: 性能、安全、完整性、模塊化設計和處理,所以有針對性的去檢查友人的代碼,當看到這個函數的時候,致命的錯誤來了: 
            分別在于: 第 13,20行 ,其分別會導致服務器崩潰和不響應

             1 
             2 tagPack* CClient::getRecvPack()
             3 {
             4     uv_mutex_lock(&_lockrecv);
             5     tagPack *t = NULL;
             6     if(_recvPackLenght < 4)
             7     {
             8         _recvOver = true;
             9         uv_mutex_unlock(&_lockrecv);
            10         return t;
            11     }
            12     tagPack* tmp = _lsRecvBuf.front();
            13     unsigned int size = *(unsigned int*)&tmp->pack[0];
            14     if(_recvPackLenght < size)
            15     {
            16         _recvOver = true;
            17         uv_mutex_unlock(&_lockrecv);
            18         return t;
            19     }
            20     BYTE* mem = new BYTE[size];
            21 
            22     unsigned int index = 0;
            23     unsigned int needSize = size;
            24     while(true)
            25     {
            26         if(tmp->size < needSize)
            27         {
            28             memcpy(&mem[index], tmp->pack, tmp->size);
            29             needSize -= tmp->size;
            30             index += tmp->size;
            31             delete tmp;
            32             _lsRecvBuf.pop_front();
            33         }
            34         else if(tmp->size == needSize)
            35         {
            36             memcpy(&mem[index], tmp->pack, tmp->size);
            37             needSize -= tmp->size;
            38             index += tmp->size;
            39             delete tmp;
            40             _lsRecvBuf.pop_front();
            41         }
            42         else
            43         {
            44             memcpy(&mem[index], tmp->pack, needSize);
            45             memcpy(tmp->pack, &tmp->pack[needSize], tmp->size - needSize);
            46             tmp->size -= needSize;
            47 
            48             needSize = 0;
            49         }
            50 
            51         if( needSize == 0 )
            52         {
            53             t = new tagPack(size);
            54             t->addBytes(mem, size);
            55             t->_sock = _sock;
            56             delete []mem;
            57             _recvPackLenght -= t->size;
            58             break;
            59         }
            60         tmp = _lsRecvBuf.front();
            61     }
            62     uv_mutex_unlock(&_lockrecv);
            63 
            64     return t;
            65 }

            13行:  開發者直接將緩存內存地址作為接收和發送地址,沒考慮 字節序問題  (htonl,ntohl) 
            20行:  邏輯不嚴密,如果用戶發送4個字節數據,且均為0 ,結果就是服務器崩潰。 

            所以codereview還是很重要的,特別是將c++應用在性能,效率關鍵部位的代碼。 







            posted on 2016-03-24 15:29 放屁阿狗 閱讀(16293) 評論(4)  編輯 收藏 引用 所屬分類: C++/Boost/STL/Template

            Feedback

            # re: 有問題的c++通信代碼的識別 2016-03-26 22:28 火紅地太陽
            這代碼能力,說實話,感覺太一般了,沒什么功底。  回復  更多評論
              

            # re: 有問題的c++通信代碼的識別 2016-03-27 00:23 放屁阿狗
            是不夠精簡,思路不夠嚴密  回復  更多評論
              

            # re: 有問題的c++通信代碼的識別[未登錄] 2016-04-12 11:36 hdqqq
            12 行沒檢查返回值
            53 行 new 一定成功嗎
              回復  更多評論
              

            # re: 有問題的c++通信代碼的識別 2016-05-06 09:36 tqt
            還new,還mutex_unlock,服.  回復  更多評論
              

            中文字幕乱码久久午夜| 国产精品久久久天天影视香蕉 | 久久精品人人做人人爽电影| 欧美黑人又粗又大久久久| 欧美va久久久噜噜噜久久| 精品国产福利久久久| 亚洲欧美一级久久精品| 97久久国产亚洲精品超碰热| 久久国产热这里只有精品| 国内精品久久久久影院薰衣草 | 日韩电影久久久被窝网| 精品无码久久久久久午夜| 欧美日韩中文字幕久久久不卡| 色欲久久久天天天综合网精品| 国产精品久久久久久久久久免费| 无码人妻少妇久久中文字幕蜜桃| 久久香蕉综合色一综合色88| 天天爽天天狠久久久综合麻豆| 开心久久婷婷综合中文字幕| 久久综合亚洲欧美成人| 亚洲国产精品无码久久久久久曰 | 亚洲婷婷国产精品电影人久久| 久久久老熟女一区二区三区| 久久久久久毛片免费看| 久久精品中文无码资源站| 色青青草原桃花久久综合| 999久久久国产精品| 久久综合亚洲欧美成人| 久久久噜噜噜久久中文字幕色伊伊 | 久久亚洲精品人成综合网| 久久毛片免费看一区二区三区| 久久精品国产免费一区| 久久久久久亚洲Av无码精品专口| 伊人久久大香线蕉亚洲五月天| 日韩久久无码免费毛片软件| 99久久99久久精品国产| 三级三级久久三级久久| 久久国产热精品波多野结衣AV| 国产成人久久精品二区三区| 国产综合久久久久久鬼色| 久久精品欧美日韩精品|