[Aooka]

Hello
I do a code :

#include < amxmodx >


public plugin_init( )
{
    register_plugin( "Score information" , "1.0" , "Aooka" );
}

public ScoreMsg( victim , attacker )
{
    if( is_user_alive( attacker ) || !is_user_alive( attacker ) && !is_user_alive( victim ) )
    {
        new szPseudoVictim[ 32 ] , szPseudoAttacker[ 32 ];
        
        get_user_name( victim , szPseudoVictim , charsmax( szPseudoVictim ) );
        get_user_name( attacker , szPseudoAttacker , charsmax( szPseudoAttacker ) );
        
        client_print( victim , print_center , "You were killed by %s" , szPseudoAttacker );
        client_print( attacker , print_center , "You killed %s" , szPseudoVictim );
    }
}

I wanted to know if it was correct and well optimized.

Thanks^^

[<VeCo>]

What is "ScoreMsg"?

[Aooka]

It's just the name of my function. I did not know what to choose ^^ Why ?

[Arkshine]

ScoreMsg comes from where ? You don't provide the full code.

[Aooka]

No it's really the complete code. I assure you.
I took a random function name

[Aooka]

So ... I just change the function name but ...

#include < amxmodx >


public plugin_init( )
{
    register_plugin( "Score information" , "1.0" , "Aooka" );
}

public ScoreInfo( victim , attacker )
{
    if( is_user_alive( attacker ) || !is_user_alive( attacker ) && !is_user_alive( victim ) )
    {
        new szPseudoVictim[ 32 ] , szPseudoAttacker[ 32 ];
        
        get_user_name( victim , szPseudoVictim , charsmax( szPseudoVictim ) );
        get_user_name( attacker , szPseudoAttacker , charsmax( szPseudoAttacker ) );
        
        client_print( victim , print_center , "You were killed by %s" , szPseudoAttacker );
        client_print( attacker , print_center , "You killed %s" , szPseudoVictim );
    }
}

[hornet]

So what's the question here?

Your function needs to be executed to actually do something.

[kramesa]

Post the full code man.

Edit: Well, try this:

#include < amxmodx >
#include < csx >

public client_death( attacker , victim ) {
    new szPseudoVictim[ 32 ] , szPseudoAttacker[ 32 ];
    
    get_user_name( victim , szPseudoVictim , charsmax( szPseudoVictim ) );
    get_user_name( attacker , szPseudoAttacker , charsmax( szPseudoAttacker ) );
    
    client_print( victim , print_center , "You were killed by %s" , szPseudoAttacker );
    client_print( attacker , print_center , "You killed %s" , szPseudoVictim );
}

[Kreation]

Originally Posted by Aooka No it's really the complete code. I assure you. I took a random function name

It's the correct code, up until the function is called, because it's never called so it will never execute. Thus, it will never do anything.

[Aooka]

I forgot my register_event ( lol ) :

#include < amxmodx >


public plugin_init( )
{
    register_plugin( "Score information" , "1.0" , "Aooka" );
    
    register_event( "DeathMsg" , "HookDeath" , "d" ); 
}

public HookDeath( victim , attacker )
{
    if( is_user_alive( attacker ) || !is_user_alive( attacker ) && !is_user_alive( victim ) )
    {
        new szPseudoVictim[ 32 ] , szPseudoAttacker[ 32 ];
        
        get_user_name( victim , szPseudoVictim , charsmax( szPseudoVictim ) );
        get_user_name( attacker , szPseudoAttacker , charsmax( szPseudoAttacker ) );
        
        client_print( victim , print_center , "You were killed by %s" , szPseudoAttacker );
        client_print( attacker , print_center , "You killed %s" , szPseudoVictim );
    }
}

So I think the code is now good ( sorry ^^ ).

And I think it's the same with this code :

#include < amxmodx >
#include < csx >

public client_death( attacker , victim ) {
    new szPseudoVictim[ 32 ] , szPseudoAttacker[ 32 ];
    
    get_user_name( victim , szPseudoVictim , charsmax( szPseudoVictim ) );
    get_user_name( attacker , szPseudoAttacker , charsmax( szPseudoAttacker ) );
    
    client_print( victim , print_center , "You were killed by %s" , szPseudoAttacker );
    client_print( attacker , print_center , "You killed %s" , szPseudoVictim );
}