Overriding JList selection

Question

Hello,
I am displaying a JList loaded with Strings inside a scrollpane. I want to make it such that clicking an item selects it, clicking it again unselects it, and multiple items can be selected at any one time. For this I wrote a MouseAdapter
public class ListListener extends MouseAdapter
  protected JList myList;
  protected String name;
  protected MyFrame parent;
  public ListListener(JList _myList, String _name, MyFrame _parent)
    myList = _myList;
    name = _name;
    parent = _parent;
  public void mouseClicked(MouseEvent e)
    int index = myList.locationToIndex(e.getPoint());
    if(e.getClickCount() == 1)
      parent.ClickItem(index, myList, name);
    else if(e.getClickCount() == 2)
      parent.DoubleClickItem(index, myList, name);
public class MyFrame extends JFrame
  protected static Vector<Integer> list1onVector = new Vector<Integer>(1, 1);
  protected static Vector<String> list1Vector = new Vector<String>(1, 1);
  protected static JList list1 = new JList();
  public MyFrame()
    //fill list1Vector
    list1.setListData(list1Vector);
    MouseListener list1Clicker = new ListListener(list1, "list1", this);
    list1.addMouseListener(list1Clicker);
  protected void ClickItem(int index, JList myList, String name)
    if(name.equalsIgnoreCase("list1")
      if(list1onVector.contains(Integer.valueOf(index)))
        list1onVector.remove(Integer.valueOf(index));
      else
        list1onVector.add(Integer.valueOf(index));
      inf[] list1Array = new int[list1onVector.size()];
      for(int i = 0; i < list1onVector.size(); i++)
        list1Array = list1onVector.get(i);
      myList.setSelectedIndices(list1Array);
      //do some processing based on what's selected
    else....
}That seems to work ok... most of the time. Every once in a while however, it will not select an entry or it will make some other entries deselected. Then on next click it shows it how it ought to be. I tried commenting out the code inside mouseClicked and selections were still going on, one at a time. So what I'm guessing is that the built in code that's handling the selections is messing with what I'm trying to do. I tried making my own JList object to deal with it
public class myJList extends JList
  public myJList()
  public void setSelectionIndex(int index)
    //do nothing here
  public void setSelectedObject(Object anObject, boolean shouldScroll)
    //do nothing here
}and using that instead of JList, but it didn't help. In fact, doesn't look like it's even going into those functions. What is actually originating the selection even, so I can cut it in the bud? Or is there a better way to do this? Any help is appreciated, thanks.

Answer 1

Well, I'll be darned, JList is in swing, not sure why i thought it was in awt.
I made the followowing change to mouseClicked, now looks like this:
Point point = e.getPoint();
int clickCount = e.getClickCount();
SafeClicks doSafeClick = new SafeClicks(myList, name, parent, point, clickCount);
SwingUtilities.invokeLater(doSafeClick);and ofcourse
protected class SafeClicks implements Runnable
  protected JList myList;
  protected String name;
  protected JFrame parent;
  protected Point point;
  protected int clickCount;
  public SafeClicks(JList _myList,
  String _name,
  JFrame _parent,
  Point _point,
  int _clickCount)
    myList = _myList;
    name = _name;
    parent = _parent;
    point = _point;
    clickCount = _clickCount;
  public void run()
    int index = myList.locationToIndex(point);
    if(clickCount == 1)
      parent.ClickItem(index, myList, name);
    else if(clickCount == 2)
     parent.DoubleClickItem(index, myList, name);
}Still the same issue. I think there IS an issue with thread timing and such, but I think the issue is that the JList object (default one) or one of its members is messing with the select at same time I am (and it's an issue if I go first it ends up bad, and if it goes first it ends up good). I think.