CODEWITHSUNDEEP
c++
softengstu
softengstu

Reputation: 57

C++ switch always goes to default

When I use switch it always prints the default value, regardless of the value entered. I included break after every case statement. How can I fix it?

// Lab 4 color.cpp
// This program lets the user select a primary color from a menu. 
#include <iostream>
#include <string>

using namespace std;

int main()
{
int choice;    // Menu choice should be 1, 2, or 3

               // Display the menu of choices
cout << "Choose a primary color by entering its number. \n\n";
cout << "1 Red \n" << "2 Blue \n" << "3 Yellow \n";

// Get the user's choice
cin >> choice;

// Tell the user what he or she picked
switch (choice)
{
case '1': cout << "\nYou picked red.\n";
    break;
case '2': cout << "\nYou picked blue.\n";
    break;
case '3' : cout << "\nYou picked yellow.\n";
    break;
default: cout << "You entered an invalid option. Please run again and choose 1, 2, or 3.\n";
    break;
} 

return 0;
}

Upvotes: 0

Views: 1445

Answers (1)

Ed Heal
Ed Heal

Reputation: 60017

In your select, your case is a character but you are selecting on an integer.

So change

case '1'

to

case 1

Alternatively change

int choice;

to 

char choice;

etc.

Upvotes: 4

Related Questions