Quick help with Tic-Tac-Toe [JCreator Pro]

Feren Silver · Dec 10, 2008

Alright. I've got my code and everything almost ready to go with my game I've been working on.

I'm only running into one problem and that is that for example player X can choose where player O played. So if someone could help me with this code it'd be greatly appreciated.

TicTacToe.java:

Code:

/**
   @Author:
      Date: Nov 28, 2006
   Teacher:
       Lab:
      Misc:
  */
// note: when you want to port your project to the web, just copy everything
//       in the "classes" folder that this project contains over to where you want

// this is an example file within this project just to show you how a project works.
// if you change the name of the main class (below it's TicTacToe), then
// you'll have to edit the TicTacToe.htm file so it matches
//
// the TicTacToe.htm file contains an "archive" parameter that
// helps the browswer find the objectdraw.jar
import objectdraw.WindowController;
import objectdraw.FilledRect;
import objectdraw.Location;
import objectdraw.*;

import java.awt.Color;
import java.awt.*;

public class TicTacToe extends WindowController 
{
  private TicTacToeSpot[][] ttt;
  private Image empty,x,y;
  private boolean isX,win;

  public void begin()
  { 
  
  
  
	isX = true;
	win = false;
    ttt = new TicTacToeSpot[3][3];
    empty = getImage("empty20.gif");
    x = getImage("x2.gif");
    y = getImage("o2.gif");
    
    for(int i = 0; i < 300; i+=100)
    {
    	for (int j = 0; j < 300; j+=100)
    	{
    		ttt[i/100][j/100] = new TicTacToeSpot(empty, i,j,canvas);
    	}
    }
  }

  public void onMouseClick(Location point)
  {
  	int r = (int)point.getX()/100;
  	int c = (int)point.getY()/100;
  	
    if (isX)
    {
    	//THIS IS WHERE THE CODE NEEDS TO BE EDITED TO FIX IT SO THAT TWO IMAGES CAN'T BE PLACED OVER ONE ANOTHER.
    	
    	ttt[r][c].setSpot('x');
    	ttt[r][c].setImage(x,r,c);
        
    }
    else
    {
        ttt[r][c].setSpot('o');
    	ttt[r][c].setImage(y,r,c);	
    }
   	checkwin();
    if (!win)isX = !isX;  
    if (win)
  	{
  		canvas.clear();
  		begin();
  	}
  	
  	int spotsLeft=0;
  	for(int q = 0; q < 3; q++)
  	{
  		for (int p = 0; p < 3; p++)
  		{
  			if (ttt[q][p].getSpot()==' ')
  			  spotsLeft++;
  		}
  		
  	}
  	if (spotsLeft==0)
  	{
  		canvas.clear();
  		begin();
  	}
  }
  
  public void checkwin()
  {
  	char spot;
  	if (isX) spot = 'x'; else  spot = 'o';
  	for (int i = 0; i < 3&& !win; i++)
  	{
  		if(ttt[i][0].getSpot()==spot &&ttt[i][1].getSpot()==spot &&ttt[i][2].getSpot()==spot)
  		{
  			win = true;
  		}
  		
  		if(ttt[0][0].getSpot()==spot &&ttt[1][1].getSpot()==spot &&ttt[2][2].getSpot()==spot)
  		{
  			win = true;
  		}
  		
        if(ttt[0][2].getSpot()==spot &&ttt[1][1].getSpot()==spot &&ttt[2][0].getSpot()==spot)
  		{
  			win = true;
  		}

  	}
  	for (int j = 0; j < 3 && !win;j++)
  	{
  		if(ttt[0][j].getSpot()==spot&&ttt[1][j].getSpot()==spot&&ttt[2][j].getSpot()==spot)
  		{
  			win = true;
  		}
  	}
 System.out.println(win);

  }
}

TicTacToeSpot.java:

Code:

import objectdraw.WindowController;
import objectdraw.FilledRect;
import objectdraw.Location;
import objectdraw.*;

import java.awt.Color;
import java.awt.*;

public class TicTacToeSpot
{
	private boolean isEmpty;
	private char currentChar;
	private VisibleImage thePic;
	private DrawingCanvas canvas;
	
	
	public TicTacToeSpot(Image pic, int x, int y, DrawingCanvas acanvas)
	{
		canvas = acanvas;
		isEmpty = true;
		currentChar = ' ';
		thePic = new VisibleImage(pic,x,y,canvas);	
	}
	
	public boolean setSpot(char cc)
	{
		if (isEmpty)
		{
			currentChar = cc;
			isEmpty = false;	
			return true;
		}
		return false;
		
	}
	
	public char getSpot()
	{	
		return currentChar;
	}
	
	public void setImage(Image pic,int x,int y)
	{
		thePic = new VisibleImage(pic,x*100,y*100,canvas);
	}
}

I haven't been able to find any guides or examples of this and would like some help. Please feel free to post here to give me an e-mail at <[email protected]>.

Thanks.

war833 · Dec 11, 2008

Just add an if statement so that if its x's turn put an x; if it's o's turn put an o. Also, don't make two theads on exactly the same thing, use your other thread.

Feren Silver · Dec 11, 2008

war833 said:
Just add an if statement so that if its x's turn put an x; if it's o's turn put an o. Also, don't make two theads on exactly the same thing, use your other thread.

My problem isn't that. It's say player X plays in the top left corner. When it's player O's turn, he has the ability to take that spot still. Plus just adding a if statement wouldn't do it I don't think. (What I've picked up from Java so far).

Swan · Dec 11, 2008

feren silver said:
My problem isn't that. It's say player X plays in the top left corner. When it's player O's turn, he has the ability to take that spot still. Plus just adding a if statement wouldn't do it I don't think. (What I've picked up from Java so far).

actually, an if statement WOULD work.

What you need to do is check if the spot that is clicked is taken, and if it is, simply return the method with the return keyword. You are just checking to see if the space is taken or not.

Feren Silver · Dec 14, 2008

Swan said:
actually, an if statement WOULD work.

What you need to do is check if the spot that is clicked is taken, and if it is, simply return the method with the return keyword. You are just checking to see if the space is taken or not.

Could you help me with the code? I haven't learned this stuff in my class, I'm doing this outside of class.

Swan · Dec 15, 2008

feren silver said:
Could you help me with the code? I haven't learned this stuff in my class, I'm doing this outside of class.

If you know how an if statement works you should be able to solve your own dilemma. It seems that you were smart enough to write all of the other code.

Feren Silver · Dec 16, 2008

Swan said:
If you know how an if statement works you should be able to solve your own dilemma. It seems that you were smart enough to write all of the other code.

My teacher gave me bits and pieces of the previous code to guide me. She's never taught us this.

SuF · Dec 16, 2008

You do not know how an if statement works? Then you should not be making this program.

Zyloch · Dec 17, 2008

I have not read this thread, but it seems to me that you need to be more proactive. People have mentioned the if clause; if you don't know how it works, look it up. The key to succeeding in computer programming is being interested and finding things out for yourself.

hampe-92 · Dec 17, 2008

Code:

if ( (1+1) == 2) //if it's true, then the code in the brackets will be executed, else it will be skipped
{
do whatever you want to do
}
else
{
blah blah
}

Code:

if ( (94 * 347) < 34694)
{
do whatever you want to do
}
else if ((94 * 347) != 32618)
{
blah blah
}
else
{
blah blah
}

and btw, http://www.letmegooglethatforyou.com/?q=java+if+statement

Feren Silver · Dec 17, 2008

SuF said:
You do not know how an if statement works? Then you should not be making this program.

Thank you for the pointless comment. I hope you feel good that you got a +1 on your post count. You can go tell your friends how cool you are now.

Zyloch said:
I have not read this thread, but it seems to me that you need to be more proactive. People have mentioned the if clause; if you don't know how it works, look it up. The key to succeeding in computer programming is being interested and finding things out for yourself.

I haven't been able to find any guides or examples of this and would like some help. Please feel free to post here to give me an e-mail at <[email protected]>.

I did look it up. Thanks though.>_<

hampe-92 said:
Code:

if ( (1+1) == 2) //if it's true, then the code in the brackets will be executed, else it will be skipped { do whatever you want to do } else { }

and btw, http://www.letmegooglethatforyou.com/?q=java+if+statement

What good did that do me? You Googled an if statement for me. That doesn't really help me. I would like if someone didn't give me a response like, "Look it up!"

SuF · Dec 17, 2008

well you have used if statements throughout the program, so if you do not know how they work, then you should not have been able to write the program

here

Code:

if(spotIsTaken = true)
{
//disable spot. :/
}

if each spot is a button, then disable the button... make a method that checks if the spot is taken, or just use 9 booleans that set to true the instant the spot gets taken, then check if the boolean is true and if it is, then do not let them take the spot...

Plus none of us want to do it for you, which is what you seem to be asking.

Your setSpot has a boolean return... whats that do? if isEmpty does what it looks like it does, sets false if the spot is taken, you just need to use that to check if the spot is taken before you change it.

i have basically just said what swan said, just slightly more detailed... i think that may be all your going to get...

hampe-92 · Dec 17, 2008

feren silver said:
What good did that do me? You Googled an if statement for me. That doesn't really help me. I would like if someone didn't give me a response like, "Look it up!"

well, besides the link, I actually did showed you how a if statement works...

war833 · Dec 17, 2008

hampe-92 said:
well, besides the link, I actually did showed you how a if statement works...

Dude, he used if statements in his program already. He clearly understands them...>_<

Zyloch · Dec 18, 2008

He does not necessarily understand them if his teacher provided most of the code for him.

To the original poster, however, I apologize. I only skimmed the thread, and I was mistaken as to what you were confused about.

I have taken a look at the code. It seems the problem is that you call setImage() after calling setSpot(). While setSpot() has code that checks whether the spot is taken already or not, setImage() does not. Either add the extra check inside there, or call setImage() directly from setSpot().

Feren Silver · Dec 18, 2008

SuF said:
well you have used if statements throughout the program, so if you do not know how they work, then you should not have been able to write the program

here

Code:

if(spotIsTaken = true) { //disable spot. :/ }

if each spot is a button, then disable the button... make a method that checks if the spot is taken, or just use 9 booleans that set to true the instant the spot gets taken, then check if the boolean is true and if it is, then do not let them take the spot...

Plus none of us want to do it for you, which is what you seem to be asking.

Your setSpot has a boolean return... whats that do? if isEmpty does what it looks like it does, sets false if the spot is taken, you just need to use that to check if the spot is taken before you change it.

i have basically just said what swan said, just slightly more detailed... i think that may be all your going to get...

Thank you. That's all the help I was asking for.

war833 said:
Dude, he used if statements in his program already. He clearly understands them...>_<

Learn to read a thread before you post useless information.

Zyloch said:
He does not necessarily understand them if his teacher provided most of the code for him.

To the original poster, however, I apologize. I only skimmed the thread, and I was mistaken as to what you were confused about.

I have taken a look at the code. It seems the problem is that you call setImage() after calling setSpot(). While setSpot() has code that checks whether the spot is taken already or not, setImage() does not. Either add the extra check inside there, or call setImage() directly from setSpot().

Thanks. You're one of the only people to actually post something productive here.

Quick help with Tic-Tac-Toe [JCreator Pro]

Feren Silver

Hero

war833

Member

Feren Silver

Hero

Swan

When They Cry...

Feren Silver

Hero

Swan

When They Cry...

Feren Silver

Hero

SuF

Legend

Zyloch

Member

hampe-92

Forum Addict

Feren Silver

Hero

SuF

Legend

hampe-92

Forum Addict

war833

Member

Zyloch

Member

Feren Silver

Hero

Similar threads