Mongo Class Maintainability improved

Controllers/API/APIInterface.cs

@@ -1,10 +1,8 @@
-using System.Collections.Generic;
-using System.Text;
-using System;
-using System.Threading.Tasks;
-using Newtonsoft.Json;
-using System.Net.Http;
 using EFB.Models;
+using Newtonsoft.Json;
+using System.Collections.Generic;
+using System.Net.Http;
+using System.Threading.Tasks;
 
 namespace EFB.Controllers.API
 {
@@ -12,10 +10,10 @@ namespace EFB.Controllers.API
     {
         private HttpClient HttpClient { get; set; }
 
-        public async Task<ResponseModel<T>> Get<T>(string Endpoint, Dictionary<string, string> Headers){
-
+        public async Task<ResponseModel<T>> Get<T>(string Endpoint, Dictionary<string, string> Headers)
+        {
+            //Create the HTTP client used for the GetRequest. 
             this.HttpClient = new HttpClient();
-
             this.HttpClient.DefaultRequestHeaders.Clear();
 
             if (Headers != null)
@@ -30,10 +28,9 @@ namespace EFB.Controllers.API
             {
                 try
                 {
+                    //Get the response from the server
                     var pendingResult = this.HttpClient.GetAsync(Endpoint);
-
                     var result = await pendingResult;
-
                     string resultString = result.Content.ReadAsStringAsync().Result;
 
                     //Assess return value (object or raw string)
@@ -43,8 +40,8 @@ namespace EFB.Controllers.API
                     {//If the user requests string for return type
                         response = JsonConvert.DeserializeObject<T>(resultString);
                     }
-
-                    return new ResponseModel<T>(){
+                    return new ResponseModel<T>()
+                    {
                         //Sender should be aware of type T becuase of Generic type
                         Result = (T)response
                     };
@@ -53,20 +50,19 @@ namespace EFB.Controllers.API
                 {
                     return new ResponseModel<T> { Error = e.Message };
                 }
-
             }
             //Returned in the event No other response has been returned
             return new ResponseModel<T> { Error = "Invalid Endpoint - Please try again later" };
-            
         }
 
-        public async Task<ResponseModel<T>> Post<T>(string Endpoint, Dictionary<string, string> Headers, HttpContent Body){
-
+        public async Task<ResponseModel<T>> Post<T>(string Endpoint, Dictionary<string, string> Headers, HttpContent Body)
+        {
+            //Create a HTTP client to allow for making Post requests
             this.HttpClient = new HttpClient();
             this.HttpClient.DefaultRequestHeaders.Clear();
 
             if (Headers != null)
-            {
+            {//Go through and add each header to the HTTP Client
                 foreach (var Header in Headers)
                 {
                     this.HttpClient.DefaultRequestHeaders.Add(Header.Key, Header.Value);
@@ -75,11 +71,11 @@ namespace EFB.Controllers.API
 
             if (Form.FormAuthenticator.ValidateEndpoint(Endpoint))
             {
-                try{//Try statement to catch errors in the process of making the request
+                try
+                {//Try statement to catch errors in the process of making the request
                     var pendingResult = this.HttpClient.PostAsync(Endpoint, Body);
                     var result = await pendingResult;
                     string resultString = result.Content.ReadAsStringAsync().Result;
-                    
                     object response = resultString;
 
                     if (typeof(T) != typeof(string))
@@ -87,28 +83,28 @@ namespace EFB.Controllers.API
                         response = JsonConvert.DeserializeObject<T>(resultString);
                     }
 
-                    return new ResponseModel<T>(){
+                    return new ResponseModel<T>()
+                    {
                         //Sender should be aware of type T becuase of Generic type
                         Result = (T)response
                     };
-                }catch(System.Exception e){
+                }
+                catch (System.Exception e)
+                {
                     return new ResponseModel<T> { Error = e.Message };
                 }
             }
             //Returned in the event No other response has been returned
             return new ResponseModel<T> { Error = "Invalid Endpoint - Please try again later" };
-
         }
-
-
-
-        public async Task<ResponseModel<T>> Put<T>(string Endpoint, Dictionary<string, string> Headers, HttpContent Body){
-
+        public async Task<ResponseModel<T>> Put<T>(string Endpoint, Dictionary<string, string> Headers, HttpContent Body)
+        {
+            //Create HTTP client to allow for HttpRequests
             this.HttpClient = new HttpClient();
             this.HttpClient.DefaultRequestHeaders.Clear();
 
             if (Headers != null)
-            {
+            {//Loop through and add each heder to the HTTP client
                 foreach (var Header in Headers)
                 {
                     this.HttpClient.DefaultRequestHeaders.Add(Header.Key, Header.Value);
@@ -117,11 +113,13 @@ namespace EFB.Controllers.API
 
             if (Form.FormAuthenticator.ValidateEndpoint(Endpoint))
             {
-                try{//Try statement to catch errors in the process of making the request
+                try
+                {//Try statement to catch errors in the process of making the request
                     var pendingResult = this.HttpClient.PutAsync(Endpoint, Body);
                     var result = await pendingResult;
                     string resultString = result.Content.ReadAsStringAsync().Result;
 
+                    //Recieve the response as a string which will be morphed in other types
                     object response = resultString;
 
                     if (typeof(T) != typeof(string))
@@ -129,20 +127,21 @@ namespace EFB.Controllers.API
                         response = JsonConvert.DeserializeObject<T>(resultString);
                     }
 
-                    return new ResponseModel<T>(){
+                    return new ResponseModel<T>()
+                    {
                         //Sender should be aware of type T becuase of Generic type
                         Result = (T)response
                     };
-                }catch(System.Exception e){
+                }
+                catch (System.Exception e)
+                {
                     return new ResponseModel<T> { Error = e.Message };
                 }
             }
             //Returned in the event No other response has been returned
             return new ResponseModel<T> { Error = "Invalid Endpoint - Please try again later" };
-
         }
 
-        
     }
 
 }

Controllers/AppController.cs

@@ -1,13 +1,8 @@
-using System;
-using System.Collections.Generic;
-using System.Diagnostics;
-using System.Linq;
-using System.Threading.Tasks;
+using EFB.Models;
+using EFB.Sessions;
+using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.Extensions.Logging;
-using EFB.Models;
-using Microsoft.AspNetCore.Http;
-using EFB.Sessions;
 
 namespace EFB.Controllers
 {
@@ -23,18 +18,17 @@ namespace EFB.Controllers
         public IActionResult Index()
         {
             //Check to see what point on the application the user is at and where they should be sent
-            UserModel User = HttpContext.Session.GetObject<UserModel>("User");
-            if (User != null)
+            UserModel user = HttpContext.Session.GetObject<UserModel>("User");
+            if (user == null)
             {
-                if (User.Route == null)
-                {
-                    return RedirectToAction("Index", "Route");
-                }else{
-                    return RedirectToAction("Index", "App");
-                }
-            }else{
                 return RedirectToAction("Index", "Home");
             }
+            if (user.Route == null)
+            {
+                return RedirectToAction("Index", "Route");
+            }
+
+            return View(user);
         }
 
         [ResponseCache(Duration = 0, Location = ResponseCacheLocation.None, NoStore = true)]

Controllers/ChartsController.cs

@@ -1,14 +1,9 @@
-using System;
-using System.Collections.Generic;
-using System.Diagnostics;
-using System.Linq;
-using System.Threading.Tasks;
+using EFB.Controllers.Form;
+using EFB.Models;
+using EFB.Sessions;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.Extensions.Logging;
-using EFB.Models;
-using EFB.Models.JSON;
-using EFB.Controllers.Form;
-using EFB.Sessions;
+using System.Threading.Tasks;
 
 namespace EFB.Controllers
 {
@@ -45,14 +40,16 @@ namespace EFB.Controllers
                     HttpContext.Session.SetObject("User", user);
                     return RedirectToAction("ViewCharts");
                 }
-            }else
+            }
+            else
             {
                 TempData["Error"] = "Invalid ICAO";
             }
             return View();
         }
 
-        public IActionResult ViewCharts(string chart){
+        public IActionResult ViewCharts(string chart)
+        {
             UserModel user = HttpContext.Session.GetObject<UserModel>("User");
             if (user != null)
             {

Controllers/FlightsimController.cs

@@ -1,14 +1,11 @@
-using System;
-using System.Collections.Generic;
-using System.Diagnostics;
-using System.Linq;
-using System.Threading.Tasks;
-using Microsoft.AspNetCore.Mvc;
-using EFB.Sessions;
-using Microsoft.Extensions.Logging;
 using EFB.Models;
-using EFB.MongoData;
 using EFB.Models.Route;
+using EFB.MongoData;
+using EFB.Sessions;
+using Microsoft.AspNetCore.Mvc;
+using Microsoft.Extensions.Logging;
+using System;
+using System.Threading.Tasks;
 
 namespace EFB.Controllers
 {
@@ -26,10 +23,18 @@ namespace EFB.Controllers
         {
             //Retrieve and Check current user status
             UserModel user = HttpContext.Session.GetObject<UserModel>("User");
-            if(user == null) return RedirectToAction("Index", "Home");
+            if (user == null)
+            {
+                TempData["Error"] = "You must be logged in before you are able to view the FlightSim Page";
+                return RedirectToAction("Index", "Home");
+            }
 
             //Retrieve the user's latest sim position and construct it into FlightsimModel
-            if (user.Route == null) return RedirectToAction("Index", "Route");
+            if (user.Route == null)
+            {
+                TempData["Error"] = "You must have a route planned before you are able to view the Flightsim page";
+                return RedirectToAction("Index", "Route");
+            }
 
             SimPositionModel latestPositionModel = await Mongo.GetLatestData(user.EMail);
 
@@ -40,7 +45,8 @@ namespace EFB.Controllers
 
         }
 
-        private IWaypoint DetermineClosest(RouteModel route, SimPosition currentPosition){
+        private IWaypoint DetermineClosest(RouteModel route, SimPosition currentPosition)
+        {
             IWaypoint closest = null;
             float closestDistance = float.MaxValue;
             //Assuming that we are on the earth for simplicity
@@ -68,7 +74,8 @@ namespace EFB.Controllers
             }
             return closest;
         }
-        private float DegreesToRadians(float degrees){
+        private float DegreesToRadians(float degrees)
+        {
             return (float)(degrees * Math.PI / 180);
         }
 

Controllers/Form/FormAuthenticator.cs

@@ -1,14 +1,12 @@
-using System;
-using System.Collections.Generic;
 using System.Linq;
-using System.Threading.Tasks;
 
 namespace EFB.Controllers.Form
 {
     public static class FormAuthenticator
     {
 
-        public static bool ValidateEMail(string EMail){
+        public static bool ValidateEMail(string EMail)
+        {
             if (EMail != null && EMail.Contains("@") && EMail.Contains(".") && !EMail.Contains(" "))
             {
                 if (EMail.Count(x => x == '@') == 1)
@@ -19,7 +17,8 @@ namespace EFB.Controllers.Form
             return false;
         }
 
-        public static bool ValidateEndpoint(string Endpoint){
+        public static bool ValidateEndpoint(string Endpoint)
+        {
             //If it contains http & :// it can be either https or http
             if (Endpoint.Contains("http") && Endpoint.Contains("://") && Endpoint.Length > 7)
             {
@@ -28,7 +27,8 @@ namespace EFB.Controllers.Form
             return false;
         }
 
-        public static bool ValidateICAOCode(string ICAO){
+        public static bool ValidateICAOCode(string ICAO)
+        {
             if (ICAO.Length == 4)
             {
                 //If the value contains a Number, then the value will return false
@@ -37,7 +37,8 @@ namespace EFB.Controllers.Form
             return false;
         }
 
-        public static bool ValidateCruiseAlt(uint CruiseAlt){
+        public static bool ValidateCruiseAlt(uint CruiseAlt)
+        {
             if (CruiseAlt > 0 && CruiseAlt < 50000)
             {
                 return true;

Controllers/HomeController.cs

@@ -1,11 +1,9 @@
 using EFB.Models;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.Extensions.Logging;
-using System;
-using System.Collections.Generic;
 using System.Diagnostics;
-using System.Linq;
-using System.Threading.Tasks;
+using Microsoft.AspNetCore.Http;
+using EFB.Sessions;
 
 namespace EFB.Controllers
 {
@@ -20,6 +18,10 @@ namespace EFB.Controllers
 
         public IActionResult Index()
         {
+            //Ensure that the user is not already logged in
+            UserModel user = HttpContext.Session.GetObject<UserModel>("User");
+            if (user != null) return RedirectToAction("Index", "App");
+
             return View();
         }
 

Controllers/NavdataController.cs

@@ -1,12 +1,8 @@
-using System;
-using System.Collections.Generic;
-using System.Diagnostics;
-using System.Linq;
-using System.Threading.Tasks;
-using Microsoft.AspNetCore.Mvc;
-using Microsoft.Extensions.Logging;
 using EFB.Models;
 using EFB.Sessions;
+using Microsoft.AspNetCore.Mvc;
+using Microsoft.Extensions.Logging;
+using System.Threading.Tasks;
 
 namespace EFB.Controllers
 {

Controllers/RouteController.cs

@@ -1,18 +1,17 @@
-using System;
-using System.Threading.Tasks;
-using System.Threading;
-using System.Collections.Generic;
-using System.Text;
-using System.Net.Http;
-using Microsoft.AspNetCore.Mvc;
-using Microsoft.Extensions.Logging;
-using Microsoft.AspNetCore.Http;
-using Newtonsoft.Json;
+using EFB.Controllers.API;
+using EFB.Controllers.Form;
 using EFB.Models;
 using EFB.Models.JSON;
 using EFB.Sessions;
-using EFB.Controllers.Form;
-using EFB.Controllers.API;
+using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.Mvc;
+using Microsoft.Extensions.Logging;
+using Newtonsoft.Json;
+using System.Collections.Generic;
+using System.Net.Http;
+using System.Text;
+using System.Threading;
+using System.Threading.Tasks;
 
 namespace EFB.Controllers
 {
@@ -46,10 +45,16 @@ namespace EFB.Controllers
         public async Task<IActionResult> New(string departure, string arrival, string cruise)
         {
             UserModel user = HttpContext.Session.GetObject<UserModel>("User");
-            if (!(user == null || user.UserToken.IsExpired()))
+            if (user == null || user.UserToken.IsExpired())
             {//If the user is still authenticated
-                if (FormAuthenticator.ValidateICAOCode(departure) && FormAuthenticator.ValidateICAOCode(arrival))
+                return RedirectToAction("Index", "Home");
+            }
+
+            if (!FormAuthenticator.ValidateICAOCode(departure) || !FormAuthenticator.ValidateICAOCode(arrival))
             {//If the user has entered valid ICAOs
+                TempData["Error"] = "Invalid Departure or Arrival ICAO";
+                return RedirectToAction("Index", "Route");
+            }
 
             uint cruiseAlt;
 
@@ -99,24 +104,23 @@ namespace EFB.Controllers
             TempData["Departure"] = departure;
             TempData["Arrival"] = arrival;
             return RedirectToAction("Index", "Route");
-
-                }
-                TempData["Error"] = "Invalid Departure or Arrival ICAO";
-                return RedirectToAction("Index", "Route");
-            }
-            return RedirectToAction("Index", "Home");
         }
 
 
         public async Task<IActionResult> Poll(string departure, string arrival, uint cruise)
         {
-            if (HttpContext.Session.GetString("User") != null)
-            {//If the user is currently logged in
+            if (HttpContext.Session.GetString("User") == null)
+            {//If the user is not currently logged in
+                TempData["Error"] = "Please login before trying to plan a route";
+                return RedirectToAction("Index", "Route");
+            }
+
             UserModel user = HttpContext.Session.GetObject<UserModel>("User");
 
-                if (user.RouteToken != null)
+            if (user.RouteToken == null)
             {//If the user has a route object (e.g, they have been to the route page)
-
+                return RedirectToAction("Index", "Route");
+            }
             //Make calls to the server to fetch route
             bool collected = false;
             int pollCount = 0;
@@ -180,23 +184,11 @@ namespace EFB.Controllers
                 user.Cruise = cruise;
                 HttpContext.Session.SetObject("User", user);
 
-                        return RedirectToAction("Index", "Route");
+                return RedirectToAction("Index", "App");
             }
 
             TempData["Error"] = $"Unable to get route after {pollCount} Attempts!";
             return RedirectToAction("Index", "Route");
-
-                }
-                else
-                {
-                    return RedirectToAction("Index", "Route");
-                }
-
-            }
-            else
-            {
-                return RedirectToAction("Index", "Route");
-            }
         }
 
 

Controllers/UserController.cs

@@ -1,15 +1,13 @@
+using EFB.Controllers.API;
+using EFB.Models;
+using EFB.Models.JSON;
+using EFB.Sessions;
+using Microsoft.AspNetCore.Mvc;
+using Microsoft.Extensions.Logging;
 using System;
 using System.Collections.Generic;
-using System.Diagnostics;
-using System.Linq;
-using System.Threading.Tasks;
-using Microsoft.AspNetCore.Mvc;
 using System.Net.Http;
-using EFB.Models.JSON;
-using Microsoft.Extensions.Logging;
-using EFB.Models;
-using EFB.Sessions;
-using EFB.Controllers.API;
+using System.Threading.Tasks;
 
 namespace EFB.Controllers
 {
@@ -28,11 +26,18 @@ namespace EFB.Controllers
             return View();
         }
 
-
-        public async Task<IActionResult> Login(string email, string password){
-
-            if (Form.FormAuthenticator.ValidateEMail(email))
+        public async Task<IActionResult> Login(string email, string password)
         {
+            //Ensure that the user is not already logged in
+            UserModel user = HttpContext.Session.GetObject<UserModel>("User");
+            if (user != null) return RedirectToAction("Index", "App");
+
+            if (!Form.FormAuthenticator.ValidateEMail(email))
+            {
+                TempData["Error"] = "Please enter a valid E-Mail";
+                return RedirectToAction("Index", "Home");
+            }
+
             //API Helper
             APIInterface API = new APIInterface();
 
@@ -55,17 +60,23 @@ namespace EFB.Controllers
                 TempData["Error"] = response.Error;
                 TempData["email"] = email;
                 return RedirectToAction("Index", "Home");
-                }else{
-
+            }
             //Type cast required but we know response will be of known type
             LoginResponse login = response.Result;
 
             //Generate User Session
-                    if (login.error == null)
+            if (login.error != null)
+            {
+                TempData["Error"] = login.error_description;
+                TempData["email"] = email;
+                return RedirectToAction("Index", "Home");
+            }
+
+            user = new UserModel
             {
-                        UserModel user = new UserModel{
                 EMail = email,
-                            UserToken = new TokenModel{
+                UserToken = new TokenModel
+                {
                     TokenValue = login.access_token,
                     Expiration = DateTime.UtcNow.AddSeconds(login.expires_in)
                 }
@@ -74,23 +85,10 @@ namespace EFB.Controllers
             //Using Session Extensions (Store the user session)
             HttpContext.Session.SetObject("User", user);
             return RedirectToAction("Index", "App");
-                    }else{
-                        TempData["Error"] = login.error_description;
-                        TempData["email"] = email;
-                        return RedirectToAction("Index", "Home");
         }
 
-                    
-                }
-
-            }else{
-                TempData["Error"] = "Please enter a valid E-Mail";
-                return RedirectToAction("Index", "Home");
-            }
-
-        }
-
-        public IActionResult Logout(){
+        public IActionResult Logout()
+        {
             HttpContext.Session.SetObject("User", null);
             return RedirectToAction("Index", "Home");
         }

EFB.csproj

@@ -7,5 +7,6 @@
     <PackageReference Include="MySql.Data" Version="*"/>
     <PackageReference Include="MongoDB.Driver" Version="*"/>
     <PackageReference Include="MongoDB.Bson" Version="*"/>
+    <PackageReference Include="libmetar" Version="*"/>
   </ItemGroup>
 </Project>

Metar/Metar.cs

@@ -0,0 +1,32 @@
+using libmetar.Services;
+using System.Collections.Generic;
+using System.Threading.Tasks;
+
+namespace EFB.Metar
+{
+    public static class Metar
+    {
+        private static MetarService metarService { get; set; } = new MetarService();
+        private static TafService tafService { get; set; } = new TafService();
+
+
+        public static async Task<string> GetMETAR(string ICAO)
+        {
+            return (await metarService.GetRawAsync(ICAO)).Raw;
+        }
+
+        public static async Task<List<string>> GetTAF(string ICAO)
+        {
+            var downloadedTAF = (await tafService.GetRawAsync(ICAO)).RawSplit;
+            List<string> TAF = new List<string>();
+
+            foreach (var line in downloadedTAF)
+            {
+                TAF.Add(line);
+            }
+            return TAF;
+        }
+
+
+    }
+}

Models/ChartModel.cs

@@ -1,18 +1,11 @@
-using System;
-using System.Threading.Tasks;
-using System.Threading;
-using System.Collections.Generic;
-using System.Text;
-using System.Net.Http;
-using Microsoft.AspNetCore.Mvc;
-using Microsoft.Extensions.Logging;
-using Microsoft.AspNetCore.Http;
-using Newtonsoft.Json;
-using EFB.Models;
-using EFB.Models.JSON;
-using EFB.Sessions;
-using EFB.Controllers.Form;
 using EFB.Controllers.API;
+using EFB.Controllers.Form;
+using EFB.Models.JSON;
+using Newtonsoft.Json;
+using System;
+using System.Collections.Generic;
+using System.Net.Http;
+using System.Threading.Tasks;
 
 namespace EFB.Models
 {
@@ -29,7 +22,8 @@ namespace EFB.Models
         public Chart[] PilotBriefing { get; set; }
 
         [JsonConstructor]
-        public ChartModel(){
+        public ChartModel()
+        {
             //Empty constructor for JSON Serialisation Purposes
         }
         public ChartModel(string ICAO, ChartList response)
@@ -45,7 +39,8 @@ namespace EFB.Models
             PilotBriefing = FillChart(response.PilotBriefing);
         }
 
-        private Chart[] FillChart(ChartsCollection collection){
+        private Chart[] FillChart(ChartsCollection collection)
+        {
             if (collection != null)
             {
                 return collection.Charts;
@@ -53,7 +48,8 @@ namespace EFB.Models
             return new Chart[] { };
         }
 
-        public static async Task<ChartList> FetchAsync(string ICAO){
+        public static async Task<ChartList> FetchAsync(string ICAO)
+        {
             Console.WriteLine("Start");
             if (FormAuthenticator.ValidateICAOCode(ICAO))
             {

Models/ErrorViewModel.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace EFB.Models
 {
     public class ErrorViewModel

Models/FlightsimModel.cs

@@ -1,7 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
 using EFB.Models.Route;
 
 namespace EFB.Models

Models/JSON/ChartResponse.cs

@@ -1,7 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
 using Newtonsoft.Json;
 
 namespace EFB.Models.JSON

Models/JSON/Login.cs

@@ -1,8 +1,4 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
 using Newtonsoft.Json;
-using System.Threading.Tasks;
 
 namespace EFB.Models.JSON
 {

Models/JSON/LoginResponse.cs

@@ -1,7 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
 using Newtonsoft.Json;
 
 namespace EFB.Models.JSON

Models/JSON/PollResponse.cs

@@ -1,7 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
 using Newtonsoft.Json;
 
 namespace EFB.Models.JSON

Models/NavdataModel.cs

@@ -1,11 +1,9 @@
+using MySql.Data.MySqlClient;
+using Newtonsoft.Json;
 using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Threading.Tasks;
-using MySql.Data.MySqlClient;
-using Newtonsoft.Json;
-using Microsoft.AspNetCore.Mvc;
-using EFB.Sessions;
 
 namespace EFB.Models
 {
@@ -19,7 +17,8 @@ namespace EFB.Models
         public string Longitude { get; set; }
 
 
-        public NavdataModel(int id, string name, string type, string latitude, string longitude){
+        public NavdataModel(int id, string name, string type, string latitude, string longitude)
+        {
             Id = id;
             Name = name;
             Type = type;
@@ -29,7 +28,8 @@ namespace EFB.Models
         }
 
         [JsonConstructor]
-        public NavdataModel(int id, string name, string type, int? frequency, string latitude, string longitude){
+        public NavdataModel(int id, string name, string type, int? frequency, string latitude, string longitude)
+        {
             Id = id;
             Name = name;
             Type = type;
@@ -38,7 +38,8 @@ namespace EFB.Models
             Longitude = longitude;
         }
 
-        public static async Task<NavdataModel[]> Populate(){
+        public static async Task<NavdataModel[]> Populate()
+        {
             string password = Environment.GetEnvironmentVariable("MySQLPassword", EnvironmentVariableTarget.User);
             MySqlConnection con = new MySqlConnection($"server=server.luke-else.co.uk;userid=root;password={password};database=EFB");
             con.Open();
@@ -67,7 +68,9 @@ namespace EFB.Models
                     navdata.Add(
                         new NavdataModel(id, name, type, frequency, latitude, longitude)
                     );
-                }else{
+                }
+                else
+                {
                     navdata.Add(
                         new NavdataModel(id, name, type, latitude, longitude)
                     );
@@ -77,7 +80,8 @@ namespace EFB.Models
             return navdata.ToArray<NavdataModel>();
         }
 
-        public static NavdataModel BinarySearch(ref NavdataModel[] data, int start, int end, string target){
+        public static NavdataModel BinarySearch(ref NavdataModel[] data, int start, int end, string target)
+        {
             int midpoint = start + ((end - start) / 2);
             target = target.ToUpper().Trim();
 
@@ -122,14 +126,19 @@ namespace EFB.Models
                 if (leftPointer == left.Length)
                 {//Take a value only from the right (left pointer had reached the end)
                     AddValue(ref combined, right[rightPointer], ref rightPointer);
-                }else if (rightPointer == right.Length)
+                }
+                else if (rightPointer == right.Length)
                 {//Take a value only from the left (right pointer has reached the end)
                     AddValue(ref combined, left[leftPointer], ref leftPointer);
-                }else{
+                }
+                else
+                {
                     if (String.Compare(left[leftPointer].Name, right[rightPointer].Name) <= 0)
                     {//Take a value from the left hand side of the list. (Left value is considered 'smaller')
                         AddValue(ref combined, left[leftPointer], ref leftPointer);
-                    }else{//Take a value from the right (right value is considered smaller)
+                    }
+                    else
+                    {//Take a value from the right (right value is considered smaller)
                         AddValue(ref combined, right[rightPointer], ref rightPointer);
                     }
                 }
@@ -139,7 +148,8 @@ namespace EFB.Models
 
         }
 
-        private static void AddValue(ref List<NavdataModel> data, NavdataModel value, ref int pointer){
+        private static void AddValue(ref List<NavdataModel> data, NavdataModel value, ref int pointer)
+        {
             pointer++;
             data.Add(value);
         }

Models/ResponseModel.cs

@@ -1,8 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
-
 namespace EFB.Models
 {
     public class ResponseModel<T>

Models/Route/IWaypoint.cs

@@ -1,8 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
-
 namespace EFB.Models.Route
 {
     public interface IWaypoint

Models/Route/NavaidModel.cs

@@ -1,8 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
-
 namespace EFB.Models.Route
 {
     public class NavaidModel : IWaypoint
@@ -17,7 +12,8 @@ namespace EFB.Models.Route
         public IWaypoint Previous { get; set; } = null;
         public bool Visited { get; set; } = false;
 
-        public NavaidModel(string name, string airway, float longitude, float latitude){
+        public NavaidModel(string name, string airway, float longitude, float latitude)
+        {
             Name = name;
             Airway = airway;
             Longitude = longitude;

Models/Route/WaypointModel.cs

@@ -1,8 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
-
 namespace EFB.Models.Route
 {
     public class WaypointModel : IWaypoint
@@ -16,7 +11,8 @@ namespace EFB.Models.Route
         public IWaypoint Previous { get; set; } = null;
         public bool Visited { get; set; }
 
-        public WaypointModel(string name, string airway, float longitude, float latitude){
+        public WaypointModel(string name, string airway, float longitude, float latitude)
+        {
             Name = name;
             Airway = airway;
             Longitude = longitude;

Models/RouteModel.cs

@@ -1,10 +1,6 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
-using EFB.Models.Route;
 using EFB.Controllers.Form;
-using System.Net.Http;
+using EFB.Models.Route;
+using System.Threading.Tasks;
 
 namespace EFB.Models
 {
@@ -19,7 +15,8 @@ namespace EFB.Models
         public WaypointModel Arrival { get; set; } = null;
         public IWaypoint Current { get; set; } = null;
         public uint Cruise { get; set; } = 0;
-        public RouteModel(string departure, string departureRoute, string arrival, string arrivalRoute, uint cruise){ 
+        public RouteModel(string departure, string departureRoute, string arrival, string arrivalRoute, uint cruise)
+        {
             if (FormAuthenticator.ValidateICAOCode(departure))
             {
                 Departure = new WaypointModel(departure, departureRoute, 0, 0);
@@ -36,16 +33,20 @@ namespace EFB.Models
             }
         }
 
-        public IWaypoint Next(){
-            if(Current.Next != null){
+        public IWaypoint Next()
+        {
+            if (Current.Next != null)
+            {
                 Current = Current.Next;
                 return Current;
             }
             return null;
         }
 
-        public IWaypoint Previous(){
-            if(Current.Previous != null){
+        public IWaypoint Previous()
+        {
+            if (Current.Previous != null)
+            {
                 Current = Current.Previous;
                 return Current;
             }
@@ -53,7 +54,8 @@ namespace EFB.Models
         }
 
         //Generate a route Object
-        public static async Task<RouteModel> StringToRoute(string departure, string arrival, uint cruise, string routeString){
+        public static async Task<RouteModel> StringToRoute(string departure, string arrival, uint cruise, string routeString)
+        {
             var navdataFetch = NavdataModel.Populate();
             string[] routeTemp = routeString.Split(" ");
 
@@ -86,7 +88,8 @@ namespace EFB.Models
                         float.Parse(currentWaypoint.Longitude),
                         float.Parse(currentWaypoint.Latitude)
                     );
-                }else
+                }
+                else
                 {//Navaid Type
                     next = new NavaidModel(
                         routeTemp[i],
@@ -123,7 +126,8 @@ namespace EFB.Models
 
 
         //Generate a route String
-        public static string ParseRoute(string route){
+        public static string ParseRoute(string route)
+        {
             route.Replace('/', ' ');
             var routeArr = route.Split(' ');
 

Models/SimPositionModel.cs

@@ -1,10 +1,6 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
-using MongoDB.Bson.Serialization.Attributes;
 using MongoDB.Bson;
-using MongoDB.Driver;
+using MongoDB.Bson.Serialization.Attributes;
+using System;
 
 namespace EFB.Models
 {
@@ -16,7 +12,8 @@ namespace EFB.Models
         public DateTime LatestPacketUpdate { get; set; }
         public SimPosition LatestPosition { get; set; } = null;
 
-        public SimPositionModel(string email, SimPosition position){
+        public SimPositionModel(string email, SimPosition position)
+        {
             EMail = email;
             LatestPacketUpdate = DateTime.Now;
             LatestPosition = position;
@@ -31,7 +28,8 @@ namespace EFB.Models
         public float Longitude { get; set; }
         public int Altitude { get; set; }
 
-        public SimPosition(float latitude, float longitude, int altitude){
+        public SimPosition(float latitude, float longitude, int altitude)
+        {
             Latitude = latitude;
             Longitude = longitude;
             Altitude = altitude;

Models/TokenModel.cs

@@ -1,7 +1,4 @@
 using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
 
 namespace EFB.Models
 {
@@ -13,7 +10,8 @@ namespace EFB.Models
         public string TokenValue { get; init; }
         public DateTime Expiration { get; init; }
 
-        public bool IsExpired(){
+        public bool IsExpired()
+        {
             //Check if the current time is beyond expiration
             if (DateTime.UtcNow > Expiration)
             {

Models/UserModel.cs

@@ -1,8 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
-
 namespace EFB.Models
 {
     public class UserModel

Models/ViewChartModel.cs

@@ -1,14 +1,5 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
-using Microsoft.AspNetCore.Mvc;
-using Microsoft.AspNetCore.Html;
-using Microsoft.AspNetCore.Razor;
-using Newtonsoft.Json;
 using EFB.Models.JSON;
-using Microsoft.AspNetCore;
-using EFB.Sessions;
+using Newtonsoft.Json;
 
 namespace EFB.Models
 {
@@ -17,7 +8,8 @@ namespace EFB.Models
         public ChartModel Charts { get; set; }
         public Chart Selected { get; set; }
 
-        public ViewChartModel(ChartModel current, string selected){
+        public ViewChartModel(ChartModel current, string selected)
+        {
             Charts = current;
             if (selected != null)
             {

Mongo/Mongo.cs

@@ -1,29 +1,29 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
-using MongoDB.Driver;
-using MongoDB.Bson;
 using EFB.Models;
+using MongoDB.Driver;
+using System;
+using System.Threading.Tasks;
 
 namespace EFB.MongoData
 {
     public class Mongo
     {
         //function that is responsible to getting the user's latest sim position from the MongoDB
-        public static async Task<SimPositionModel> GetLatestData(string email){
+        public static async Task<SimPositionModel> GetLatestData(string email)
+        {
+            //Create mongo client to allow connection to the database
             MongoClient client = new MongoClient(
                 Environment.GetEnvironmentVariable("MongoDBConnectionString", EnvironmentVariableTarget.User)
             );
             MongoDatabaseBase database = (MongoDatabaseBase)client.GetDatabase("EFB");
             MongoCollectionBase<SimPositionModel> collection = (MongoCollectionBase<SimPositionModel>)database.GetCollection<SimPositionModel>("Simdata");
 
+            //Create filter to select specifically the records where the Email is the same as what we specified
             FilterDefinition<SimPositionModel> filter = Builders<SimPositionModel>.Filter.Eq(x => x.EMail, email);
             var data = await collection.FindAsync<SimPositionModel>(filter).Result.ToListAsync();
             if (data.Count > 0)
-            {
+            {//Return the first item in the list (Should only be 1)
                 return data[0];
-            }
+            }//Or return null if not present
             return null;
         }
     }

Program.cs

@@ -1,11 +1,5 @@
 using Microsoft.AspNetCore.Hosting;
-using Microsoft.Extensions.Configuration;
 using Microsoft.Extensions.Hosting;
-using Microsoft.Extensions.Logging;
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
 
 namespace EFB
 {

Sessions/SessionExtensions.cs

@@ -8,7 +8,8 @@ namespace EFB.Sessions
         public static void SetObject(this ISession session, string key, object value)
         {//Sets the object of a session to Object
             session.SetString(key, JsonConvert.SerializeObject(value, Formatting.None,
-                new JsonSerializerSettings(){
+                new JsonSerializerSettings()
+                {
                     ReferenceLoopHandling = ReferenceLoopHandling.Ignore
                 }
             ));

Startup.cs

@@ -1,13 +1,8 @@
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Hosting;
-using Microsoft.AspNetCore.HttpsPolicy;
 using Microsoft.Extensions.Configuration;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Hosting;
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
 
 namespace EFB
 {

Views/App/Index.cshtml

@@ -0,0 +1,48 @@
+@using EFB.Metar;
+@model EFB.Models.UserModel;
+@{
+    ViewData["Title"] = "Welcome";
+}
+
+<div class="row d-flex">
+    <div class="card-body col-md-12 bg-primary">
+        <div class="container jumbotron">
+            <h3>Current Session - @Model.EMail</h3>
+
+            <br />
+            <br />
+
+            
+            <div class="row d-flex">
+                <div class="col-md-6">
+                    <h4>Departure</h4>
+                    @Model.Departure
+
+                    <br />
+
+                    @await Metar.GetMETAR(Model.Departure)
+
+                    <h4>Cruise</h4>
+                    @Model.Cruise ft
+                </div>
+                <div class="col-md-6">
+                    <h4>Arrival</h4>
+                    @Model.Arrival
+
+                    <br />
+
+                    @await Metar.GetMETAR(Model.Arrival)
+                </div>
+            </div>
+
+            <br />            
+
+            <h4>Route</h4>
+            @Model.Route
+
+        </div>
+    </div>
+</div>
+
+
+

Views/Charts/ViewCharts.cshtml

@@ -1,4 +1,5 @@
 @using Newtonsoft.Json;
+@using EFB.Metar;
 @model EFB.Models.ViewChartModel;
 @{
     ViewData["Title"] = "Welcome";
@@ -80,6 +81,10 @@
                     </div>
                 </form>
             }
+
+            <br />
+            <h4>Current Weather</h4>
+            @await Metar.GetMETAR(@Model.Charts.ICAO);
         </div>
     </div>
 
@@ -99,4 +104,6 @@
             }
         </div>
     </div>
+
+
 </div>