Code Reference

[ZERO]

This thread archives tips and tricked used in coding war3 races to help improve efficiency of the code used and thus reduce lag, improve performance and prevent crashes. As no true guide serves for making war3 races this thread serves to place important code snippets in one place on a post by post basis.

---------- Post added at 03:04 PM ---------- Previous post was at 02:56 PM ----------

Get weapon to prevent ability damage stacking, use for all ability's that require hit detection regardless of internal function use for consistency:

OLD

public PlayerHurtEvent(Handle:event,const String:name[],bool:dontBroadcast)
{
    decl String:weapon[64];
    GetEventString(event,"weapon",weapon,63);

CORRECT

public OnWar3EventPostHurt(victim,attacker,dmgamount)
{
    decl String:weapon[64];
    GetEventString(W3GetVar(SmEvent),"weapon",weapon,63);

Advantages:
Removes the need for a hook event like:

HookEvent( "player_hurt", PlayerHurtEvent );

Also removes the need for all of the following code:

new userid=GetEventInt(event,"userid");
        new attacker_userid=GetEventInt(event,"attacker");
        
        new victim=GetClientOfUserId(userid);
        new attacker=GetClientOfUserId(attacker_userid);
        
        new dmg=GetEventInt(event,"dmg_health");

Lastly as seen it additionally removes the problem of dmg being a Float value and thus the need for RoundToFloor() to be used to convert back to an Integer.

[ZERO]

This is the method to be used with timers to prevent errors:

In Variable Declaration:

//timers
new Handle:NewTimer[MAXPLAYERS+1];

In Program Include this Method: Must be included to remove the timer to prevent errors

//CLEAR TIMERS
public OnClientDisconnect(client)
{
    new Handle:Timer;
    Timer = NewTimer[client];
    if(Timer != INVALID_HANDLE)
    {
        KillTimer(NewTimer);
        Timer = INVALID_HANDLE;
    }
}

In Method Using Timer: This is used to create the timer

NewTimer[client] = CreateTimer(LENGTH_OF_TIMER,NAME_OF_METHOD,GetClientUserId(client));

In Program Include this Method: Place code to run when timer ends within this method too

public Action:NAME_OF_METHOD(Handle:timer,any:userid)
{
    //Get the current client of the target userid
    new client = GetClientOfUserId(userid);
    
    //Code to run when timer ends
    Timer[client] = INVALID_HANDLE;
    
    //If client is a valid player
    if(client!=0)
    {
        
    }
}

New userid based tracking ensures that the client target at the end of the timer is the intended one. This can protect against crashes related to a client reference value changing unexpectedly.

[ZERO]

This is a better order for if statemets in OnWar3EventPostHurt

//Hit based abilities
public OnWar3EventPostHurt(victim,attacker,dmg)
{
    //Better order:
    
    //Confirm valid trigger
    if(victim>0&&attacker>0&&victim!=attacker)
    {
        //Confirm that it is for this race
        new race_attacker=War3_GetRace(attacker);
        if(race_attacker==thisRaceID && !Hexed(attacker,false) && !W3HasImmunity(victim,Immunity_Skills))
        {
            decl String:weapon[64];
            GetEventString(W3GetVar(SmEvent),"weapon",weapon,63);
            
            //Will only trigger from things not listed below
            if(!StrEqual(weapon,"hegrenade",false)&&!StrEqual(weapon,"athena_relm",false)&&!StrEqual(weapon,"athena_spear",false))
            {
                //abilities below
            }
        }
    }
}

Advantages: Causes less if statements to be read on normal event triggers. The ones that are read are less costly to check. Thus the speed of the overall function should be improved.

[ZERO]

Below is the switch statements for optimal dmg stack prevention on hit based abilities:

//Binary check for mult skill trigger
                new skillcheck = 0;
                
                //Relm Level
                new relm_level=skill_level_relm[attacker];    
                
                //Spear Level
                new spear_level=skill_level_spear[attacker];
                
                //Relm Prop
                if(relm_level>0 && Math_GetRandomFloat( 0.0, 1.0 ) <= RelmChanse[relm_level])
                {
                    skillcheck+=1;
                }
                
                //Spear Prop
                if(spear_level>0 && Math_GetRandomFloat( 0.0, 1.0 ) <= SpearChanse[spear_level])
                {
                    skillcheck+=2;
                }
                
                switch (skillcheck)
                {
                    //Runs if Relm was selected
                   case 1:
                   {
                      Relm(victim,attacker,dmg,relm_level);
                   }
                   //Runs if Spear was selected
                   case 2:
                   {
                      Spear(victim,attacker,dmg,spear_level);    
                   }
                   //Runs if both were selected and selects 1
                   case 3:
                   {
                        new chance = Math_GetRandomInt( 1, 2 );
                        switch (chance)
                        {
                            //Runs if Relm was selected
                            case 1:
                            {
                                Relm(victim,attacker,dmg,relm_level);
                            }
                            //Runs if Spear was selected
                            case 2:
                            {
                                Spear(victim,attacker,dmg,spear_level);
                            }
                            //Checks for erros
                            default:
                            {
                                PrintToChat(attacker,"[ERROR] Out of bounds vale for stacked skill, please report.");
                            }
                        }
                    }
                }

Note that in this code the abilities themselves are methods which are called by the appropriate switch. An example of a method is below:

public Relm(any:victim,any:attacker,any:dmg,any:skill_level)
{
//skill code here
}

This methodology makes it impossible for the dmg stack prevention to effect the probability of a skill actually occurring. Also the use of switches dramatically improves performance which is especially important for a function that can be called very often. Also note the use of && vs nested ifs for when a new variable creation is not needed. This reduces the number of calls.

[ZERO]

Below is the best method for tracking the level of a given ability:

//Better tracking of current level for increased performance!
public OnSkillLevelChanged(client,race,skill,newskilllevel)
{
    if(race==thisRaceID)
    {
        if(skill==SKILL_RELM) //1
        {
            skill_level_relm[client]=newskilllevel;
        }
        else if(skill==SKILL_SPEAR) //2
        {
            skill_level_spear[client]=newskilllevel;
        }
        else if(skill==SKILL_IMMU) //3
        {
            skill_level_immu[client]=newskilllevel;
        }
        else if(skill==SKILL_HELM) //4
        {
            skill_level_helm[client]=newskilllevel;
        }
        else if(skill==ULT_GAZE) //5
        {
            skill_level_gaze[client]=newskilllevel;
        }
    }
}

I would use a switch if I could but the variables are not constant (compiler throws error). Optimizing this function is very important as it is called at the start of every round, when ever a player changes races and when ever a players skill level changes.

[ZERO]

Below is the code NOT to be used for removing weapons:

if( ValidPlayer( client, true ) )
        {
            new wep1 = GetPlayerWeaponSlot( client, CS_SLOT_PRIMARY );
            
            if( IsValidEdict( wep1 ) )
            {
                CS_DropWeapon(client, wep1, true, true);
                AcceptEntityInput(wep1, "Kill");
            }
}

This removes the need for the bad:

W3DropWeapon(client,ent);
                UTIL_Remove(ent);

Use stocks from zerostocks instead such as:

KnifeOnly(client);

[ZERO]

Color table: https://www.doctormckay.com/morecolors.php