JavaFX Boolean ändern Table View

[Samofan]

Hallo,

ich brauche noch einmal Eure Hilfe. Ich hab das Internet durchsucht und auch etwas dazu gefunden, nur weiß ich nicht, wie ich das in meinem Programm umgesetzt bekomme. Ich habe eine Spalte in einer JavaFX Table View, die einen Boolean als Bild anzeigt, welcher aus einer Datenbank gelesen wird. Jetzt hätte ich gerne, dass der Benutzer mit einem Doppelklick auf eine Zelle die Frage gestellt bekommt, ob er den Wert ändern möchte. Es reicht auch erstmal wenn auf der Konsole "double clicked" oder sowas erscheint. Ab da dürfte ich dann weiter kommen. Meine Cell Factory sieht so aus:

remoteColumn.setCellFactory(col -> new TableCell<Data, Boolean>()
    {
      private final ImageView imageView = new ImageView();


      {
        // initialize ImageView + set as graphic
        imageView.setFitWidth(20);
        imageView.setFitHeight(20);
        setGraphic(imageView);
      }

      @Override
      protected void updateItem(Boolean item, boolean empty)
      {
        if (empty || item == null)
        {
          // no image for empty cells
          imageView.setImage(null);
        }
        else
        {
          // set image for non-empty cell
          imageView.setImage(item ? imageTrue : imageFalse);
        }
      }

      @Override
      public Bounds sceneToLocal(Bounds sceneBounds)
      {
        // TODO: Auto-generated method stub
        return super.sceneToLocal(sceneBounds);
      }
    });

Ich weiß jetzt nicht, ob ich ein MouseListener in diese CellFactory einbauen soll oder ob ich eine neue anlegen soll. Ihr merkt, ich bin ein bisschen ratlos. Die Seiten habe ich mir grade schon angeschaut:
https://stackoverflow.com/questions...-a-tableview-cell-in-javafx?noredirect=1&lq=1
https://stackoverflow.com/questions/26563390/detect-doubleclick-on-row-of-tableview-javafx
Ich will wirklich nur diese eine Spalte überprüfen und nicht die komplette Zeile, da ich in anderen Spalten auch weitere Booleans habe, die ebenfalls mit dieser Methode verändert werden sollen. Ich hoffe, dass das verständlich ist

 

[MoxxiManagarm]

Ich bin kein Profi in JavaFX. Aber...

https://stackoverflow.com/questions/26563390/detect-doubleclick-on-row-of-tableview-javafx

Du hast einen Link mit TableRow. Eine TableRow ist ein Parent. Eine TableCell kannst du sicherlich mit https://docs.oracle.com/javase/8/javafx/api/javafx/scene/Parent.html#getChildren--
erreichen. Eine TableCell ist auch ein Node, damit dürfte das Event auch hier registriert werden können.

Best guess:

TableView<Item> table = new TableView<>();
       table.setRowFactory(tv -> {
           TableRow<Item> row = new TableRow<>();
           TableCell<Item, Boolean> cell = row.getChildren().get(columnOfTheBoolean);
           cell.setOnMouseClicked(event -> {
               if (event.getClickCount() == 2 && (! cell.isEmpty()) ) {
                   // do something here
               }
           });
           return row ;
       });

 

[Samofan]

Ich bin kein Profi in JavaFX. Aber...[...]

Ich danke dir schonmal. Ich werde das mal versuchen. Könnte etwas Zeit in Anspruch nehmen da ich auch noch andere Baustellen habe. Wenn mein Code funktioniert poste ich ihn hier mal. Das mit dem Parent ist auf jeden Fall ein guter Tipp.

 

[Samofan]

Ich habe es jetzt mit

tableView.setRowFactory(tv ->
    {
      TableRow<Data> row = new TableRow<>();
      TableCell<Data, Boolean> cell = row.getChildren().get(remoteColumn);
       cell.setOnMouseClicked(event -> {
               if (event.getClickCount() == 2 && (! cell.isEmpty()) ) {
                   System.out.println("Test");
               }
           });
           return row ;
       });

versucht aber leider klappt es nicht so ganz, wie ich will. Ich bekomme einfach nicht die Zelle der Zeile. Als Fehler bekomme ich:

getChildren() has protected access in Parent

und

incompatible types: TableColumn<Data,Boolean> cannot be converted to int

. Wenn ich "TableCell" rausnehme und sage

cell.setOnMouseClicked(event ->...

funktioniert alles wunderbar. Wie komme ich an die Zelle ran?

 

[Robat]

Zum Bleistift:

column.setCellFactory(tableCell -> {
    TableCell<Wrapper, String> cell = new TableCell<>() {
        @Override
        protected void updateItem(String item, boolean empty) {
            super.updateItem(item, empty);
            setText(empty ? "" : item);
        }
    };
    cell.setOnMouseClicked(event -> {
        if(event.getClickCount() == 2) {
            System.out.println("Cell [" + cell.getText() + "] clicked");
        }
    });
    return cell;
});

 

[Samofan]

So hat es geklappt aber die Spalte zeigt die Bilder, die ich vorher drin hatte nicht mehr an und der Inhalt ist null...

remoteColumn.setCellFactory(tableCell ->
    {
      TableCell<Data, Boolean> cell = new TableCell()
      {
        protected void updateItem(String item, boolean empty)
        {
          super.updateItem(item, empty);
          setText(empty ? "" : item);
        }
      };
      cell.setOnMouseClicked(event ->
      {
        if (event.getClickCount() == 2)
        {
          System.out.println("Cell [" + cell.getText() + "] clicked");
        }
      });
      return cell;
    });

 

[Robat]

Du darfst es auch nicht einfach nur rein kopieren - das war nur ein Beispiel
Deine updateItem() Methode muss schon so bleiben wie sie war

 

[Samofan]

In etwa so? Weiß grade leider echt nicht weiter...

remoteColumn.setCellFactory(col -> new TableCell<Data, Boolean>()
    {
      private final ImageView imageView = new ImageView();


      {
        // initialize ImageView + set as graphic
        imageView.setFitWidth(20);
        imageView.setFitHeight(20);
        setGraphic(imageView);

//dieser Teil funktioniert nicht. Ich kann kein MouseEvent auf die Spalte setzen
        col.onMouseClicked(event ->
        {
          if (event.getClickCount() == 2)
          {
            System.out.println("Cell [" + col.getText() + "] clicked");
          }
        });
      }

      @Override
      protected void updateItem(Boolean item, boolean empty)
      {
        if (empty || item == null)
        {
          // no image for empty cells
          imageView.setImage(null);
        }
        else
        {
          // set image for non-empty cell
          imageView.setImage(item ? imageTrue : imageFalse);
        }
      }

      @Override
      public Bounds sceneToLocal(Bounds sceneBounds)
      {
        // TODO: Auto-generated method stub
        return super.sceneToLocal(sceneBounds);
      }

      @Override
      public Point2D sceneToLocal(Point2D scenePoint)
      {
        // TODO: Auto-generated method stub
        return super.sceneToLocal(scenePoint);
      }
    });

Das "col.onMouseClicked(event ->" funktioniert leider nicht

 

[Robat]

remoteColumn.setCellFactory(col -> {
    TableCell<Data, Boolean> cell = new TableCell<Data, Boolean>()
    {
        @Override
        protected void updateItem(Boolean item, boolean empty)
        {
            final ImageView imageView = new ImageView();
            imageView.setFitHeight(20);
            imageView.setFitWidth(20);

            if (empty || item == null)
            {
                // no image for empty cells
                imageView.setImage(null);
            }
            else
            {
                // set image for non-empty cell
                imageView.setImage(item ? imageTrue : imageFalse);
            }
        }
        @Override
        public Bounds sceneToLocal(Bounds sceneBounds)
        {
            return super.sceneToLocal(sceneBounds);
        }
    };
    cell.onMouseClicked(event ->
    {
        if (event.getClickCount() == 2)
        {
            System.out.println("Cell [" + col.getText() + "] clicked");
        }
    });
});

 

[Samofan]

Vielen, vielen Dank! So klappt es. Das ist mir leider noch etwas zu hoch. Deswegen wusste ich echt nicht was zu tun war. Dann werde ich mich jetzt mal dran machen, den Code zu verstehen

    remoteColumn.setCellFactory(col ->
    {
      TableCell<Data, Boolean> cell = new TableCell<Data, Boolean>()
      {
        @Override
        protected void updateItem(Boolean item, boolean empty)
        {
          final ImageView imageView = new ImageView();
          imageView.setFitHeight(20);
          imageView.setFitWidth(20);
          setGraphic(imageView);
          if (empty || item == null)
          {
            // no image for empty cells
            imageView.setImage(null);
          }
          else
          {
            // set image for non-empty cell
            imageView.setImage(item ? imageTrue : imageFalse);
          }
        }
        @Override
        public Bounds sceneToLocal(Bounds sceneBounds)
        {
          return super.sceneToLocal(sceneBounds);
        }
        @Override
        public Point2D sceneToLocal(Point2D scenePoint)
        {
          // TODO: Auto-generated method stub
          return super.sceneToLocal(scenePoint);
        }
      };
      cell.setOnMouseClicked(event ->
      {
        if (event.getClickCount() == 2)
        {
          System.out.println("Cell [" + col.getText() + "] clicked");
        }
      });
      return cell;
    });

 

[mihe7]

Dann werde ich mich jetzt mal dran machen, den Code zu verstehen

Und lager den Code aus.

 

[Samofan]

Und lager den Code aus.

Du meinst in eine andere Klasse, auf die ich in meiner "initialize"-Methode zugreife?

 

[mihe7]

Ja, zum Beispiel.

 

[Samofan]

Habe meine ganze initialize-Methode in einzelne Methoden zusammengefasst und aufgeräumt. So ist der Code viel besser lesbar. Danke. Durch euch werde ich jeden Tag ein Stück besser

 

[mihe7]

So ist der Code viel besser lesbar

Nicht nur das: kleine, einfache Einheiten sind leichter zu testen und helfen dabei, Code zu vermeiden (DRY-Prinzip oder: der beste Code ist der, den man nicht schreibt )