Well, what HK suggested is a lot better, but even so it's Kind-Of-Evil to use magic numbers like that instead of hard-coded constants. If you think that the cleric class is #3, you should hardcode a CLASS_CLERIC constant as 3, and then never refer to '3' but instead of CLASS_CLERIC. Not only will it make it obvious what you actually mean, but if the cleric class number ever changes for some reason, you will only have one place to change, instead of 'x'.