Thread: some help needed :/

1. Keep on top of the indentation -> SourceForge.net: Indentation - cpwiki

> while(koord[3][3]!='X');
What is this?
It's an array overrun, there is no [3][3] element of your array.

Presumably, the game is over when
- there is a line of 'X' or 'O'
- 9 moves have been made.

Use srand(time(NULL)) to seed the PRNG with the current time (so the numbers will be different each time you run the program). Then call rand() to generate a random number. Finally, find a way to make this random number either 0 or 1 (look into the modulos operator, or come up with your own algorithm, modulos introduces a slight bias), then you can pick a starting player based upon this value.

Also, use Google more, this question has been asked and answered each and every day since the beginning of time. Google is your best friend, if you need to solve a common problem but forgot how? Google. If you can't remember the exact syntax for some library you're using? Google. If the compiler is giving you incomprehensible errors? Google.

Well now you have either 0 or 1, randomly chosen each time the program runs. And you want either 1 or the other player to start. So obviously you should map 0 and 1 to your 2 players and make the chosen one start.

At the moment both players take a turn for each iteration of your loop. I would redesign the program so that either one or the other (Depending on who's turn it is) takes their turn, during the loop, then on the next iteration the other one goes, and so forth. Also if you split your program into smaller functions the code becomes easier to read and write. The turn system could then work something like this:

Code:

if(CurrentTurn == "Player")
{
    PlayerTurn();
    CurrentTurn = "Computer";
}
else
{
    ComputerTurn();
    CurrentTurn = "Player";
}

With something like the above code, inside your main game loop, it would be trivial to make either one or the other player start off, just set CurrentTurn to whoever is starting, before entering the loop. Also, having one turn per iteration makes more sense i think, but you'll have to redesign it yourself.

Well first of all, as i said, my example needs to be placed in a loop. Second of all, it was just an example, you cannot just copy paste it and expect it to work. You have to declare the variables and implement the functions. Did you redesign your loop to only handle 1 turn per iteration?

There will be no handouts, not from me atleast. Giving you the code doesn't help and you'll be back in a day or two with similar questions, and before i know it, i'll be working full time for you as a programmer. Show some effort, then i'll help.

Also, what did you intend with this line:

Code:

0 = "pc";

Did you write the original program yourself? Because if you are making mistakes like this it seems you need to take a step back and learn some more basic stuff before attempting this game.