You are onto something. Avoid directly setting the AC, hitroll and damroll, ie. don't use "GET_HITROLL(ch) = 1" because this is what's getting you into trouble.
You might not want to give bonuses on every level up - this will eventually lead to some really OP characters.
Hitroll and damroll bonuses are better when getting higher.
Code:
+int class_xxx_level_bonus[][3] = {
+// HR, DR, AC
+ {1, 0, 0},
+ {0, 0, -5},
+ {0, 1, 0},
+ {1, 0, 0},
+ {0, 0, -5},
+ {0, 1, 0},
+ {1, 0, 0},
+ {0, 0, -5},
+ {0, 1, 0},
+ {1, 0, 0},
+ {0, 0, -5},
+ {0, 1, 0},
+ {1, 0, 0},
+ {0, 0, -5},
+ {0, 1, 0},
+ {1, 0, 0},
+ {0, 0, -5},
+ {0, 1, 0} // expand until you have MAX_LEVEL lines.
+};
/* This function controls the change to maxmove, maxmana, and maxhp for each
* class every time they gain a level. */
void advance_level(struct char_data *ch)
{
int add_hp, add_mana = 0, add_move = 0, i;
add_hp = con_app[GET_CON(ch)].hitp;
switch (GET_CLASS(ch)) {
+ case CLASS_XXX:
+ add_hp += rand_number(10, 15);
+ add_mana = 0;
+ add_move = rand_number(1, 3);
+ GET_HITROLL(ch) += class_xxx_level_bonus[GET_LEVEL(ch)][0];
+ GET_DAMROLL(ch) += class_xxx_level_bonus[GET_LEVEL(ch)][1];
+ GET_AC(ch) += class_xxx_level_bonus[GET_LEVEL(ch)][2];
+ break;
case CLASS_MAGIC_USER:
note how this uses += everywhere? This means that the correct value will be saved in the player files, after subtracting/adding anything the player is wearing or affected by.
