Three questions:
1. Why not use a while statement instead of a for..next statement?
2. Is need_to_choose_another a function or variable? if a varibale how
does it decide which quests to check?
3. The way the code is written now, if I remember corretly, you will
never reach your else statements. Only the first line after an
unbracketed if or else statement is read when that statement is true, if
I remember right. When your if statement is false it skips
need_to_choose another and hits your break statement. Here are two ways
to fix:
switch(number_range(0,1))
{
case 0:
if (this_quest_completed) { /*Add bracket */
need_to_choose_another;
break;
} /*Add bracket */
else { /*Add bracket */
please_go_kill_mob_a;
break;
} /*Add bracket */
case
OR:
switch(number_range(0,1))
{
case 0:
if (this_quest_completed)
need_to_choose_another; /*delete offending break */
else
please_go_kill_mob_a;
break; /* Reposition to reflect code */
case 1:
if (this_quest_completed) /*delete offending break */
need_to_choose_another;
else
please_go_give_item_to_mob_b;
break; /* Reposition to reflect code */
}
--
Jonathan L. Potter
Snafu Life.
Shoddy-Hack Artists for Life.
01000101 -- go ahead take a byte ;)
