StreamWriter, StreamReader для сохранения и загрузки, логика нажатия на загрузку в меню

AntiAircraftGun/AntiAircraftGun/ExtentionAntiAircraftGun.cs

@@ -7,7 +7,7 @@ using System.Threading.Tasks;
 namespace AntiAircraftGun
 {
     /// <summary>
-    /// Расширение для класса DrawningCar
+    /// Расширение для класса DrawingAntiAircraftGun
     /// </summary>
     internal static class ExtentionAntiAircraftGun
     {

AntiAircraftGun/AntiAircraftGun/FormMapWithSetAntiAircraftGuns.cs

@@ -243,7 +243,18 @@ namespace AntiAircraftGun
         /// <param name="e"></param>
         private void LoadToolStripMenuItem_Click(object sender, EventArgs e)
         {
-            // TODO продумать логику
+            if (openFileDialog.ShowDialog() == DialogResult.OK)
+            {
+                if (_mapsCollection.LoadData(openFileDialog.FileName))
+                {
+                    MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
+                    ReloadMaps();
+                }
+                else
+                {
+                    MessageBox.Show("Не загрузилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                }
+            }
         }
     }
 }

AntiAircraftGun/AntiAircraftGun/MapsCollection.cs

@@ -74,16 +74,6 @@ namespace AntiAircraftGun
             }
         }
         /// <summary>
-        /// Метод записи информации в файл
-        /// </summary>
-        /// <param name="text">Строка, которую следует записать</param>
-        /// <param name="stream">Поток для записи</param>
-        private static void WriteToFile(string text, FileStream stream)
-        {
-            byte[] info = new UTF8Encoding(true).GetBytes(text);
-            stream.Write(info, 0, info.Length);
-        }
-        /// <summary>
         /// Сохранение информации по орудиям в хранилище в файл
         /// </summary>
         /// <param name="filename">Путь и имя файла</param>
@@ -94,12 +84,12 @@ namespace AntiAircraftGun
             {
                 File.Delete(filename);
             }
-            using (FileStream fs = new(filename, FileMode.Create))
+            using (StreamWriter sw = new StreamWriter(filename))
             {
-                WriteToFile($"MapsCollection{Environment.NewLine}", fs);
+                sw.WriteLine($"MapsCollection");
                 foreach (var storage in _mapStorages)
                 {
-                    WriteToFile($"{storage.Key}{separatorDict}{storage.Value.GetData(separatorDict, separatorData)}{Environment.NewLine}", fs);
+                    sw.WriteLine($"{storage.Key}{separatorDict}{storage.Value.GetData(separatorDict, separatorData)}");
                 }
             }
             return true;
@@ -115,44 +105,33 @@ namespace AntiAircraftGun
             {
                 return false;
             }
-            string bufferTextFromFile = "";
+            using (StreamReader sr = new StreamReader(filename))
-            using (FileStream fs = new(filename, FileMode.Open))
             {
-                byte[] b = new byte[fs.Length];
+                string str;
-                UTF8Encoding temp = new(true);
+                _mapStorages.Clear();
-                while (fs.Read(b, 0, b.Length) > 0)
+                str = sr.ReadLine();
+                if (!str.Contains("MapsCollection"))
                 {
-                    bufferTextFromFile += temp.GetString(b);
+                    return false;
                 }
-            }
+                while ((str = sr.ReadLine()) != null)
-            var strs = bufferTextFromFile.Split(new char[] { '\n', '\r' }, StringSplitOptions.RemoveEmptyEntries);
-            if (!strs[0].Contains("MapsCollection"))
-            {
-                //если нет такой записи, то это не те данные
-                return false;
-            }
-            //очищаем записи
-            _mapStorages.Clear();
-            for (int i = 1; i < strs.Length; ++i)
-            {
-                var elem = strs[i].Split(separatorDict);
-                AbstractMap map = null;
-                switch (elem[1])
                 {
-                    case "SimpleMap":
+                    var elem = str.Split(separatorDict);
-                        map = new SimpleMap();
+                    AbstractMap map = null;
-                        break;
+                    switch (elem[1])
+                    {
+                        case "SimpleMap":
+                            map = new SimpleMap();
+                            break;
+                        case "MySecondMap":
+                            map = new SimpleMap2();
+                            break;
+                    }
+                    _mapStorages.Add(elem[0], new MapWithSetAntiAircraftGunsGeneric<IDrawingObject, AbstractMap>(_pictureWidth, _pictureHeight, map));
+                    _mapStorages[elem[0]].LoadData(elem[2].Split(separatorData, StringSplitOptions.RemoveEmptyEntries));
                 }
-                switch (elem[2])
+                return true;
-                {
-                    case "SimpleMap2":
-                        map = new SimpleMap2();
-                        break;
-                }
-                _mapStorages.Add(elem[0], new MapWithSetAntiAircraftGunsGeneric<IDrawingObject, AbstractMap>(_pictureWidth, _pictureHeight, map));
-                _mapStorages[elem[0]].LoadData(elem[2].Split(separatorData, StringSplitOptions.RemoveEmptyEntries));
             }
-            return true;
         }
     }
 }