[reinert]

What is the best and most efficient way to detect double kill (a.k.a. 2 kills with 1 bullet)

[Bugsy]

Check timing between 2 kills? If 0 or < 1.0 (or some small value) then double kill. get_gametime() returns float value. You can also check if one weapon fire resulted in multiple kills. Im sure there are plenty of ways.

Untested

PHP Code:

new Float:g_fLastKill[ 33 ];

//When kill is made
g_fLastKill[ id ] = get_gametime();

//do this check first before the above line
if ( ( get_gametime() - g_fLastKill[ id ] ) < 0.25 )
     //double kill? 


[minimiller]

I used something like this a while ago

PHP Code:

#include <amxmodx>
#include <hamsandwich>

#define FUNKYTHING 871

new giKills[33];

public plugin_init()
{
    RegisterHam(Ham_Killed, "player", "fwdPlayerDeath", 0);
}

public fwdPlayerDeath(iVictim, iKiller)
{
    if(giKills[iKiller]++ == 2)
    {
         giKills[iKiller] = 0;
         
         //Your code here
         
         return;
    }
    
    set_task(0.1, "funcResetCount", iKiller + FUNKYTHING);
}

public funcResetCount(id)
{
    giKills[id - FUNKYTHING] = 0;
} 


[reinert]

Do I've to use return in giKills checking ?
If yes why ?

Also, can I check it in

this event:

PHP Code:

register_message(get_user_msgid("DeathMsg"), "Event_DeathMsg") 


[Hunter-Digital]

You could use Bugsy's method to detect serval quick kills, not only double kill by adding a task with 0.26 and removing it at the same time, that way if the player keeps killing people, it won't show "double kill", "multi kill", etc, it will just show the final result if he stopped.

[Bugsy]

I think it should be if ( ++giKills[ iKiller ] == 2 ).

You don't have to return a value but if you want to use HAM_IGNORED. You can use DeathMsg instead. Just use register_event( "DeathMsg" , ...

[reinert]

PHP Code:

register_event("DeathMsg","Event_DeathMsg","a") 


Like this ?

Well I'm using none of the returns only the return PLUGIN_HANDLED; on the end of the public function, and in the beggining where I check if(!is_user_connected(killer))

[Bugsy]

Add "1>0" condition

[minimiller]

i only returned it to stop the set_task doing whats already done

and i think giKills[iKiller]++ and ++giKills[iKiller] will both work

[Bugsy]

You will get different results with each:

Var++ returns the value prior to incrementing, so the first time it'll be 0, second time 1.

++Var returns the value post-increment, first would be 1, second 2