Discussion :

[paesportcomputing]

Bonjour,

J'ai un problème avec la gestion d'une liste observable utilisée dans un TableView.

Voici un bout de code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
private ObservableList<WrapperRoom> DB = FXCollections.observableArrayList();
 
//Charge la liste DB à partir de requête SQL...
DB = loadDB();
 
public void addTeam(Team t){
        WrapperRoom wr;
        for(Room r:t.getRooms()){
            wr = new WrapperRoom();
            wr.setDate(LocalDate.now());
            wr.setNameTeam(t.getNameTeam());
            wr.setChipNbr(t.getId());
            wr.setGameTime(LocalTime.parse("00:00"));
            wr.setRoom(new Room(r.getNameRoom()));
            DB.add(wr);
        }
}
 
public void deleteTeam(Team t){
        t.getRooms().forEach((Room r) -> {
            Predicate<WrapperRoom> wrPredicate = wr -> (wr.getDate().compareTo(LocalDate.now())==0)&&(wr.getNameTeam().equals(t.getNameTeam()))&&(wr.getRoom().getNameRoom().equals(r.getNameRoom()))&&(wr.getGameTime().equals(r.getGameTime()))&&(wr.getChipNbr().equals(t.getId()));
            System.out.println("DB size : "+DB.size());
            DB.removeIf(wrPredicate);
            System.out.println("DB size : "+DB.size());
        });
}

Donc, j'ai une liste observable DB. Quand j'ajoute un objet 'WrapperRoom' (-> addTeam) dans ma liste, mon TableView est bien mit à jour directement. Mais dès que je veux enlever un objet à l'aide de la méthode 'removeIf' (-> deleteTeam), mon (mes) objet n'est pas retiré. C'est bien les objets qui ne sont pas retirés de ma liste car j'ai testé avec un DB.size() avant et après l'appel à 'removeIf' et ma liste a tjrs le même nombre d'objets !
Si je ferme/réouvre mon application et que je veux retirer un objet, ça fonctionne.

Quelqu'un à une idée ?

Merci d'avance.

[bouye]

Ben oui mais jamais tu ne retires des trucs de la table.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
final WrapperRoom[] toRemove = t.getRooms()
    .stream()
    .filter(wr -> (wr.getDate().compareTo(LocalDate.now())==0)&&(wr.getNameTeam().equals(t.getNameTeam()))&&(wr.getRoom().getNameRoom().equals(r.getNameRoom()))&&(wr.getGameTime().equals(r.getGameTime()))&&(wr.getChipNbr().equals(t.getId()))
    .toArray(WrapperRoom[]::new);
// Clear DB.
Arrays.stream(toRemove).forEach(wr -> {
    [...]
});
// Clear table.
t.getRooms().removeAll(toRemove);

[paesportcomputing]

Bonjour et merci pour ton intervention.

Mais attention que je souhaite retirer des objets la liste DB et pas de t.getRooms.
En réalité, t.getRooms est une liste de Rooms. Pour chaque Room de cette liste -> t.getRooms().forEach((Room r) -> {...});, je vérifie dans la liste DB si des WrapperRoom conrrespondent au critères d'une Room. Les critères sont dans le Predicate. Si oui, je retire ce WrapperRoom.
Et comme dit dans mon message : Si je ferme/réouvre mon application et que je veux retirer un objet, ça fonctionne.

[bouye]

Tu as bien vérifié dans ce cas que toRemove contient quelques chose à la fin du stream ? Est-ce que ton prédicat est correct ?

Et du coté de WrapperRoom (on a pas son code), les propriété de la classe sont-elles observables pour que les lignes de la table se mettent à jour quand le contenu de ces objets change ?
Parce que là clairement tu effectues une action sur ta BD mais aucune action sur la table ou son contenu.

[paesportcomputing]

J'ai déclaré ma variable 'wrPredicate' à l'extérieur de la boucle for et Bizarrement, ça fonctionne.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
public void deleteTeam(Team t){
        // Permet de supprimer les WrapperRoom dans le DB_TableView
        Predicate<WrapperRoom> wrPredicate;
        for(Room r:t.getRooms()){
            wrPredicate = wr -> (wr.getDate().compareTo(LocalDate.now())==0)&&(wr.getNameTeam().equals(t.getNameTeam()))&&(wr.getRoom().getNameRoom().equals(r.getNameRoom()))&&(wr.getGameTime().equals(r.getGameTime()))&&(wr.getChipNbr().equals(t.getId()));
            DB.removeIf(wrPredicate);
        }
        IEscapeRoom_TeamsDAO escapeRoom_TeamsDAO = AbstractDAOFactory.getFactory().createEscapeRoom_TeamsDAO();
        escapeRoom_TeamsDAO.deleteTeam(t);
        teams.remove(t);
    }

Encore une petite question bouye : Je dois rafraîchir de temps en temps un TableView quand des données ont été modifiées dans ma DB. Je fais comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
if (data != null) {
        data.clear();
}
data.setAll(facade.getAllTeams());

Est-ce la bonne manière de procéder ?

[bouye]

Ben le soucis est toujours le meme : si les objets du modele ont des proprietes FX ou Beans observables, la mise a jour serait automatique : l'objet est mis a jour avec de nouvelles valeurs -> automatiquement la table set met a jour. Si ce sont des POJO, elle ne peut pas vraiment savoir ce qui a change ou pas sauf en modifiant completement le contenu de la liste (ce que fait ton code).

Apres il semble qu'il y ait eut un bug dans les version anciennes : JavaFX 2.1 TableView refresh items sur StackOverflow mentionne un bug qui etait ete resolu dans JavaFX 8_60.

Cette meme version a egalement introduit une nouvelle methode refresh() pour forcer la maj, mais c'est une methode privee .

EDIT - cette methode revient a faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

tableView.getProperties().put("tableRefreshKey", Boolean.TRUE);

[paesportcomputing]

Effectivement, je n'ai pas pensé préciser ma situation :
J'ai 2 pgm bien distinct mais qui utilisent la même DB.
A un moment, il y a un pgm qui met à jour des datas , qui sont affichées dans un Tableview du second pgm, dans la DB. Dès lors, je souhaite mettre à jour mon tableView avec les 'nouvelles' données de la DB.

Ma liste utilisée par mon TableView est bien Observable mais dans ce cas précis, c'est cette liste (plus précisément mon tableView) que je veux mettre à jour avec les nouvelles données de la DB.