JavaFX Как правильно обрабатывать события при использовании пользовательских фабрик ячеек?

Question

Я действительно извиняюсь, если этот вопрос не имеет большого смысла с вашей точки зрения, но с моей точки зрения, это действительно важно для написания более чистого и более понятного кода.

У меня есть A.fxml, AController.java (класс контроллера для A.fxml). У меня есть TableView с определенными фабриками ячеек. Я предпочитаю определять свои фабрики ячеек в отдельном классе, чтобы я мог использовать их в случае необходимости. Я предпочитаю писать весь мой код обработки событий в моем классе контроллера. Но если я использую собственные фабрики ячеек, то я вынужден написать обработку событий в самом классе фабрики ячеек.

Есть ли способ обработки пользовательских событий фабрики ячеек в самом классе моего контроллера? или хотя бы просто выбросить событие из пользовательского класса фабрики ячеек в мой класс контроллера и обработать?

Заранее спасибо.

Answer 1

Вы можете передать объект на фабрику, которая определяет, когда следует открыть контекстное меню, и который готовит меню.

Пример:

public interface CellContextMenuProvider<S, T> {

    /**
     * Prepares the context menu for opening.
     * @param cell the cell the menu was requested for
     * @param menu the menu to prepare
     */
    public void prepareContextMenu(TableCell<S, T> cell, ContextMenu menu);

    /**
     * Checks, if a cell continaing a certain item should have an active context
     * menu.
     * @param empty if the cell is empty
     * @param item the item of the cell
     * @return {@literal true} iff the context menu should be enabled.
     */
    public boolean enableContextMenu(boolean empty, T item);

    /**
     * Prepares the intial menu. This menu must not be empty, otherwise it won't
     * be shown when it's requested for the first time.
     * @param menu the menu to prepare
     */
    public void prepareInitialContextMenu(ContextMenu menu);

}

public class CellFactory<S, T> implements Callback<TableColumn<S, T>, TableCell<S, T>> {

    private final CellContextMenuProvider<S, T> menuProvider;
    private final ContextMenu contextMenu;

    public CellFactory(@NamedArg("menuProvider") CellContextMenuProvider<S, T> menuProvider) {
        this.menuProvider = menuProvider;

        if (menuProvider == null) {
            this.contextMenu = null;
        } else {
            this.contextMenu = new ContextMenu();
            menuProvider.prepareInitialContextMenu(contextMenu);
        }

        this.menuEventHandler = evt -> {
            if (this.contextMenu != null) {
                TableCell<S, T> source = (TableCell<S, T>) evt.getSource();
                this.menuProvider.prepareContextMenu(source, this.contextMenu);
            }
        };
    }

    public CellFactory() {
        this(null);
    }

    private final EventHandler<ContextMenuEvent> menuEventHandler;

    @Override
    public TableCell<S, T> call(TableColumn<S, T> param) {
        TableCell<S, T> result = new TableCell<S, T>() {

            @Override
            protected void updateItem(T item, boolean empty) {
                super.updateItem(item, empty);
                setText(Objects.toString(item, ""));
                setContextMenu(menuProvider != null && menuProvider.enableContextMenu(empty, item) ? contextMenu : null);
            }

        };
        result.setOnContextMenuRequested(menuEventHandler);
        if (menuProvider != null && menuProvider.enableContextMenu(true, null)) {
            result.setContextMenu(contextMenu);
        }
        return result;
    }

}

public class AController {

    @FXML
    private TableView<Item<Integer>> table;

    public void initialize() {
        for (int i = 0; i < 100; i++) {
            table.getItems().add(new Item<>(i));
        }
    }    

    public CellContextMenuProvider<Item<Integer>, Integer> getMenuProvider() {
        return new CellContextMenuProvider<Item<Integer>, Integer>() {

            private final MenuItem item = new MenuItem("Say Hello World");

            {
                item.setOnAction(evt -> System.out.println("Hello World"));
            }

            @Override
            public void prepareContextMenu(TableCell<Item<Integer>, Integer> cell, ContextMenu menu) {
            }

            @Override
            public void prepareInitialContextMenu(ContextMenu menu) {
                menu.getItems().setAll(item);
            }

            @Override
            public boolean enableContextMenu(boolean empty, Integer item) {
                // only for odd items
                return !empty && (item % 2) != 0;
            }

        };
    }

}

A.fxml

<TableView fx:id="table" prefHeight="400.0" prefWidth="600.0" xmlns="http://javafx.com/javafx/8.0.60" xmlns:fx="http://javafx.com/fxml/1" fx:controller="fxml.table.AController">
  <columns>
      <TableColumn prefWidth="159.0" text="C1">
          <cellValueFactory>
              <PropertyValueFactory property="value" />
          </cellValueFactory>
          <cellFactory>
              <CellFactory menuProvider="$controller.menuProvider"/>
          </cellFactory>
      </TableColumn>
  </columns>
</TableView>

Примечание: если контекстное меню всегда одинаково, вы также можете добавить EventHandler свойства к фабрике и использовать их как, например, onAction атрибуты для кнопок, которые позволят вам передавать обработчики событий контроллера, что приведет к сокращению / упрощению кода.

Answer 2

Не было бы возможно дать вашей фабрике клеток FunctionalInterface в качестве параметра при обработке вашего события?(не уверен, что это хорошая идея)

Я представляю ваш код следующим образом:

контроллер:

myTableView.setCellFactory(new MyOwnCellFactory<>(() -> { 
    // event handling 
}));

MyOwnCellFactory:

public MyOwnCellFactory(MyFunctionalInterface myInterface) {
    functionalInterface = myInterface;
}

// something something

functionalInterface.handleEvent();

FunctionalInterface:

@FunctionalInterface
public interface MyFunctionalInterface {
    public void handleEvent();
}

Не уверен, правильно ли я понимаю вашу идею. Не проверял код, просто выписал его из головы.