SetVisible(false) doesn't work with JDialog

Question

Hi evry one in this forum, i am using JDialog to get some inputs from user, when the user click on the ok button, i start processing and the JDialog must be invisible, for that i use myJdialog.setVisible(false) methode, but the JDialog is still visible, i may be use the wrong component or there is a problem.
I write some thing like this:
actionPerformed(){
//getinputs and make some controls
if(test){
this.setVisible(false);
//some traitments
//some traitments
}I think there is no thing wrong, not? what happen?

Answer 1

I am sorry, this is some thing complicated:
public class OpenKeyStore
    extends JDialog
    implements ActionListener, KeyListener, WindowListener {
  JPanel jPanel1 = new JPanel();
  Border border1;
  JLabel lprivateKey = new JLabel();
  JLabel lkeyPass = new JLabel();
  JTextField tkeyStorePath = new JTextField();
  JPasswordField tkeyPass = new JPasswordField();
  JButton bvalidate = new JButton();
  JButton bopenKeyStore = new JButton();
  JButton breset = new JButton();
  GridBagLayout gridBagLayout1 = new GridBagLayout();
  JFileChooser jfc = new JFileChooser();
  JOptionPane jop = new JOptionPane();
  private UploadParameters uploadParameters;
  private OpenRequest openRequest;
  private OpenResponse openResponse;
  private CheckCertRequest checkCertRequest;
  private CheckCertResponse checkCertResponse;
  private WaitBox waitBox;
  UploadApplet uploadApplet;
  public OpenKeyStore(Frame frame, String title, boolean modal) {
    super(frame, title, modal);
    try {
      jbInit();
    catch (Exception ex) {
      ex.printStackTrace();
  public OpenKeyStore(UploadApplet uploadApplet) {
    this(null, "", false);
    this.uploadApplet = uploadApplet;
  private void jbInit() throws Exception {
    uploadParameters = new UploadParameters();
    border1 = new EtchedBorder(EtchedBorder.RAISED, Color.white,
                               new Color(148, 145, 140));
    this.setModal(true);
    this.setTitle("Ouvrir");
    jPanel1.setBorder(border1);
    jPanel1.setLayout(gridBagLayout1);
    jPanel1.setSize(400, 140);
    lprivateKey.setText("Cl� priv�e :");
    lkeyPass.setText("Mot de passe : ");
    bopenKeyStore.setActionCommand("openKeyStore");
    bopenKeyStore.setText("Ouvrir");
    bopenKeyStore.setMnemonic(KeyEvent.VK_O);
    bopenKeyStore.addKeyListener(this);
    bopenKeyStore.addActionListener(this);
    bvalidate.setActionCommand("bvalidate");
    bvalidate.setText("Valider");
    bvalidate.setMnemonic(KeyEvent.VK_V);
    bvalidate.addKeyListener(this);
    bvalidate.addActionListener(this);
    breset.setActionCommand("breset");
    breset.setText("R�etablir");
    breset.setMnemonic(KeyEvent.VK_R);
    breset.addKeyListener(this);
    breset.addActionListener(this);
    tkeyStorePath.setText("C:\\y.p12");
    tkeyStorePath.addKeyListener(this);
    tkeyPass.setText("y");
    tkeyPass.addKeyListener(this);
    this.getContentPane().add(jPanel1, BorderLayout.CENTER);
    this.getContentPane().setSize(410, 150);
    jPanel1.add(lprivateKey, new GridBagConstraints(0, 0, 1, 1, 0.0, 0.0
        , GridBagConstraints.WEST, GridBagConstraints.NONE,
        new Insets(5, 5, 5, 5), 0, 0));
    jPanel1.add(lkeyPass, new GridBagConstraints(0, 1, 1, 1, 0.0, 0.0
                                                 , GridBagConstraints.WEST,
                                                 GridBagConstraints.NONE,
                                                 new Insets(5, 5, 5, 5), 0, 0));
    jPanel1.add(tkeyStorePath, new GridBagConstraints(1, 0, 1, 1, 10.0, 0.0
        , GridBagConstraints.WEST, GridBagConstraints.HORIZONTAL,
        new Insets(5, 5, 5, 5), 150, 0));
    jPanel1.add(tkeyPass, new GridBagConstraints(1, 1, 1, 1, 10.0, 0.0
                                                 , GridBagConstraints.WEST,
                                                 GridBagConstraints.HORIZONTAL,
                                                 new Insets(5, 5, 5, 5), 200, 0));
    jPanel1.add(bvalidate, new GridBagConstraints(1, 2, 1, 1, 0.0, 0.0
                                                  , GridBagConstraints.EAST,
                                                  GridBagConstraints.NONE,
                                                  new Insets(5, 5, 5, 5), 0, 0));
    jPanel1.add(bopenKeyStore, new GridBagConstraints(2, 0, 1, 1, 0.0, 0.0
        , GridBagConstraints.CENTER, GridBagConstraints.NONE,
        new Insets(5, 5, 5, 5), 0, 0));
    jPanel1.add(breset, new GridBagConstraints(2, 2, 1, 1, 0.0, 0.0
                                               , GridBagConstraints.CENTER,
                                               GridBagConstraints.NONE,
                                               new Insets(5, 5, 5, 5), 0, 0));
    this.initFileChooser();
    this.tkeyStorePath.requestFocus();
    this.pack();
    Rectangle screenRect = this.getGraphicsConfiguration().getBounds();
    this.setLocation(
        screenRect.x + screenRect.width / 2 - this.getSize().width / 2,
        screenRect.y + screenRect.height / 2 - this.getSize().height / 2);
    this.show();
  public void actionPerformed(ActionEvent e) {
    if (e.getActionCommand().equals("openKeyStore")) {
      this.showOpenFileChooser();
      return;
    if (e.getActionCommand().equals("bvalidate")) {
      this.acte();
      return;
    if (e.getActionCommand().equals("breset")) {
      this.reset();
      return;
  public void keyPressed(KeyEvent e) {
    if (e.getKeyCode() == KeyEvent.VK_ENTER) {
      if (e.getSource() == this.bopenKeyStore ||
          e.getSource() == this.tkeyStorePath) {
        this.showOpenFileChooser();
        return;
      if (e.getSource() == this.bvalidate ||
          e.getSource() == this.tkeyPass) {
        this.acte();
        return;
      if (e.getSource() == this.breset) {
        this.reset();
        return;
  public void keyReleased(KeyEvent e) {}
  public void keyTyped(KeyEvent e) {
  private void initFileChooser() {
    jfc.setFileFilter(new javax.swing.filechooser.FileFilter() {
      public boolean accept(File f) {
        return (f.getName().endsWith(".p12") || f.isDirectory());
      public String getDescription() {
        return "(.p12) fichier key store";
    jfc.setDialogTitle("Selectionnez un fichier .p12");
    jfc.setMultiSelectionEnabled(false);
    jfc.setDialogType(JFileChooser.OPEN_DIALOG);
    jfc.setFileSelectionMode(JFileChooser.FILES_ONLY);
  private void showOpenFileChooser() {
    int returnVal = jfc.showOpenDialog(this);
    if (returnVal == JFileChooser.APPROVE_OPTION && jfc.getSelectedFile() != null &&
        jfc.getSelectedFile().exists()) {
      this.tkeyStorePath.setText(jfc.getSelectedFile().getAbsolutePath());
      this.tkeyPass.requestFocus();
    else {
      this.tkeyStorePath.requestFocus();
  private void reset() {
    this.tkeyStorePath.setText("");
    this.tkeyStorePath.requestFocus();
    this.tkeyPass.setText("");
  private void acte() {
//waitBox = new WaitBox();
    openRequest = new OpenRequest();
    openRequest.setStorePath(this.tkeyStorePath.getText());
    if (!openRequest.isValide()) {
      jop.showMessageDialog(null,
                            "S.V.P v�rifiez le chemin de votre cl�!",
                            "Echec...", jop.ERROR_MESSAGE);
      this.tkeyStorePath.requestFocus();
      this.tkeyStorePath.selectAll();
      return;
    openRequest.setStorePass(new String(this.tkeyPass.getPassword()));
    openRequest.setReciverCert(this.uploadParameters.getReciverCert());
    OpenAction openAction = new OpenAction(this.openRequest);
    try {
      while (openResponse == null) {
        Thread.sleep(100);
        openResponse = (OpenResponse) openAction.getResponse();
    catch (Exception e) {
      e.printStackTrace();
    if (openResponse.getSenderPK() == null) {
      jop.showMessageDialog(null,
          "S.V.P entrez une cl� valide, \n ou verifiez votre mot de passe !",
          "Echec...", jop.ERROR_MESSAGE);
      this.tkeyStorePath.requestFocus();
      this.tkeyStorePath.selectAll();
      return;
    if (openResponse.getCaCert() == null) {
      this.setVisible(false);
      jop.showMessageDialog(null,
          "Votre cl� n'est pas valide.\n contactez votre administrateur!",
          "Echec...", jop.ERROR_MESSAGE);
      this.gotoPreviousPage();
      return;
    if (this.uploadParameters.getCipher() && openResponse.getReciverCert() == null) {
this.setVisible(false);//*********************Does not work
jop.showMessageDialog(null,
          "Vous ne disposez pas de certificat pour votre correspondant!",
          "Echec...", jop.ERROR_MESSAGE);
      this.gotoPreviousPage();
      return;
    this.setVisible(false);//*********************Does not work
    if (this.uploadParameters.getCipher()) {
      String compte;
      while (true) {
        compte = (String) JOptionPane.showInputDialog(
            this, "S.V.P. entrez le compte de votre correspondant : ",
            "Customized Dialog", JOptionPane.PLAIN_MESSAGE, null, null, "");
        if (compte == null) {
          //gotoprevious page
          return;
        this.checkCertRequest.setCommunName(compte);
        if (this.checkCertRequest.isValide()) {
          this.checkCertRequest.setReciverCert(this.openResponse.getReciverCert());
          this.checkCertRequest.setCaCert(this.openResponse.getCaCert());
          CheckCertAction checkCertAction = new CheckCertAction(this.
              checkCertRequest);
          try {
            while (this.checkCertResponse == null) {
              Thread.sleep(100);
              this.checkCertResponse = (CheckCertResponse) checkCertAction.
                  getResponse();
          catch (Exception e) {
            e.printStackTrace();
          if (this.checkCertResponse.getReciverCertState()) {
            return;
          else {
            jop.showMessageDialog(null,
                "L'identit� de votre correspondant n'a pas pu etre v�rifier!",
                "Echec...", jop.ERROR_MESSAGE);
  public void windowActivated(WindowEvent e) {}
  public void windowClosed(WindowEvent e) {}
  public void windowDeactivated(WindowEvent e) {}
  public void windowDeiconified(WindowEvent e) {}
  public void windowIconified(WindowEvent e) {}
  public void windowOpened(WindowEvent e) {}
  public void windowClosing(WindowEvent e) {
    this.gotoPreviousPage();
  public void gotoPreviousPage() {
}