bug « 爱好玩

很久很久以前

刚从业时,是做Java后端的
我们知道Integer是int的包装类,2个int类型的值是可以通过a == b来进行比较的
于是想当然以为Integer也是可以这么比较

一次事故就要开始

一个简单的充值验证,需要比较充值数额是否相等,用的Integer类
我们测试的时候用的是1块钱(10金币),比较金币拿了2个Integer来做==判断
10金币顺利通过测试,100金币也是,随之上线

事故的爆发

产品上线后,从后台查询发现,只有10金币和100金币的充值记录(但按经验看来是不合理的)
也有玩家陆续反应100块(1000金币)充值失败

原因

Java中<128的Integer取的是常量,内存地址是一样的,>=128时,则是重新开辟了一块内存
又因为==是比较内存地址用的,比较值相等应该用a.equals(b)

测试代码

package test;

public class Test {
    public static void main(String[] args) {
        Integer goldA = 100;
        Integer goldB = 100;
        System.out.println(goldA == goldB);
        goldA = 1000;
        goldB = 1000;
        System.out.println(goldA == goldB);
    }
}

package test;

public class Test {

public static void main(String[] args) {

Integer goldA = 100;

Integer goldB = 100;

System.out.println(goldA == goldB);

goldA = 1000;

goldB = 1000;

System.out.println(goldA == goldB);

}

输出

true
false

1 2	true false

问题的产生

项目是用ccs1.6做的UI,在某个需求中,需要把一个Sprite的SpriteFrame clone一份,去构造另一个
代码类似于

local sprite = cc.ui.UIImage.new("file1.png")
sprite:pos(x,y):addTo(self)

local sprite2 = cc.ui.UIImage.new(sprite:clone():getSpriteFrame())
sprite2:pos(x2,y2):addTo(self)

local sprite = cc.ui.UIImage.new("file1.png")

sprite:pos(x,y):addTo(self)

local sprite2 = cc.ui.UIImage.new(sprite:clone():getSpriteFrame())

sprite2:pos(x2,y2):addTo(self)

这样并没有问题
但是如果sprite的SpriteFrame被set过一个新的,那sprite2得到的SpriteFrame,依然是老的

怎么回事?

查看源码,我们发现Node:clone()方法被重写了(在NodeEx.lua里)

function Node:clone()
    local cloneNode = self:createCloneInstance_()

    cloneNode:copyProperties_(self)
    cloneNode:copySpecialPeerVal_(self)
    cloneNode:copyCloneWidgetChildren_(self)

    return cloneNode
end

function Node:clone()

local cloneNode = self:createCloneInstance_()

cloneNode:copyProperties_(self)

cloneNode:copySpecialPeerVal_(self)

cloneNode:copyCloneWidgetChildren_(self)

return cloneNode

end

粗读一下代码,先克隆一个实例,然后克隆基础属性,最后递归把子节点也克隆了

那问题就应该出现在self:createCloneInstance_()上
查阅发现

function UIImage:createCloneInstance_()
    return UIImage.new(unpack(self.args_))
end

function UIImage:createCloneInstance_()

return UIImage.new(unpack(self.args_))

end

好吧…直接用构造参数返回一个实例是什么鬼

既然出事了,就弄弄大

我们发现所有UI组件的createCloneInstance_方法都有这个问题
写一段比较简单的代码

local label = cc.ui.UILabel.new({text = "before"})
label:pos(x,y):addTo(self)
label:setString("after")

local label2 = label:clone()
label2:pos(x2,y2):addTo(self)

local label = cc.ui.UILabel.new({text = "before"})

label:pos(x,y):addTo(self)

label:setString("after")

local label2 = label:clone()

label2:pos(x2,y2):addTo(self)

可以看到,屏幕上显示label显示after,label2显示before

如果不用UI组件呢?

我试了一下,直接使用Sprite和LabelTTF是没有问题的,那这个就可以定性为quick的bug了

发现问题

项目从3.3rc1升级到3.5后,很多触摸事件都变得不正常了

于是

在cocoachina上发现了这篇文章
已经有童鞋在github上提交了pr戳这里

然后

在编译simulator的时候,windows.h找不到,结果发现win7+vs2013是需要windows8.1的sdk百度盘

最后

问题解决了

BUG表现为,cocos2dx的工程下,在能输入文字的地方,点击iOS键盘上的"话筒"按钮,进行听写,此时APP直接崩溃

解决方法:

在CCDirectorCaller.mm文件里修改 doCaller方法


-(void) doCaller: (id) sender
{
    [EAGLContext setCurrentContext:[[EAGLView sharedEGLView] context]];
    cocos2d::CCDirector::sharedDirector()-&gt;mainLoop();
}

-(void) doCaller: (id) sender

{

[EAGLContext setCurrentContext:[[EAGLView sharedEGLView] context]];

cocos2d::CCDirector::sharedDirector()->mainLoop();

}

添加头文件


#import <opengles eagl.h="">
#import &quot;EAGLView.h&quot;
</opengles>

#import <opengles eagl.h="">

#import "EAGLView.h"

</opengles>

再次实验后问题消失

环境:cocos2dx 2.1rc0-x-2.1.4
ios7.0.6

https://github.com/boyu0/cocos2d-x/commit/c3778d8123537f0e991e98c2d89d3e577e3120f1

一	二	三	四	五	六	日
« 8月
1	2	3	4	5	6	7
8	9	10	11	12	13	14
15	16	17	18	19	20	21
22	23	24	25	26	27	28
29	30

爱好玩

Stay Hungry, Stay Foolish.

Tag Archives: bug

[Java]Integer比较容易遇到的坑