AirplaneWithRadar/ProjectAirplaneWithRadar/CollectionGenericObjects/StorageCollection.cs

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

AirplaneWithRadar/ProjectAirplaneWithRadar/Exceptions/CollectionOverflowException.cs

@@ -1,21 +0,0 @@
-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

@@ -1,21 +0,0 @@
-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

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