Checking a player's job?

[michaelsoftman]

I was just wondering if there's an easier way to check a player's job in the source than what I've been using, which is this, for example, to determine if they're 3rd classes:

if( bl->type == BL_PC && pc_mapid2jobid(((TBL_PC*)bl)->class_, ((TBL_PC*)bl)->status.sex) >= JOB_RUNE_KNIGHT )

But I would like to know if there's an easier way to just determine a single class, or will I have to write this statement out for every class I plan to check for?

Edited January 2, 2013 by michaelsoftman

[ICEROCK]

check jobs for what? if its an npc theres an easy way.

[michaelsoftman]

It's not an NPC, it's a source edit.

I'm messing around with the Dark Collector class (turning it into a usable class out of boredom) and so I configuring class specific drops for them. I could do

if( bl->type == BL_PC && pc_mapid2jobid(((TBL_PC*)bl)->class_, ((TBL_PC*)bl)->status.sex) == JOB_DARK_COLLECTOR )

I just want to know if this is the most efficient way or not. I'm not too familiar with some of the source code commands.

[CandyCandy]

I was just wondering if there's an easier way to check a player's job in the source than what I've been using, which is this, for example, to determine if they're 3rd classes:

if( bl->type == BL_PC && pc_mapid2jobid(((TBL_PC*)bl)->class_, ((TBL_PC*)bl)->status.sex) >= JOB_RUNE_KNIGHT )

But I would like to know if there's an easier way to just determine a single class, or will I have to write this statement out for every class I plan to check for?

Try this?

if((sd->class_&MAPID_BASEMASK) == MAPID_DARK_COLLECTOR)

or this

if(sd->class_ == MAPID_DARK_COLLECTOR)

Saw lot's of those in skill.c checks.

Edited January 2, 2013 by CandyCandy

[michaelsoftman]

if(sd->class_ == MAPID_DARK_COLLECTOR)

Seems to be working perfectly for me, thanks a lot!