//FIGHTING STUFF

void dungeon()
{
srand(GetTickCount());
//BASIC MOVE COMMANDS

char pMove;
int fight(); 
top:

if (player.pX == 5 && player.pY == 5)
{
fight();
}



cout <<"Your Position is ("<<player.pX<<","<<player.pY<<")"<<endl;
    cout <<"---> ";
    cin >> pMove;
    
    switch (pMove)
    {
    case 'l':
    
    player.pX -= 1;
    goto top;
    
    case 'r':
    
    player.pX += 1;
    goto top;
    
    case 'u':
    
    player.pY += 1;
    goto top;
    
    case 'd':
    
    player.pY -= 1;
    goto top;
    
    case 'm':
    
    Draw (↦);
    goto top;
    
    case 'c':
    
    system("cls");
    goto top;
    
    case 'h':
    help();
    break;
    
    case 'q':
    
    char yn[3];
    cout <<"Are you sure your wish to leave? ";
    cin >> yn;
    
    if (yn[0] = 'y')
    {
    system("cls");
    Draw (&tag);
    return;
    }
    
    else 
    {
    goto top;
    }
    
    default:
    
    cout <<" ------INVALID COMMAND------\n";
    
    goto top;
    }
    }

    
    
//BASIC MOVE COMMADS

  
Under invalid command it repeats continously until you close it or until the computer crashes..Also
      
class PERSON
{
    public:
    int pX;
    int pY;
    char pName[8], pClass[8];
    int strength, energy, dex, armor, vit, gold, dam, wep, kills, kill;
};  
  

  // I believe you want to do either this:	if(kill = 5) {		kill = 0; // reset kill		/* update player statistics */	}// or this:	if((kill%5) == 0) {		/* update player statistics */	}  

  void dungeon() { // your dungeon function, modified for better structure	char pMove;	char yn[4] = "yes"; // you had 3, but it should be 4(room for "yes" and null zero)	int fight(); // this doesn''t look right....	// a do-while lood does what you want, just a little differently	do {			if (player.pX == 5 && player.pY == 5) {			fight();		}				cout << "Your Position is (" << player.pX << "," << player.pY << ")" << endl;		cout << "---> ";		cin >> pMove;			switch (pMove) {		case ''l'':			player.pX -= 1;			break; // note the breaks			case ''r'':			player.pX += 1;			break;			case ''u'':			player.pY += 1;			break;			case ''d'':			player.pY -= 1;			break;			case ''m'':					Draw (↦);			break;			case ''c'':			system("cls");			break;			case ''h'':			help();			break; // why did you put break here and nowhere else?			case ''q'':					cout <<"Are you sure your wish to leave? ";			cin >> yn;					if (yn[0] = ''y'') {				system("cls");				Draw (&tag);				return;			}			// don''t need an else with that return			break;			default:			cout <<" ------INVALID COMMAND------\n";			break;		}	}}  

    class PERSON{    public:    int pX;    int pY;    char pName[8], pClass[8];    int strength, energy, dex, armor, vit, gold, dam, wep, kills;    void kill(void); // just like a normal function(so far)};...// later, perhaps after mainvoid PERSON::kill(void) {	kill++; // you killed something!	if((kill % 5) == 0) { // if kill is a multiple of 5          strength += 3; // in a member function, you don't need to say class.variable, just variable		// update everything here	}	// do anything else you want}