Still having trouble with the checkWinner

Question

it's making it so that every empty array spot is equal so the checkWinner method is screwing up, any ideas on how to fix this?
import javax.swing.*;
import java.text.*;
import java.applet.*;
import java.io.*;
import java.awt.*;
import java.lang.*;
import java.util.*;
import java.math.*;
import java.sql.*;
import java.security.*;
import java.beans.*;
import javax.crypto.*;
import javax.net.*;
import org.omg.CORBA.*;
import java.awt.color.*;
import java.util.jar.*;
public class TwoDArray
     String [] [] board=new String [3] [3];
     int x;
     public TwoDArray()
          for(int x=0;x<3;x++)
               for(int y=0;y<3;y++)
                    board[x][y]="";
     public void getMove(int row, int col, String letter)
          if(board[row][col].equalsIgnoreCase(""))
               board[row][col]=letter;
          x++;
     public void printBoard()
          for(int x=0;x<3;x++)
               System.out.print("|");
               for(int y=0;y<3;y++)
                    System.out.print(board[x][y]);
               System.out.println("|");
     public boolean checkWinner()
          boolean win=false;
          if(x>=3)
               if(board[0][0].equalsIgnoreCase(board[0][1])&&board[0][0].equalsIgnoreCase(board[0][2]))
                    win=true;
                    return win;
               if(board[0][0].equalsIgnoreCase(board[1][0])&&board[0][0].equalsIgnoreCase(board[2][0]))
                    win=true;
                    return win;
               if(board[0][1].equalsIgnoreCase(board[1][1])&&board[0][1].equalsIgnoreCase(board[2][1]))
                    win=true;
                    return win;
               if(board[0][2].equalsIgnoreCase(board[1][2])&&board[0][2].equalsIgnoreCase(board[2][2]))
                    win=true;
                    return win;
               if(board[1][0].equalsIgnoreCase(board[1][1])&&board[1][0].equalsIgnoreCase(board[1][2]))
                    win=true;
                    return win;
               if(board[2][0].equalsIgnoreCase(board[2][1])&&board[2][0].equalsIgnoreCase(board[2][2]))
                    win=true;
                    return win;
               if(board[0][0].equals(board[1][1])&&board[0][0].equalsIgnoreCase(board[2][2]))
                    win=true;
                    return win;
               if(board[2][0].equals(board[1][1])&&board[2][0].equalsIgnoreCase(board[0][2]))
                    win=true;
                    return win;
          return win;     
     public static void main(String[] args)
          boolean winner=false;     
          String player1="X";
          String player2="O";
          String instructions="Welcome to Tic-Tac-Toe.";
          for (int i = 0;i<instructions.length() ;i++ )
                System.out.print(instructions.charAt(i));
          try
          Thread.sleep(99);
          }catch(InterruptedException e) {e.printStackTrace();}
          System.out.println();
          TwoDArray board=new TwoDArray();
          String input1=JOptionPane.showInputDialog("Enter the row you wish to move in.");
          int rowmove=Integer.parseInt(input1);
          String input2=JOptionPane.showInputDialog("Enter the column you wish to move in.");
          int colmove=Integer.parseInt(input2);
          board.getMove(rowmove,colmove,player1);
          board.printBoard();
          while(winner==false)
               String input3=JOptionPane.showInputDialog("Enter the row you wish to move in.");
               int rowmove2=Integer.parseInt(input3);
               String input4=JOptionPane.showInputDialog("Enter the column you wish to move in.");
               int colmove2=Integer.parseInt(input4);
               board.getMove(rowmove2,colmove2,player2);
               board.printBoard();
               board.checkWinner();
               if(board.checkWinner()==true)
                    System.out.println("O WINS");
                    System.exit(0);
String input5=JOptionPane.showInputDialog("Enter the row you wish to move in.");
          int rowmove3=Integer.parseInt(input5);
String input6=JOptionPane.showInputDialog("Enter the column you wish to move in.");
          int colmove3=Integer.parseInt(input6);
          board.getMove(rowmove3,colmove3,player1);
               board.printBoard();
               board.checkWinner();
               if(board.checkWinner()==true)
               System.out.println("X WINS");
                    System.exit(0);
}

Answer 1

if(board.checkWinner()==true)can be:
if(board.checkWinner())
if(winner==false)can be:
if(!winner)It isn't actually causing trouble right now, but the "x" in the constructor and in printBoard hide the member variable "x". If you aren't careful, you will use the wrong "x" somewhere and confuse yourself. Besides, the member variable "x" is a bad name. It should be "numberOfMoves" or something like that.
Once you fix what scsi-boy said with checkWinner, take out all but the last "return winner;" line, and change it to "if...else if...else if...else if..." It is cleaner to have only one return statement. And, once one "if" is true, you don't need to check the rest.