If else statement won't work.

[Jack816]

No matter what I put in for "room" it outputs "the number of students exceeds the limit." Could someone explain why it is doing that?

  #include <iostream>

using namespace std;

int main()
{
  const int LH312 = 42;
  const int CAS41 = 44;
  const int KH133 = 38;
  int room;
  int amountOfPeople;
  {//Gathers room number and amount of people
      cout << "Choose a room" << endl;
      cout << "1. Leigh Hall Room 312" << endl;
      cout << "2. College of Arts and Sciences Room 41" << endl;
      cout << "3. Kolbe Hall Room 133" << endl;
      cout << "Option:";
      cin >> room;
      cout << "How many people are in the room?" << endl << "Number of People:";
      cin >> amountOfPeople;
  }
  if (room == 1)
  {
      if (amountOfPeople <= LH312)
      {
         cout << "It is legal to hold your class";
      }
      else (amountOfPeople > LH312);
      {
          cout << "The amount of students exceeds the limit";
      }
  }
  else if (room == 2)
  {
      if (amountOfPeople <= CAS41)
      {
          cout << "It is legal to hold your class";
      }
      else (amountOfPeople > CAS41);
      {
          cout << "The amount of students exceeds the limit";
      }
  }
  else (room == 3);
  {
      if (amountOfPeople <= KH133)
      {
          cout << "It is legal to hold your class";
      }
      else (amountOfPeople > KH133);
      {
          cout << "The amount of students exceeds the limit";
      }
  }
    return 0;
}

[Peter87]

The else part of an if statement should not have a condition.

else (amountOfPeople > LH312);

[Jack816]

Thanks that partially solved my problem. For some reason the if-else outputs the text twice.

[Jack816]

#include <iostream>

using namespace std;

int main()
{
  const int LH312 = 42;
  const int CAS41 = 44;
  const int KH133 = 38;
  int room;
  int amountOfPeople;
  {//Gathers room number and amount of people
      cout << "Choose a room" << endl;
      cout << "1. Leigh Hall Room 312" << endl;
      cout << "2. College of Arts and Sciences Room 41" << endl;
      cout << "3. Kolbe Hall Room 133" << endl;
      cout << "Option:";
      cin >> room;
      cout << "How many people are in the room?" << endl << "Number of People:";
      cin >> amountOfPeople;
  }
  if (room == 1)
  {
      if (amountOfPeople <= LH312)
      {
         cout << "It is legal to hold your class";
      }
      else
      {
          cout << "The amount of students exceeds the limit";
      }
  }
  else if (room == 2)
  {
      if (amountOfPeople <= CAS41)
      {
          cout << "It is legal to hold your class";
      }
      else
      {
          cout << "The amount of students exceeds the limit";
      }
  }
  else (room == 3);
  {
      if (amountOfPeople <= KH133)
      {
          cout << "It is legal to hold your class";
      }
      else
      {
          cout << "The amount of students exceeds the limit";
      }
  }
    return 0;
}

[Jack816]

That didn't work. I put that semi-colon there because the following line gets the error: "expected ';' before '{' token."

[Jack816]

That was the issue. I thought you were supposed to use "else" before the last condition instead of "else if." Thank you for the help.

[Peter87]

You could add another else if you want to print something in case the user inputs a number that is not equal to one of 1, 2 or 3.

...
else if (room == 3)
{
	...
}
else
{
	cout << "Sorry, we don't have such a room.";
}

[Farshad Aroon]

Why you didn't use Switch-case? Its faster and more readable.

switch(room){
case 1:
    if (amountOfPeople <= LH312)
        cout << "It is legal to hold your class";
    else
        cout << "The amount of students exceeds the limit";
    break;

case 2:
    if (amountOfPeople <= CAS41)
        cout << "It is legal to hold your class";
    else
        cout << "The amount of students exceeds the limit";
    break;

case 3:
    if (amountOfPeople <= KH133)
        cout << "It is legal to hold your class";
    else
        cout << "The amount of students exceeds the limit";
    break;
default:
    cout<<"The input was invalid!"<<endl;
}