This code is not modular and if you more than, let's say 30 different warriors, your fight function will be too long and complex. Try Adding to Warrior "attack" function and its subclasses if needed. More