сравнение с строковых литералов результатов в неопределенном поведении, ошибка

Question

#include <iostream> //include header files 

using namespace std; 

int main() //start of main body 

{ 

int num; //declaring integer 

int control=1; //declaring integer 

while(control==1)//start of loop, as long as condition is true 
{ 
    cout << "Press 1 for coffee" << endl; //writes to standard o/p 
    cout << "Press 2 for tea" << endl; 
    cout << "Press 3 for hot chocolate" << endl; 
    cout << "Press 4 for soft drink" << endl; 
    cout << "Press 5 to exit" << endl; 
    cin >> num; 


if (num == 1) //code to execute if integer = 1 
{ 
    cout << "coffee selected" << endl; 
} 
else if (num == 2) //code to execute integer = 2 
{ 
    cout << "tea selected" << endl; 
} 
else if (num == 3) //code to execute if integer = 3 
{ 
    cout << "hot chocolate selected" << endl; 
} 
else if (num == 4) //code to execute if integer = 4 
{ 
    cout << "soft drink selected" << endl; 
} 
else if (num == 5) //code to execute if integer = 5 
{ 
    cout << "Exit Program" << endl; 
    control=0; 
} 


} 

}

вот мой ammended код это работает. Однако я не был уверен в инициализации целого числа num, поэтому я оставил его, но код все еще выполняется и работает правильно.

Answer 1

Вы сравниваете целые числа с строками здесь num == "1". Вместо этого используйте num == 1

Answer 2

Ошибка в том, что вы сравниваете в int (num), строковые литералы, такие как "1". Этот конкретный строковый литерал имеет тип const char[2], который распадается на const char*, следовательно, ошибка компилятора.

Вы должны сравнивать числа с числами, например

if (num == 1) { .... } 

Answer 3

Вы пытаетесь сравнить номер с строкового литерала. Не хороший план. Попробуйте

if(num==1)

не

if(num=="1")

Во-вторых, num не определено. Попробуйте установить для него значение.

Компилятор C++ даст вам ошибки, поскольку вы не можете сравнивать два разных типа данных. Подробнее о сравнении here

Answer 4

Вам не нужно, чтобы menu был string, так как вы просите пользователей ввести целые числа. Просто используйте

if (num == 1) 

не

if (num == "1") 

Answer 5

целое число сравниваемых со строкой :)

num == "1" 

Не допускается, кстати я думаю, что вместо "номер" вы намеревались "меню", который является строкой?

Answer 6

сначала, ваш номер неинициализирован. , когда вы будете это изменить, то и изменить сравнение с num == "1" к num == 1

В настоящее время вы кладете вход в menu переменных, когда вы делаете getline (cin, menu, '\n'); так что вы должны изменить это, если вы хотите сохранить вход в num.

рядом, это очень хороший код, я бы выбрал 4)