OO Conception avec UndoManager et Listeners sur un composant JComponent

Question

Lors de la création d'une interface utilisateur graphique dans Swing, je suis toujours confronté à un problème de conception qui met fin au piratage. Je voudrais connaître la bonne pratique à ce sujet.

Prenons un exemple simple.

J'ai un composant JComponent sur lequel j'aimerais implémenter des actions annuler/rétablir, et quelques actions qui ajoutent/suppriment d'autres composants. Donc, je voudrais commencer à écrire:

public class MyComponent extends JComponent { 
    private UndoManager undo = new UndoManager(); 
    public void addComponent(JComponent comp) { 
     this.add(comp);//Add the component to the view 
     undo.addEdit(new ComponentAddedEdit(comp));//make this action undoable 
     fireComponentAdded(comp);//warn the listeners that the action occured 
    } 
} 

Puis commence le problème. Dans mon ComponentAddedEdit je pense à quelque chose comme:

public class ComponentAddedEdit extends AbstractUndoableEdit { 
    private final JComponent comp; 
    public ComponentAddedEdit(JComponent comp) {this.comp = comp;} 
    @Override 
    public void undo() throws CannotUndoException { 
     MyComponent.this.removeComponent(comp); 
    } 
    @Override 
    public void redo() throws CannotRedoException { 
     MyComponent.this.addComponent(comp); 
    } 
} 

Bien sûr, cela ne fonctionne pas parce que l'action redo va créer une nouvelle Edit à UndoManager. Je dois donc créer une nouvelle méthode comme ça:

public void addComponentNoUndo() { 
     this.add(comp);//Add the component to the view 
     fireComponentAdded(comp);//warn the listeners that the action occured 
} 

En fin de compte, juste pour l'action « ajouter », je me retrouve avec 3 méthodes avec des noms similaires: add, addComponent et addComponentNoUndo. Si j'ai plus d'actions, plus complexes, cela peut devenir très confus. Alors, comment ferais-tu ça?

Answer 1

Je ne suis pas vraiment dans le swing, donc je ne peux pas vous donner une réponse de mise en œuvre spécifique, mais voir ce tutorial on memento pattern.

Vous avez sûrement besoin de l'adapter à vos besoins, mais je pense que c'est la voie à suivre. Essayez d'écrire votre propre classe "state" plutôt que d'utiliser simplement une chaîne de caractères.

Je veux alors annuler quelque chose, charger l'ancien état. Si vous ne voulez pas que l'utilisateur puisse annuler quelque chose, ne chargez tout simplement pas l'état précédent, en disant à l'utilisateur que ce n'est pas possible. Pensez à ajouter un drapeau booléen "undoable" à votre classe "state".

EDIT

quelque chose comme

class state extends JComponent { 

    private boolean undoable; 
    private JComponent toSave; 

    public state (boolean undoable, JComponent toSave, ...) { 
     this.undoable = undoable; 
     this.toSave = toSave; 
     ... 
    } 
... 
}