19

Luaから C++ にテーブルをロードしようとしていますが、うまくいきません。私は最初の繰り返しをうまくやっていますが、lua_next への 2 回目の呼び出しでクラッシュします。何か案は？

ルアファイル: 

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: 正確なコードを新しいプロジェクトにコピーしようとしましたが、周囲のすべてのクラスと、あちこちに含める必要のないものをスキップして動作します。しかし、ここではそうではありません。

Edit3: もう少し 絞り込みました。2D エンジンとしてhgeを使用しています。以前のすべてのコードを関数テストに入れました。

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場合、2 番目の引数はテーブルのインデックスにする必要があります。テーブルをスタックにプッシュしているだけなので

lua_getglobal(L, "level");

その後、スタックは次のようになります

-1: テーブル「レベル」

（+1スタックがダウンして読み取られるため、そうではありません）。それからあなたは電話します

lua_pushnil(L);

だからあなたのスタックは

-1: キー (なし)
-2: テーブル「レベル」

あなたのテーブルは に-2あるので、 を呼び出すときlua_nextは、インデックスを使用する必要があります-2。最後に、各反復の後、スタックは次のようになります。

-1: 値
-2: キー
-3: テーブル「レベル」

そのため、値を ( で) 読み取ってから-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);

2 回目の編集に基づいて編集する

外部のものを削除すると機能しますが、再度追加すると機能しないため、何らかの方法でスタック (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 に答える