[^SmileY]

Hi, i having problems on compile a test plugin:

PHP Code:

#include <amxmodx>
#include <csstats>
#include <csx>

#include <PugCore>
#include <PugStocks>

enum _:pStats
{
    Auth[MAX_AUTHID_LENGTH],
    Name[MAX_NAME_LENGTH],
    Stats[STATSX_MAX_STATS],
    Body[MAX_BODYHITS],
    Objective[STATSX_MAX_OBJECTIVE]
};

new g_Stats[33][pStats];

public plugin_init()
{
    register_plugin("Pug Mod (Stats)",PUG_VERSION,PUG_AUTHOR);
    
    register_concmd("teste","cmdTeste");
}

public cmdTeste(id)
{
    new StatsNum = get_statsnum();
    
    for(new i;i < StatsNum;i++)
    {
        get_stats2(i,g_Stats[i][Objective]);
        
        server_print("(%s) (%s) Shots %d",g_Stats[i][Name],g_Stats[i][Auth],g_Stats[i][STATSX_SHOTS]);
    }
} 


AMX Mod X Compiler 1.9.0.5235
Copyright (c) 1997-2006 ITB CompuPhase
Copyright (c) 2004-2013 AMX Mod X Team

Error: Array sizes do not match, or destination array is too small on line 32

1 Error.
Could not locate output file D:\Steam\steamapps\common\Half-Life\cstrike\addons\amxmodx\plugins\PugStats.amx (compile failed).

I know that g_Stats[i][Objective] is same as g_Stats[i][4], and get_stats2 needs to be STATSX_MAX_OBJECTIVE, but how use like an struct?

Thanks for help

[CrazY.]

#include <amxmodx>
#include <csstats>

new g_Objective[33][STATSX_MAX_OBJECTIVE];

public plugin_init()
{
    register_plugin("Plugin", "Version", "Author");

    new i, iStatsNum;
    iStatsNum = min(get_statsnum(), sizeof g_Objective);

    for (i = 0; i < iStatsNum; i++)
    {
        get_stats2(i, g_Objective[i]);
        server_print("Stats[%d]^nTOTAL_DEFUSIONS=%d^nBOMBS_DEFUSED=%d^nBOMBS_PLANTED=%d^nBOMB_EXPLOSIONS=%d^n", i, g_Objective[i][STATSX_TOTAL_DEFUSIONS], g_Objective[i][STATSX_BOMBS_DEFUSED], g_Objective[i][STATSX_BOMBS_PLANTED], g_Objective[i][STATSX_BOMB_EXPLOSIONS])
    }
}

[^SmileY]

@edon1337 Will not solve the error

@CrazY. The main idea is use enum, thanks anyway

Ps.
Any ideias?

[OciXCrom]

Originally Posted by edon1337 PHP Code: enum _:pStats  {      Auth[MAX_AUTHID_LENGTH],      Name[MAX_NAME_LENGTH],      Stats[STATSX_MAX_STATS],      Body[MAX_BODYHITS],      Objective[STATSX_MAX_OBJECTIVE]  };  --> PHP Code: enum _:pStats  {      Auth[64],      Name[64],      Stats[64],      Body[64],      Objective[64]  };

What's the difference? Why are you suggesting him to use magic numbers (also with wrong values) instead of the newly defined constants in 1.9? That is very bad advice.

On topic: try get_stats2(i, g_Stats[][Objective]).

[Bugsy]

get_stats() and get_stats2() retrieve data from csstats.dat, it is not always associated with a player who is actively on your server. I am pointing this out because you have this: new g_Stats[33][pStats];

Take a look at this: https://forums.alliedmods.net/showpo...80&postcount=5

[^SmileY]

Originally Posted by Bugsy get_stats() and get_stats2() retrieve data from csstats.dat, it is not always associated with a player who is actively on your server. I am pointing this out because you have this: new g_Stats[33][pStats]; Take a look at this: https://forums.alliedmods.net/showpo...80&postcount=5

Thanks, but is not this, i plan to reset stats in every map changes, also current map will not have more players enter/exiting than 33 etc. Soo it is not for each player, but create a struct for it.

I take a look in your plugin, also i need to store in array first since i will try to send using curl, mysql or sockets to a web server

Thanks again

[Bugsy]

Your usage looks correct to me, but the compiler is temperamental sometimes when passing enumerator sized arrays to a function. Save yourself the headache and use individual arrays for stats and stats2, as I did in my linked plugin/thread. There's no need/benefit that I can think of to pack it all into an array.

get_stats2(i,g_Stats[i][Objective]);

[^SmileY]

Yes, that will be the solution.

Thanks again

[edon1337]

Originally Posted by OciXCrom What's the difference? Why are you suggesting him to use magic numbers (also with wrong values) instead of the newly defined constants in 1.9? That is very bad advice. On topic: try get_stats2(i, g_Stats[][Objective]).

I didn't know the values of these defines that he used so I randomly used 64.
I thought he had the same problem as me in this code:

If for example you change AIRDROP_CLASSNAME to 63 meanwhile the destination is declared as 64 it would throw same error as him, just a little misunderstanding.

PHP Code:

enum _:Config
{
    AIRDROP_CLASSNAME[ 64 ],
    AIRPLANE_CLASSNAME[ 64 ],
    GAME_SOUNDS_FILE[ 64 ],
    GAME_MODELS_FILE[ 64 ],
    GAME_WEAPONS_FILE[ 64 ],
    PLANE_ORIGINS_FILE[ 64 ],
    GAME_CUSTOM_SPAWN_POINTS_FILE[ 64 ],
    ARROW_ICON_SPRITE[ 64 ],
    AUTO_POINTS_FILE[ 64 ],
    ZONE_ORIGINS_FILE[ 64 ]
};

ReadConfig( )
{
    new szFilename[ 256 ];
    formatex( szFilename, charsmax( szFilename ), "%s/%s/%s", g_szConfigsDir, PATH_MANAGMENT, CSBR_Config_File );
    
    new iFile = fopen( szFilename, "rt" );
    
    if( iFile )
    {
        new szData[ 128 ], szValue[ 64 ], szKey[ 64 ];
        }
        ...
} 


[^SmileY]

get_stats2 have a fixed array for parameter. that is why im having troubles.

The solution is use as separated array, or just use a copy() to copy into the struct.