What's wrong with my switch function?

Hi,
I am very new to C++... in fact I started last night at 10:00. I started so I can make a extra credit function for math (if anyone wants to help me out with that, email me). Anyways, I am having alot of fun, but I can't seem to get "switch" statements to work. Whenever I input the selection of the case I want to run, the window crashes.
Here's my source code of a simple program I was trying to write:

Code:

---

#include <iostream>
using namespace std;
int main()
{
  int x;
  int y;
  int selection;
  cout<<"Please enter 2 numbers: ";
  cin>> x >> y;
  cin.ignore();
  cout<<"What would you like to do with these numbers:\n";
  cout<<"1. Add them\n";
  cout<<"2. Subtract them\n";
  cout<<"Selection: ";
  cin>> selection;
  switch (selection) {
        case 1:
              return x + y;
              break;
        case 2:
              return x - y;
              break;
        default:
              cout<<"Error\n";
              break;
              } 
  cin.get();
}

---

I got a simpler switch statement to work, but whenever I try more advanced stuff, it crashes. Can someone help me? Am I missing a measly equal sign or period somewhere?

Thanks!

P.S. I'm using Dev-C++ 5

Since you're main() function is an int, it will terminate when you return an integer. It returns, though, to the system, not the output stream. You want:

Code:

---

case 1:
    cout << x + y;
    break;
case 2:
.
.
.

---

That way, the result will be printed to the output screen instead.
Then, when you want the program to terminate, use

Code:

---

return 0;

---

at the end of int main() to signifiy a successfull run of the program.

Why don't you show us this "more advanced stuff"

Of course I have to question why you are returning in your switch statement when it is inside of main(). Since it'll most likely not be zero you are returning you are telling the OS that an error occured in your program.

thank you, decrypt. But I replaced return with cout and the window still crashes.

Quote:

---

Originally Posted by Patton

Hi,
I am very new to C++... in fact I started last night at 10:00.

---

There is no reason that you should be doing programs with switch statements if you're that new. Don't read through the tutorials and immediately say "I understand it, next" and go onto the next tutorial. It's clear in your program that you still don't know many of the basics. Take your time with the tutorials. Read the first few and try programs with those for a few days. Then move on.

Quote:

---

Originally Posted by Patton

thank you, decrypt. But I replaced return with cout and the window still crashes.

---

The window doesn't crash, it just simply closes because that's what Dev-Cpp does when it gets a return on a program. Put a cin.ignore() after you get the input for selection and remember to return an int.

Quote:

---

Originally Posted by Patton

thank you, decrypt. But I replaced return with cout and the window still crashes.

---

If what you mean by the "window crashes" is that the window closes and the cin.get() seems not to have any effect then the answer is:
there is still a newline in cin's buffer and you could put another cin.get() to the end of your program. But a far better solution would be to call a commandline program from the commandline.
Kurt

It works now with the cin.ignore(), thanks
I think I tried putting that in there, but the returns were still there so it didn't matter
Thanks again.