changes from the reviewers implemented, some more error handling implemented with the load function of the show method

This commit is contained in:
Rithvik Sriram 2025-12-05 14:12:05 +01:00
commit 4414328fe0

View file

@ -11,8 +11,8 @@ public class Navigation {
/**
* Constructs a Navigation Object that uses a fxml object and Pane that will allow to switch UI screens
* @param x
* @param y
* @param x - a MyFXML Object
* @param y - a Pane Object
*/
public Navigation(MyFXML x, Pane y){
this.myFXML = x;
@ -22,14 +22,19 @@ public class Navigation {
/**
* The show function takes in an FxmlPath and changes the current pane to the value given by the path
* using the load function of the myFXML object
* @param fxmlpath
* @param fxmlpath - FXML path provided to show the next pane
*/
public void show(String fxmlpath){
var result = myFXML.load(Object.class, fxmlpath); //stores the result of the load method in result
Parent uiRoot = result.getValue();
screen.getChildren().setAll(uiRoot);
try{
var result = myFXML.load(Object.class, fxmlpath); //stores the result of the load method in result
Parent uiRoot = result.getValue();
screen.getChildren().setAll(uiRoot);
}
catch(Exception e) { //catches any failures thrown by the load method
System.err.println("Failed to load FXML file: " + fxmlpath);
throw new IllegalArgumentException("Cannot load FXML file: " + fxmlpath, e);
}
}