ScrollEvent auf TableView dauert zu viel von der CPU-Zeit

Question

Ich habe eine TableView mit dem Namen tableVerre und ich möchte jede Zeile für ein Kriterium überprüft haben (stock Spaltenwert) und führen Sie einige Code auf sie als ich scroll, also schrieb ich dieser Code, aber es macht das Programm eine Menge CPU-Zeit zu verbrauchen, ich bin nicht vertraut mit Lambda-Ausdrücke so gibt es eine einfachere Möglichkeit, dies zu schreiben? :

tableVerre.addEventFilter(ScrollEvent.ANY, new EventHandler<ScrollEvent>() { 

    @Override 
    public void handle(ScrollEvent scrollEvent) { 

    stock.setCellFactory(column -> { 

    return new TableCell<VerreFX, Number>() { 
     @Override 
     protected void updateItem(Number item, boolean empty) { 
     super.updateItem(item, empty); 

      TableRow<VerreFX> currentRow = getTableRow(); 

      if (empty || item == null) { 
       setText(""); 
       setGraphic(null); 
       currentRow.setStyle(tableVerre.getStyle()); 
      } else { 
       setText(getItem().toString()); 
      } 

      if (!isEmpty()) { 
       if ((int) item == 0 && st.getVerresBOX()) currentRow.setStyle("-fx-background-color:lightcoral"); 
      } 
     } 
    }; 
}); 

      } 
    }); 

Answer 1

Die Tabellenansicht werden die Zellen beim Scrollen des Nutzers wiederverwenden und wird updateItem automatisch auf die Zellen rufen, wenn sie für neue Objekte wiederverwendet werden. Daher sollten Sie die Zellenfactory nur einmal festlegen und dann nur die Tabellenansicht ausführen lassen, um die Aufgabe auszuführen, für die sie vorgesehen ist. Sie können die Zellenfactory in der Methode initialize() festlegen, wenn Sie FXML verwenden, oder nur dort, wo Sie die Tabelle und die Spalten andernfalls erstellen.

Ihre Zellimplementierung ist nicht ganz korrekt: Da eine Zelle wiederverwendet werden kann, um beliebige zwei verschiedene Elemente beliebig anzuzeigen, müssen Sie alle möglichen Bedingungen berücksichtigen. Wenn in Ihrer Implementierung eine Zelle einen Artikel anzeigt, für den item.intValue()==0 und dann erneut verwendet wird, um einen Artikel anzuzeigen, für den item.intValue() != 0 angezeigt wird, wird der Stil nicht korrekt aktualisiert.

Beachten Sie auch, dass Sie eine Number in eine int "umwandeln" sollten, indem Sie intValue() aufrufen.

TableColumn<VerreFX, Number> stock ; 

// ... 

stock.setCellFactory(column -> new TableCell<VerreFX, Number>() { 
    @Override 
    protected void updateItem(Number item, boolean empty) { 
     super.updateItem(item, empty); 

     TableRow<VerreFX> currentRow = getTableRow(); 

     if (empty || item == null) { 
      setText(""); 
      setGraphic(null); 
      currentRow.setStyle(tableVerre.getStyle()); 
     } else { 
      setText(getItem().toString()); 
     } 

     if (!isEmpty()) { 
      if (item.intValue() == 0 && st.getVerresBOX()) { 
       currentRow.setStyle("-fx-background-color:lightcoral"); 
      } else { 
       currentRow.setStyle(tableVerre.getStyle()); 
      } 
     } 
    } 
}); 

Sie sollten den Scroll-Event-Handler vollständig entfernen können.

Answer 2

Erstens, Sie haben nicht über die CPU-Zeit hinweg mit Ihnen gearbeitet, da Sie nicht alle Scrollsituationen abdecken konnten. Wenn der Benutzer mit der Taste Down/Up blättert oder die Bildlaufleiste verwendet, wird das Scroll-Ereignis nicht ausgelöst. So müssen Sie zwei weitere EventFilter hinzufügen, die erste wird das Scrollen mit der Bildlaufleiste behandeln.

tableVerre.addEventFilter(MouseEvent.MOUSE_CLICKED,(
      MouseEvent event)-> 
    { 
     if ((event.getTarget() instanceof TableColumnHeader) | event.isDragDetect()) { 
      System.err.println("Mouse Draged : " + event.toString()); 

      stock.setCellFactory((TableColumn<VerreFX, Number> column) -> { 

       return new TableCell<VerreFX, Number>() { 
        @Override 
        protected void updateItem(Number item, boolean empty) { 
         super.updateItem(item, empty); 

         TableRow<VerreFX> currentRow = getTableRow(); 

         if (empty || item == null) { 
          setText(""); 
          setGraphic(null); 
          currentRow.setStyle(tableVerre.getStyle()); 
         } else { 
          setText(getItem().toString()); 
         } 

         if (!isEmpty()) { 
          if ((int) item == 0 && st.getVerresBOX()) { 
           currentRow.setStyle("-fx-background-color:lightcoral"); 
          } 
         } 
        } 
       }; 
      }); 
     } 
    }); 

Und der zweite wird das Scrollen mit der Tastatur Taste DOWN/UP behandeln.

tableVerre.addEventFilter(KeyEvent.KEY_PRESSED,new EventHandler<KeyEvent>(){ 
    @Override 
    public void handle(KeyEvent event) { 
     if (event.getCode() == KeyCode.DOWN | event.getCode() == KeyCode.UP) { 
      stock.setCellFactory(column -> { 

       return new TableCell<VerreFX, Number>() { 
        @Override 
        protected void updateItem(Number item, boolean empty) { 
         super.updateItem(item, empty); 

         TableRow<VerreFX> currentRow = getTableRow(); 

         if (empty || item == null) { 
          setText(""); 
          setGraphic(null); 
          currentRow.setStyle(tableVerre.getStyle()); 
         } else { 
          setText(getItem().toString()); 
         } 

         if (!isEmpty()) { 
          if ((int) item == 0 && st.getVerresBOX()) { 
           currentRow.setStyle("-fx-background-color:lightcoral"); 
          } 
         } 
        } 
       }; 
      }); 
     } 
     System.err.println("Key Pressed : " + event.toString()); 
    } 
});