simple question- not ruinning as expected

[jonezy]

hi, im just trying to make a simple program, just to see wat i can do.

i want to no y this code doesnt run as expected, after the last line the program quits, without leaving time to view wat is supposed to be on the screen

help please

 #include <iostream>

using namespace std; // So we can see cout and endl

int main()
{ 
  int x = 0;  //  declare variables
  int answer;
  int y = 0;
  


  
  while ( x < 100000 ) { // While x is less than 100000
    cout<< x <<endl;
    x++; 
    x++;            // Update x so the condition can be met eventually
  }
cout<< "Total files placed in your computer = " ;
cout << "99899 \n";
cout << "Happy holidays!.\n";

cout<< "Do you think im telling the truth?\n";
cout<< "1 = yes\n2 = no\n" ;
cin>> answer; //stores variable in answer
  if (answer = 0) {
  cout<< "Well if your not going to believe me......\n maybe i should add some more... would you like that? \n";
  
}

  //program should wait here, waiting for an input or enter stroke to quit
   

  cin.get();}

[Daved]

Common problem. The code "cin >> answer" reads an integer from the input stream into the variable answer. The newline from when you hit Enter is still in the input stream. The cin.get() call reads in that newline instead of waiting for you to hit Enter again. Use two cin.get() calls, or add a cin.ignore() call before the cin.get() to ignore the newline that is still in the stream.

[7stud]

while ( x < 100000 ) { // While x is less than 100000

Do you really think that comment is necessary? Comments like that are as bad as no comments for difficult code.

[jonezy]

Think i must be doing something still wrong.
or missing out something uve said.

#include <iostream>

using namespace std; // So we can see cout and endl

int main()
{ 
  int x = 0;  //  declare variables
  int answer;
  int y = 0;
  


  
  while ( x < 100000 ) { 
    cout<< x <<endl;
    x++; 
    x++;            // Update x so the condition can be met eventually
  }
cout<< "Total files placed in your computer = " ;
cout << "99899 \n";
cout << "Happy holidays!.\n";

cout<< "Do you think im telling the truth?\n";
cout<< "1 = yes\n2 = no\n" ;
cin.get(); //tried it here after the new line
cin>> answer; //stores variable in answer

  if (answer = 0) {
  cout<< "Well if your not going to believe me......\n maybe i should add some more... would you like that? \n";
  
}

  //program should wait here, waiting for an input or enter stroke to quit
   

cin.get(); tried it here ( only 1 at a time)
  cin.get();}

what am i missing out / doing wrong

all help appreciated

[Lithorien]

Here's some pretty heavily commented code for you. I cleaned up your text formatting, and tried to illustrate some concepts you should know.

#include <iostream>

int main()
{
	// Let's replace your while() loop with a for() loop so it's more intuitive to read.
	
	// In a for() loop, you can declare a variable that will only be in that loop's scope.
	// This is a GoodThing(tm). It helps your program not use as much memory.
	
	for (int x = 0; x < 10000; x++)
	{ 
		std::cout << x << std::endl;
	}
	
	// std::cout is overloaded. You can string together multiple <<'s, and it helps readability most of the time.
	
	std::cout << "\nTotal files placed in your computer = " << "9999\n";
	std::cout << "Happy holidays!\n\n";
	
	std::cout << "Do you think I'm telling the truth?\n\n";
	
	std::cout << "1 = yes\n";
	std::cout << "2 = no\n\n";
	
	std::cout << ">> "; // It's usually a good idea to let the user know you expect input.
	
	// Don't have an empty cin.get() here, you'll swallow your number with no effect.
	
	int answer; // It's usually a good idea to declare variables near the point of first use.
	
	std::cin >> answer;
	
	// Since we're just going to keep this simple, we'll assume the user plays nice and doesn't put
	// anything but a number in from standard input.
	
	// Since the only possible answer that wouldn't display the message in the if{} block is 1,
	// then we can just check for a non-1 answer.
	
	if (answer != 1)
	{
		std::cout << "\nWell if you're not going to believe me. . .\n";
		std::cout << ". . .maybe I should add some more. Would you like that?\n";
	}
	
	std::cin.ignore(80, '\n'); // This will flush the newline from stdin (the input buffer).
	std::cin.get();

	return(0); // ALWAYS RETURN A VALUE FROM MAIN(). It doesn't matter if main() implicitly returns or not.
}

[jonezy]

Thanks for that.
Yeah i can understand most of that now and its proved helpfull
the only thing is i tried adding an 'else if (answer = 1)' line, but compiler reckons there is something wrong

all help has and will be apreciated

#include <iostream>

int main()
{
	// Let's replace your while() loop with a for() loop so it's more intuitive to read.
	
	// In a for() loop, you can declare a variable that will only be in that loop's scope.
	// This is a GoodThing(tm). It helps your program not use as much memory.
	
	for (int x = 0; x < 10000; x++)
	{ 
		std::cout << x << std::endl;
	}
	
	// std::cout is overloaded. You can string together multiple <<'s, and it helps readability most of the time.
	
	std::cout << "\nTotal files placed in your computer = " << "9999\n";
	std::cout << "Happy holidays!\n\n";
	
	std::cout << "Do you think I'm telling the truth?\n\n";
	
	std::cout << "1 = yes\n";
	std::cout << "2 = no\n\n";
	
	std::cout << ">> "; // good idea to let the user know you expect input.
	
	// Don't have an empty cin.get() here, you'll swallow your number with no effect.
	
	int answer; // good idea to declare variables near the point of first use.
	
	std::cin >> answer;
	
	// Since we're just going to keep this simple, we'll assume the user plays nice and doesn't put
	// anything but a number in from standard input.
	
	// Since the only possible answer that wouldn't display the message in the if{} block is 1,
	// then we can just check for a non-1 answer.
	
	if (answer != 1)
	{
		std::cout << "\nWell if you're not going to believe me. . .\n";
		std::cout << ". . .maybe I should add some more. Would you like that?\n";
	}
	
	std::cin.ignore(80, '\n'); // This will flush the newline from stdin (the input buffer).
	
	std::cout << "\n\n1 = yes\n";
	std::cout << "2 = no\n\n";
	
	std::cout << ">> "; // good idea to let the user know you expect input.
    
    else if (answer = 1)
	{
         for (int y = 0; y < 100000; y++)
         {
             std::cout << y << std::endl;
             }
         std::cout << "\nGood....Thats very good. For i have now removed them all!\n";
         std::cout << "Press enter to quit\n";
         std::cin.get();
             }
    std::cout << "\n\nAnd again please....\n\n"
    std::cin.get();

	return(0); // ALWAYS RETURN A VALUE FROM MAIN(). It doesn't matter if main() implicitly returns or not.
}

[Lithorien]

else if (answer = 1)

Wrong. = is the assignment operator, == is the comparison operator.

[jonezy]

o lol, i thought when i saw that , that it was a silly little mistake, however,
still recons theres something roung. even when i add the extra =

#include <iostream>

int main()
{
	// Let's replace your while() loop with a for() loop so it's more intuitive to read.
	
	// In a for() loop, you can declare a variable that will only be in that loop's scope.
	// This is a GoodThing(tm). It helps your program not use as much memory.
	
	for (int x = 0; x < 10000; x++)
	{ 
		std::cout << x << std::endl;
	}
	
	// std::cout is overloaded. You can string together multiple <<'s, and it helps readability most of the time.
	
	std::cout << "\nTotal files placed in your computer = " << "9999\n";
	std::cout << "Happy holidays!\n\n";
	
	std::cout << "Do you think I'm telling the truth?\n\n";
	
	std::cout << "1 = yes\n";
	std::cout << "2 = no\n\n";
	
	std::cout << ">> "; // good idea to let the user know you expect input.
	
	// Don't have an empty cin.get() here, you'll swallow your number with no effect.
	
	int answer; // good idea to declare variables near the point of first use.
	
	std::cin >> answer;
	
	// Since we're just going to keep this simple, we'll assume the user plays nice and doesn't put
	// anything but a number in from standard input.
	
	// Since the only possible answer that wouldn't display the message in the if{} block is 1,
	// then we can just check for a non-1 answer.
	
	if (answer != 1)
	{
		std::cout << "\nWell if you're not going to believe me. . .\n";
		std::cout << ". . .maybe I should add some more. Would you like that?\n";
	}
	
	std::cin.ignore(80, '\n'); // This will flush the newline from stdin (the input buffer).
	
	std::cout << "\n\n1 = yes\n";
	std::cout << "2 = no\n\n";
	
	std::cout << ">> "; // good idea to let the user know you expect input.
    
    else if (answer == 1)
	{
         for (int y = 0; y < 100000; y++)
         {
             std::cout << y << std::endl;
             }
         std::cout << "\nGood....Thats very good. For i have now removed them all!\n";
         std::cout << "Press enter to quit\n";
         std::cin.get();
             }
    std::cout << "\n\nAnd again please....\n\n"
    std::cin.get();

	return(0); // ALWAYS RETURN A VALUE FROM MAIN(). It doesn't matter if main() implicitly returns or not.
}

[JoshR]

Here is your fixed program, I'm pretty confident that this is how you want it to run.

#include <iostream>

int main()
{
	for (int x = 0; x < 10000; x++)
		std::cout << x << std::endl;
	
	std::cout << "\nTotal files placed in your computer = " << "9999\n";
	std::cout << "Happy holidays!\n\n";
	
	std::cout << "Do you think I'm telling the truth?\n\n";
	
	std::cout << "1 = yes\n";
	std::cout << "2 = no\n\n";
	
	std::cout << ">> "; 
	
	int answer;
	
	std::cin >> answer;
	std::cin.get();	//catch enter input
	
	if (answer != 1)
	{
		std::cout << "\nWell if you're not going to believe me. . .\n";
		std::cout << ". . .maybe I should add some more. Would you like that?\n";	
        
    std::cout << "\n\n1 = yes\n";
	std::cout << "2 = no\n\n";
	
	std::cout << ">> ";
	std::cin >> answer;   //you forgot to get the answer for the 2nd question
	std::cin.get();            //catch the enter input
    }

    if (answer == 1)
   {
         for (int y = 0; y < 100000; y++)
         {
             std::cout << y << std::endl;
         }
         std::cout << "\nGood....Thats very good. For i have now removed them all!\n";
         std::cout << "Press enter to quit\n";
         std::cin.get();
    }
    std::cout << "\n\nAnd again please....\n\n";
    std::cin.get();

    return 0;
}

[jonezy]

nope sorry lol.
this is my fault- by half completing a prgram.
the

else if (answer == 1)

line was supposed to do wateva i put in- if the original answer was 1. i left the

if

only half completed, because i knew this worked and would add more later.

if this makes no sense, tell me and i will annotate code

[JoshR]

then there is no use for the else part, because it isnt paired up with an if statement. plus dont you want the input for the second question?

[JoshR]

Instead of ignore, you can use cin.get() to catch the return key after each input.

[Krak]

Lemme add a few touch-ups for you sir.

#include <iostream>

using std::cout;
using std::endl;

int main()
{ 
  int x=0,y=0,answer=0; //  declare variables
  
  while ( x < 100000 ) { // While x is less than 100000
    cout<< x <<endl;
    x+=2;  // Update x so the condition can be met eventually
  }
      cout<< "Total files placed in your computer = 99899 \n" ;
      cout << "Happy holidays!.\n";

      cout<< "Do you think im telling the truth?\n";
      cout<< "1 = yes\n2 = no"<<endl; //Flushing ze buffer is good, ya?
      cin>> answer; //stores variable in answer
           if (answer == 0) {
                 cout<< "Well if you're not going to believe me......\n
                        maybe i should add some more... would you like that? \n";
  
              }

  //program should wait here, waiting for an input or enter stroke to quit
   
  cin.ignore();
  cin.get();
  return 0;
}

[jonezy]

Im not making my self clear enough, apoligies.

I hope this code will help explain myself.

#include <iostream>

int main()
{
	// Let's replace your while() loop with a for() loop so it's more intuitive to read.
	
	// In a for() loop, you can declare a variable that will only be in that loop's scope.
	// This is a GoodThing(tm). It helps your program not use as much memory.
	
	for (int x = 0; x < 10000; x++)
	{ 
		std::cout << x << std::endl;
	}
	
	// std::cout is overloaded. You can string together multiple <<'s, and it helps readability most of the time.
	
	std::cout << "\nTotal files placed in your computer = " << "9999\n";
	std::cout << "Happy holidays!\n\n";
	
	std::cout << "Do you think I'm telling the truth?\n\n";
	
	std::cout << "1 = yes\n";
	std::cout << "2 = no\n\n";
	
	std::cout << ">> "; // good idea to let the user know you expect input.
	
	
	int answer; // good idea to declare variables near the point of first use.
	
	std::cin >> answer;
	
	
	
	if (answer != 1)
	{
		std::cout << "\nWell if you're not going to believe me. . .\n";
		std::cout << ". . .maybe I should add some more. Would you like that?\n";
	}
	
	std::cin.ignore(80, '\n'); // This will flush the newline from stdin (the input buffer).
	
	std::cout << "\n\n1 = yes\n";
	std::cout << "2 = no\n\n";
	
	std::cout << ">> "; 
	//this is half completed, i was going to cacth variable, and act upon it later.
    
    else if (answer == 1)
    // this else if, is still acting upon the first variable , x,.
    //the first stated code to be completed if ,x, was not 1.
    // now i am stating code which should be completed, if ,x, does = 1
	{
         for (int y = 0; y < 100000; y++)
         {
             std::cout << y << std::endl;
             }
         std::cout << "\nGood....Thats very good. For i have now removed them all!\n";
         std::cout << "Press enter to quit\n";
         std::cin.get();
             }
    std::cout << "\n\nAnd again please....\n\n"
    std::cin.get();

	return(0); // ALWAYS RETURN A VALUE FROM MAIN().

[JoshR]

does it work the way you want it to?