一、问题的起因
问题首先是在Quick-x中发现的,原因是希望为CCNode添加一个Lua自定义方法就会出错:
function CCNode:myMethod() end
运行错误提示是 [LUA ERROR] stack overflow,在Player上运行时甚至直接抛出异常提示框必须直接关闭程序。
进一步调试,有以下结果:
1.此问题只出现在导出的子类中,如CCObject就不会有问题,而包括CCNode在内所有继承CCObject的类就有问题
2.并不仅仅是新加方法有问题,新加成员变量也是有问题的
因此,这一问题可以描述为:tolua++导出的C++子类,在Lua中无法添加新成员。
二、问题的分析
由于是无法添加新成员,首先考虑的是__newindex元方法是否有问题。
分析tolua++的代码,可以看到在tolua_classevents函数中,将__newindex设置为了class_newindex_event函数:
static int class_newindex_event (lua_State* L) { int t = lua_type(L,1); if (t == LUA_TUSERDATA) { //此处代码略去 } else if (t== LUA_TTABLE) { module_newindex_event(L); } return 0; }
当CCObject和它的子类添加新成员时,这里的t值是LUA_TTABLE,因此会调用module_newindex_event函数:
static int module_newindex_event (lua_State* L) { lua_pushstring(L,".set"); lua_rawget(L,-4); if (lua_istable(L,-1)) { //此处代码略去 } /* call old newindex meta event */ if (lua_getmetatable(L,1) && lua_getmetatable(L,-1)) { lua_pushstring(L,"__newindex"); lua_rawget(L,-2); if (lua_isfunction(L,-1)) { lua_pushvalue(L,1); lua_pushvalue(L,2); lua_pushvalue(L,3); lua_call(L,3,0); } } lua_settop(L,3); lua_rawset(L,-3); return 0; }
可以看出这里是取metatable里的.set表,并对新的成员进行赋值。
第一个判断,CCObject和它的子类都不会进入,没有影响。
但第二个判断情况就不同了,CCObject不会进入,它的子类是会进入的。从代码写法上看,这里似乎是希望调用其父类的__newindex,但这样是很有问题的。
首先现在这样的写法,取到的仍然是子类自己的__newindex,因此会再调用一次class_newindex_event函数,而class_newindex_event自然又会调用module_newindex_event,结果又回到了这段代码当中!这样,函数不断递归调用,永无休止,直到堆栈溢出为止,这就是出错为stack overflow的原因。
其次我还没有想明白的是,即使是父类的__newindex,这里似乎也没有调用的必要。因为子类要是定义一个新成员是在父类里生效的话,那问题就大了。所以这里我认为直接调用lua_rawset应该就可以了。
三、临时解决方案
基于以上分析,我认为问题就在于module_newindex_event的处理上。我尝试不调用module_newindex_event,而直接对新成员进行赋值操作,即修改class_newindex_event函数的相关代码如下:
else if (t== LUA_TTABLE) { // module_newindex_event(L); lua_settop(L,3); lua_rawset(L,-3); }
修改的这两句相当于在lua里面直接调用rawset对新成员进行赋值操作。
修改后,在Lua中运行以下代码:
function CCNode:testFunc() print("----------CCNode.testFunc------Entry now!!!") end function MainScene:ctor() local s = display.newSprite("Logo.png", display.cx, display.cy) self:addChild(s) s:testFunc() end
上述代码是可以正常运行并得到预想中的输出的。
不过目前这只是临时方案。我不知道tolua++原来这样写代码是不是另有原因。后面我将再仔细考虑清楚,这样的修改会不会有什么负作用。
(2014.5.30加注:quick-x的2.2.3版本目前已经采用了这一修改方案)