关于 2dx v3.7 UIScale9Sprite的bug

关于 2dx v3.7 UIScale9Sprite的bug

刚把引擎从js binding v3.0升级到v3.7，发现了一些bug，这里先说说关于scale9sprite的

1. 关于capInsets

https://github.com/cocos2d/cocos2d-x/issues/13560
问题描述：使用cocosbuilder创建的九宫格图片，如果capInserts没有设置（即ZERO），那么九宫格使用的应该是默认缩放，但是结果却是没有缩放。
builder中：

运行结果（上图使用builder默认inserts产生的错误九宫格，下图正常九宫格缩放）：

• 解决方案：

bool Scale9Sprite::updateWithSprite(Sprite* sprite, const Rect& textureRect, bool rotated, const Vec2 &offset, const Size &originalSize, const Rect& capInsets)
{
    //...

    // Set the given rect's size as original size
     _spriteRect = rect;
     _offset = offset;
     _spriteFrameRotated = rotated; 
     _originalSize = size;
     _preferredSize = size; 
    // if(!capInsets.equals(Rect::ZERO))  //此处的if判断应该去掉，直接赋值给_capInsetsInternal 
     { 
         _capInsetsInternal = capInsets; 
     } 
     if (_scale9Enabled) 
     { 
         this->createSlicedSprites(); 
     }

    //...
}

• 原因：
  setSpriteFrame时，默认传入的capInsets是zero，注意_insetRight ,_insetBottom 的值

    void Scale9Sprite::setSpriteFrame(SpriteFrame * spriteFrame, const Rect& capInsets)
    {
        Sprite * sprite = Sprite::createWithTexture(spriteFrame->getTexture());
        this->updateWithSprite(sprite,
                               spriteFrame->getRect(),
                               spriteFrame->isRotated(),
                               spriteFrame->getOffset(),
                               spriteFrame->getOriginalSize(),
                               capInsets);

        // Reset insets
        this->_insetLeft = capInsets.origin.x; // == 0
        this->_insetTop = capInsets.origin.y;// == 0
        this->_insetRight = _originalSize.width - _insetLeft - capInsets.size.width; //== width
        this->_insetBottom = _originalSize.height - _insetTop - capInsets.size.height;// ==height
    }

然后cocosbuilder加载scale9sprite时，顺序调用setInsetLeft(0), setInsetTop(0), setInsetRight(0), setInsetBottom(0)
解析ccb的代码如下：

void Scale9SpriteLoader::onHandlePropTypeFloat(Node * pNode, Node * pParent, const char * pPropertyName, float pFloat, CCBReader * ccbReader) {
    if(strcmp(pPropertyName, PROPERTY_INSETLEFT) == 0) {
        ((cocos2d::ui::Scale9Sprite *)pNode)->setInsetLeft(pFloat);
    } else if(strcmp(pPropertyName, PROPERTY_INSETTOP) == 0) {
        ((cocos2d::ui::Scale9Sprite *)pNode)->setInsetTop(pFloat);
    } else if(strcmp(pPropertyName, PROPERTY_INSETRIGHT) == 0) {
        ((cocos2d::ui::Scale9Sprite *)pNode)->setInsetRight(pFloat);
    } else if(strcmp(pPropertyName, PROPERTY_INSETBOTTOM) == 0) {
        ((cocos2d::ui::Scale9Sprite *)pNode)->setInsetBottom(pFloat);
    } else {
        NodeLoader::onHandlePropTypeFloat(pNode, pParent, pPropertyName, pFloat, ccbReader);
    }
}

4个函数都会调用updateCapInset，并在其中调用setCapInsets，在setCapInsets中调用updateWithSprite和重新计算_insetRight ,_insetBottom的值，
那么问题来了，执行到setInsetRight(0)时：

    void Scale9Sprite::setCapInsets(const Rect& capInsets)
    {
        Size contentSize = this->_contentSize;
        this->updateWithSprite(this->_scale9Image,
                               _spriteRect,
                               _spriteFrameRotated,
                               _offset,
                               _originalSize,
                               capInsets); // 这里传入的capInsets = (0,0, origin.width,0)，此值不等于zero，在updateWithSprite中会赋值给_capInsetsInternal
        this->_insetLeft = capInsets.origin.x;
        this->_insetTop = capInsets.origin.y;
        this->_insetRight = _originalSize.width - _insetLeft - capInsets.size.width;
        this->_insetBottom = _originalSize.height - _insetTop - capInsets.size.height;
        this->setContentSize(contentSize);
    }

然后再在最后一次调用 setInsetBottom(0)时，capInsets==zero，因为有if(!capInsets.equals(Rect::ZERO))判断，不会覆盖_capInsetsInternal，那么_capInsetsInternal就成了一个错误的临时值(0,0, origin.width,0)，其实我们要的应该是最后一次调用后的值zero，即使用默认的九宫格缩放

    // If there is no specified center region
    if ( _capInsetsInternal.equals(Rect::ZERO) )
    {
        // log("... cap insets not specified : using default cap insets ...");
        _capInsetsInternal = Rect(width /3, height /3, width /3, height /3);
    }

当_capInsetsInternal == (0,0, origin.width,0)时，默认值也不会被使用，那么九宫格缩放当然是错误的啦。
所以在updateWithSprite中，// if(!capInsets.equals(Rect::ZERO)) //此处的if判断应该去掉，直接赋值给_capInsetsInternal ，避免多次setCapInset时使用中间的临时值。

---

=======================华丽的分割线=======================

---

2. 创建的九宫格图片偏移了几个像素

https://github.com/cocos2d/cocos2d-x/issues/13564
如图（右为普通sprite，左为scale9sprite）：

createSlicedSprites接口中计算offsetPosition时：

void Scale9Sprite::createSlicedSprites()
    {
        float width = _originalSize.width;
        float height = _originalSize.height;

    Vec2 offsetPosition(ceil(_offset.x + (_originalSize.width - _spriteRect.size.width) / 2),
                            ceil(_offset.y + (_originalSize.height - _spriteRect.size.height) / 2));
    //...
   }

此处计算偏移时为什么要向上取整，总之我不是很明白，希望有人能解释一下，谢谢。
- 导致的问题：
生成的九宫格图片偏移了几个像素，这个问题一般情况下不明显，我是在使用plist拼图时发现， 九宫格图片下方出现了其他图片的条纹，自己图片的上方少了几排像素。
然后找到上面的代码，去掉ceil后，发现恢复了一些，如图（左图还是能看到上方的角少了点）：

把ceil换成floor后，发现正常了，如图（texturepacker打开shape outlines）：

那么问题来了：
按我的理解此处不应该使用ceil，也不应该使用floor，但为什么使用floor后才是正常的。有空的朋友可以使用texturepacker打个纹理图集测试一下，把shape outlines勾选上就能看到图片的矩形区域，看看你们创建出来的scale9sprite矩形区域是否发生了偏移。

---

=======================华丽的分割线=======================

---