[TandborsteN]

Hello!
Im trying to detect when a player jumps and i want to have so low CPU as possible.
So what way is the best way?

My code:

public client_PreThink(id)
{
    new New_Button = get_user_button(id);
    new Old_Button = get_user_oldbutton(id);
    if((New_Button & IN_JUMP) && !(Old_Button & IN_JUMP) && is_user_alive(id))
    {
        //ACTION CODE
    }
}

[Exolent[jNr]]

Move to FM_CmdStart.
Use get_uc() for buttons instead of get_user_button()
Check if player is on ground (get_entity_flags() & FL_ONGROUND).

[TandborsteN]

Can you PLEASE show me an example?
Doesen't get anything of what you mean.

[Exolent[jNr]]

Search for FM_CmdStart. You can find funciton examples here: http://www.amxmodx.org/funcwiki.php

[TandborsteN]

Okey thank you very much, u are awsome!

[TandborsteN]

So something like this?

public FM_CmdStart_Pre(id, uc_handle)
{
    new button = get_uc(uc_handle, UC_Buttons);
    new oldbutton = pev(id, pev_oldbuttons);
    new flags = pev(id, pev_flags);
    if(button & IN_JUMP && !(oldbutton & IN_JUMP) && flags & FL_ONGROUND)
    {
        //CODE
    }
}

[bibu]

Exolent isn't ham better?

PHP Code:

RegisterHam(Ham_Player_Jump, "player", "Player_Jump") 


[Exolent[jNr]]

Originally Posted by TandborsteN So something like this? public FM_CmdStart_Pre(id, uc_handle) { new button = get_uc(uc_handle, UC_Buttons); new oldbutton = pev(id, pev_oldbuttons); new flags = pev(id, pev_flags); if(button & IN_JUMP && !(oldbutton & IN_JUMP) && flags & FL_ONGROUND) { //CODE } }

Yes.

Originally Posted by bibu Exolent isn't ham better? PHP Code: RegisterHam(Ham_Player_Jump, "player", "Player_Jump")

Ham would be better to do the same check in this case, but you would still have to check it like in FM_CmdStart because it isn't called only when a player jumps.

[abdul-rehman]

Originally Posted by TandborsteN So something like this? public FM_CmdStart_Pre(id, uc_handle) { new button = get_uc(uc_handle, UC_Buttons); new oldbutton = pev(id, pev_oldbuttons); new flags = pev(id, pev_flags); if(button & IN_JUMP && !(oldbutton & IN_JUMP) && flags & FL_ONGROUND) { //CODE } }

Ya but its better if you would make the variables static since this forward sometimes get called very quickly