Need help with code?

You need to 'seed' the random number generator before using it. Find some documentation for the srand() function to get you going.

EDIT: That'll teach me to skim read, I see you're already seeding! Have you tried stepping each line using your debugger, and checking each value is what you expect?

Quote:

Original post by NateTheGreat22 The problem is that the computer keeps guessing the same number an infinite amount of times.

Look carefully at where the computer makes its random guess (hint: it only does so once, instead of every time through the loop).

Thanks guys I got it 2 work but I still have a question

When the computer guesses a number it tends to guess a number a certain amount of times before it goes to its next guess. Is there a way to stop it from doing that??

Here's the code and try it on your compiler to have a first-person point of view of
what I mean. (P.S. - How do you format code on a forum properly?)



#include <iostream>
#include <cstdlib>
#include <ctime>

using namespace std;

int main()
{
srand(static_cast<unsigned int>(time(0))); //seed random number generator
int CompGuess = rand() % 10 + 1; //random # bet. 1 and 10

int tries = 0;
int secretnumber;

cout << "\tWelcome to Guess My number V 1.1" << endl;

cout << "\nPlease input a number from (1 - 10): ";
cin >> secretnumber;

cout << "\nNow lets see how long the AI takes to guess your number....hehehehe!!!" << endl;

while (CompGuess != secretnumber)
{

srand(static_cast<unsigned int>(time(0)));
int CompGuess = rand() % 10 + 1;

cout << "\nHmmm is your number " << CompGuess << " ???\n";
++tries;

if (CompGuess != secretnumber)
{
cout << "\nTry again!";
}
else
{
cout << "\nTook it long enough....\n";
cout << "The AI guessed your number in " << tries << " try / tries....WOW!";
break;
}


}
}

Seed the random number generator once at the beginning of the program, not every loop.

.........wow....smh real embarrassing :$, thanks bro

Quote:

Original post by NateTheGreat22 (P.S. - How do you format code on a forum properly?)

wrap your code with [ source ] or [ code ] tags. No spaces :D

You put your code text between these tags: [ source ] and [ /source ] (without spaces). If you want to specifically use a certain syntax highlighting then you use instead [ source lang="" ] and [ /source ], with the desired language between the "". For example, source lang ="cpp" is for C++. Like so:



You can see a list of the different tags here: http://www.gamedev.net/community/forums/faq.asp#tags

Anyway, from looking at your source code it seems like you seed it only ones. I take it you edited it after nobodynews commented on it? Anyway, I would suggest you copy swiftcoder's reply and paste it in your source code as a comment, and use it as a reference. Then read your code line by line with no rush and make sure you understand what's happening.


EDIT: Ninja'd.

[Edited by - Metallon on January 2, 2011 1:57:58 AM]

@ Metallon

I actually did use swiftcoder's post as a ref. and I know have everything running smoothly

Thanks