Brain explosion

[raulkolk]

Hey dudes, please help me out with this code.

#include <windows.h>
#include <ctime>
#include <conio.h>
#include <iostream>
#include <string>
 
using namespace std;
 
 
 
int main ()
{
cout << "Hello!" << endl;
 
int amount = 0;
int primary = 0;
int turn = 0;
int time = 1;

const int SEC = time;
time = 2;
clock_t delay = SEC *CLOCKS_PER_SEC;
clock_t start = clock();
 
         while(clock() - start < delay){
		if(clock() - start == delay){
			cout << turn << endl;
			cout << "lol" << endl;
           }
 
while (primary == 0){ 
if(amount >= 1) { 
	if (turn == 0) {
		turn = turn + 1;
	}
}
 
if(amount >= 2) {
	if (turn == 1) {
		turn = turn + 1;
      }
}
           
if(amount >= 3) {
	if (turn == 2) {
		turn = turn - 2;
				}
			}
		}
	}
}

For some reason the loop doesn't start and it only prints me "Hello!".

Any help is appreciated.

[Zaita]

If you want to wait for a period of time before executing some code, I'd suggest you use the sleep() method. You're code is putting the application into a high-demand loop for that period of time (using 100% of core/cpu).

clock() - start == delay is likely to never be true. Think more about the flow of logic you want to have in your application and design it accordingly.

It'd also be helpful if you told us what your application is suppose to do. It's currently got very poor indentation which makes the flow of logic hard to follow.

[raulkolk]

Thanks dude! Exactly what I needed.

I'll keep that in mind ;)

[Athar]

Your loop should look like this (this is only regarding formatting):

while (primary==0)
{ 
  if (amount>=1 && turn==0)turn++;
  if (amount>=2 && turn==1)turn++;
  if (amount>=3 && turn==2)turn-=2;
}

There are few things that can make source code more unreadable than braces (both curly and regular ones).
Avoid them whenever possible. The same goes for vast amounts of unnecessary whitespace.

[raulkolk]

Yea, I know. I'm just too lazy :)
Thanks.