[Pat234]

The program runs but for some reason it ignores the if statement and goes above 200 ( i even tried changing it to 250 but still didn't work)
Compiler being used: Dev-C++

#include <iostream>
#include <windows.h>

using namespace std;

int main()
{
   int start_time = GetTickCount(); //i want this to happen once
   int end_time, total_time; 


   while(true)
   {
	 end_time = GetTickCount();
	 total_time = (end_time - start_time)/ 10;
	 
		   cout << 200 << endl
		   << total_time << endl; 
	 if(total_time == 200)	//for some reason, this if statement gets ignored?
	 {
	   cout << "Perfect" <<endl;  
	   system("PAUSE");
	   return 0;  
	 }
	 
	 else if(total_time > 200)
	 {
				   
		 cout << "WHAT THE F-" << endl;
		 Sleep(10000);
		 return 0;
	 }
	 system("cls");
   }

   system("PAUSE");
   return 0;

}
 

This post has been edited by Pat234: 17 January 2009 - 10:24 PM

[no2pencil]

Try the following, & see what it's seeing.

total_time=0;
while(total_time < 200) {
     end_time = GetTickCount();
     total_time = (end_time - start_time)/ 10;
     
     cout << total_time << endl
     if(total_time == 200) {
     ...
     }
     
     else if(total_time > 200) {
     ...
     }
}

** Renamed title to be more descriptive of topic **

[Pat234]

no2pencil, on 17 Jan, 2009 - 09:30 PM, said:

Try the following, & see what it's seeing.

i don't get it?

[no2pencil]

Pat234, on 17 Jan, 2009 - 11:43 PM, said:

no2pencil, on 17 Jan, 2009 - 09:30 PM, said:

Try the following, & see what it's seeing.

i don't get it?

Looking at your code you don't see why the conditional statements are not reacting the way you think they should.

So output the value & see what the conditional statements are actually comparing. Check the values visually & make the correct change to the code.

[OliveOyl3471]

It worked for me, the way I thought it should, according to what I see in the code.

[Pat234]

no2pencil, on 17 Jan, 2009 - 09:49 PM, said:

Pat234, on 17 Jan, 2009 - 11:43 PM, said:

no2pencil, on 17 Jan, 2009 - 09:30 PM, said:

Try the following, & see what it's seeing.

i don't get it?

Looking at your code you don't see why the conditional statements are not reacting the way you think they should.

So output the value & see what the conditional statements are actually comparing. Check the values visually & make the correct change to the code.

i did by using

cout << 200 << endl
		   << total_time << endl; 

but i still don't understand why its doing that

OliveOyl3471, on 17 Jan, 2009 - 09:53 PM, said:

It worked for me, the way I thought it should, according to what I see in the code.

yeah, i read the code now and it looks like it makes sense, but when i compile & run it, it does not go as planned

[no2pencil]

Try the following :

#include <iostream>
#include <windows.h>

using namespace std;

int main()
{
   int start_time = GetTickCount(); //i want this to happen once
   int end_time, total_time;


   while(true)
   {
     end_time = GetTickCount();
     total_time = (end_time - start_time)/ 10;
     
           cout << 200 << endl
           << total_time << endl;
  
     if(total_time > 200)
     {
                   
         cout << "WHAT THE F-" << endl;
         Sleep(10000);
         return 0;
     }
     if(total_time == 200)    //for some reason, this if statement gets ignored?
     {
       cout << "Perfect" <<endl;  
       system("PAUSE");
       return 0;  
     }
     system("cls");
   }

   system("PAUSE");
   return 0;

}

I think it's in your else statement.

[Pat234]

i tried that as well with no sucess

[no2pencil]

So what values are being displayed when you run this?

[OliveOyl3471]

Pat234, on 18 Jan, 2009 - 12:06 AM, said:

yeah, i read the code now and it looks like it makes sense, but when i compile & run it, it does not go as planned

So, what do you expect it to output?

This post has been edited by OliveOyl3471: 17 January 2009 - 11:29 PM

[Pat234]

it will count up to 201, then my 2nd if statement occurs printing text and pausing the screen showing:
200
201

[OliveOyl3471]

no2pencil, on 18 Jan, 2009 - 12:09 AM, said:

I think it's in your else statement.

That's what I thought, too. The else will never run.

Pat234, on 18 Jan, 2009 - 12:29 AM, said:

it will count up to 201, then my 2nd if statement occurs printing text and pausing the screen showing:
200
201

That's funny, for me it outputs the 200 on top like yours, but counts up to 200 and stops, for the 2nd line and outputs Perfect on the third line.

This post has been edited by OliveOyl3471: 17 January 2009 - 11:33 PM

[no2pencil]

The code solution that I provided works perfectly for me, 100% of the time. I've ran it like 30 times & not had it pass 200 once.

Quote

198
198
200
Perfect

[Pat234]

i'll check it out try it again 2morrow
NEEEEED SLEEEEEP
and thanks for the help, cant for 2morrow

[Pat234]

okay so i ran...

#include <iostream>
#include <windows.h>

using namespace std;

int main()
{
   int start_time = GetTickCount(); //i want this to happen once
   int end_time, total_time;


   while(true)
   {
	 end_time = GetTickCount();
	 total_time = (end_time - start_time)/ 10;
	 
		   cout << 200 << endl
		   << total_time << endl;
  
	 if(total_time > 200)
	 {
				   
		 cout << "WHAT THE F-" << endl;
		 Sleep(10000);
		 return 0;
	 }
	 if(total_time == 200)	//for some reason, this if statement gets ignored?
	 {
	   cout << "Perfect" <<endl;  
	   system("PAUSE");
	   return 0;  
	 }
	 system("cls");
   }

   system("PAUSE");
   return 0;

}
 

and it still is not working for some odd reason
it still prints out :
200
201
WHAT THE F-

could it be vista? lol, im out of ideas
i'll have to test this on my friend's laptop then
Glad to see it works, even tho its working on other people's computers and not mine

This post has been edited by Pat234: 18 January 2009 - 12:57 PM