There'sn't nothing wrong with that if you want the game to end after one battle (assuming battle() returns after one battle) or you could use a game loop wich continues untill the player choses to...