JList selection error

Question

Hi!
I haveing great problems with getting the JList to function properly and dont understand why.
The problem is that no selection of the JList can be performed at all, etc no highligting at all.
Anyone who sees the error? This is my first time buliding this kind of object, so any suggestions to improvements is also welcome!
Best regards Wargrammer
code:
public class GUI2 implements ListSelectionListener{
private MyListModel JposListModel = new MyListModel();
private JList JposAdapterList     = new JList(JposListModel);
      public GUI2(){
                     JposAdapterList.setCellRenderer(render);
     JposAdapterList.setBackground(Color.BLACK);
     JposAdapterList.setFocusable(true);
     JposAdapterList.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
     JposAdapterList.setSelectedIndex(0);
     JposAdapterList.addListSelectionListener(this);
    public void update(){
          JposListModel.updateList();
class MyListModel extends DefaultListModel{
       void updateList(){
     Vector results = testResultModel.getTestResults();
     this.removeAllElements();
     for (int i = 0; i < results.size(); i++){                    
                    this.addElement(results.get(i));
     this.setSize(results.size());                              
public class CellRenderer extends DefaultListCellRenderer {
            public Component getListCellRendererComponent(JList list,
     Object value,
     int index,                                        boolean isSelected,
     boolean hasFocus) {
                    JLabel label = new JLabel();
     if (value instanceof ObjectImpl) {
                  ObjectImpl temp = (ObjectImpl)value;
                  label.setText(temp.getTypeId() + " : " + temp.getLogicalName());
                   if(temp.getStatus()< 0){
          label.setForeground(Color.RED);
       return label;
}

Answer 1

The basic problem is that you are not setting the fg/bg colours according to whether the cell is selected in your ListCellRenderer. If you look at the source code for DefaultListCellRenderer then you will see that it does this.
More serious, though, is that you are extending DefaultListCellRenderer, which is itself a label and a suitable object to return but creating a new JLabel every time the renderer is called and returning that instead. This is not only pointless but wasteful.
Try changing your code to either call the super class method and then do your own stuff on this or get the necessary highlight code ideas from the JDK source and extend JLabel yourself. In both cases, realise that the label is reused by swing whenever it wants to paint a cell, and you don't want to create a new one.