Confirming some script

[Zagreuz]

just want to confirm if this is the right syntax for the item script.

bonus bAllStats,10; if(BaseClass == Job_Sniper) { bonus2 bSubRace,RC_DemiHuman,30; bonus2 bSkillAtk,"SN_SHARPSHOOTING",300; } else if(BaseClass==Job_Bard || BaseClass == Job_Clown) { bonus2 bSubRace,RC_DemiHuman,10; bonus2 bSkillAtk,"CG_ARROWVULCAN",300;

its a helm that was use for sniper and clown , i want to make it increase the skill damage, by using if & else

[Mystery]

You can always make the item only available for Archer class. Thus, changing to :

bonus bAllStats,10; if(BaseClass == Job_Sniper) { bonus2 bSubRace,RC_DemiHuman,30; bonus2 bSkillAtk,"SN_SHARPSHOOTING",300; } else { bonus2 bSubRace,RC_DemiHuman,10; bonus2 bSkillAtk,"CG_ARROWVULCAN",300; }

_{There are other ways to change the above method.}

Other then that, try your way.

Edited December 23, 2011 by Mysterious

[Zagreuz]

well, i already make the item exclusive to sniper's and clowns, so is my problem at the "else if" change to "else" only ?

[JayPee]

I guess your just missing a closing curly brace

bonus bAllStats,10; if(BaseClass == Job_Sniper) { bonus2 bSubRace,RC_DemiHuman,30; bonus2 bSkillAtk,"SN_SHARPSHOOTING",300; } else if(BaseClass==Job_Bard || BaseClass == Job_Clown) { bonus2 bSubRace,RC_DemiHuman,10; bonus2 bSkillAtk,"CG_ARROWVULCAN",300};

[Zagreuz]

this is the whole line of the script

{ bonus bAllStats,10; if(BaseClass == Job_Sniper) { bonus2 bSubRace,RC_DemiHuman,30; bonus2 bSkillAtk,"SN_SHARPSHOOTING",300; } else if(BaseClass==Job_Bard || BaseClass == Job_Clown) { bonus2 bSubRace,RC_DemiHuman,10; bonus2 bSkillAtk,"CG_ARROWVULCAN",300; } },{},{}

still having a missing curly brace ?

Edited December 23, 2011 by LordOfHeRO

[JayPee]

I dont know if is really BaseClass exist. the only Code i found in the script commands that check jobs is

BaseJob

This is sneaky, apparently meant for baby class support. This will supposedly equal Job_Acolyte regardless of whether the character is an acolyte or a baby acolyte, for example.

and

Class

Current Job

Edited December 23, 2011 by JayPeeMateo

[Sneaky]

BaseClass doesn't exist; Should be Class instead.

here:

{ bonus bAllStats,10; if(Class == 4012) { bonus2 bSubRace,RC_DemiHuman,30; bonus2 bSkillAtk,"SN_SHARPSHOOTING",300; } else if(Class == 19 || Class == 4020) { bonus2 bSubRace,RC_DemiHuman,10; bonus2 bSkillAtk,"CG_ARROWVULCAN",300; } },{},{}

4012 = sniper

19 = bard

4020 = clown

You can check the ids in db/const.txt even though using job_class will still work.

[Zagreuz]

say . . . its the BaseClass problem ey ! , i try Slim info first ! , thanks Slim and JayPeeMateo

p/s : it works , thanks Guys ! , really been helping, topic close i guess ?

Edited December 24, 2011 by LordOfHeRO

[Emistry]

BaseClass doesnt Exist...

but BaseJob Exist ~

BaseJob     - This is sneaky, apparently meant for baby class support.
             This will supposedly equal Job_Acolyte regardless of whether the 
             character is an acolyte or a baby acolyte, for example.

to check current Class..we use..

Class       - Current job

[Sneaky]

say . . . its the BaseClass problem ey ! , i try Slim info first ! , thanks Slim and JayPeeMateo

p/s : it works , thanks Guys ! , really been helping, topic close i guess ?

No problem, I won't close it but, I'll mark it as Solved instead.