Server GUI

[Theo van Loon]

Hello all, i have a simple GUI for my server which displays just a single line and a button which enables the user to close the server.

GUI looks fine but ones i have dragged the gui i can't seem to click the button and it is below my button, and text seems to be displayed below the original text, so i have to click below the original button and there seems to be the button.

It seems like once i've dragged the GUI it repaints below the original fields and all goes wrong.

Very strange in order for you to be able to help i'll show a couple of snippets of my code :

 
ServerGUI :
 
public class ServerGUI extends JDialog implements ServerGUIControl
{
   private JTextField serverText;
   private JPanel buttonPanel;
   private JPanel messagePanel;
   private JButton closeButton;
 
   /**
    * Constructor that initializes the client GUI and sets up the listeners
    * for the event handling methods.
    *
    * @param _serverGUIController The controller object for this GUI.
    */
   public ServerGUI(ServerGUIController _serverGUIController)
   {
      init();
      addListeners(_serverGUIController);
      pack();
   }
 
   /* (non-Javadoc)
    * This method initializes the server GUI.
    */
   private void init()
   {
 
      setTitle(GUIConstants.TITLE_SERVER);
      setDefaultCloseOperation(DO_NOTHING_ON_CLOSE);
      setResizable(false);
      
      Container contentPane = getContentPane();
      contentPane.setLayout(new BorderLayout());
 
      buildServerText();
 
      buildMessagePanel();
 
      buildButtonPanel();
 
      contentPane.add(buttonPanel, BorderLayout.CENTER);
      contentPane.add(messagePanel, BorderLayout.NORTH);
   }
 
   /* (non-Javadoc)
    * This method builds the textarea for the server message.
    */
   private void buildServerText()
   {
      serverText = new JTextField();
      
      serverText.setEditable(false);
      serverText.setFocusable(false);
      serverText.setBorder(null);
 
      // Setting the Font of the text to bold.
      Font currentFont = serverText.getFont();
      Font boldFont = new Font(currentFont.getName(), Font.BOLD, currentFont.getSize());
      serverText.setFont(boldFont);
 
      // Storing null in background means use the background of the ServerGUI. 
      serverText.setBackground(null);
   }
 
   /* (non-Javadoc)
    * This method builds the panel for the server message and
    * populates it with the server text area.
    */
   private void buildMessagePanel()
   {
 
      // Building message panel.
      messagePanel = new JPanel();
      messagePanel.setLayout(new GridBagLayout());
      messagePanel.setBorder(BorderFactory.createEmptyBorder(10, 10, 10, 10));
 
      GridBagConstraints messageConstraints;
      messageConstraints = new GridBagConstraints();
      messageConstraints.gridx = 0;
      messageConstraints.gridy = 0;
      messageConstraints.weightx = 1.0;
      messageConstraints.fill = GridBagConstraints.HORIZONTAL;
      messageConstraints.anchor = GridBagConstraints.WEST;
 
      messagePanel.add(serverText, messageConstraints);
 
      messageConstraints = new GridBagConstraints();
      messageConstraints.gridx = 0;
      messageConstraints.gridy = 0;
      messageConstraints.weighty = 1.0;
      messageConstraints.gridheight = 20;
 
      messagePanel.add(new JPanel(), messageConstraints);
      messagePanel.setPreferredSize(new Dimension(450, 75));
   }
 
   /* (non-Javadoc)
    * This method builds the buttonpanel.
    */
   private void buildButtonPanel()
   {
      buttonPanel = new JPanel();
      closeButton = new JButton(GUIConstants.BUTTON_CLOSE);
      closeButton.setActionCommand(GUIConstants.ACTION_COMMAND_CLOSE);
      closeButton.setMnemonic(KeyEvent.VK_C);
      closeButton.setToolTipText(GUIConstants.TOOL_TIP_SERVER_CLOSE);
 
      buttonPanel.setLayout(new FlowLayout(FlowLayout.RIGHT));
      buttonPanel.add(closeButton);
   }
 
   /** 
    * @see suncertify.server.ServerGUIControl#addListeners(suncertify.server.ServerGUIController)
    */
   public void addListeners(ServerGUIController _serverGUIController)
   {
      // Add window listener. 
      addWindowListener(_serverGUIController);
 
      // Add action and keylistener for button.
      closeButton.addActionListener(_serverGUIController);
      closeButton.addKeyListener(_serverGUIController);
   }
 
   // Getter methods
}

Code in the Controller :

 
public class ServerGUIController extends WindowAdapter implements ActionListener, KeyListener
{
   private ServerGUI serverGUI;
   private DatabaseRMIServer databaseRMIServer;
   private String serverMessage;
 
   /**
    * Constructor that creates the {@link ServerGUI}.
    *
    * @param _databaseRMIServer A reference to the {@link DatabaseRMIServer}
    *        object.
    */
   public ServerGUIController(DatabaseRMIServer _databaseRMIServer)
   {
      databaseRMIServer = _databaseRMIServer;
 
      if (databaseRMIServer != null)
      {
         // Server is set up successfully.
         serverMessage = MessageConstants.SERVERRUNNING;
         serverGUI = new ServerGUI(this);
         serverGUI.getServerText().setText(serverMessage);
         serverGUI.setVisible(true);
      }
      else
      {
         // Server is not set up successfully.
         JOptionPane.showMessageDialog(
            null,
            MessageConstants.SERVER_PROPERTIES_ERROR,
            GUIConstants.TITLE_ERROR_MESSAGE,
            JOptionPane.ERROR_MESSAGE);
      }
   }
 
   /**
    * This event handling method responds to the user clicking the close
    * button which causes the server to be closed.
    *
    * @param e The ActionEvent object.
    *
    * @see java.awt.event.ActionListener#actionPerformed(java.awt.event.ActionEvent)
    */
   public void actionPerformed(ActionEvent e)
   {
      if (e.getActionCommand().equals(GUIConstants.ACTION_COMMAND_CLOSE))
      {
         closeServer();
      }
   }
 
   /* (non-Javadoc)
    * Close the server and exit the application.
    */
   private void closeServer()
   {
      // Closing the server.
      if (showConfirmOption() == JOptionPane.OK_OPTION)
      {
         databaseRMIServer.closeServer();
         serverGUI.dispose();
         System.exit(0);
      }
   }
 
   /* (non-Javadoc)
    * This method requests the user to confirm the server should be closed.
    */
   private int showConfirmOption()
   {
      int selOption =
         JOptionPane.showConfirmDialog(
            serverGUI,
            "Are you sure you want to close the server?",
            GUIConstants.TITLE_WARNING_MESSAGE,
            JOptionPane.OK_CANCEL_OPTION,
            JOptionPane.WARNING_MESSAGE);
 
      return selOption;
   }
 
   /* (non-Javadoc)
    * @see java.awt.event.KeyListener#keyTyped(java.awt.event.KeyEvent)
    */
   public void keyTyped(KeyEvent e)
   {
      // Do nothing
   }
 
   /* (non-Javadoc)
    * @see java.awt.event.KeyListener#keyPressed(java.awt.event.KeyEvent)
    */
   public void keyPressed(KeyEvent e)
   {
      if (e.getKeyCode() == KeyEvent.VK_ENTER)
      {
         if (e.getSource() == serverGUI.getCloseButton())
         {
            closeServer();
         }
      }
   }
 
   /* (non-Javadoc)
    * @see java.awt.event.KeyListener#keyReleased(java.awt.event.KeyEvent)
    */
   public void keyReleased(KeyEvent e)
   {
      // Do nothing
   }
}
  

So that's all the code, the problem occurs when dragging the window to another position, so focus is lost. Same thing is happening to my proerties gui.

Can anybody help me out please?

[peter wooster]

An old debugging trick will come in handy on this one. Reduce the problem to the simplest example that still shows the incorrect behaviour. At that point you should be able to figure it out.

One question I have is why are you extending JDialog? I'd have a class the "has" a JFrame, and use that to build the GUI.

[Reza Rahman]

Theo, not to be too picky, but I'd be careful of posting too much code. The Sun non-disclosure stuff is pretty tricky...personally, I think the server-side GUI should be as minimal as:

1. Getting/confirming the startup options.
2. Showing some startup status information in a "startup screen" type deal.
3. Giving the user the abilty to shut-down with one click (on a link on the
startup screen somewhere?).