[ghost95v]

Hello, i tested a simple plugin on my server, but im in doubt if this part of code (below) its wrong for "get_user_name" for sender, and target of Cmd...

also i got a Crash, when i runned this plugin.

PHP Code:

new TargetName[32], szName[32]
    get_user_name(id, szName, 31)  // im in doubt if i can user twice "get_user_name" like this case?
    get_user_name(TargetPlayer, TargetName, charsmax(TargetName))
    // the function to give health here...
    
    DoSayTextMessage(0, "^x04[CS]^x01 [ADMIN]^x04 %s^x01 gaved Health Pack to^x03 %s", szName, TargetName) 


[Natsheh]

Post the full code?

No need to be shy we all men here?

[HamletEagle]

You can use as many get_user_name as you want, the crash is not from that. You should use charsmax for szName string-array size as you do for TargetName, apart of that everything is fine.
The crash is not from getting the name, you should post your full code or start debugging in order to find what's wrong. We don't know how your DoSayTextMessage works so it may cause a crash aswell.

So, post more code if you want help.

[ghost95v]

Originally Posted by Natsheh No need to be shy we all men here?

Ive liked girls to be here...

Ok, well...

here its the part of registered concmd

PHP Code:

public plugin_init() {
        register_concmd("amx_example", "cmdDo_it", ADMIN_CFG, "<target>");
} 


here its the part of the cmdDo_it

PHP Code:

public cmdDo_it(id, level, cid)
{
    if(!cmd_access(id, level, cid, 2))
    {
        console_print(id, "CS] Access is denied!")
        return PLUGIN_HANDLED
    }
        
    new iTarget[32]
    read_argv(1, iTarget, charsmax(iTarget))
    
    new TargetPlayer = cmd_target(id, iTarget, 2)
    
    if(!TargetPlayer)
    {
        console_print(id, "[CS] Cannot find that name, be more specific!")
        return PLUGIN_HANDLED
    }
    
    if(!is_user_alive(TargetPlayer))
    {
        console_print(id, "[CS] Cannot give to dead player!")
        return PLUGIN_HANDLED
    }
    
    if(read_argc() != 2)
    return PLUGIN_HANDLED
        new TargetName[32], szName[32] 
        get_user_name(id, szName, 31)  // im in doubt if i can user twice "get_user_name" like thiscase? 
        get_user_name(TargetPlayer, TargetName, charsmax(TargetName)) 
        // the function to give health here... 
     
         DoSayTextMessage(0, "^x04[CS]^x01 [ADMIN]^x04 %s^x01 gaved Health Pack to^x03 %s", szName, TargetName)  
} 


here its the part of "DoSayTextMessage", other parts are Alright, ive runned always fine.

PHP Code:

DoSayTextMessage( iTarget, cMessage[ ], any: ... )
{
    static cBuffer[ 192 ];
    vformat( cBuffer, 191, cMessage, 3 );

    message_begin( MSG_ONE_UNRELIABLE, gmsgSayText, _, iTarget );
    write_byte( iTarget );
    write_string( cBuffer );
    message_end( );
} 


[Natsheh]

The parameters of say text msg ( iTarget, const cMessage[], ...)

y forget to add const to the sec parameter!

[ghost95v]

Originally Posted by Natsheh The parameters of say text msg ( iTarget, const cMessage[], ...) y forget to add const to the sec parameter!

the both variables "iTarget" and "cMessage" are const's... not needed to add const
its same shit, like you do "new const" when you can do just const