Advance level drawing problem with Jframe and JPanel need optimize sol?

Question

Dear Experts,
I m trying to create a GUI for puzzle game following some kind of "game GUI template", but i have problems in that,so i tried to implement that in various ways after looking on internet and discussions about drawing gui in swing, but i have problem with both of these, may be i m doing some silly mistake, which is still out of my consideration. please have a look at these two and recommend me one of them, which is running without problems (flickring and when you enlarge window the board draw copies (tiled) everywhere,
Note: i don't want to inherit jpanel or Jframe
here is my code : import java.awt.BorderLayout;
public class GameMain extends JFrame {
     private static final long serialVersionUID = 1L;
     public int mX, mY;
     int localpoints = 0;
     protected static JTextField[][] squares;
     protected JLabel statusLabel = new JLabel("jugno");
     Label lbl_score = new Label("score");
     Label lbl_scorelocal = new Label("local score");
     protected static TTTService remoteTTTBoard;
     // Define constants for the game
     static final int CANVAS_WIDTH = 800; // width and height of the game screen
     static final int CANVAS_HEIGHT = 600;
     static final int UPDATE_RATE = 4; // number of game update per second
     static State state; // current state of the game
     private int mState;
     // Handle for the custom drawing panel
     private GameCanvas canvas;
     // Constructor to initialize the UI components and game objects
     public GameMain() {
          // Initialize the game objects
          gameInit();
          // UI components
          canvas = new GameCanvas();
          canvas.setPreferredSize(new Dimension(CANVAS_WIDTH, CANVAS_HEIGHT));
          this.setContentPane(canvas);
          this.setDefaultCloseOperation(EXIT_ON_CLOSE);
          this.pack();
          this.setTitle("MY GAME");
          this.setVisible(true);
     public void gameInit() {     
     // Shutdown the game, clean up code that runs only once.
     public void gameShutdown() {
     // To start and re-start the game.
     public void gameStart() {
     private void gameLoop() {
     public void keyPressed(KeyEvent e) {
     public void keyTyped(KeyEvent e) {
     public void gameKeyReleased(KeyEvent e) {
          PuzzleBoard bd = getBoard();
          for (int row = 0; row < 4; ++row) {
               for (int col = 0; col < 4; ++col) {
                    if (e.getSource() == squares[row][col]) {
                         if (bd.isOpen(col, row)) {
                              lbl_score.setText("Highest Score = "
                                        + Integer.toString(bd.getPoints()));
                              setStatus1(bd);
                              pickSquare1(col, row, squares[row][col].getText()
                                        .charAt(0));
     protected void pickSquare1(int col, int row, char c) {
          try {
               remoteTTTBoard.pick(col, row, c);
          } catch (RemoteException e) {
               System.out.println("Exception: " + e.getMessage());
               e.printStackTrace();
               System.exit(1);
     // method "called" by remote object to update the state of the game
     public void updateBoard(PuzzleBoard new_board) throws RemoteException {
          String s1;
          for (int row = 0; row < 4; ++row) {
               for (int col = 0; col < 4; ++col) {
                    squares[row][col].setText(new_board.ownerStr(col, row));
          lbl_score.setText("Highest Score = "
                    + Integer.toString(new_board.getPoints()));
          setStatus1(new_board);
     protected void setStatus1(PuzzleBoard bd) {
          boolean locals = bd.getHave_winner();
          System.out.println("local win" + locals);
          if (locals == true) {
               localpoints++;
               System.out.println("in condition " + locals);
               lbl_scorelocal.setText("Your Score = " + localpoints);
          lbl_score
                    .setText("Highest Score = " + Integer.toString(bd.getPoints()));
     protected PuzzleBoard getBoard() {
          PuzzleBoard res = null;
          try {
               res = remoteTTTBoard.getState();
          } catch (RemoteException e) {
               System.out.println("Exception: " + e.getMessage());
               e.printStackTrace();
               System.exit(1);
          return res;
     /** Custom drawing panel (designed as an inner class). */
     class GameCanvas extends JPanel implements KeyListener {
          /** Custom drawing codes */
          @Override
          public void paintComponent(Graphics g) {
               // setOpaque(false);
               super.paintComponent(g);
               // main box; everything placed in this
               // JPanel box = new JPanel();
               setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));
               // add(statusLabel, BorderLayout.NORTH);
               // set up the x's and o's
               JPanel xs_and_os = new JPanel();
               xs_and_os.setLayout(new GridLayout(5, 5, 0, 0));
               squares = new JTextField[5][5];
               for (int row = 0; row < 5; ++row) {
                    for (int col = 0; col < 5; ++col) {
                         squares[row][col] = new JTextField(1);
                         squares[row][col].addKeyListener(this);
                         if ((row == 0 && col == 1) || (row == 2 && col == 3)
                         || (row == 1 && col == 4) || (row == 4 && col == 4)
                                   || (row == 4 && col == 0))
                              JPanel p = new JPanel(new BorderLayout());
                              JLabel label;
                              if (row == 0 && col == 1) {
                                   label = new JLabel("1");
                                   label.setHorizontalAlignment(JLabel.LEFT);
                                   label.setVerticalAlignment(JLabel.TOP);
                              else if (row == 4 && col == 0) {// for two numbers or
                                   // two
                                   // blank box in on row
                                   label = new JLabel("2");
                                   label.setHorizontalAlignment(JLabel.LEFT);
                                   label.setVerticalAlignment(JLabel.TOP);
                              else if (row == 1 && col == 4) {
                                   label = new JLabel("3");
                                   label.setHorizontalAlignment(JLabel.LEFT);
                                   label.setVerticalAlignment(JLabel.TOP);
                              else if (row == 4) {
                                   label = new JLabel("4");
                                   label.setHorizontalAlignment(JLabel.LEFT);
                                   label.setVerticalAlignment(JLabel.TOP);
                              else {
                                   label = new JLabel("5");
                                   label.setHorizontalAlignment(JLabel.LEFT);
                                   label.setVerticalAlignment(JLabel.TOP);
                              label.setOpaque(true);
                              label.setBackground(squares[row][col].getBackground());
                              label.setPreferredSize(new Dimension(label
                                        .getPreferredSize().width, squares[row][col]
                                        .getPreferredSize().height));
                              p.setBorder(squares[row][col].getBorder());
                              squares[row][col].setBorder(null);
                              p.add(label, BorderLayout.WEST);
                              p.add(squares[row][col], BorderLayout.CENTER);
                              xs_and_os.add(p);
                         } else if ((row == 2 && col == 1) || (row == 1 && col == 2)
                                   || (row == 3 && col == 3) || (row == 0 && col == 3)) {
                              xs_and_os.add(squares[row][col]);
                              // board[ row ][ col ].setEditable(false);
                              // board[ row ][ col ].setText("");
                              squares[row][col].setBackground(Color.RED);
                              squares[row][col].addKeyListener(this);
                         } else {
                              squares[row][col] = new JTextField(1);
                              // squares[row][col].addActionListener(this);
                              squares[row][col].addKeyListener(this);
                              xs_and_os.add(squares[row][col]);
               this.add(xs_and_os);
               this.add(statusLabel);
               this.add(lbl_score);
               this.add(lbl_scorelocal);
          public void keyPressed(KeyEvent e) {
          public void keyReleased(KeyEvent e) {
               gameKeyReleased(e);
          public void keyTyped(KeyEvent e) {
     // main
     public static void main(String[] args) {
          SwingUtilities.invokeLater(new Runnable() {
               @Override
               public void run() {
                    new GameMain();
  thanks a lot for your time , consideration and efforts.
jibby
Edited by: jibbylala on Sep 20, 2010 6:06 PM

Answer 1

jibbylala wrote:
thanks for mentioning as i wasn't able to write complete context here.Yep thanks camickr. I think that Darryl's succinct reply applies here as well.