How can i get an instance of JavaFX controller

Question

I'm stuck in my code. I'm writting a library system, which contains many of controllers. This is my main controller fxml file:

As you can see every tab has its own controller. I want to send data from one to another included controller (from Readers.fxml controller to Borrows.fxml controller. My code: READER CONTROLLER

    @FXML
void onCRentActionButton(ActionEvent event) throws IOException {
    if(CTable.getSelectionModel().getSelectedItem()==null)
    {
        Alert alert = new Alert(Alert.AlertType.ERROR);
        alert.setTitle("Błąd !!!");
        alert.setHeaderText("Nie możesz edytować użytkownika, jeśli żadnego"
                + "nie zaznaczyłeś w tabeli!!");
        alert.showAndWait();
    }
    else
    {
        FXMLLoader loader = new FXMLLoader(getClass().getResource("/fxml/Borrows.fxml"));
        loader.load();
        logger.info("Kontroller pliku Borrows.fxml to: "+ loader.getController().getClass());
        BorrowController controller = loader.getController();
        ReaderBorrow rb;
        int id=CTable.getSelectionModel().getSelectedItem().getId();

        rb = SQLController.getInstance().getReaderToBorrow(id);
        controller.setReader(rb);
}

BORROW CONTROLLER

public void setReader(ReaderBorrow rb)
{
    logger.info("Ustawianie nazwiska: " + rb.getSurname());
    this.WSurname.setText(rb.getSurname());
    logger.info("Odczyt wartości: " + WSurname.getText());
}

I have a good information that data is send succesfully. And WSurname.getText() method returns a good information. But in my view WSurname TextField is still empty. I think, when I call getController method in ReadersController i get another instance of BorrowController. How can I set data into TextField correct ?

James_D · Accepted Answer

You have the wrong instance of BorrowController: in onCRentActionButton() you load Borrows.fxml again, getting a new copy of the UI, and then you get the controller associated with that new copy. Since that new copy of the UI is never displayed, you're updating something you can't see.

Instead, inject the controllers for the different tabs into the main controller. According to the documentation, you can inject the controller by using the fx:id for the with "Controller" appended.

So in MainController, add:

@FXML
private BorrowController borrowController ;
@FXML
private ReaderController readerController ;

add fx:ids to the s:

To link the controllers you will need a reference to borrowController in ReaderController:

public class ReaderController {

    private BorrowController borrowController ;

    public void setBorrowController(BorrowController borrowController) {
        this.borrowController = borrowController ;
    }

    // ...
}

which you need to set up in the initialize() method of your main controller:

public class MainController {

    @FXML
    private BorrowController borrowController ;
    @FXML
    private ReaderController readerController ;

    // ...

    public void initialize() {
        readerController.setBorrowController(borrowController);
        // ...
    }

    // ...
}

and then finally you can just do

@FXML
void onCRentActionButton(ActionEvent event) throws IOException {
    if(CTable.getSelectionModel().getSelectedItem()==null) {
        Alert alert = new Alert(Alert.AlertType.ERROR);
        alert.setTitle("Błąd !!!");
        alert.setHeaderText("Nie możesz edytować użytkownika, jeśli żadnego"
                + "nie zaznaczyłeś w tabeli!!");
        alert.showAndWait();
    } else {
        ReaderBorrow rb;
        int id=CTable.getSelectionModel().getSelectedItem().getId();

        rb = SQLController.getInstance().getReaderToBorrow(id);
        borrowController.setReader(rb);
    }
}

If you use a MVC approach and keep the state in a model class, you can avoid all the coupling among these controllers.

How can i get an instance of JavaFX controller

Answers (1)

Related Questions