Изменение обычных Exception

AirplaneWithRadar/ProjectAirplaneWithRadar/CollectionGenericObjects/StorageCollection.cs

@@ -1,6 +1,7 @@
 using System.IO;
 using System.Text;
 using ProjectAirplaneWithRadar.Drawnings;
+using ProjectAirplaneWithRadar.Exceptions;
 
 namespace ProjectAirplaneWithRadar.CollectionGenericObjects
 {
@@ -98,10 +99,10 @@ namespace ProjectAirplaneWithRadar.CollectionGenericObjects
         /// </summary>
         /// <param name="filename">Путь и имя файла</param>
         /// <returns>true - сохранение прошло успешно, false - ошибка при сохранении данных</returns>
-        public bool SaveData(string filename)
+        public void SaveData(string filename)
         {
-            if(_storages.Count == 0)
-                return false;
+            if (_storages.Count == 0)
+                throw new InvalidDataException("В хранилище отсутсвуют коллекции для сохранения");
 
             if(File.Exists(filename))
                 File.Delete(filename);            
@@ -136,7 +137,6 @@ namespace ProjectAirplaneWithRadar.CollectionGenericObjects
                     sw.Write(_separatorItems);
                 }
             }
-            return true;
         }
 
         /// <summary>
@@ -144,11 +144,11 @@ namespace ProjectAirplaneWithRadar.CollectionGenericObjects
         /// </summary>
         /// <param name="filename">Путь и имя файла</param>
         /// <returns>true - загрузка прошла успешно, false - ошибка при загрузке данных</returns>
-        public bool LoadData(string filename)
+        public void LoadData(string filename)
         {
             if (!File.Exists(filename))
             {
-                return false;
+                throw new FileNotFoundException("Файл не существует");
             }
 
             using (FileStream fs = new(filename, FileMode.Open))
@@ -158,12 +158,12 @@ namespace ProjectAirplaneWithRadar.CollectionGenericObjects
                 string str = sr.ReadLine();
                 if (str == null || str.Length == 0)
                 {
-                    return false;
+                    throw new InvalidDataException("В файле нет данных");
                 }
 
                 if (!str.Equals(_collectionKey))
                 {
-                    return false;
+                    throw new InvalidOperationException("В файле неверные данные");
                 }
                 _storages.Clear();
 
@@ -179,7 +179,7 @@ namespace ProjectAirplaneWithRadar.CollectionGenericObjects
                     ICollectionGenericObjects<T>? collection = StorageCollection<T>.CreateCollection(collectionType);
                     if (collection == null)
                     {
-                        return false;
+                        throw new InvalidOperationException("Не удалось создать коллекцию");
                     }
 
                     collection.MaxCount = Convert.ToInt32(record[2]);
@@ -189,14 +189,22 @@ namespace ProjectAirplaneWithRadar.CollectionGenericObjects
                     {
                         if (elem?.CreateDrawningAirplane() is T airplane)
                         {
-                            if (collection.Insert(airplane) == -1)
-                                return false;
+                            try
+                            {
+                                if (collection.Insert(airplane) == -1)
+                                {
+                                    throw new InvalidOperationException("Объект не удалось добавить в коллекцию: " + record[3]);
+                                }
+                            }
+                            catch (CollectionOverflowException ex)
+                            {
+                                throw new ArgumentOutOfRangeException("Коллекция переполнена", ex);
+                            }
                         }
                     }
                     _storages.Add(record[0], collection);
                 }
             }
-            return true;
         }
 
         /// <summary>

AirplaneWithRadar/ProjectAirplaneWithRadar/Exceptions/CollectionOverflowException.cs

@@ -0,0 +1,21 @@
+using System.Runtime.Serialization;
+
+namespace ProjectAirplaneWithRadar.Exceptions    
+{
+    /// <summary>
+    /// Класс, описывающий ошибку переполнения коллекции
+    /// </summary>
+    [Serializable]
+    internal class CollectionOverflowException : ApplicationException
+    {
+        public CollectionOverflowException(int count) : base("В коллекции превышено допустимое количество: " + count) { }
+
+        public CollectionOverflowException() : base() { }
+
+        public CollectionOverflowException(string message) : base(message) { }
+
+        public CollectionOverflowException(string message, Exception exception) : base(message, exception) { }
+
+        protected CollectionOverflowException(SerializationInfo info, StreamingContext contex) : base(info, contex) { }
+    }
+}

AirplaneWithRadar/ProjectAirplaneWithRadar/Exceptions/ObjectNotFoundException.cs

@@ -0,0 +1,21 @@
+using System.Runtime.Serialization;
+
+namespace ProjectAirplaneWithRadar.Exceptions
+{
+    /// <summary>
+    /// Класс, описывающий ошибку, что по указанной позиции нет элемента
+    /// </summary>
+    [Serializable]
+    internal class ObjectNotFoundException : ApplicationException
+    {
+        public ObjectNotFoundException(int i) : base("Не найден объект по позиции " + i) { }
+
+        public ObjectNotFoundException() : base() { }
+
+        public ObjectNotFoundException(string message) : base(message) { }
+
+        public ObjectNotFoundException(string message, Exception exception) : base(message, exception) { }
+
+        protected ObjectNotFoundException(SerializationInfo info, StreamingContext contex) : base(info, contex) { }
+    }
+}

AirplaneWithRadar/ProjectAirplaneWithRadar/Exceptions/PositionOutOfCollectionException.cs

@@ -0,0 +1,21 @@
+using System.Runtime.Serialization;
+
+namespace ProjectAirplaneWithRadar.Exceptions
+{
+    /// <summary>
+    /// Класс, описывающий ошибку выхода за границы коллекции
+    /// </summary>
+    [Serializable]
+    internal class PositionOutOfCollectionException : ApplicationException
+    {
+        public PositionOutOfCollectionException(int i) : base("Выход за границы коллекции. Позиция " + i) { }
+
+        public PositionOutOfCollectionException() : base() { }
+
+        public PositionOutOfCollectionException(string message) : base(message) { }
+
+        public PositionOutOfCollectionException(string message, Exception exception) : base(message, exception) { }
+
+        protected PositionOutOfCollectionException(SerializationInfo info, StreamingContext contex) : base(info, contex) { }
+    }
+}

AirplaneWithRadar/ProjectAirplaneWithRadar/FormAirplaneCollection.cs

@@ -249,13 +249,14 @@ namespace ProjectAirplaneWithRadar
         {
             if (saveFileDialog.ShowDialog() == DialogResult.OK)
             {
-                if (_storageCollection.SaveData(saveFileDialog.FileName))
+                try
                 {
+                    _storageCollection.SaveData(saveFileDialog.FileName);
                     MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
                 }
-                else
+                catch(Exception ex)
                 {
-                    MessageBox.Show("Не сохранилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                    MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
                 }                
             }
         }
@@ -269,14 +270,15 @@ namespace ProjectAirplaneWithRadar
         {
             if (openFileDialog.ShowDialog() == DialogResult.OK)
             {
-                if (_storageCollection.LoadData(openFileDialog.FileName))
+                try
                 {
+                    _storageCollection.LoadData(openFileDialog.FileName);
                     MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
                     RefreshListBoxItems();
                 }
-                else
+                catch(Exception ex)
                 {
-                    MessageBox.Show("Не сохранилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                    MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
                 }
             }
         }