diff --git a/src/main/java/seng202/group9/GUI/BarGraphController.java b/src/main/java/seng202/group9/GUI/BarGraphController.java index c151226..14cd66a 100644 --- a/src/main/java/seng202/group9/GUI/BarGraphController.java +++ b/src/main/java/seng202/group9/GUI/BarGraphController.java @@ -36,7 +36,7 @@ public class BarGraphController extends Controller { public void build_graph(){ //Takes routes from the full dataset. current_routes = currentdata.getRoutes(); - datasetup(current_routes); + //datasetup(current_routes); //Builds series needed for the graph. XYChart.Series seriesArivals = new XYChart.Series(); XYChart.Series seriesDeparts = new XYChart.Series(); @@ -83,11 +83,11 @@ public class BarGraphController extends Controller { name = entry.getCountryName(); } if (useddata.containsKey(name)){ - int temp = useddata.get(name); - useddata.replace(name,temp+1); + //int temp = useddata.get(name); + //useddata.replace(name,temp+1); }else { Integer temp = 1; - useddata.put(name,temp); + //useddata.put(name,temp); } } } @@ -95,12 +95,12 @@ public class BarGraphController extends Controller { private void datasetupCustomRoute(ArrayList current_routes){ //Takes out the specified field (Currently departure airport and arrival airport) then adds to the used data dict. - if(currentsession.getSelectedgraphagainst() = ""){ + //if(currentsession.getSelectedgraphagainst() = ""){ - } + //} } - +/** for (Route entry : current_routes){ String departs = entry.getDepartureAirport(); String arives = entry.getArrivalAirport(); @@ -125,7 +125,7 @@ public class BarGraphController extends Controller { useddata.put(arives,temp); } } - +*/ /** * Takes the current dataset then loads the data to the graph using build graph. */ @@ -161,7 +161,7 @@ public class BarGraphController extends Controller { else{ d = currentdata.getAirlines(); } - datasetupCustomarAirline(d); + //datasetupCustomarAirline(d); } else if (temp == "Routes") { ArrayList d = new ArrayList(); diff --git a/src/main/java/seng202/group9/GUI/PieChartErrorController.java b/src/main/java/seng202/group9/GUI/PieChartErrorController.java index 3b2c89a..33f9547 100644 --- a/src/main/java/seng202/group9/GUI/PieChartErrorController.java +++ b/src/main/java/seng202/group9/GUI/PieChartErrorController.java @@ -14,7 +14,7 @@ public class PieChartErrorController extends Controller { } public void ignoredWarning() { - currentSession.setForceGraph(Boolean.TRUE); + currentSession.setForceGraph(true); replaceSceneContent(SceneCode.AIRPORT_ANALYSER); } public void backToSafety(){ diff --git a/src/main/java/seng202/group9/GUI/PieChooserController.java b/src/main/java/seng202/group9/GUI/PieChooserController.java index b6be1d7..72254a4 100644 --- a/src/main/java/seng202/group9/GUI/PieChooserController.java +++ b/src/main/java/seng202/group9/GUI/PieChooserController.java @@ -47,7 +47,7 @@ public class PieChooserController extends Controller{ currentsession.setSelectedDataToGraph(datatypechooser.getSelectionModel().getSelectedItem().toString()); currentsession.setSelectedgraphagainst(graph_options.getSelectionModel().getSelectedItem().toString()); currentsession.setUsefilter(usefilter.isSelected()); - currentsession.setForceGraph(Boolean.FALSE); + currentsession.setForceGraph(false); replaceSceneContent(SceneCode.AIRPORT_ANALYSER); } @@ -64,6 +64,7 @@ public class PieChooserController extends Controller{ changeTables(); } }); + graph_options.setItems(airportOptions); graph_options.getSelectionModel().selectFirst(); allOptions.add(airportOptions); diff --git a/src/main/java/seng202/group9/GUI/PieGraphController.java b/src/main/java/seng202/group9/GUI/PieGraphController.java index eb80465..75c82c8 100644 --- a/src/main/java/seng202/group9/GUI/PieGraphController.java +++ b/src/main/java/seng202/group9/GUI/PieGraphController.java @@ -44,7 +44,7 @@ public class PieGraphController extends Controller { pieChartData.add(new PieChart.Data(airport,temp)); } //Gives the data to the graph. - if (useddata.keySet().size() > 50 && currentsession.getForceGraph()){ + if (useddata.keySet().size() > 250 && currentsession.getForceGraph()){ replaceSceneContent(SceneCode.CHART_ERROR); } else{ @@ -66,7 +66,7 @@ public class PieGraphController extends Controller { else if (currentsession.getSelectedgraphagainst() == "ICAO") { name = entry.getICAO(); } - else if (currentsession.getSelectedgraphagainst() == "IATA_FFA") { + else if (currentsession.getSelectedgraphagainst() == "IATA FFA") { name = entry.getIATA_FFA(); } else if (currentsession.getSelectedgraphagainst() == "City") { diff --git a/src/main/resources/pie_graph_chooser.fxml b/src/main/resources/pie_graph_chooser.fxml index 843111d..e2fd2f4 100644 --- a/src/main/resources/pie_graph_chooser.fxml +++ b/src/main/resources/pie_graph_chooser.fxml @@ -1,5 +1,10 @@ + + + + + @@ -14,58 +19,47 @@ - + - - - + + + - + - + - - - - - - - - - + + + + + - - + + + + + + + + + - - - - - - - - - - - - - - - - + + - + + - -