19

我正在尝试将表从Lua加载到 C++，但我无法正确处理。我很好地完成了第一次迭代，但是在第二次调用 lua_next 时它崩溃了。有任何想法吗？

Lua 文件： 

level = { 1, 2, 3, }

C++ 文件 - 首先我这样做：

lua_getglobal( L, "level" );
for( lua_pushnil( L ); lua_next( L, 1 ); lua_pop( L, -2 ) )
{
    if( lua_isnumber( L, -1 ) ) {
        int i = (int)lua_tonumber( L, -1 );
        //use number
    }
}
lua_pop( L, 1 );

然后我尝试了参考手册：

lua_getglobal( L, "level" );
int t = 1;
lua_pushnil( L );
while( lua_next( L, t ) ) {
    printf( "%s - %s", 
        lua_typename( L, lua_type( L, -2 ) ),
        lua_typename( L, lua_type( L, -1 ) ) );
    lua_pop( L, 1 );
}
lua_pop( L, 1 );

最后是这个：

lua_getglobal( L, "level" );
lua_pushnil( L );

lua_next( L, 1 );
if( lua_isnumber( L, -1 ) ) {
    int i = (int)lua_tonumber( L, -1 );
    //use number fine
}
lua_pop( L, 1 );

lua_next( L, 1 ); //crashes

etc...

自然 L 是一个 lua_State* ，我正在初始化它并解析文件。

编辑： 为了回应 Jesse Beder 的回答，我用记录器尝试了这段代码，但我仍然无法让它工作。

Log::Get().Write( "engine", "stack size: %i", lua_gettop( L ) );

lua_getglobal(L, "level");
if( lua_istable( L, -1 ) )
    Log::Get().Write( "engine", "-1 is a table" );

lua_pushnil(L);
if( lua_isnil( L, -1 ) )
    Log::Get().Write( "engine", "-1 is now nil" );
if( lua_istable( L, -2 ) )
    Log::Get().Write( "engine", "-2 is now table" );

int pred = lua_next( L, -2 );
Log::Get().Write( "engine", "pred: %i", pred );
while( pred ) {
    Log::Get().Write( "engine", "loop stuff" );
    if( lua_isnumber( L, -1 ) ) {
        int i = (int)lua_tonumber( L, -1 );
        //use number
        Log::Get().Write( "engine", "num: %i", i );
    }
    Log::Get().Write( "engine", "stack size: %i", lua_gettop( L ) );
    if( lua_istable( L, -3 ) )
        Log::Get().Write( "engine", "-3 is now table" );

    lua_pop( L, 1 );
    Log::Get().Write( "engine", "stack size: %i", lua_gettop( L ) );
    if( lua_istable( L, -2 ) )
        Log::Get().Write( "engine", "-2 is now table" );

    pred = lua_next( L, -2 );
    Log::Get().Write( "engine", "pred: %i", pred );
}
lua_pop( L, 1 );

这给出了这个输出：

stack size: 0
-1 is a table
-1 is now nil
-2 is now table
pred: 1
loop stuff
num: 1
stack size: 3
-3 is now table
stack size: 2
-2 is now table

杰西，你所说的一切似乎都是正确的。但它仍然无法进入下一次迭代。

Edit2： 我试图将确切的代码复制到一个新项目中，跳过所有周围的类和我不想在这里和那里包含的东西。但在这里它没有，它只会在调用 lua_next 后存活下来。

Edit3： 我现在进一步缩小了范围。我使用hge作为我的 2D 引擎。我把之前的所有代码都放在了函数测试中：

test(); //works
if( hge->System_Initiate() )
{       
    test(); //fails
    hge->System_Start();
}

据我了解，hge 对 lua 没有任何作用。 这是我做的一个小测试的源代码。hge 1.81 的来源在这里。

Edit4： 问题的大小已经失控，但无济于事。这是我能够将其简化为的最小代码。

extern "C"
{
    #include <lua/lua.h>
    #include <lua/lualib.h>
    #include <lua/lauxlib.h>
}
#include <hge\hge.h>

bool frame_func()
{   
    return true;
}

bool render_func()
{
    return false;
}

void test()
{
    lua_State *L = lua_open();
    luaL_openlibs( L );

    if( luaL_dofile( L, "levels.lua" ) ) {
        lua_pop( L, -1 );
        return;
    }
    lua_getglobal(L, "level");
    lua_pushnil(L);

    while( lua_next( L, -2 ) ) {
        if( lua_isnumber( L, -1 ) ) {
            int i = (int)lua_tonumber( L, -1 );
            //use number
        }
        lua_pop( L, 1 );
    }
    lua_pop( L, 1 );

    lua_close( L );
}

int main()
{
    HGE *hge = hgeCreate( HGE_VERSION );

    hge->System_SetState( HGE_FRAMEFUNC, frame_func );
    hge->System_SetState( HGE_RENDERFUNC, render_func );
    hge->System_SetState( HGE_WINDOWED, true );
    hge->System_SetState( HGE_SCREENWIDTH, 800 );
    hge->System_SetState( HGE_SCREENHEIGHT, 600 );
    hge->System_SetState( HGE_SCREENBPP, 32 );

    //test(); //works

    if( hge->System_Initiate() )
    {       
        test(); //fails
        hge->System_Start();
    }

    hge->Release();

    return 0;
}
4

3 回答 3

29

调用lua_next时，第二个参数应该是表的索引。因为你只是把桌子推到堆栈上

lua_getglobal(L, "level");

在那次调用之后，你的堆栈看起来像

-1：表“级别”

（不是+1，因为堆栈被向下读取）。然后你打电话

lua_pushnil(L);

所以你的堆栈将是

-1：键（无）
-2：表“级别”

你的桌子在-2，所以当你打电话时lua_next，你应该使用索引-2。最后，在每次迭代之后，您的堆栈应如下所示：

-1：值
-2：钥匙
-3：表“级别”

所以你想读取值（at -1）然后弹出它（所以只需弹出一次），然后调用lua_next以获取下一个键。所以这样的事情应该有效：

lua_getglobal(L, "level");
lua_pushnil(L);

while(lua_next(L, -2)) {  // <== here is your mistake
    if(lua_isnumber(L, -1)) {
        int i = (int)lua_tonumber(L, -1);
        //use number
    }
    lua_pop(L, 1);
}
lua_pop(L, 1);

根据您的第二次编辑进行编辑

由于它在您删除外部内容时有效，但在您重新添加时无效，我最好的猜测是您以某种方式破坏了堆栈（C++ 堆栈或 lua 堆栈）。仔细查看您的指针，尤其是在您操作 lua 状态时。

于 2009-09-17T14:27:32.400 回答
1

阅读 LUA 手册lua_next你会发现直接在键上使用 lua_tostring 可能会出现问题，即你必须检查键的值然后决定使用 lua_tostring 或 lua_tonumber。所以你可以试试这段代码：

   std::string key
   while(lua_next(L, -2) != 0){ // in your case index may not be -2, check

    // uses 'key' (at index -2) and 'value' (at index -1)

    if (lua_type(L, -2)==LUA_TSTRING){ // check if key is a string
         // you may use key.assign(lua_tostring(L,-2));
    }
    else if (lua_type(L, -2)==LUA_TNUMBER){ //or if it is a number
         // this is likely to be your case since you table level = { 1, 2, 3, }
         // don't declare field ID's
         // try:
         // sprintf(buf,"%g",lua_tonumber(L,-2));
         // key.assign(buf);
    }
    else{
        // do some other stuff
    }
    key.clear();
    lua_pop(L,1)
   }

希望能帮助到你。

于 2014-02-27T20:54:49.653 回答
0

你为什么要lua_pop(L, 1)在参考手册的版本末尾做额外的事情？我可以看到这可能是个问题，因为您弹出的堆栈深度超出了堆栈深度。

于 2009-09-17T13:51:23.230 回答