[~Ice*shOt]

Need help, how to make load points with loop? Without loop they are loading fine.

PHP Code:

Load_Data( iPlayer )
{
    new szKey[ 64 ];
    new szData[ 64 ];

    formatex( szKey, charsmax( szKey ), "%s", g_szAuthID[ iPlayer ] );

    //for( new i = 0; i < g_iTotal; i++ )
    //    formatex( szData, charsmax( szData ), "%d", g_iPoints[ iPlayer ] );

    nvault_get( g_iVault, szKey, szData, charsmax( szData ) );

    new szPoints[ 8 ];

    parse( szData, szPoints, charsmax( szPoints ) );
    
    for( new i = 0; i < g_iTotal; i++ )
    {
        g_iPoints[ iPlayer ][ i ] = str_to_num( szPoints );
        log_amx( "%d ", g_iPoints[ iPlayer ][ i ] );
    }

} 


[PartialCloning]

1. What is g_iTotal? Where do you set its value?
2. What does the log print out?

[ironskillz1]

Originally Posted by ~Ice*shOt Need help, how to make load points with loop? Without loop they are loading fine. PHP Code: Load_Data( iPlayer ) {     new szKey[ 64 ];     new szData[ 64 ];     formatex( szKey, charsmax( szKey ), "%s", g_szAuthID[ iPlayer ] );     //for( new i = 0; i < g_iTotal; i++ )     //    formatex( szData, charsmax( szData ), "%d", g_iPoints[ iPlayer ] );     nvault_get( g_iVault, szKey, szData, charsmax( szData ) );     new szPoints[ 8 ];     parse( szData, szPoints, charsmax( szPoints ) );          for( new i = 0; i < g_iTotal; i++ )     {         g_iPoints[ iPlayer ][ i ] = str_to_num( szPoints );         log_amx( "%d ", g_iPoints[ iPlayer ][ i ] );     } }

Why are you trying to load points with a loop.

Tell us instead what you are trying todo and we can help you easier

[Bugsy]

You shouldn't post code without including all variables that are referenced. You should also include more information about what you are trying to do.

"how to make load points with loop? Without loop they are loading fine."

How many items are stored in each vault entry?
Is g_iPoints a 2-dimensional array and sized large enough?
What is the value of g_iTotal?

You should make sure g_iTotal is a constant variable and that its value is never larger than the size of the second dimension of g_iPoints or else you can run into index out of bounds errors.

[~Ice*shOt]

Saving works fine, for example I set points g_iPoints[ iPlayer ][ 3 ] it printout

L 12/03/2015 - 18:13:14: [test.amxx] 0 L 12/03/2015 - 18:13:14: [test.amxx] 0 L 12/03/2015 - 18:13:14: [test.amxx] 0 L 12/03/2015 - 18:13:14: [test.amxx] 200 L 12/03/2015 - 18:13:14: [test.amxx] 0

but when it's loading it printout

L 12/03/2015 - 18:13:14: [test.amxx] 0 L 12/03/2015 - 18:13:14: [test.amxx] 0 L 12/03/2015 - 18:13:14: [test.amxx] 0 L 12/03/2015 - 18:13:14: [test.amxx] 0 L 12/03/2015 - 18:13:14: [test.amxx] 0

g_iTotal value is always 5.

PHP Code:

new g_iTotal;

new g_iPoints[33][5];

public native_register( iPlugin, iParams )
{
    new eData[ Data ];

    get_string( 1, eData[ Name ], charsmax( eData[ Name ] ) );

    eData[ Cost ] = get_param( 2 );

    ArrayPushArray( g_aData, eData );
    g_iTotal++;

    return ( g_iTotal - 1 );
}

Save_Data( iPlayer )
{
    new szKey[ 64 ];
    new szData[ 64 ];

    formatex( szKey, charsmax( szKey ), "%s", g_szAuthID[ iPlayer ] );

    for( new i = 0; i < g_iTotal; i++ )
    {
        formatex( szData, charsmax( szData ), "%d", g_iPoints[ iPlayer ][ i ] );

        log_amx( "%d", g_iPoints[ iPlayer ][ i ] );
    }

    nvault_set( g_iVault, szKey, szData );
} 


P.S. no errors, nothing.

[Bugsy]

In your save, try "%d " instead of "%d". How else is parse supposed to know where numbers are separate? Your load function also needs to be fixed

[jimaway]

you overwrite szData in the for loop with every iteration.
when you use nvault_set only the last value is present in there.

[Bugsy]

PHP Code:

new iPos

iPos += formatex( szData[ iPos ] , charsmax( szData ) - iPos , "%d ", g_iPoints[ iPlayer ][ i ] ); 


You need to do something like this, with iPos defined outside of the loop. You also need to fix your load function

[Bugsy]

PHP Code:

Save_Data( iPlayer )
{
    new szData[ 64 ] , iPos;

    //You do not need to use format for this. You only need to use format when you need to actually format the string.
    //Instead, just pass g_szAuthID[] directly to the nvault native.
    //formatex( szKey, charsmax( szKey ), "%s", g_szAuthID[ iPlayer ] );

    for( new i = 0; i < g_iTotal; i++ )
    {
        //formatex( szData, charsmax( szData ), "%d", g_iPoints[ iPlayer ][ i ] );
        iPos += formatex( szData[ iPos ], charsmax( szData ) - iPos , "%d ", g_iPoints[ iPlayer ][ i ] );
        log_amx( "%d", g_iPoints[ iPlayer ][ i ] );
    }

    //nvault_set( g_iVault, szKey, szData );
    nvault_set( g_iVault , g_szAuthID[ iPlayer ] , szData );
}

Load_Data( iPlayer )
{
    new szData[ 64 ] , szPts1[ 6 ] , szPts2[ 6 ] , szPts3[ 6 ] , szPts4[ 6 ] , szPts5[ 6 ];

    nvault_get( g_iVault, g_szAuthID[ iPlayer ] , szData , charsmax( szData ) );

    parse( szData , szPts1 , charsmax( szPts1 ) , szPts2 , charsmax( szPts2 ) , szPts3 , charsmax( szPts3 ) , szPts4 , charsmax( szPts4 ) , szPts5 , charsmax( szPts5 )  );

    g_iPoints[ iPlayer ][ 0 ] = str_to_num( szPts1 );
    g_iPoints[ iPlayer ][ 1 ] = str_to_num( szPts2 );
    g_iPoints[ iPlayer ][ 2 ] = str_to_num( szPts3 );
    g_iPoints[ iPlayer ][ 3 ] = str_to_num( szPts4 );
    g_iPoints[ iPlayer ][ 4 ] = str_to_num( szPts5 );
} 


[~Ice*shOt]

Hey everyone, thanks a lot for your replies and help

I made some testing, with this:

PHP Code:

    case 1: g_iPoints[ iPlayer ][ 0 ] += 100;
    case 2: g_iPoints[ iPlayer ][ 2 ] += 200;
    case 3: g_iPoints[ iPlayer ][ 4 ] += 300;

Save_Data( iPlayer )
{
    new szData[ 64 ];

    for( new i = 0; i < g_iTotal; i++ )
    {
        formatex( szData, charsmax( szData ), "%d", g_iPoints[ iPlayer ][ i ] );

        log_amx( "[%i] %d", i, g_iPoints[ iPlayer ][ i ] );
    }

    nvault_set( g_iVault, g_szAuthID[ iPlayer ], szData );
}

Load_Data( iPlayer )
{
    new szData[ 64 ];

    nvault_get( g_iVault, g_szAuthID[ iPlayer ], szData, charsmax( szData ) );

    new  szPoints[ 8 ];

    parse( szData, szPoints, charsmax( szPoints ) );

    for( new i = 0; i < g_iTotal; i++ )
    {
        g_iPoints[ iPlayer ][ i ] = str_to_num( szPoints );
        log_amx( "[%i] %d", i, g_iPoints[ iPlayer ][ i ] );
    }
} 


Save:

L 12/05/2015 - 13:532: [test.amxx] [0] 100 L 12/05/2015 - 13:532: [test.amxx] [1] 0 L 12/05/2015 - 13:532: [test.amxx] [2] 200 L 12/05/2015 - 13:532: [test.amxx] [3] 0 L 12/05/2015 - 13:532: [test.amxx] [4] 300

Load:

L 12/05/2015 - 13:537: [test.amxx] [0] 300 L 12/05/2015 - 13:537: [test.amxx] [1] 300 L 12/05/2015 - 13:537: [test.amxx] [2] 300 L 12/05/2015 - 13:537: [test.amxx] [3] 300 L 12/05/2015 - 13:537: [test.amxx] [4] 300

So I would be grateful if we done it in my way, can anyone fix load function? It loads only last element how we see.