Guessing Game - Problem

I can't see why this wouldn't work. Either I'm completely missing something or this book is missing something.

Code:

---

#include <stdio.h>
#include <windows.h>

main()
{
     
      int iRandom = 0;
      int iResponse = 0;
      srand(time());
     
      iRandom = (rand() % 6) + 1;
     
      printf("\nPlease enter a number netween 1 and 6: ");
      scanf("%d", &iResponse);
     
      if (iResponse == iRandom)
        printf("\nCongratulations, you guessed correct!\n");
      else {
        printf("\nUnlucky, good luck next time!\n");
        printf("\nThe correct number was %d!\n", iRandom);
      }
       
      Sleep(5000);
     
}

---

Thanks.

rand and srand appear in stdlib.h. time appears in time.h. I'm not quite sure from your question what exactly you're confused about (did the book do something in a different way? did it not compile? etc?) so I don't know what else to say.

It didn't say to include stdlib.h and time.h for one, even with them included it still does not compile.

Quote:

---

Originally Posted by Slynet

It didn't say to include stdlib.h and time.h for one, even with them included it still does not compile.

---

I'm completely confident that some day - maybe next week, maybe tomorrow, maybe next month - you'll paste up the error codes you received.

I'm sure you'll manage that, some day. :)

Ha, I'll take your sarcasm as humor for now. :)

Sorry, it's quite late here, bit brain dead. :P

So, when I compile it comes up with the errors:

Code:

---

C:\Documents and Settings\User\Desktop\1.c In function `main':
11 C:\Documents and Settings\User\Desktop\1.c too few arguments to function `time'

---

Error on line 11, srand(time());

I double checked with my compiler, and it says (once I included time.h) that time does not take zero arguments (which is true -- it takes one argument, which is conventionally NULL in this circumstance).

Ok, so I've never used this function before and this is the example in the book. But if if use:

Code:

---

srand(time(0))

---

It works fine. Can someone explain to me what exactly it's doing, please.

Also now it's compiling without stdlib.h and time.h? So I need them, I don't need them?

stdlib.h and time.h are headers -- so there's no way for the compiler to tell that you are missing an argument if the header is not included. If you're using some really fancy IDE, it may be including things for you.

So how have you gotten this far without looking time() up in the index of your textbook, or typing "man time" online and getting this page?

In the post with the errors, that included the headers stdlib.h and time.h, which was advised.

In the post after I stated "it is now compiling", as opposed to before. In the successful compilation post all I done was to change srand(time()) to srand(time(0)), which compiled, then took away stdlib.h and time.h, and it still compiled.

Hope that makes more sense. :)

And yes, I'm using Dev-C++ with Mingw/GCC 3.4.2.

Some compilers (like Turbo C/C++ of old), are smart enough to include some parts of a header, if you need it, and don't include the right header file for it.

It will not do that for everything you need, so get in the habit of using the right headers. Other compilers will just give you some error, and not go looking for anything to help your code deal with a missing header.

Quote:

---

Originally Posted by Slynet

In the post with the errors, that included the headers stdlib.h and time.h, which was advised.

In the post after I stated "it is now compiling", as opposed to before. In the successful compilation post all I done was to change srand(time()) to srand(time(0)), which compiled, then took away stdlib.h and time.h, and it still compiled.

Hope that makes more sense. :)

And yes, I'm using Dev-C++ with Mingw/GCC 3.4.2.

---

Right. So the compiler cannot determine there's a problem unless you include the headers. If you didn't include the headers, then it won't know what to look for, so it will just take whatever you've got typed. If it doesn't work, you won't know until you crash your computer when you try to run the program (or, nowadays, "would you like to submit this error report to Microsoft?" when time takes some random number off the stack (since you didn't pass it one) and tries to use it as its argument).

Ok, I get it, thanks. It's really annoying that this book doesn't include them two headers, if you're going to teach someone, teach them properly.

Quote:

---

Originally Posted by Slynet

Ok, I get it, thanks. It's really annoying that this book doesn't include them two headers, if you're going to teach someone, teach them properly.

---

You'd think so, wouldn't you... Unfortunately, we do see it not so rarely that books either take short-cuts, or do not use up-to-date information...

--
Mats