Revize 127a92b1

Cosmetic changes, documentation adding, warnings fixing.

Server/ServerApp/Program.cs

@@ -48 +48 @@
 			}
 			Console.WriteLine("Config parsing successful.");
 
-			//logger.Debug("ahoj");
-			//logger.Warn("ahoj2");
-			//Console.ReadLine();
 
 			// data download init
 			Console.WriteLine("Downloading open data...");
 			DataDownloader dd = DataDownloaderInit(config);
 			Console.WriteLine("Data downloaded and saved.");
 
-			//WeatherAsStringTest(dd);
-			//Console.ReadLine();
-
-			// xml building test
-			//XMLTest();
-
-
-			// json parser test
-			//JSONParserTest();
 
 			// model init
 			Console.WriteLine("Training the predictor...");
@@ -85 +73 @@
 
         }
 
-
+		/// <summary>
+		/// Initializes DataDownloader. Downloads default span of data and saves the files.
+		/// </summary>
+		/// <param name="config">The Config instance</param>
+		/// <returns>New instance of DataDownloader</returns>
 		private static DataDownloader DataDownloaderInit(Config config)
 		{
-			//test scenario -data download:
+			
 			DataDownloader dd = new DataDownloader(config.DataRootDir, config.DataWebsite, config.DownloadedFilesNaming, config.WeatherSite);
 			dd.OverwriteExisting = false;
 			List<string> savedFiles = new List<string>();
 
-			// -> 12-2019 to exclude corona? But we lose a lot of data
+			// -> 12-2019 to exclude corona period which could mess with the predictor
 			savedFiles.AddRange(dd.DownloadData(DataType.JIS, DataFormat.CSV, new DataDownload.Date(1, 2017), new DataDownload.Date(12, 2019)));
 			savedFiles.AddRange(dd.DownloadData(DataType.STROJE, DataFormat.CSV, new DataDownload.Date(1, 2017), new DataDownload.Date(12, 2019)));
 			savedFiles.AddRange(dd.DownloadData(DataType.POCASI, DataFormat.CSV, new DataDownload.Date(1, 2017), new DataDownload.Date(12, 2019)));
@@ -130 +122 @@
 		}
 
 
-		private static void XMLTest()
-		{
-			Response response = Response.Randomize();
-			var xml = XmlCommunication.Serialize(response);
-			//xml = "haha";
-			Response response1 = new Response();
-			Console.WriteLine(xml);
-			try
-			{
-				Response responseDeserialized = XmlCommunication.Deserialize<Response>(xml);
-			}
-			catch(Exception e)
-			{
-				Console.WriteLine("bad format");
-			}
-			
-
-			Console.WriteLine("------");
-
-
-			Request request = Request.Randomize();
-			xml = XmlCommunication.Serialize(request);
-			Console.WriteLine(xml);
-			Request requestDeserialized = XmlCommunication.Deserialize<Request>(xml);
-		}
-
-
 		private static void JSONParserTest()
 		{
             // FIXME pass the right references to the JsonParser constructor
@@ -175 +140 @@
 			Console.WriteLine(dd.DownloadWeatherPrediction());
 		}
 
+		/// <summary>
+		/// Initializes and trains the predictor if need be.
+		/// </summary>
+		/// <param name="dd">The DataDownloader instance.</param>
+		/// <returns>New instance of the predictor.</returns>
 		private static IPredictionController PredictionControllerInit(DataDownloader dd)
 		{
 			IDataParser p = new DataParser(dd);
@@ -187 +157 @@
 			return predictionController;
 		}
 
+		/// <summary>
+		/// Initializes waiting for the clients' requests.
+		/// </summary>
+		/// <param name="predictionController">The initialized and trained instance of the predictor.</param>
+		/// <param name="config">The Config class instance.</param>
 		private static void ConnectionInit(IPredictionController predictionController, Config config)
 		{
 			ConnectionListenerAsync cl = new ConnectionListenerAsync(int.Parse(config.Port), predictionController);
 			cl.StartListening();
 
-			//HttpRequestHandler hrh = new HttpRequestHandler(int.Parse(config.Port));
-			//hrh.ListenAsynchronously();
 		}
 
-
+		/// <summary>
+		/// Parses the configuration file and extracts valuable information from it.
+		/// </summary>
+		/// <param name="args">The arguments passed to the program.</param>
+		/// <returns>Filled instance of the Config class, null if there was a problem.</returns>
 		private static Config FillConfigInfo(string[] args)
 		{
 
@@ -215 +192 @@
 			catch(Exception ex)
 			{
 				Console.WriteLine("Could not open " + fullPathConfig);
+				Console.WriteLine(ex.Message);
 				return null;
 			}
 			
@@ -256 +234 @@
 			bool success = int.TryParse(extractedConfigInfo.Port, out parsedPort);
 			if (!success)
 			{
-				Console.WriteLine("Configured port " + extractedConfigInfo.Port + " is not an integer number! Abort.");
+				Console.WriteLine("Configured port " + extractedConfigInfo.Port + " is not an integer! Abort.");
 				return null;
 			}