[cocos2d-x]CCHttpClient的一个bug
公司的新游戏《我是大官人》马上就要大规模PR了，一切都已经准备就绪，这时测试部门却反馈了一个小问题，打开游戏的时候，偶尔会卡在启动界面，提示：正在连接服务器...然后就没反应了，这个问题发生的概率很低，大概3%左右，而且退出重新打开游戏就好了，&应该是网络不好造成的&，大家并没有太重视这个bug，但是老板不放心，&就算是网络问题，也不应该卡住，如果是新玩家碰到这种情况就直接流失了，这个问题得查一下。& 看来这不是一个小问题，于是这个bug分配给了我。
花了一些时间重现，我发现问题出在CCHttpClient，熟悉cocos2d－x的同学知道，这是一个异步的网络库，它的工作原理是这样的：HttpClient内部有一个工作线程，游戏主线程调用send()函数，将http请求压入一个队列，然后唤醒工作线程就返回了。工作线程被唤醒后，将http请求从队列中取出，再调用libcurl进行处理，然后再通过回调将结果返回主线程。bug重现的时候，主线程注册的回调没有触发，游戏就卡住了。我估计是libcurl将工作线程卡住了，于是在工作线程中加入了一些打印进行验证，结果却大意料：工作线程根本就没有触发！
又是一个多线程同步的问题！这是工作线程的代码片段：
// Worker thread
static void* networkThread(void *data)
CCHttpRequest *request = NULL;
while (true)
if (need_quit)
// step 1: send http request if the requestQueue isn't empty
request = NULL;
pthread_mutex_lock(&s_requestQueueMutex); //Get request task from queue
if (0 != s_requestQueue-&count())
request = dynamic_cast(s_requestQueue-&objectAtIndex(0));
s_requestQueue-&removeObjectAtIndex(0);
// request's refcount = 1 here
pthread_mutex_unlock(&s_requestQueueMutex);
if (NULL == request)
// Wait for http request tasks from main thread
pthread_cond_wait(&s_SleepCondition, &s_SleepMutex);
// step 2: libcurl sync access
工作线程空闲的时候，通过pthread_cond_wait()挂起
send()函数的代码：
//Add a get task to queue
void CCHttpClient::send(CCHttpRequest* request)
if (false == lazyInitThreadSemphore())
if (!request)
++s_asyncRequestC
request-&retain();
pthread_mutex_lock(&s_requestQueueMutex);
s_requestQueue-&addObject(request);
pthread_mutex_unlock(&s_requestQueueMutex);
// Notify thread start to work
pthread_cond_signal(&s_SleepCondition);
我仔细的看了一遍，发现这句话很奇怪：
pthread_cond_wait(&s_SleepCondition, &s_SleepMutex);
这里用了一个互斥锁s_SleepMutex，而这个锁没有其他代码使用。这很可能是一个错误：一个互斥锁如果只有一处代码使用，那么只有一种可能，这段代码会在多个线程中执行。而httpClient的线程显然只有一个。google了一番，果然是这个互斥锁用错了。大家可以参考知乎上的一篇文章：/question/
里面详细解释了为什么没有正确的加锁会导致信号量丢失。
这里是修改后的代码，不再使用单独的s_SleepMutex, 用requestQueueMutex代替
// Worker thread
static void* networkThread(void *data)
CCHttpRequest *request = NULL;
while (true)
if (need_quit)
// step 1: send http request if the requestQueue isn't empty
request = NULL;
pthread_mutex_lock(&s_requestQueueMutex); //Get request task from queue
while (0 == s_requestQueue-&count()) {
pthread_cond_wait(&s_SleepCondition, &s_requestQueueMutex);
request = dynamic_cast(s_requestQueue-&objectAtIndex(0));
s_requestQueue-&removeObjectAtIndex(0);
// request's refcount = 1 here
CCLog(s_SleepCondition notified);
pthread_mutex_unlock(&s_requestQueueMutex);
(window.slotbydup=window.slotbydup || []).push({
id: '2467140',
container: s,
size: '1000,90',
display: 'inlay-fix'
(window.slotbydup=window.slotbydup || []).push({
id: '2467141',
container: s,
size: '1000,90',
display: 'inlay-fix'
(window.slotbydup=window.slotbydup || []).push({
id: '2467142',
container: s,
size: '1000,90',
display: 'inlay-fix'
(window.slotbydup=window.slotbydup || []).push({
id: '2467143',
container: s,
size: '1000,90',
display: 'inlay-fix'
(window.slotbydup=window.slotbydup || []).push({
id: '2467148',
container: s,
size: '1000,90',
display: 'inlay-fix'今天上传文件的时候用了MultipartEntityBuilder，添加了httpcore等2个依赖库，编译运行的时候，studio出现一个奇怪的错误提示：
Duplicate files copied in APK META-INF/DEPENDENCIES
File 1: /home/bluelife/.gradle/caches/modules-2/files-2.1/org.apache.httpcomponents/httpmime/4.3.1/f7899276dddd01d8a42ecfe27e7031fcf9824422/httpmime-4.3.5.jar
File 2: /home/bluelife/.gradle/caches/modules-2/files-2.1/org.apache.httpcomponents/httpmime/4.3.1/f7899276dddd01d8a42ecfe27e7031fcf9824422/httpmime-4.3.5.jar不知道哪里出错，最后Google找到了解决方法。需要在build.gradle文件里添加如下配置：
packagingOptions {
exclude 'META-INF/DEPENDENCIES'
exclude 'META-INF/NOTICE'
exclude 'META-INF/LICENSE'
exclude 'META-INF/LICENSE.txt'
exclude 'META-INF/NOTICE.txt'
顺利编译。（注，我目前使用的版本是最新的candy版0.8.11）
阅读(...) 评论()