Это сравнение строк не работает

Question

Итак, когда я ввожу месяц, который хотел бы использовать, например, декабрь, и я ставлю 722 в качестве часов, программа говорит: «Количество введенных часов не может превышать количество часов в течение месяца 720 ". Есть ли способ исправить? Я не хочу делать заявления за каждый месяц, я чувствую, что есть более простой способ. Это также моя первая программа колледжа

int userPackage, userHours; //Declaring integer variables 
double savings, savings2, total; //Declaring double value 
string userMonth; 
cout<<"\tHello.\nEnter the number of the package you have\n1) Package A\n2) Package B\n3) Package C\n"; //Prompts the user for their package in a menu like fashion 
cin>>userPackage; //gets package 
if(userPackage > 3 || userPackage < 1) //Error output for numbers that don't match packages 
{ 
    cout<<"Error, invalid choice"; 
    return 0; 
} 

cout<<"Enter the number of hours you have been online."; //Propmts the user for the number of hours they've been online 
cin>>userHours; //gets hours 
cout<<"Enter the month (by name): "; 
cin>>userMonth; 
cout<<"\n"; 
if(userMonth == "January","March","May","July","August","October","December") 
{ 
    if (userHours > 744) 
    { 
     cout<<"The amount of hours you entered cannot exceed the amount of hours within the month 744"; 
     return 0; 
    } 
} 
if(userMonth == "April", "June", "September", "November"); 
{ 
    if(userHours > 720) 
    { 
     cout<<"The amount of hours you entered cannot exceed the amount of hours within the month 720"; 
     return 0; 
    } 
} 
if(userMonth == "February"); 
{ 
    if (userHours > 672) 
    { 
     cout<<"The amount of hours you entered cannot exceed the amount of hours within the month 672"; 
     return 0; 
    } 
} 

Answer 1

if(userMonth == "January","March","May","July","August","October","December")

Это не делать то, что вы думаете, что делает (т.е. не сравнить userMonth каждой из строк. (предполагается, что вы также хотите использовать else if, даже если ваш код не соответствует):

if (userMonth == "January" || 
    userMonth == "March" || 
    userMonth == "July" || 
    userMonth == "August" || 
    userMonth == "October" || 
    userMonth == "December") 
{ 
    ... 
} 
else if (userMonth == "April" || 
    userMonth == "June" || 
    userMonth == "September" || 
    userMonth == "November") 
{ 
} 
else if (userMonth == "February") 
{ 
} 

Примечание: Это также чувствительные к регистру сравнения (т. Е. «Январь» не эквивалентен «январю» или другим различиям в случае), и может быть лучше преобразовать все ко всем нижним или всем верхним корпусам.

if(userMonth == "April", "June", "September", "November");
// problematic trailing semi-colon ^

Это завершает if заявление и следующий блок выполняется безоговорочно. Поэтому, когда вводится 722, это всегда больше 720, и вы получаете сообщение, которое видите.

У вас есть эта же ошибка в логике if для «Февраля».