Battleship/Battleship/CollectionGenericObjects/StorageCollection.cs

@@ -93,12 +93,11 @@ public class StorageCollection<T>
     /// Сохранение информации по кораблям в хранилище в файл
     /// </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;
+            throw new Exception("В хранилище отсутствуют коллекции для сохранения");
         }
 
         if (File.Exists(filename))
@@ -134,19 +133,17 @@ public class StorageCollection<T>
                 sb.Clear();    
             }
         }
-        return true;
     }
 
     /// <summary>
     /// Загрузка информации по кораблям в хранилище из файла
     /// </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 Exception("Файл не существует");
         }
 
         using (StreamReader sr = new StreamReader(filename))
@@ -154,7 +151,7 @@ public class StorageCollection<T>
             string? str;
             str = sr.ReadLine();
             if (str != _collectionKey.ToString())
-                return false;
+                throw new Exception("В файле неверные данные");
             _storages.Clear();
             while ((str = sr.ReadLine()) != null)
             {
@@ -167,7 +164,7 @@ public class StorageCollection<T>
                 ICollectionGenericObjects<T>? collection = StorageCollection<T>.CreateCollection(collectionType);
                 if (collection == null)
                 {
-                    return false;
+                    throw new Exception("Не удалось определить тип коллекции:" + record[1]);
                 }
 
                 collection.MaxCount = Convert.ToInt32(record[2]);
@@ -178,13 +175,12 @@ public class StorageCollection<T>
                     if (elem?.CreateDrawningShip() is T ship)
                     {
                         if (collection.Insert(ship) == -1)
-                            return false;
+                            throw new Exception("Объект не удалось добавить в коллекцию: " + record[3]);
                     }
                 }
                 _storages.Add(record[0], collection);
             }
         }
-        return true;
     }
 
     /// <summary>

Battleship/Battleship/Exceptions/CollectionOverflowException.cs

@@ -0,0 +1,21 @@
+using System.Runtime.Serialization;
+
+
+namespace Battleship.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) { }
+}

Battleship/Battleship/Exceptions/ObjectNotFoundException.cs

@@ -0,0 +1,21 @@
+using System.Runtime.Serialization;
+
+
+namespace Battleship.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) { }
+}

Battleship/Battleship/Exceptions/PositionOutOfCollectionException.cs

@@ -0,0 +1,21 @@
+using System.Runtime.Serialization;
+
+
+namespace Battleship.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) { }
+}