关于cocos2dx widget窗口中的关闭按钮 删除自身 的时候崩溃bug

回调函数里判断是弹起事件时会调用UIButton的removeFromParent方法，这时会导致游戏崩溃，代码中断在void Widget::onTouchEnded(Touch *touch, Event *unusedEvent)方法里的releaseUpEvent()调用处。

响应UIButton的触摸弹起事件时，删除UIButton控件崩溃

修改代码解决方法1

[plain] view plain copy

1. 解决办法   
2. void Widget::releaseUpEvent()  
3. {  
4.     if (_touchEventListener && _touchEventSelector)  
5.     {  
6.         (_touchEventListener->*_touchEventSelector)(this,TOUCH_EVENT_ENDED);  
7.     }  
8.       
9.     if (_touchEventCallback) {  
10.         _touchEventCallback(this, TouchEventType::ENDED);  
11.     }  
12. }  
13. 换个位置就好了  
14. 调用callback完以后释放本对象，但代码没执行完 继续调用Listener，这时释放的地址很可能不为空，就导致出错了。   

2.不需要修改引擎代码

释放的时候改用

改用 widget->runAction(RemoveSelf::create()); 

代替原来的removeFromParent()

原理:我们可以看看引擎对removeSelf的实现

[cpp] view plain copy

1. //  
2. // Remove Self  
3. //  
4. RemoveSelf * RemoveSelf::create(bool isNeedCleanUp /*= true*/)   
5. {  
6.     RemoveSelf *ret = new RemoveSelf();  
7.   
8.     if (ret && ret->init(isNeedCleanUp)) {  
9.         ret->autorelease();  
10.     }  
11.   
12.     return ret;  
13. }  
14.   
15. bool RemoveSelf::init(bool isNeedCleanUp) {  
16.     _isNeedCleanUp = isNeedCleanUp;  
17.     return true;  
18. }  
19.   
20. void RemoveSelf::update(float time) {  
21.     CC_UNUSED_PARAM(time);  
22.     _target->removeFromParentAndCleanup(_isNeedCleanUp);  
23. }  
24.   
25. RemoveSelf *RemoveSelf::reverse() const  
26. {  
27.     return RemoveSelf::create(_isNeedCleanUp);  
28. }  
29.   
30. RemoveSelf * RemoveSelf::clone() const  
31. {  
32.     // no copy constructor  
33.     auto a = new RemoveSelf();  
34.     a->init(_isNeedCleanUp);  
35.     a->autorelease();  
36.     return a;  
37. }  

弹起事件中不是直接调用remove,而是设置一个标志，在update中检测到标志再remove. 

3.修改源码

void Widget::pushDownEvent()
{
+ this->retain();
if (_touchEventCallback) {
_touchEventCallback(this, TouchEventType::BEGAN);
}
void Widget::pushDownEvent()
{
(_touchEventListener->*_touchEventSelector)(this,TOUCH_EVENT_BEGAN);
}
+ this->release();
}

void Widget::moveEvent()
{
+ this->retain();
if (_touchEventCallback) {
_touchEventCallback(this, TouchEventType::MOVED);
}
void Widget::moveEvent()
{
(_touchEventListener->*_touchEventSelector)(this,TOUCH_EVENT_MOVED);
}
+ this->release();
}

void Widget::releaseUpEvent()
{
-
+ this->retain();
if (_touchEventCallback) {
_touchEventCallback(this, TouchEventType::ENDED);
}
void Widget::releaseUpEvent()
{
(_touchEventListener->*_touchEventSelector)(this,TOUCH_EVENT_ENDED);
}
+ this->release();
}

void Widget::cancelUpEvent()
{
+ this->retain();
if (_touchEventCallback)
{
_touchEventCallback(this, TouchEventType::CANCELED);
  void Widget::cancelUpEvent()
{
(_touchEventListener->*_touchEventSelector)(this,TOUCH_EVENT_CANCELED);
}
-
+ this->release();
}