Every time I run this program, I get the number the user inputted printed back on the screen infinite amount of times. According to my CS teacher, he wants us to use basics, no arrays or strings. The user is suppose to type in several numbers followed by -999. Say I type in 7 10 14 13 -999 into the line, and press enter, I get 7 repeating on the screen until I press Ctrl + C to exit. It's suppose to go through the loop, print the number out if it's an even and then check the system input stream and grab the next number on it. But it does this...any help would be appreciated. Ohh sorry for the poor code structure.

#include <iostream>
using namespace std;
const int SENTINEL = -999;

// This program is suppose to have the user input a list of numbers followed by
// -999 as the sentinel. It will take the numbers the user inputted and check 
// whether or not they are even. If they are, it will count how many are even, 
// and output that number. It will also output the total data set size

int main () {
    cout << "List a set of numbers followed by the number -999" << endl;
    int x;
    int ct = 0;
    int en = 0;
    int mod = 0; //
    cin >> x;
    // I put the cin here so it can run it through the while condition before
    // the loop body. That way, when the user enters -999, ct won't increment 
    // by 1
    while (x != SENTINEL) {
          mod = x % 2;
          if (mod == 0) {
             cout << x;
    cout << "The total amount of even numbers were: " << en << endl;
    cout << "The total amount of numbers in the data set were: " << ct << endl;
    return 0;
9 Years
Discussion Span
Last Post by CloudKill9

You need to watch your brace style. (The while end-brace should be at the same indent as the while keyword, based on your other braced blocks.)

The loop never ends because you forgot to cin >> x; at the end of your loop also.

Hope this helps.

This question has already been answered. Start a new discussion instead.
Have something to contribute to this discussion? Please be thoughtful, detailed and courteous, and be sure to adhere to our posting rules.