cocos2dx在android下是采用Paint来生成图片然后在CCLabelTTF里显示的,它具体的代码都在java类Cocos2dxBitmap里,生成完成之后会调用一个jni函数将结果传给cpp层,cpp层靠一个static变量来与java层交换数据,具体如下
BitmapDC &dc = sharedBitmapDC(); CC_BREAK_IF(! dc.getBitmapFromJava(pText,nWidth,nHeight,eAlignMask,pFontName,nSize)); // assign the dc.m_pData to m_pData in order to save time m_pData = dc.m_pData; CC_BREAK_IF(! m_pData);
这里有个问题,sharedBitmapDC是一直共用的,它的dc.m_pData永远保存的是上一次的数据,如果某种原因java层调用的时候失败了,那么cpp层继续的话,就会拿到上上次的数据,但是这个数据是由cpp层负责free的,于是就会出现double free导致程序崩溃。我们在实际中就碰到了这个情况,当时崩溃的栈告诉我们的是CCImage析构里出现问题
CCImage::~CCImage() { CC_SAFE_DELETE_ARRAY(m_pData); #if (CC_TARGET_PLATFORM == CC_PLATFORM_WINRT) || (CC_TARGET_PLATFORM == CC_PLATFORM_WP8) CC_SAFE_DELETE(m_ft); #endif }
可是单纯看CCImage的代码是看不问题的,而且如果这里有问题,那麻烦大了。最终检查了之后还是确认cpp部分可能会隐藏bug,于是将上面CCImage里修改如下
BitmapDC &dc = sharedBitmapDC(); CC_BREAK_IF(! dc.getBitmapFromJava(pText,nSize)); // assign the dc.m_pData to m_pData in order to save time m_pData = dc.m_pData; dc.m_pData = NULL; CC_BREAK_IF(! m_pData);
这样一来,cpp层肯定不会再出现double free了,于是再测,这个时候发现java层抛异常了,之前的异常由于程序退出根本就没打印得出来。再去追查java层,发现具体原因是
setContentSize之后,java层根据ContentSize判断出来,字体根本放不进去,于是它认为创建出来的Bitmap的高度是0,而Bitmap根本不允许创建高度为0的,于是异常了。我解决的办法是至少让它显示出来一行一列,这样我们看到显示就知道哪里出问题了,而不是直接崩溃。
从这个问题再一次印证了,fail early,fail loud的编程习惯,否则一个简单问题就变成复杂问题了。