我为什么把 Rocket Through游戏中的Rocket::collideWithSides()写成这样？

本文是学习cocos2d-x by Example Beginner's Guide 第5章 Rocket Through的副产物。

我们首先来看看本书的作者是怎么写Rocket::collideWithSides()的，代码如下：

bool Rocket::collidedWithSides() {
    
    CCSize screenSize = CCDirector::sharedDirector()->getWinSize();
    
    
    if (m_tPosition.x > screenSize.width - _radius) {
        m_tPosition.x =  screenSize.width - _radius;
        _rotationOrientation = ROTATE_NONE;
        _vector = ccp (this->getVector().x * -1, this->getVector().y);
        this->setRotationFromVector();
        return true;
    }
    
    if (m_tPosition.x < _radius) {
        m_tPosition.x =  _radius;
       	_rotationOrientation =  ROTATE_NONE;
        _vector =  ccp (this->getVector().x * -1, this->getVector().y);
        this->setRotationFromVector();
        return true;
        
    }
    
    if (m_tPosition.y < _radius) {
        m_tPosition.y = _radius;
       	_rotationOrientation = ROTATE_NONE;
        _vector =  ccp (this->getVector().x, this->getVector().y * -1);
        this->setRotationFromVector();
        return true;
    }
    
    if (m_tPosition.y > screenSize.height - _radius) {
        m_tPosition.y =  screenSize.height - _radius;
       	_rotationOrientation = ROTATE_NONE;
        _vector =  ccp (this->getVector().x, this->getVector().y * -1);
        this->setRotationFromVector();
        return true;
    }
    
    return false;
}

上面的 Rocket::collidedWithSides()方法，是检测火箭碰壁有的，如果碰壁，就重设火箭的状态。作为教学用例，效果非常棒。

我打算改写有两个原因

一：可是我深受CharlesPetzold 影响，觉得可以改写的更简练一些。if的复合语句作用有4点

1.重设火箭的坐标

2.设置旋转方向为ROTATE_NONE

3.设置新Vector

4.调用setRotationFromVector()

最后，最后就返回啦。

二：返回分支太多，打算改写成一个 return 出口。

好吧，是时候动手了。

让我从简单的返回分支开始,改写后

bool Rocket::collideWithSides() {
    ...
    if (m_tPosition.x > screenSize.width - _radius) {
        ...

        bResult = true;
    }
    
    if (m_tPosition.x < _radius) {
        ...
        
        bResult = true;        
    }
    
    if (m_tPosition.y < _radius) {
        ...
        
        bResult = true;
    }
    
    if (m_tPosition.y > screenSize.height - _radius) {
        ...
        
        bResult = true;
    }
    
    return bResult;
}

看着真不错，可是总觉得哪里不对劲耶？

4个if {},每次都执行一遍。

还是默默地，改过来吧。

既然改写return出口是白忙活，那就看看 if 复合语句

上面已经概括 if 复合语句的使用，下面来写个private属性的方法，实现上面的4个功能

声明：

private:
    bool collideWithSide(float& side,const float newSide,const cocos2d::CCPoint& point,int rotationOrientation = ROTATE_NONE);

实现：

bool Rocket::collideWithSide(float& side,const float newSide,const cocos2d::CCPoint& point,int rotationOrientation) {
    side = newSide;
    this->_rotationOrientation = rotationOrientation;
    _vector = ccp(point.x, point.y);
    this->setRotationFromVector();
    return true;
}

collideWithSide方法的返回值，做成bool，以方便collideWithSides() 调用。 /*此方法名为collideWithSides,后面有S，以区别collideWithSide*/

bool Rocket::collideWithSides(void) {
    CCSize screen = CCDirector::sharedDirector()->getWinSize();
    CCPoint pos = this->getVector();
    if (m_tPosition.x < _radius) {
        return collideWithSide(m_tPosition.x,_radius, ccp(pos.x * -1, pos.y));
    }
    if (m_tPosition.x + _radius > screen.width) {
        return collideWithSide(m_tPosition.x,screen.width - _radius, ccp(pos.x * -1, pos.y));
    }
    if (m_tPosition.y < _radius) {
        return collideWithSide(m_tPosition.y,_radius, ccp(pos.x, pos.y * -1));
    }
    if (m_tPosition.y + _radius > screen.height) {
        return collideWithSide(m_tPosition.y, screen.height - _radius, ccp(pos.x, pos.y));
    }
    return false;
}

正如你现在所看到的，这些是我改写的过程。欢迎大家来拍砖。