Hello everyone,

I've been getting frustrated over this code so I thought I'd pick some other people's brains. Essentially my mud has a dual class system that allows the players to add a second class later on in the game, and then switching back and forth between the two (essentially swaping out the class information with a command) - this systems works great for levelling up 2 seperate classes but the problem comes in when someone is say - a warrior / mage. When the warrior class is the primary class they cannot cast spells and vice versa. I've written a piece of code that is also a piece of ---- apparently because its not working. This is under the do_cast section in magic.c

            if (xIS_SET(ch->act, PLR_MULTICLASS)) {
                int class = ch->mclass;
            if ( ( sn = find_spell( ch, arg1, TRUE) ) < 0
            || ( ch->level < skill_table[sn]->skill_level[ch->class] ||
                ch->mclasslvl < skill_table[sn]->skill_level[class]) ) {
                send_to_char( "You have yet to master this spell..\n\r", ch );
                return;  } }

Help would be amazing right about now

What exactly is printed out when warrior-mages try to cast spells?

What does the find_spell function do? .........................
David Haley
Head Coder, Legends of the Darkstone
BabbleMUD Project
http://david.the-haleys.org

gives them the "You have yet to master this spell.."

Find_spell

int find_spell( CHAR_DATA *ch, const char *name, bool know )
{
    if ( IS_NPC(ch) || !know )
        return bsearch_skill( name, gsn_first_spell, gsn_first_skill-1 );
    else
        return ch_bsearch_skill( ch, name, gsn_first_spell, gsn_first_skill-1 );
}
        

which points to ch_bsearch_skill


ch_bsearch_skill_prefix which I added this:

      if(xIS_SET(ch->act, PLR_MULTICLASS)) {
    for (;
    {
     	sn = (first + top) >> 1;
//      int class = ch->mclass;
        
        /* if ( LOWER(name[0]) == LOWER(skill_table[sn]->name[0])
        &&  !str_prefix(name, skill_table[sn]->name)
        &&   ch->pcdata->learned[sn] > 0
        && (  ch->level >= skill_table[sn]->skill_level[ch->class]
        ||   ch->mclasslvl >= skill_table[sn]->skill_level[class] ) )
                return sn; */
                
    if( LOWER(name[0] == LOWER(skill_table[sn]->name[0])
        && !str_prefix(name, skill_table[sn]->name)
        && ch->pcdata->learned[sn] > 0
        && ( ch->mclasslvl >= skill_table[sn]->skill_level[ch->mclass]
        && skill_table[sn]->skill_level[ch->mclass] )) )
                return sn;
            

	if (first >= top)
            return -1;

        if (strcmp( name, skill_table[sn]->name) < 1)
            top = sn - 1;
        else
            first = sn + 1;
    } }

  

        && ( ch->mclasslvl >= skill_table[sn]->skill_level[ch->mclass]
        && skill_table[sn]->skill_level[ch->mclass] )

This is probably what's getting you. Is mclass set to what you expect? .........................
David Haley
Head Coder, Legends of the Darkstone
BabbleMUD Project
http://david.the-haleys.org

Yes, for instance mage is 0 - so either of these should be acceptable

ch->class = 0
ch->mclass = 0

(either one results in being a mage)

Why are they both the same, and are you sure that that is what is happening as you run the code? (E.g., have you tried printing out the variables, or running it through a debugger.)

And the code there only shows mclass -- not class. Where is 'class' coming from? .........................
David Haley
Head Coder, Legends of the Darkstone
BabbleMUD Project
http://david.the-haleys.org

The stock code is there first, then it runs through the second part I put if the multiclass bit has been set. I've ran through the debugger but had no luck... I can try I could print the variables but I'm about 99% sure what they will say... I think for some reason it just won't except ch->mclass in the place of ch->class. That is the part that is giving me a very hard time trying to trace the problem.

When debugging problems you don't understand, make as few assumptions as possible -- if your assumptions were all correct, you wouldn't be having the problem in the first place...!

So go back to basics and verify that all variables involved contain what they're supposed to contain. .........................
David Haley
Head Coder, Legends of the Darkstone
BabbleMUD Project
http://david.the-haleys.org

            if (!xIS_SET(ch->act, PLR_MULTICLASS)) {
            if ( ( sn = find_spell( ch, arg1, TRUE ) ) < 0
            || ( !IS_NPC(ch) && ch->level < skill_table[sn]->skill_level[ch->class] ) )
            {
                send_to_char( "You aren't ready to do this.\n\r", ch );
                return;
            } }
    
            if (xIS_SET(ch->act, PLR_MULTICLASS)) {
            if ( ( sn = find_spell( ch, arg1, TRUE ) ) < 0
             || (( !IS_NPC(ch) && ch->level < skill_table[sn]->skill_level[ch->class]
             &&  !IS_NPC(ch) && ch->mclasslvl < skill_table[sn]->skill_level[ch->mclass] )) )
            {
                        send_to_char( "You aren't ready to do this.\n\r", ch );
    
                //if (xIS_SET(ch->act, PLR_MULTICLASS))
                //        pager_printf(ch, "debug: not working...\n\r";
    
                return;
            } }

I got it! Damn stupid syntax will kill you every time! I should have been using an && instead of an ||

That is not code you had pasted before... having the actual code you're looking at usually makes debugging easier for us. .........................
David Haley
Head Coder, Legends of the Darkstone
BabbleMUD Project
http://david.the-haleys.org