PHP Code:
for(new i=0;i<3;i++)
sOrigin[id][i] = Origin[i]
// ->
// this works at least in amxx 1.8, former versions give you a tagmismatch if i remember correct
sOrigin[id] = Origin
also, stuck always used to be the former checkpoint you created. imagine falling down. your near to the finish, fall, gocheck and get stucked. you say /stuck and get teleported to the start. look again at paddys code, he also made the last cp the stuck origin before a saved the new cp origin on creating a checkpoint.
furthermore you could add mroe features for your checkpoints like checkpointing while ducking and/or disallowing it in the air. there are a few kz maps where you can jump down somewhere, checkpoint->gocheck to reset your velocity and lan safly (shortcut).
PHP Code:
// connorr tested this and found out that it actually works
// for checkpoint while being ducked
new const Float:VEC_DUCK_HULL_MIN[3] = { -16.0, -16.0, -18.0 };
new const Float:VEC_DUCK_HULL_MAX[3] = { 16.0, 16.0, 18.0 };
stock set_user_ducking( plr )
{
set_pev( plr, pev_flags, pev( plr, pev_flags ) | FL_DUCKING );
engfunc( EngFunc_SetSize, plr, VEC_DUCK_HULL_MIN, VEC_DUCK_HULL_MAX );
}
// for not checkpointing while being in air
if( !( pev( plr, pev_flags ) & FL_ONGROUND ) )
you do not reset checked[id] which means a new player might be able to use another palyers checkpoints.
the noblock var itself is unnecessary in your plugin. you set it twice to true (putinserver and disconnect) and the only time is used in your plugin is the following:
PHP Code:
// since noblock is always true your semiclip cvar has no effect on the var
if(get_pcvar_num(cvar_kzmenu_semiclip) == 1 || noblock[x])
also, cuz you added a cvar for semiclip you should add a check in gocheck whether the gocheckorigin is near to another player so the player wont get stucked (paddy does this in his plugin).
//edit: you check the cvar for semiclip twice in the semiclip task which means your cvar really works
PHP Code:
if(get_pcvar_num(cvar_kzmenu_semiclip) == 1 || noblock[x])
{
// dont delete the code inside which is done here but the if/else clause becomes unnecessary
}
else if(!solid[x])
{
set_pev(x, pev_solid, SOLID_BBOX);
}
// stop edit
PHP Code:
// the condition inside a loop (in this case i<= get_maxplayers() should always be cached
for(new i=1;i<=get_maxplayers();i++)
// ->
for(new i=1;i<=g_iMaxPlayers;i++)
// g_iMaxPlayers should be a global var (you could use a static in here too but since youre new to coding... forget about a static :D)
// if you leave your code as it is currently the for loop will work like this
create var i
set var to 1
get_maxpalyers()
check whether condition is true
run the code inside the loop
increment i (set var to i+1)
get_maxpalyers()
check whether condition is true
run the code inside the loop
increment i (set var to i+1)
get_maxpalyers()
check whether condition is true
run the code inside the loop
increment i (set var to i+1)
get_maxpalyers()
check whether condition is true
run the code inside the loop
increment i (set var to i+1)
...
// as you can see, get_maxplayers() is done a lot of times, caching it will leave this out and only check whetehr the contion is true
kinda good for your first plugin
__________________
Linear Mode
