Merge pull request #141 from wwweert123/VisDG

Add to DG and fix bugs

docs/DeveloperGuide.md

@@ -99,8 +99,8 @@ the data one upon exiting the program with the `exit` command.
 
 This feature is implemented with the help of [XChart](https://knowm.org/open-source/xchart/), a simple charting library for Java by Knowm.
 
-By typing in the vis command with the appropriate arguments (/s and /t), users will be able to visualize their income or expense 
-using visualization tools (Piechart, Bar Chart)
+By typing in the vis command with the appropriate arguments (`/s` and `/t`), users will be able to visualize their 
+income or expense using visualization tools (Piechart, Bar Chart or Radar Chart)
 
 Demo: 
 
@@ -132,7 +132,7 @@ specified cashflow entry according to type using a Hashmap which is returned and
 
 Visualizer's Role: 
 
-According to the chart type (Pie/Bar) argument and the Hashmap obtained from the categorizer passed in, 
+According to the chart type (Pie/Bar/Radar) argument and the Hashmap obtained from the categorizer passed in, 
 the visualizer displays the specified visualization chart by calling the charting library Xchart.
 
 ### Class Diagram

docs/UserGuide.md

@@ -541,7 +541,7 @@ Use watchlist command to view updated Watchlist
 
 ### Visualizing your cashflow: `vis`
 
-Using this command to visualize your income or expenses in a pie chart or bar chart
+Using this command to visualize your income or expenses in a pie chart, bar chart or radar chart
 
 Format: `vis /t TYPE /c TOOL`
 
@@ -550,10 +550,11 @@ Format: `vis /t TYPE /c TOOL`
 | Income Cashflows `Income`   |
 | Expense Cashflows `Expense` |
 
-| Tool `/c`      |
-|----------------|
-| PieChart `pie` |
-| BarChart `bar` |
+| Tool `/c`          |
+|--------------------|
+| PieChart `pie`     |
+| BarChart `bar`     |
+| RadarChart `radar` |
 
 Example of usage: `vis /t expense /c pie`
 
@@ -565,14 +566,22 @@ Displaying piechart for expense
 
 ![](images/vis/visOutput.png)
 
-Example of usage: `vis /t expense /c bar`
+Example of usage: `vis /t income /c bar`
 
 ```
 Displaying barchart for income
 ```
 
 ![](images/vis/barOuput.png)
 
+Example of usage: `vis /t income /c radar`
+
+```
+Displaying radarchart for income
+```
+
+![](images/vis/radarOutput.png)
+
 ### Exiting the program: `exit`
 
 Exits the program.

docs/diagrams/vis/visualisationClass.puml

@@ -27,9 +27,10 @@ class Categorizer #HoneyDew {
 }
 
 class Visualizer #Beige {
-+<u>displayChart(chart: String, cashFlowByCat: Map, type: String)
-+ <u>displayPieChart(cashFlowByCat: Map, type: String)
-+ <u>displayBarChart(cashFlowByCat: Map, type: String)
++<u>displayChart(chart: String, cashflowByCat: HashMap, type: String)
++ <u>displayPieChart(cashflowByCat: HashMap, type: String)
++ <u>displayBarChart(cashflowByCat: HashMap, type: String)
++ <u>displayRadarChart (cashflowByCat: HashMap, type: String)
 }
 
 "{abstract}\nCommand" <|-- VisCommand

src/main/java/seedu/financialplanner/visualisations/Visualizer.java

@@ -89,7 +89,7 @@ public static void displayBarChart (HashMap<String, Double> cashflowByCat, Strin
             keys.add(set.getKey());
             values.add(set.getValue());
         }
-        chart.addSeries("Expense", keys, values);
+        chart.addSeries(StringUtils.capitalize(type), keys, values);
 
         logger.log(Level.INFO, "Displaying Bar Chart");
         JFrame swHR = new SwingWrapper<>(chart).displayChart();
@@ -112,7 +112,7 @@ public static void displayRadarChart (HashMap<String, Double> cashflowByCat, Str
 
         radarChart.getStyler().setSeriesColors(sliceColors);
 
-        String[] keys = new String[0];
+        String[] keys;
         switch (type) {
         case "income":
             keys = IncomeType.getNames(IncomeType.class);