Discussion :

[JohnLocke]

Bonjour tous le monde,
j'ai un problème avec le tableView de javaFX, j'ai besoin de pouvoir y ajouter des éléments, et d'en supprimer, pour ajouter je n'ai pas eu de problème, mais quand je tente de supprimer un élément sélectionné, rien ne se passe

Code :

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

1
2
3
4
5
6
7
8
9
 
@FXML
public void delbanlist(ActionEvent event) {
ObservableList<banned> data = tableView.getItems();
data.remove(tableView.getSelectionModel().getSelectedItems());
System.out.println(String.valueOf(tableView.getSelectionModel().getSelectedItems()));
tableView.refresh();
tableView.setItems(data);
}

Si quelqu'un à une idée, merci

[bouye]

Deux choses :

1. tableView.getSelectionModel().getSelectedItems() retourne une nouvelle liste observable qu'on nommera A .
2. data.remove(A) : aucune des versions de remove() de List ou ObservableList ne prend de liste d'objets en paramètre donc là du coup tu essaies de retirer A en tant que valeur. Comme A n'est pas une des valeurs contenues dans la liste des données de la table, rien ne se passe.

-> Donc tu dois retirer les éléments contenus dans A, pas A elle-même.

Et enfin puisque tu manipules directement la liste de la table et que c'est une liste observable, tu n'as pas besoin de faire tableView.setItems(data);

[JohnLocke]

Deux choses :

1. tableView.getSelectionModel().getSelectedItems() retourne une nouvelle liste observable qu'on nommera A .
2. data.remove(A) : aucune des versions de remove() de List ou ObservableList ne prend de liste d'objets en paramètre donc là du coup tu essaies de retirer A en tant que valeur. Comme A n'est pas une des valeurs contenues dans la liste des données de la table, rien ne se passe.

-> Donc tu dois retirer les éléments contenus dans A, pas A elle-même.

Et enfin puisque tu manipules directement la liste de la table et que c'est une liste observable, tu n'as pas besoin de faire tableView.setItems(data);

Je vois, cependant, quand j'ajoute un élément "0.0.0.0" et que je fais un "data.remove("0.0.0.0");" il ne se passe rien non plus, j'imagine qu'il faut donc choisir l'index d'un élément j'ai donc besoin de récupérer l'ID de l'élément sélectionné et de faire un data.remove(id); ?

[bouye]

Aucun problème pour ma part :

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
final TableView<String> table = new TableView();
table.setColumnResizePolicy(TableView.CONSTRAINED_RESIZE_POLICY);
final TableColumn<String, String> testColumn = new TableColumn("Test");
testColumn.setCellValueFactory(feature -> new SimpleStringProperty(feature.getValue()));
table.getColumns().setAll(testColumn);
table.getItems().add("0.0.0.0");
final Button button = new Button();
button.setText("Delete");
button.setOnAction(actionEvent -> table.getItems().remove("0.0.0.0"));
VBox root = new VBox();
root.getChildren().setAll(button, table);        
Scene scene = new Scene(root, 300, 250);        
primaryStage.setTitle("Test");
primaryStage.setScene(scene);
primaryStage.show();

Tu as probablement un soucis dans ton code.

[JohnLocke]

Aucun problème pour ma part :

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
final TableView<String> table = new TableView();
table.setColumnResizePolicy(TableView.CONSTRAINED_RESIZE_POLICY);
final TableColumn<String, String> testColumn = new TableColumn("Test");
testColumn.setCellValueFactory(feature -> new SimpleStringProperty(feature.getValue()));
table.getColumns().setAll(testColumn);
table.getItems().add("0.0.0.0");
final Button button = new Button();
button.setText("Delete");
button.setOnAction(actionEvent -> table.getItems().remove("0.0.0.0"));
VBox root = new VBox();
root.getChildren().setAll(button, table);        
Scene scene = new Scene(root, 300, 250);        
primaryStage.setTitle("Test");
primaryStage.setScene(scene);
primaryStage.show();

Tu as probablement un soucis dans ton code.

Je vois, pour ajouter des éléments dans mon tableau je m'y prend ainsi:

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
 
	public void addbanlist(ActionEvent event) {
		String ipaddress;
		String reasonforban;
		TextInputDialog dialogip = new TextInputDialog("0.0.0.0");
		dialogip.setTitle("");
		dialogip.setContentText("IP:");
		Optional<String> ip = dialogip.showAndWait();
		ipaddress = ip.get();
		TextInputDialog dialogreason = new TextInputDialog("Flood.");
		dialogreason.setTitle("");
		dialogreason.setContentText("Reason:");
		Optional<String> reason = dialogreason.showAndWait();
		reasonforban = reason.get();
		ObservableList<banned> data = tableView.getItems();
		String date = getdate();
        data.add(new banned(ipaddress,reasonforban,date));
        ipAddrTable.setCellValueFactory(new PropertyValueFactory<banned, String>("ipaddress"));
        reasonTable.setCellValueFactory(new PropertyValueFactory<banned, String>("reasonforban"));
        dateTable.setCellValueFactory(new PropertyValueFactory<banned, String>("date"));
        tableView.setItems(data);
        tableView.refresh();
	}

je vois vraiment pas ou se trouve mon erreur

[bouye]

Qu'est ce que fait la méthode refresh() et d'où sort-elle ?

EDIT1 - Marrant une méthode publique que je n'ai jamais utilisée et qui n'est pas dans la javadoc chez Oracle.

EDIT2 - ok ajoutée dans la 1.8.0u60. Ceci explique pourquoi je ne l'ai jamais utilisée. Et la doc chez Oracle date de la u40 (il y a un ticket sur le bug system à propos du fait qu'elle ne soit pas à jour).

Callingrefresh() forces the TableView control to recreate and repopulate the cells necessary to populate the visual bounds of the control. In other words, this forces the TableView to update what it is showing to the user. This is useful in cases where the underlying data source has changed in a way that is not observed by the TableView itself.