Wondering why compile takes so long, what am I doing wrong

[millerizi]

Just starting out and trying to figure out where I am doing this wrong, takes more than 2 seconds to compile, what am I doing wrong, just starting out and messing around with the while loop. Thanks.

#include <iostream>

using namespace std;

int main()
{
    int num;
    int total;
    int counter = 1;
    int enter;

    cout << "How many times do you want to play?\n"
         << "Enter Number Here: ";
    cin >> num;

    total= num;

    cout << "You want to play to " << total <<" times.";

    while (counter <= total){
            cout << "\nEnter the Number one.\n";
            cin >> enter;
            ++counter;
    }

}

[sasauke]

How fast do you wanna go? lol It takes around a second for me. It depends on your computer

[millerizi]

ha ha. does the code look ok, I am curious if I missing anything, have the variable enter seems kind of odd, I was wondering if there was a better way to do it.

[sasauke]

Well, I'm not sure what you're trying to do, so I can't tell you if there's anything wrong.

Although cin >> num; total= num; kind of stands out. You don't really need a variable called num. You can just input total.

And don't forget to "return 0;" in the end. int main() is a function and it must have a return value. :P

#include <iostream>

using namespace std;

int main()
{
	int total;
	int counter = 1;
	int enter;

	cout << "How many times do you want to play?\n" << "Enter Number Here: ";
	cin >> total;

	cout << "You want to play to " << total << " times.";

	while (counter <= total) 
	{
		cout << "\nEnter the Number one.\n";
		cin >> enter;
		++counter;
	}
	return 0;
}

[millerizi]

awesome, that is what I was looking for, I don't know what I want either :0) just trying to figure out the logic and make sure I am writing the cleanest I can, and I see what you are saying, why make something equal to another variable when its not going to change, DOH. thank you.

[ZippyEthan]

Nothing is wrong you could add "stdafx.h" but I don't know if your IDE would allow it stdafx is a precompiled header thing which will make it so the compiler doesn't have to start from scratch and re compile everything.

[MiiNiPaa]

don't forget to "return 0;"

Main function is a special case and is threated as returning 0 if no explicit return clause found.

you could add "stdafx.h"

You should actually add stuff you need in that header for it to be of any use.