jessie puls

Actually I fixed the problem..I should have been creating a new thread instead of just trying to call play again. If you wanted to see the rest of the code though there is a link in my previous message to download it in a zip file. Anyway I got it fixed now and it works right. Now I just want to make my computer player a little bit smarter because as it is right now it just picks an open spot randomly. anyway thanks for the input. This is an assignment for a class and we haven't really covered threading yet, so I messed that up, but I was approaching the assignment slightly differently than most of my classmates who were doing the bulk of their work in action listeners on buttons. I didn't really like the way that looked so I decided not to do it that way ( which I believe is what the teacher originally intended since we haven't covered threading in any great detail yet ). Thanks again.

jessie

something like this maybe?

public void paint( Graphics g) { for ( int i = 0; i<100; i++) { g.setColor( new Color( new Random().nextInt(256), new Random().nextInt(256), new Random().nextInt(256) ) ); g.fillRect( new Random().nextInt(), new Random().nextInt(), new Random().nextInt(), new Random().nextInt()); g.fillOval( new Random().nextInt(), new Random().nextInt(), new Random().nextInt(), new Random().nextInt()); } }

you would probably want to set some maximum value for the randoms generated on the size and position of the shapes.
[ November 24, 2004: Message edited by: jessie puls ]

I am not sure why it is freezing up. I know where, but not why. Here is the code where the problem is coming from. I have the line of code that is causing the problem commented out right now. The first one is where it is being called the second is what it is calling. Also if you want to look at all the code you can from http://student.cns.uni.edu/~pulsj/assign9.zip If anyone can help at all I would be very greatful. Thanks!

jessie

import java.awt.*; import java.awt.event.*; public class GuiGameWindow extends CloseableFrame { public final int width = 500; public final int height = 400; private char[][] grid; private Label label; private GameBoard gameBoard; private int firstOrSecond; private CheckboxMenuItem first, second; public GuiGameWindow( GameBoard x ) { makeWindow(); gameBoard = x; firstOrSecond = 1; grid = gameBoard.getGridValues(); } public void makeWindow() { setSize( width, height ); setResizable( false ); setTitle( "Tic Tac Toe" ); MenuBar myMenuBar = new MenuBar(); Menu gameMenu = new Menu( "Game" ); Menu orderMenu = new Menu( "Do you want to Move" ); MenuItem newGame = new MenuItem( "New Game" ); newGame.addActionListener( new menuActionListener( newGame ) ); MenuItem quitGame = new MenuItem( "Quit" ); quitGame.addActionListener( new menuActionListener( quitGame ) ); first = new CheckboxMenuItem( "first", true ); first.addActionListener( new menuActionListener( first ) ); second = new CheckboxMenuItem( "second", false ); second.addActionListener( new menuActionListener( second ) ); gameMenu.add( newGame ); gameMenu.add( orderMenu ); gameMenu.addSeparator(); gameMenu.add( quitGame ); orderMenu.add( first ); orderMenu.add( second ); myMenuBar.add( gameMenu ); setMenuBar( myMenuBar ); label = new Label( "Tic Tac Toe", Label.CENTER ); label.setBackground( Color.darkGray ); label.setForeground( Color.yellow ); add( "South", label ); } public void paint( Graphics g ) { g.setColor( Color.darkGray ); g.drawLine( getWidth() / 3 + 25, 80, getWidth() / 3 + 25, getHeight() - 80 ); g.drawLine( 2 * getWidth() / 3 - 25, 80, 2 * getWidth() / 3 - 25, getHeight() - 80 ); g.drawLine( 80, getHeight() / 3 + 25, getWidth() - 80, getHeight() / 3 + 25 ); g.drawLine( 80, 2 * getHeight() / 3 - 25, getWidth() - 80, 2 * getHeight() / 3 - 25 ); if ( grid != null ) { g.drawString( "" + grid[0][0], 140, 120 ); g.drawString( "" + grid[0][1], 250, 120 ); g.drawString( "" + grid[0][2], 360, 120 ); g.drawString( "" + grid[1][0], 140, 200 ); g.drawString( "" + grid[1][1], 250, 200 ); g.drawString( "" + grid[1][2], 360, 200 ); g.drawString( "" + grid[2][0], 140, 280 ); g.drawString( "" + grid[2][1], 250, 280 ); g.drawString( "" + grid[2][2], 360, 280 ); } } public void setGridValues( char[][] x ) { grid = x; repaint(); } public void SetMessage( String message ) { label.setText( message ); } public int getFirstOrSecond() { return firstOrSecond; } private class menuActionListener implements ActionListener { MenuItem menuItem; public menuActionListener( MenuItem x ) { menuItem = x; } public void actionPerformed( ActionEvent e ) { if ( menuItem.getActionCommand().equals( "Quit" ) ) System.exit( 1 ); if ( menuItem.getActionCommand().equals( "New Game" ) ) { gameBoard.reset(); //this is the line that causes it to freeze up...I don't know why though. //gameBoard.connection().game().play(); } /*as far as I could tell there ins't a CheckboxGroup for CheckboxMenuItems so *I set the states of them explicitly every time. I suppose I could have *built my own, but this seemed easier... */ if ( menuItem.equals(first) ) { firstOrSecond = 1; first.setState(true); second.setState(false); } if ( menuItem.equals(second) ) { firstOrSecond = 2; second.setState(true); first.setState(false); } repaint(); } } } import java.io.*; public class TicTacToeGame { private GameBoard gameBoard; private Player player1, player2, player; private Connection connection; public TicTacToeGame( GameBoard x, Player y, Connection z ) { gameBoard = x; connection = z; setup( y ); connection.addPlayers(player1, player2); connection.setGame(this); player1.setConnection( connection ); player2.setConnection( connection ); gameBoard.setConnection(connection); play(); } public void play() { connection.gameBoard().setMessage("Tic Tac Toe"); player1.setCharacter( 'x' ); player2.setCharacter( 'o' ); try { player = player1; while ( true ) { gameBoard.drawGrid(); while ( true ) { player.activate(); if ( placement( player.move() , player.getCharacter() ) ) break; } if ( gameBoard.horizontalWin() || gameBoard.verticalWin() || gameBoard.diagonalWin() ) { gameBoard.drawGrid(); gameBoard.setMessage( "game over " + player.getCharacter() + " wins!" ); break; } if ( gameBoard.isFull() ) { gameBoard.drawGrid(); gameBoard.setMessage( "game over nobody wins" ); break; } player = switchPlayer( player ); } } catch ( Exception e ) { gameBoard.setMessage( "error: " + e ); } if ( gameBoard.playAgain() ) { player = player1; player1 = player2; player2 = player; play(); } } protected Player switchPlayer( Player player ) { if ( player.equals( player1 ) ) return player2; return player1; } public Player player() { return player; } public boolean placement( int i, char j ) throws IOException { int x, y; if ( i % 3 == 0 ) { x = i / 3 - 1; y = 2; } else { x = i / 3; y = i % 3 - 1; } if ( !gameBoard.isOpenSlot( x, y ) ) return false; gameBoard.placeChar( x, y, j ); return true; } public void setup( Player x ) { if ( gameBoard.getFirstOrSecond() == 1 ) { player1 = x; player2 = new ComputerPlayer(); } else { player1 = new ComputerPlayer(); player2 = x; } } }



Again thanks.