[Natsheh]

PHP Code:

new Top_fragers[10], count;
for(new i, j; i < pnum; i++)
{
      for(j = 0; j < pnum; j++)
      {
              if(get_user_frags(players[i]) <
                            get_user_frags(players[j])
               {
                       Top_fragers[count] = players[j];
               }
       }
       count++;
} 


[fysiks]

Do you have a question?

[Natsheh]

Originally Posted by fysiks Do you have a question?

Yeah how to get top fragers correctly

[edon1337]

You didn't provide enough information, furthermore your code won't even compile.

[Bugsy]

I would use a 2D array loaded with player-id's and frag counts. Then sort the array by frags descending and then your 0-9 indecies are your top 10 fraggers.

Here's an example where you can set the variable number of top to retrieve.

PHP Code:


#include <amxmodx>

new const Version[] = "0.1";

enum _:TopXData
{
    txdUserID,
    txdFrags
}

public plugin_init() 
{
    register_plugin( "TopX Demo" , Version , "bugsy" );
    
    register_concmd( "topX" , "ShowTopX" );
}

public ShowTopX()
{
    TopXFraggers( 10 );
}

public TopXFraggers( iNumFraggers )
{
    new iPlayers[ 32 ] , iNum , iPlayer;
    new iTopXData[ 32 ][ TopXData ] , szName[ 32 ];
    
    get_players( iPlayers , iNum );
    
    for ( new i = 0 ; i < iNum ; i++ ) 
    {
        iPlayer = iPlayers[ i ];
        
        iTopXData[ i ][ txdUserID ] = iPlayer;
        iTopXData[ i ][ txdFrags ] = get_user_frags( iPlayer );
        
        //Can be used for testing instead of get_user_frags()
        //iTopXData[ i ][ txdFrags ] = random_num( 1 , 100 );
    }
    
    SortCustom2D( iTopXData , iNum , "stats_custom_compare" );
    
    iNumFraggers = clamp( iNumFraggers , 1 , iNum );
    
    for ( new i = 0 ; i < iNumFraggers ; i++ )
    {
        get_user_name( iTopXData[ i ][ txdUserID ] , szName , charsmax( szName ) );
        server_print( "%d = %s [ %d frags ]" , i + 1 , szName , iTopXData[ i ][ txdFrags ] );
    }
}

public stats_custom_compare( elem1[] , elem2[] )
{
    if ( elem1[ 1 ] > elem2[ 1 ] ) 
        return -1;
    else if ( elem1[ 1 ] < elem2[ 1 ] )
        return 1;
    
    return 0;
} 


[Natsheh]

Originally Posted by Bugsy I would use a 2D array loaded with player-id's and frag counts. Then sort the array by frags descending and then your 0-9 indecies are your top 10 fraggers. Here's an example where you can set the variable number of top to retrieve. PHP Code: #include <amxmodx> new const Version[] = "0.1"; enum _:TopXData {     txdUserID,     txdFrags } public plugin_init()  {     register_plugin( "TopX Demo" , Version , "bugsy" );          register_concmd( "topX" , "ShowTopX" ); } public ShowTopX() {     TopXFraggers( 10 ); } public TopXFraggers( iNumFraggers ) {     new iPlayers[ 32 ] , iNum , iPlayer;     new iTopXData[ 32 ][ TopXData ] , szName[ 32 ];          get_players( iPlayers , iNum );          for ( new i = 0 ; i < iNum ; i++ )      {         iPlayer = iPlayers[ i ];                  iTopXData[ i ][ txdUserID ] = iPlayer;         iTopXData[ i ][ txdFrags ] = get_user_frags( iPlayer );                  //Can be used for testing instead of get_user_frags()         //iTopXData[ i ][ txdFrags ] = random_num( 1 , 100 );     }          SortCustom2D( iTopXData , iNum , "stats_custom_compare" );          iNumFraggers = clamp( iNumFraggers , 1 , iNum );          for ( new i = 0 ; i < iNumFraggers ; i++ )     {         get_user_name( iTopXData[ i ][ txdUserID ] , szName , charsmax( szName ) );         server_print( "%d = %s [ %d frags ]" , i + 1 , szName , iTopXData[ i ][ txdFrags ] );     } } public stats_custom_compare( elem1[] , elem2[] ) {     if ( elem1[ 1 ] > elem2[ 1 ] )          return -1;     else if ( elem1[ 1 ] < elem2[ 1 ] )         return 1;          return 0; }

Thanks solved by using ArraySort...
Which i didnt know about..