closing a JDialog

[Hasnain S]

i have search here but non have my problem....ahem

in my custom dialog that opens up when u click ok or cancel it simply goes
addCust.dialog.setVisible(false);

this way the window disappears by in my taskbar i can still see that java session alive

how do i kill it

i tried
dialog.setVisible(false);
dialog.dispose();

and then
dialog.dispose();

but it doesnt work

i want that session KILLED!!!

[Jesper de Jong]

Try:

System.exit(0);

[Hasnain S]

i already did....it killed the whole [program

i just want the dialog to die....it disappears but stays in the taskbar
[ July 22, 2006: Message edited by: Hasnain SS ]

[Hasnain S]

well i still havent solved the problem so if anyone can help...please do so...


bump

[Michael Dunn]

what shows in the taskbar is the dialog's parent, not the dialog.

can you post a simple demo program, so we can see exactly how you are doing this

[Hasnain S]

i could send it coz the program code is too big to post here

i am pretty sure that the ghost is of the Jdialog

pm me if possible

[Hasnain S]

i have sent at ur email

plz chek it

[Michael Dunn]

the part creating the extra taskbar item is caused by the code in addCust
going around in circles i.e. you seem to be creating a separate dialog/frame
within the dialog itself, instead of using the class as a dialog (extends JDialog)

a couple of changes in mainFrame
1) starting at line 587

    
class AddListener implements ActionListener {
    public void actionPerformed(ActionEvent e)
    {
    if (nameList.getSelectedIndex() == -1)
    {
    ArrayList<Item> selectedNames = new ArrayList<Item>();
//selectedNames = addCust.showDialog(centerPanel);//<----------------------------------------------------------------------
addCust ac = new addCust(mainFrame.this);//<----------------------------------------------------------------------
//selectedNames = addCust.showDialog(mainFrame.this);//<----------------------------------------------------------------------
selectedNames = ac.showDialog();//<----------------------------------------------------------------------
    if (selectedNames != null)
    {
 
    Item received = selectedNames.get(0);
    String type = received.getType();
       String namer = received.getName();
       String citi = received.getCity();
    int pbox = received.getBox();
    int fone = received.getTelephone();
    int fax = received.getFax();
    String nett = received.getNetAddress();
 
    typeModel.addElement(type);
    nameModel.addElement(namer);
    cityModel.addElement(citi);
    boxModel.addElement(pbox);
    telModel.addElement(fone);
    faxModel.addElement(fax);
    adyModel.addElement(nett);
    }
    else
    {
    //Do nothing if the null Array Pointer is passed(Canel button is clicked).
    errorField.setText("Add new Item cancelled.");
    }
  }
  else
  {
  JOptionPane.showMessageDialog(null,"Please Unselect a field before adding new customers.","Error : Unselect",JOptionPane.ERROR_MESSAGE);
 
  }
}
}

2) added a 'main' to mainFrame

    
 public static void main(String[] args)
 {
   mainFrame mf = new mainFrame();
   mf.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
   mf.setBounds(100,100,400,300);
   mf.setVisible(true);
 }
 

and this is the modified addCust code

    
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import java.util.ArrayList;
 
public class addCust extends JDialog implements ActionListener {
    private static addCust dialog;
    private static ArrayList<Item> value;
    private JTextField nameField;
    private JTextField boxxField;
    private JTextField teleField;
    private JTextField faxField;
    private JTextField netField;
    private JTextField typeField;
 
    private JComboBox cityCombo;
    private Item item;
 

    //public static ArrayList showDialog(Component frameComp)
    //public static ArrayList showDialog(JFrame frameComp)
    public static ArrayList showDialog()//<----------------------------------------------------------------------all of this section
    {
        //Frame frame = JOptionPane.getFrameForComponent(frameComp);
        //dialog = new addCust(frame);
        //dialog = new addCust(frameComp);
        //dialog.setSize(500,360);
        //dialog.setResizable(false);
        //dialog.setVisible(true);
 
        return value; //Passes back an ArrayList with the Item Object
    }
 
//private addCust(Frame frame) {//<----------------------------------------------------------------------
public addCust(JFrame frame) {//<----------------------------------------------------------------------note: public
        super(frame, true);
 
    Container contentPane = getContentPane();
    contentPane.setLayout(null);
 

    final JButton okButton = new JButton("OK");
   okButton.setBounds(30,250,100,20);
       okButton.setActionCommand("OK");
        okButton.addActionListener(this);
        getRootPane().setDefaultButton(okButton);
 
        getContentPane().add(okButton);
 

    JButton can = new JButton("Cancel");
    can.setBounds(150,250,100,20);
    getContentPane().add(can);
    can.setActionCommand("Cancel");
        can.addActionListener(this);
 

    // Setting up Labels and Fields
    JLabel req = new JLabel("* Indicates Required Fields");
    req.setBounds(30,280,150,65);
    getContentPane().add(req);
 
/////////////////////////////
 
JLabel typer = new JLabel("*Customer Type : ");
    typer.setBounds(30,0,120,65);
    getContentPane().add(typer);
 
    typeField = new JTextField();
    typeField.setText("N/A");
    typeField.setBounds(130,20,150,25);
    getContentPane().add(typeField);
 
    ///////////////////////////////
 
    JLabel nam = new JLabel("*Name : ");
    nam.setBounds(30,30,120,65);
    getContentPane().add(nam);
 
    nameField = new JTextField();
    nameField.setText("N/A");
        nameField.setBounds(130,50,200,25);
        getContentPane().add(nameField);
 

    //////////////////////////////
 
    JLabel citi = new JLabel("City : ");
    citi.setBounds(30,60,120,65);
    getContentPane().add(citi);
 
    String[] citiStrings = { "Abu Dhabi", "Dubai", "Sharjah", "Ajman", "Umm Al-Qaiwain", "Ras Al-Khaimah", "Fujairah", "Oman" };
 
    cityCombo = new JComboBox(citiStrings);
    cityCombo.setBounds(130,80,100,25);
    getContentPane().add(cityCombo);
 
    //////////////////////////////
 
    JLabel upc = new JLabel("Post Box No: ");
    upc.setBounds(30,90,120,65);
    getContentPane().add(upc);
 
    boxxField = new JTextField();
    boxxField.setText("0");
    boxxField.setBounds(130,110,100,25);
    getContentPane().add(boxxField);
    //////////////////////////////
 

    JLabel des = new JLabel("Telephone : ");
    des.setBounds(30,120,120,65);
    getContentPane().add(des);
 
    teleField = new JTextField();
    teleField.setText("0");
    teleField.setBounds(130,140,150,25);
    getContentPane().add(teleField);
 
    JLabel mesg = new JLabel("Numbers Only. No Spaces allowed");
    mesg.setBounds(290,120,200,65);
    getContentPane().add(mesg);
 
    ///////////////////////////////
 
    JLabel fax = new JLabel("Fax : ");
    fax.setBounds(30,150,120,65);
    getContentPane().add(fax);
 
    faxField = new JTextField();
    faxField.setText("0");
    faxField.setBounds(130,170,150,25);
    getContentPane().add(faxField);
 
    JLabel mesg2 = new JLabel("Numbers Only. No Spaces allowed");
    mesg2.setBounds(290,150,200,65);
    getContentPane().add(mesg2);
 
    ///////////////////////////////
 
    JLabel net = new JLabel("Net Address : ");
    net.setBounds(30,180,120,65);
    getContentPane().add(net);
 
    netField = new JTextField();
    netField.setText("N/A");
    netField.setBounds(130,200,200,25);
    getContentPane().add(netField);
 
    ///////////////////////////////
 

 
    item = new Item();
        //pack();//<----------------------------------------------------------------------
setSize(500,360);//<----------------------------------------------------------------------
setResizable(false);//<----------------------------------------------------------------------
setVisible(true);//<----------------------------------------------------------------------
    }
    //**********Time for Action
public void actionPerformed(ActionEvent e)
    {
        if ("OK".equals(e.getActionCommand()))
        {
        item.setType(typeField.getText());
        item.setName(nameField.getText());
        item.setCity(cityCombo.getSelectedItem().toString());
 
        item.setBox(Integer.parseInt(boxxField.getText()));
        item.setTelephone(Integer.parseInt(teleField.getText()));
        item.setFax(Integer.parseInt(faxField.getText()));
        value = new ArrayList<Item>();
    item.setNet(netField.getText());
    value.add(item);
//addCust.dialog.setVisible(false);//<----------------------------------------------------------------------
dispose();//<----------------------------------------------------------------------
        }
        else
        {
//addCust.dialog.setVisible(false);//<----------------------------------------------------------------------
dispose();//<----------------------------------------------------------------------
        }
 
}
 
}

[Hasnain S]

ok it didnt work

plz bear with me...im going to list the changes...

No.1=================================

//selectedNames = addCust.showDialog(centerPanel);//<-------cancelled

addCust ac = new addCust(mainFrame.this);//<----------added

//selectedNames = addCust.showDialog(mainFrame.this); //<---this wasnt there when i put it it give me error
selectedNames = ac.showDialog();//<-----i added this

No.2==================================

why did u add a main to the mainFrame? I already had one in dFreeze.java

thats how i learnt it......to keep main away from the code....

anyway i added it in the end just before the last }...and surprisngly it compiled and ran normally

No.3==================================
//public static ArrayList showDialog(Component frameComp)
//public static ArrayList showDialog(JFrame frameComp)
public static ArrayList showDialog()//

i did not understand this part so i removed the first line, the second wasnt there anyway, and added the last one.....

then changed from private to public and in the same line from Frame to JFrame

removed pack(); and added the three lines....

replaced addCust.dialog.setVisible(false) with dispose();

----------------------------------------------------------------
****************************************************************
-----------------------------------------------------------------

i believe thats it....if i missed out something please correct me....or if the code you edited works fine on ur comp then just send the edited files back.....thx for helping me out
[ July 24, 2006: Message edited by: Hasnain SS ]

[Hasnain S]

can u also tell me why is it that when i load a csv file i get 3 new ghost windows opened?

edit : and here is a pic of the above problem

http://img153.imageshack.us/img153/7374/untitledyj8.png
[ July 24, 2006: Message edited by: Hasnain SS ]

[Michael Dunn]

left out one other change in mainFrame, line 102
//this.setVisible(true);//comment out

thought I had checked the 8 files for main(), but missed it in dFreeze.java
that's the only reason I added it. can now be removed from mainFrame.java

the other 2 code changes are just straight swaps
1) listener class
2) addCust.java

the extra change above stops all the taskbar entries, but there's too much
code for me to wade through to see if it's a fix, or just a mask.

[Hasnain S]

WOW cant believe that one line made the difference

THANKS ALOT Michael Dunn, you really helped me out here....



[ July 24, 2006: Message edited by: Hasnain SS ]