Java - JList: Custom ListSelectionListener question (SSCCE inclus)

Question

Je suis en train de mettre des actions sur un ListSelectionListener personnalisé et bien que tout compile bien passé lorsque je sélectionne en fait un composant du JList il ne fonctionne pas.

Voici un extrait de code:

public class ListSelectionHandler implements ListSelectionListener 
{ 
    ListCustomObject o; 

    @Override 
    public void valueChanged(ListSelectionEvent e) 
    { 
      o = (ListCustomObject) app.MainWindow.jList1.getModel() 
        .getElementAt(e.getFirstIndex()); 

      new app.actions.Actions().createSetEdgeColorTo(o.getColor()); 
    } 

} 

L'action que je fais appel, travaille et il n'y a pas d'erreur lors de la compilation. Mais rien ne se passe réellement.

Je sais que je ne suis pas beaucoup de détails dans y compris ce code, je veux juste demander si je fais une erreur logique dans cet événement.

Merci d'avance!

EDIT: Ajout de l'action et l'initialisation de JList:

public Action createSetEdgeColorTo(Color color) 
    { 
     return new SetEdgeColorTo(color); 
    } 

    class SetEdgeColorTo extends AbstractAction 
    { 

     Color color; 

     SetEdgeColorTo(Color color) 
     { 

      super("Set new Edge Color"); 
      this.color = color; 
     } 

     @Override 
     public void actionPerformed(ActionEvent evt) 
     { 
      app.graph.GraphEdit.view.getGraph2D().getDefaultEdgeRealizer() 
               .setLineColor(color); 
      app.graph.GraphEdit.view.getGraph2D().updateViews(); 
     } 
    } 

et

JList jList1 = new javax.swing.JList(); 
ListSelectionModel listSelectionModel = jList1.getSelectionModel(); 
listSelectionModel.addListSelectionListener(new app.jlist 
              .ListSelectionHandler()); 

EDIT 3: Reworked SSCCE:

import java.awt.event.ActionEvent; 
import javax.swing.AbstractAction; 
import javax.swing.DefaultListModel; 
import javax.swing.JList; 
import javax.swing.ListSelectionModel; 
import javax.swing.event.ListSelectionListener; 
import javax.swing.event.ListSelectionEvent; 
import javax.swing.Action; 
import javax.swing.JFrame; 

public class SSCCE 
{ 

    static JList jList1; 

    public static void main(String[] args) 
    { 

     JFrame frame = new JFrame(); 
     jList1 = new JList(); 
     ListSelectionModel listSelectionModel = jList1.getSelectionModel(); 
     listSelectionModel.addListSelectionListener(
       new ListSelectionHandler()); 
     DefaultListModel listModel = new DefaultListModel(); 
     jList1.setModel(listModel); 
     listModel.addElement("String"); 
     listModel.addElement("String two"); 

     frame = new JFrame(); 
     frame.setDefaultCloseOperation(1); 
     frame.add(jList1); 
     frame.pack(); 
     frame.setVisible(true); 

    } 
} 

class ListSelectionHandler implements ListSelectionListener 
{ 
    @Override 
    public void valueChanged(ListSelectionEvent e) 
    { 

     System.out.println("" + e.getFirstIndex()); 
     new Actions().createTestAction(); 
    } 
} 

class Actions 
{ 

    public Action createTestAction() 
    { 
     return new TestAction(); 
    } 

    class TestAction extends AbstractAction 
    { 

     TestAction() 
     { 
      super("Test Action"); 
     } 

     @Override 
     public void actionPerformed(ActionEvent evt) 
     { 
      System.out.println("Test Action Fired!"); 
     } 
    } 
} 

Ce SSCCE décrit le problème exact avec un échantillon TestAction qui ne tire pas encore .

Answer 1

Alors, qu'essayez-vous exactement de faire?

Si vous essayez de définir la couleur d'une ligne qui a été sélectionné, ce code doit être fait dans le moteur de rendu. En général chaque fois que vous jouez avec des couleurs dans un moteur de rendu dont vous avez besoin du chèque:

if (! isSelected) 
    // do you custom rendering 

De cette façon, la ligne affichera toujours la valeur par défaut soulignant que vous sélectionnez des lignes différentes.

Et bien sûr, un SSCCE doit encore être affiché parce que nous avons aucune idée de ce que votre action personnalisée tente de faire et ne peut donc pas faire des suggestions réelles.

Modifié:

Il ne fait rien parce que tout votre code est en train de faire est la création de l'action. Si vous voulez invoquer l'action alors le code devrait être quelque chose comme:

new Actions().createTestAction().actionPerformed(null); 

En réalité, il n'y a pas besoin de créer une action réelle, il vous suffit d'appeler directement une méthode qui fait ce que vous voulez. La raison pour laquelle vous créez une action serait si vous vouliez ajouter l'Actdion à un JButton ou un JMenuItem afin que l'utilisateur puisse cliquer sur le composant pour invoquer l'Action.