Лабораторная работа №7 (исправленная)

WinFormsAppExcavator/WinFormsAppExcavator/CollectionGenericObjects/StorageCollection.cs

@@ -139,73 +139,60 @@ public class StorageCollection<T>
     /// </summary>
     /// <param name="filename">Путь и имя файла</param>/// <returns>true - загрузка прошла успешно, false - ошибка при загрузке 
     ///данных</returns>
-    public bool LoadData(string filename)
+    public void LoadData(string filename)
     {
         if (!File.Exists(filename))
         {
             throw new Exception("Файл не существует");
         }
-
+        using (StreamReader fs = File.OpenText(filename))
-        string bufferTextFromFile = "";
-        using (FileStream fs = new(filename, FileMode.Open))
         {
-            byte[] b = new byte[fs.Length];
+            string str = fs.ReadLine();
-            UTF8Encoding temp = new(true);
+            if (str == null || str.Length == 0)
-            while (fs.Read(b, 0, b.Length) > 0)
             {
-                bufferTextFromFile += temp.GetString(b);
+                throw new Exception("В файле нет данных");
             }
-        }
+            if (!str.StartsWith(_collectionKey))
-
-        string[] strs = bufferTextFromFile.Split(new char[] { '\n', '\r' }, StringSplitOptions.RemoveEmptyEntries);
-        if (strs == null || strs.Length == 0)
-        {
-            throw new Exception("В файле нет данных");
-        }
-
-        if (!strs[0].Equals(_collectionKey))
-        {
-            throw new Exception("В файле неверные данные");
-        }
-
-        _storages.Clear();
-        foreach (string data in strs)
-        {
-            string[] record = data.Split(_separatorForKeyValue, StringSplitOptions.RemoveEmptyEntries);
-            if (record.Length != 4)
             {
-                continue;
+                throw new Exception("В файле неверные данные");
             }
-
+            _storages.Clear();
-
+            string strs = "";
-            CollectionType collectionType = (CollectionType)Enum.Parse(typeof(CollectionType), record[1]);
+            while ((strs = fs.ReadLine()) != null)
-            ICollectionGenericObjects<T>? collection = StorageCollection<T>.CreateCollection(collectionType) ??
-                throw new Exception("Не удалось определить тип коллекции:" + record[1]);
-            collection.MaxCount = Convert.ToInt32(record[2]);
-
-            string[] set = record[3].Split(_separatorItems, StringSplitOptions.RemoveEmptyEntries);
-            foreach (string elem in set)
             {
-                if (elem?.CreateDrawningExcavatorEmpty() is T excavator)
+                string[] record = strs.Split(_separatorForKeyValue, StringSplitOptions.RemoveEmptyEntries);
+                if (record.Length != 4)
                 {
-                    try
+                    continue;
+                }
+                CollectionType collectionType = (CollectionType)Enum.Parse(typeof(CollectionType), record[1]);
+                ICollectionGenericObjects<T>? collection = StorageCollection<T>.CreateCollection(collectionType);
+                if (collection == null)
+                {
+                    throw new Exception("Не удалось создать коллекцию");
+                }
+                collection.MaxCount = Convert.ToInt32(record[2]);
+                string[] set = record[3].Split(_separatorItems, StringSplitOptions.RemoveEmptyEntries);
+                foreach (string elem in set)
+                {
+                    if (elem?.CreateDrawningExcavatorEmpty() is T excavator)
                     {
-                        if (collection.Insert(excavator) != -1)
+                        try
                         {
-                            throw new Exception("Объект не удалось добавить в коллекцию: " + record[3]);
+                            if (collection.Insert(excavator) == -1)
+                            {
+                                throw new Exception("Объект не удалось добавить в коллекцию: " + record[3]);
+                            }
+                        }
+                        catch (CollectionOverflowException ex)
+                        {
+                            throw new Exception("Коллекция переполнена", ex);
                         }
                     }
-                    catch (CollectionOverflowException ex)
-                    {
-                        throw new Exception("Коллекция переполнена", ex);
-                    }
                 }
+                _storages.Add(record[0], collection);
             }
-
-            _storages.Add(record[0], collection);
         }
-
-        return true;
     }
 
     /// <summary>