Have patience, it's only been a few hours.
That long list of if statements in your growUp function doesn't look too good, normally if you end up using more than 2-3 if statements in a row a loop could do it better (especially when they are all going up by the same amount). Something like this should work(it can be modified how you like).
1 2 3 4 5 6 7
|
for(int i = 1; i < i < 17; i++)
{
if( sec + clock() > (i * 10000) && sec + clock() < (i * 100000) )
{
age = i;
}
}
|
I see quite a few goto statements in your code. These don't need to be used 99.99% of the time and just make it much harder to keep control of program flow and make it harder for people to read your code later on. There are usually better ways to accomplish things using while/do while loops. Try redoing some parts of code that rely on that.
You are using system calls quite often too, in my opinion system("cls") is one of the harder ones to avoid, but seeing as you are using windows.h anyway you might want to look up a better way to clear the screen (system calls aren't efficient and aren't liked by virus scanners).
There are some minor things I see like
happines = happines - 35;
is usually written as
happines -= 35;
just because it looks cleaner, but that's all personal preference.
Keep an eye on your indenting (which looks pretty good for the most part), it can make a big difference when people are reading through your code. Stuff like this doesn't look good.
1 2 3 4 5
|
if (progres == 'y')
{
fEnd = false;
break;
}
|
Definitely a good start just some cleaning up needed in certain places, keep working at it!