网狐棋牌框架-WHDataQueue队列的问题

研究网狐的内核框架发现经常有以下错误:

SocketEngine Index=0,RountID=18,OnRecvCompleted 发生“数据包效验码错误”异常


因为不是必现的问题,非常不好排查,而网狐的东西也相对较稳定,不知如何下手,然后就仔细看内核相关的代码,看到了内存数据队列;WHDataQueue队列的实现其实还是非常巧妙的,用变量标志就在一个内存块上实现了环形的队列,越是巧妙的东西,就越容易出错,也就越需要打磨,虽然研究网狐框架代码的人很多,但不知道为什么我在网上各种找资料,就是找不到这个问题的解决方案,可能这是个偶发的问题,在服务端跑的时候,也极少出现,就算出现也只是影响某个玩家,所以问题就被隐藏了吧。


		int nIndex = 0;
		TCHAR szInfo[4096] = TEXT("");
		int nLen = 0;
		 
		//写入0字节, 让gDataQueue内存块大小为 40, 实际上是写入4字节,因为里面还有个数据头
		rand_str(szInfo, 0); 
		gDataQueue.InsertData(++nIndex, szInfo, 0);
		tagDataHead head;
		//取出,之后 m_dwDataQueryPos 变成4
		gDataQueue.DistillData(head, szInfo, sizeof(szInfo)); 

		//写入16个字节
		rand_str(szInfo, 7); 
		nLen = CountStringBuffer(szInfo); 
		gDataQueue.InsertData(++nIndex, szInfo, nLen);
		std::cout << nLen <<", " << nIndex << endl;

		//取出,之后 m_dwDataQueryPos 变成24, m_dwDataSize=0
		gDataQueue.DistillData(head, szInfo, sizeof(szInfo));
		std::cout << head.wDataSize << ", " << head.wIdentifier<< endl; 

		std::cout << "////////////////////////////////" << endl;

		//写入20字节, 
		//因为写入到结束位置,写入位置+写入长度大于总长,查询位置大于写入长度,
		//基于以上三点原因,说明可以写从头开始再入,所以会把 m_dwInsertPos 设置为0
		rand_str(szInfo, 9); 
		nLen = CountStringBuffer(szInfo);
		gDataQueue.InsertData(++nIndex, szInfo, nLen);
		std::cout << nLen << ", " << nIndex << endl;

		//再写入10字节
		rand_str(szInfo, 3);
		nLen = CountStringBuffer(szInfo);
		//写入10字节之后, 
		//实际上gDataQueue就变成m_dwDataQueryPos = 24(因为没有读取操作)
		//m_dwDataSize = 38, m_dwInsertPos = 38
		//到这一步貌似也还没有问题,但是如果后续的操作不是读取而是继续写入,那么就有出现问题
		gDataQueue.InsertData(++nIndex, szInfo, nLen);
		std::cout << nLen << ", " << nIndex << endl;

		rand_str(szInfo, 28);
		nLen = CountStringBuffer(szInfo);
		//为了测试问题,这一步继续写入,因为总长是40,再写入,则会重新分配空间
		//重新分配空间时,需要将原有内存数据拷贝到新内存,问题就出现这个拷贝上,步骤如下:
		//1. 判断是否存在末尾还有未读完的数据:m_dwTerminalPos-m_dwDataQueryPos=14
		//2. 先将末尾数据拷贝:CopyMemory(pNewQueueServiceBuffer, m_pDataQueueBuffer + m_dwDataQueryPos, dwPartOneSize);
		//3. 再从头开始拷贝:CopyMemory(pNewQueueServiceBuffer + dwPartOneSize, m_pDataQueueBuffer, m_dwInsertPos);
		//这个步骤理论上是没有错的,但是结合上面的写入数据的顺序,则忽略了一个细节: 写入的数据长度已经覆盖了查询标记
		gDataQueue.InsertData(++nIndex, szInfo, nLen);
		std::cout << nLen << ", " << nIndex << endl;

		std::cout << "////////////////////////////////////" << endl;

		gDataQueue.DistillData(head, szInfo, sizeof(szInfo));
		std::cout << head.wDataSize << ", " << head.wIdentifier << endl;
		gDataQueue.DistillData(head, szInfo, sizeof(szInfo));
		std::cout << head.wDataSize << ", " << head.wIdentifier << endl;
		gDataQueue.DistillData(head, szInfo, sizeof(szInfo)); 
		std::cout << head.wDataSize << ", " << head.wIdentifier << endl;


修复代码:

在bool CWHDataQueue::RectifyBuffer(DWORD dwNeedSize)函数中加上一行:

	try
	{
		//缓冲判断
		if ((m_dwDataSize+dwNeedSize)>m_dwBufferSize) throw 0;

		//重新开始
		if ((m_dwInsertPos==m_dwTerminalPos)&&((m_dwInsertPos+dwNeedSize)>m_dwBufferSize)) 
		{
			if (m_dwDataQueryPos>=dwNeedSize) m_dwInsertPos=0;
			else throw 0;
		}

		//缓冲判断
		if ((m_dwInsertPosm_dwDataQueryPos)) throw 0;

		////////////////////////////////////////////////////
		//头追上尾或尾追上头
		if (m_dwInsertPos + dwNeedSize > m_dwDataQueryPos && m_dwDataQueryPos >= m_dwInsertPos)
		{
			//尾追上头
			if (m_dwDataSize > 0) throw 0;
		}
		////////////////////////////////////////////////////
	}



你可能感兴趣的:(开源库,游戏服务器,网络通信)