Dealing with Jlist

[Raj kalaria]

I have one frame in which i have a Jlist box.
AllGroupList
the AllGroupList populates some user name dynamically.
Now i have a edit button for that Jlist box.
The edit button will open a dialog.
The dialog will have two list box and search button to search new users so we can add users.
1) AvailableGrp_list
2) currentgrpmembership_listbox

currentgrpmembership_listbox is filled up when i call the dialog(by passing frame list model)

AvailableGrp_list is filled up by the search criteria

My problem is that when ever i add an element in the dialog from AvailableGrp_list to currentgrpmembership_listbox it gets added but at the same time the element also gets added in parent frame list AllGroupList.

why is it so?
Is this the correct way to pass a list from one frame to other dialog if we need the dialog to be initialized by a list from calling frame?

Once the user selects the users to be added, i want to load my AllGroupList(present in frame) with currentgrpmembership_listbox (which the user will upadte from dialog) can some one help in that also?

Frame code

  private void AllGroupEdit_btnActionPerformed(java.awt.event.ActionEvent evt) {
System.out.println("calling2");
    AddGroupDialog dy = new AddGroupDialog(new javax.swing.JFrame(),true, (DefaultListModel) AllGroupList.getModel());
dy.setVisible(true);
}
  

Dialog Code

   public class AddGroupDialog extends javax.swing.JDialog {
 
    /** Creates new form AddGroupDialog */
    public AddGroupDialog(java.awt.Frame parent, boolean modal, DefaultListModel cm) {
        super(parent, modal);
        initComponents();
        customint();
       currentgrpmembership_listbox.setModel(cm);
    } 
 private void Add_btnActionPerformed(java.awt.event.ActionEvent evt) {
 DefaultListModel adddlm =  (DefaultListModel) currentgrpmembership_listbox.getModel();
 Object selected[] = AvailableGrp_list.getSelectedValues();
 for (int i = 0, n = selected.length; i < n; i++) {     
               System.out.println("Adding element "+  selected[i]);
               adddlm.addElement((String) selected[i]);
           }
 
   
   
   
 
}
 
private void Remove_btnActionPerformed(java.awt.event.ActionEvent evt) {
   DefaultListModel delete_dlm =  (DefaultListModel) currentgrpmembership_listbox.getModel();
   Object selected[] = currentgrpmembership_listbox.getSelectedValues();
 for (int i = 0, n = selected.length; i < n; i++) {     
                System.out.println("Deleting element "+  selected[i]);
               delete_dlm.removeElement((String) selected[i]);
           }
 
}

[ November 14, 2008: Message edited by: Raj kalaria ]
[ November 14, 2008: Message edited by: Raj kalaria ]

[Reinhard Horn]

Hi Ray,

it seems to me that you need to create a second model for currentgrpmembership_listbox. You share pass your model to your target list, which means it is beeing shared by the listbox in the frame.

Kind Regards