Users input check moved to a new method

Author: AleksaMCode

CacheSimulator/MainWindow.xaml.cs

@@ -54,19 +54,8 @@ private async void StartSimulation(object sender, RoutedEventArgs e)
         {
             logNumberOfWrites = 0;
 
-            if (traceFileFullPaths == null)
-            {
-                MessageBox.Show("Please insert the trace file first.", "Warning", MessageBoxButton.OK, MessageBoxImage.Warning);
-                return;
-            }
-            else if (ramFileFullPath == null)
-            {
-                MessageBox.Show("Please insert the RAM file first.", "Warning", MessageBoxButton.OK, MessageBoxImage.Warning);
-                return;
-            }
-            else if (cacheSize.Text == "" || cacheLineSize.Text == "")
+            if (!SimulationStartCheck())
             {
-                MessageBox.Show("Please fill out all of the cache parameters with values.", "Warning", MessageBoxButton.OK, MessageBoxImage.Warning);
                 return;
             }
 
@@ -206,6 +195,27 @@ private async void StartSimulation(object sender, RoutedEventArgs e)
         }
 
 
+        private bool SimulationStartCheck()
+        {
+            if (traceFileFullPaths == null)
+            {
+                MessageBox.Show("Please insert the trace file first.", "Warning", MessageBoxButton.OK, MessageBoxImage.Warning);
+                return false;
+            }
+            else if (ramFileFullPath == null)
+            {
+                MessageBox.Show("Please insert the RAM file first.", "Warning", MessageBoxButton.OK, MessageBoxImage.Warning);
+                return false;
+            }
+            else if (cacheSize.Text == "" || cacheLineSize.Text == "")
+            {
+                MessageBox.Show("Please fill out all of the cache parameters with values.", "Warning", MessageBoxButton.OK, MessageBoxImage.Warning);
+                return false;
+            }
+
+            return true;
+        }
+
         private CacheConfigurationBuilder GetCacheConfigBuilder()
         {
             var cacheConfigBuilder = new CacheConfigurationBuilder();