help finishing this program

Quote:

---

when i build and run this is what is displayed on startup:

---

I take this as a question.
The answer:

Code:

---

void DescribeExits(int RoomNumber)
{
  unsigned int exitdir;
  unsigned int exitcount = 0;
  for (exitdir = 0; exitdir < 4; exitdir++)
  {
      if (Room[RoomNumber].Exit[exitdir] > 0) exitcount++;
  }
  if (exitcount == 1) printf("This room has %u exit: \n", exitcount);
  else printf("This room has %u exits: \n", exitcount);
  if (Room[RoomNumber].Exit[0] != NO_EXIT) printf("  North: %s\n", Room[Room[RoomNumber].Exit[0]].Description);
  if (Room[RoomNumber].Exit[1] != NO_EXIT) printf("  North: %s\n", Room[Room[RoomNumber].Exit[1]].Description);
  if (Room[RoomNumber].Exit[2] != NO_EXIT) printf("  North: %s\n", Room[Room[RoomNumber].Exit[2]].Description);
  if (Room[RoomNumber].Exit[3] != NO_EXIT) printf("  North: %s\n", Room[Room[RoomNumber].Exit[3]].Description);
  printf("\n");
}

---

You print North for each exit.
When counting the exits you should compare for != NO_EXIT as well

Kurt

> if (Room[RoomNumber].Exit[0] != NO_EXIT) printf(" North: %s\n", Room[Room[RoomNumber].Exit[0]]
What about Room[Room[Room[ or Room[Room[Room[Room[Room[Room[ ?

There are still a lot of 0 to 3 all over the place.
You've learnt about enum, so maybe

Code:

---

enum {
    North,
    East,
    South,
    West,
    MAX_DIRN
};

---

Then

Code:

---

const char *dirns[] = { "North", "East", "South", "West" };

---

And finally (phew)

Code:

---

for ( dirn = 0 ; dirn < MAX_DIRN ; dirn++ ) {
    if ( Room[RoomNumber].Exit[dirn] != NO_EXIT)
        printf("  %s: %s\n", dirns[dirn], Room[RoomNumber].Description );
}

---