ProjectBomber/ProjectBomber/CollectionGenericObject/AbstractCompany.cs

@@ -13,7 +13,7 @@ public abstract class AbstractCompany
     /// <summary>
     /// Размер места (ширина)
     /// </summary>
-    protected readonly int _placeSizeWidth = 180;
+    protected readonly int _placeSizeWidth = 220;
     /// <summary>
     /// Размер места (высота)
     /// </summary>
@@ -33,7 +33,8 @@ public abstract class AbstractCompany
     /// <summary>
     /// Вычисление максимального количества элементов, который можно разместить в окне
     /// </summary>
-    private int GetMaxCount => _pictureWidth * _pictureHeight / (_placeSizeWidth * _placeSizeHeight);
+    
+    private int GetMaxCount => (_pictureWidth * _pictureHeight / (_placeSizeWidth * _placeSizeHeight));
     /// <summary>
     /// Конструктор
     /// </summary>

ProjectBomber/ProjectBomber/CollectionGenericObject/ICollectionGenericObject.cs

@@ -10,27 +10,37 @@ public interface ICollectionGenericObject<T>
 where T : class
 {
     /// <summary>
-    /// Количество объектов в коллекции
+    /// колличество объектов в коллекции
     /// </summary>
     int Count { get; }
+
     /// <summary>
 	/// Установка максимального количества элементов
+	/// </summary>
 	int MaxCount { get; set; }
 
+    /// <summary>
+    /// Добавление объекта в коллекцию
+    /// </summary>
+    /// <param name="obj">Добавляемый объект</param>
+    /// <returns>true - вставка прошла удачно, false - вставка не удалась</returns>
     int Insert(T obj);
+
     /// <summary>
     /// Добавление объекта в коллекцию на конкретную позицию
     /// </summary>
     /// <param name="obj">Добавляемый объект</param>
     /// <param name="position">Позиция</param>
     /// <returns>true - вставка прошла удачно, false - вставка не удалась</returns>
-    bool? Insert(T obj, int position);
+    int Insert(T obj, int position);
+
     /// <summary>
     /// Удаление объекта из коллекции с конкретной позиции
     /// </summary>
     /// <param name="position">Позиция</param>
     /// <returns>true - удаление прошло удачно, false - удаление не удалось</returns>
-    T? Remove(int position);
+    T Remove(int position);
+
     /// <summary>
     /// Получение объекта по позиции
     /// </summary>
@@ -42,6 +52,7 @@ where T : class
 	/// Получение типа коллекции
 	/// </summary>
 	CollectionType GetCollectionType { get; }
+
     /// <summary>
     /// Получение объектов коллекции по одному
     /// </summary>

ProjectBomber/ProjectBomber/CollectionGenericObject/ListGenericObjects.cs

@@ -1,4 +1,5 @@
 using ProjectAirBomber.CollectionGenericObject;
+using ProjectAirBomber.Exceptions;
 using System;
 using System.Collections.Generic;
 using System.Linq;
@@ -10,22 +11,23 @@ namespace ProjectAirBomber.CollectionGenericObject;
 public class ListGenericObjects<T> : ICollectionGenericObject<T>
     where T : class
 {
-    private readonly List<T?> _collection;
-    private int _maxCount;
-    public int Count => _collection.Count;
-    public CollectionType GetCollectionType => CollectionType.List;
 
-    public int MaxCount
+    /// <summary>
-    {
+    /// Список объектов, которые храним
-        get
+    /// </summary>
-        {
+    private readonly List<T?> _collection;
-            return _maxCount;
+
-        }
+    /// <summary>
-        set
+    /// Максимально допустимое число объектов в списке
-        {
+    /// </summary>
-            if (value > 0) _maxCount = value;
+    private int _maxCount;
-        }
+
-    }
+    public int Count => _collection.Count;
+
+    public int MaxCount { set { if (value > 0) { _maxCount = value; } } get { return Count; } }
+
+
+    public CollectionType GetCollectionType => CollectionType.List;
 
     /// <summary>
     /// Конструктор
@@ -37,53 +39,46 @@ public class ListGenericObjects<T> : ICollectionGenericObject<T>
 
     public T? Get(int position)
     {
-        if (position >= 0 && position < Count)
+        // TODO проверка позиции
-        {
+        if (position >= Count || position < 0) throw new PositionOutOfCollectionException(position);
         return _collection[position];
     }
-        else
-        {
-            return null;
-        }
-
-    }
 
     public int Insert(T obj)
     {
-        if (Count == _maxCount) { return -1; }
+        // TODO проверка, что не превышено максимальное количество элементов
+        // TODO вставка в конец набора
+        if (Count == _maxCount) throw new CollectionOverflowException(Count);
         _collection.Add(obj);
         return Count;
     }
 
     public int Insert(T obj, int position)
     {
-        if (position < 0 || position >= Count || Count == _maxCount)
+        // TODO проверка, что не превышено максимальное количество элементов
-        {
+        // TODO проверка позиции
-            return -1;
+        // TODO вставка по позиции
-        }
+        if (Count == _maxCount) throw new CollectionOverflowException(Count);
+        if (position >= Count || position < 0) throw new PositionOutOfCollectionException(position);
         _collection.Insert(position, obj);
         return position;
-
     }
 
-    public T? Remove(int position)
+    public T Remove(int position)
     {
-        if (position >= Count || position < 0) return null;
+        // TODO проверка позиции
-        T? obj = _collection[position];
+        // TODO удаление объекта из списка
-        _collection?.RemoveAt(position);
+        if (position >= Count || position < 0) throw new PositionOutOfCollectionException(position);
+        T obj = _collection[position];
+        _collection.RemoveAt(position);
         return obj;
     }
 
     public IEnumerable<T?> GetItems()
     {
-        for (int i = 0; i < _collection.Count; i++)
+        for (int i = 0; i < Count; ++i)
         {
             yield return _collection[i];
         }
     }
-
-    bool? ICollectionGenericObject<T>.Insert(T obj, int position)
-    {
-        throw new NotImplementedException();
-    }
 }

ProjectBomber/ProjectBomber/CollectionGenericObject/MassiveGenericObject.cs

@@ -1,8 +1,4 @@
-using System;
+using ProjectAirBomber.Exceptions;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
 
 namespace ProjectAirBomber.CollectionGenericObject;
 
@@ -10,10 +6,11 @@ public class MassiveGenericObject<T> : ICollectionGenericObject<T>
 where T : class
 {
     /// <summary>
-    /// Массив объектов, которые храним
+    /// массив объектов, которые храним
     /// </summary>
     private T?[] _collection;
     public int Count => _collection.Length;
+
     public int MaxCount
     {
         get
@@ -36,9 +33,7 @@ where T : class
         }
     }
 
-
     public CollectionType GetCollectionType => CollectionType.Massive;
-
     /// <summary>
     /// Конструктор
     /// </summary>
@@ -46,73 +41,87 @@ where T : class
     {
         _collection = Array.Empty<T?>();
     }
-    public T? Get(int position) // получение с позиции
+
+    public T? Get(int position)
     {
-        if (position < 0 || position >= _collection.Length) // если позиция передано неправильно
+        // TODO проверка позиции
-            return null;
+        if (position >= _collection.Length || position < 0)
+        { return null; }
         return _collection[position];
     }
-    public int Insert(T obj) // вставка объекта на свободное место
+
+    public int Insert(T obj)
     {
-        for (int i = 0; i < _collection.Length; ++i)
+        // TODO вставка в свободное место набора
+        int index = 0;
+        while (index < _collection.Length)
         {
-            if (_collection[i] == null)
+            if (_collection[index] == null)
             {
-                _collection[i] = obj;
+                _collection[index] = obj;
-                return i;
+                return index;
             }
+
+            index++;
         }
-        return -1;
+        throw new CollectionOverflowException(Count);
     }
-    public int Insert(T obj, int position) // вставка объекта на место
+
+    public int Insert(T obj, int position)
     {
-        if (position < 0 || position >= _collection.Length) // если позиция переданна неправильно
+        // TODO проверка позиции
-            return -1;
+        // TODO проверка, что элемент массива по этой позиции пустой, если нет, то
-        if (_collection[position] == null)//если позиция пуста
+        //		ищется свободное место после этой позиции и идет вставка туда
+        //		если нет после, ищем до
+        // TODO вставка
+        if (position >= _collection.Length || position < 0)
+        { throw new PositionOutOfCollectionException(position); }
+
+        if (_collection[position] == null)
         {
             _collection[position] = obj;
             return position;
         }
-        else
+        int index;
+
+        for (index = position + 1; index < _collection.Length; ++index)
         {
-            for (int i = position; i < _collection.Length; ++i) //ищем свободное место справа
+            if (_collection[index] == null)
             {
-                if (_collection[i] == null)
+                _collection[position] = obj;
-                {
+                return position;
-                    _collection[i] = obj;
-                    return i;
             }
         }
-            for (int i = 0; i < position; ++i) // иначе слева
+
+        for (index = position - 1; index >= 0; --index)
         {
-                if (_collection[i] == null)
+            if (_collection[index] == null)
             {
-                    _collection[i] = obj;
+                _collection[position] = obj;
-                    return i;
+                return position;
             }
         }
+        throw new CollectionOverflowException(Count);
     }
-        return -1;
+
-    }
+    public T Remove(int position)
-    public T? Remove(int position) // удаление объекта, зануляя его
     {
-        if (position < 0 || position >= _collection.Length || _collection[position] == null)
+        // TODO проверка позиции
-            return null;
+        // TODO удаление объекта из массива, присвоив элементу массива значение null
-        T? temp = _collection[position];
+        if (position >= _collection.Length || position < 0)
+        { throw new PositionOutOfCollectionException(position); }
+        if (_collection[position] == null)
+        { throw new ObjectNotFoundException(position); }
+        T obj = _collection[position];
         _collection[position] = null;
-        return temp;
+        return obj;
     }
 
     public IEnumerable<T?> GetItems()
     {
-        for (int i = 0; i < _collection.Length; i++)
+        for (int i = 0; i < _collection.Length; ++i)
         {
             yield return _collection[i];
         }
     }
-
-    bool? ICollectionGenericObject<T>.Insert(T obj, int position)
-    {
-        throw new NotImplementedException();
-    }
 }

ProjectBomber/ProjectBomber/CollectionGenericObject/PlaneHangar.cs

@@ -4,6 +4,7 @@ using System.Linq;
 using System.Text;
 using System.Threading.Tasks;
 using ProjectAirBomber.Drawnings;
+using ProjectAirBomber.Exceptions;
 
 namespace ProjectAirBomber.CollectionGenericObject;
 
@@ -15,45 +16,45 @@ public class PlaneHangar : AbstractCompany
     /// <param name="picWidth"></param>
     /// <param name="picHeight"></param>
     /// <param name="collection"></param>
-    public PlaneHangar(int picWidth, int picHeight, ICollectionGenericObject<DrawningPlane> collection) : base(picWidth, picHeight, collection) { }
+    public PlaneHangar(int picWidth, int picHeight, ICollectionGenericObject<DrawningPlane> collection) : base(picWidth, picHeight, collection)
+    {
+    }
     protected override void DrawBackgound(Graphics g)
     {
+        int width = _pictureWidth / _placeSizeWidth;
+        int height = _pictureHeight / _placeSizeHeight;
         Pen pen = new(Color.Black, 3);
-        int posX = 0;
+        for (int i = 0; i < width; i++)
-        for (int i = 0; i < _pictureWidth / _placeSizeWidth; i++)
         {
-            int posY = 0;
+            for (int j = 0; j < height + 1; ++j)
-            g.DrawLine(pen, posX, posY, posX, posY + _placeSizeHeight * (_pictureHeight / _placeSizeHeight));
-            for (int j = 0; j <= _pictureHeight / _placeSizeHeight; j++)
             {
-                g.DrawLine(pen, posX, posY, posX + _placeSizeWidth - 30, posY);
+                g.DrawLine(pen, i * _placeSizeWidth, j * _placeSizeHeight, i * _placeSizeWidth + _placeSizeWidth - 15, j * _placeSizeHeight);
-                posY += _placeSizeHeight;
+            }
+        }
     }
-            posX += _placeSizeWidth;
 
-        }
-    }
     protected override void SetObjectsPosition()
     {
-        int posX = _pictureWidth / _placeSizeWidth - 1;
+        int count = 0;
-        int posY = 0;
+        for (int y = 5; y + 50 < _pictureHeight; y += 170)
-        for (int i = 0; i < _collection?.Count; i++)
         {
-            if (_collection.Get(i) != null)
+            for (int x = 5; x + 200 < _pictureWidth; x += _placeSizeHeight + 50)
             {
-                _collection?.Get(i)?.SetPictureSize(_pictureWidth, _pictureHeight);
+                //_collection?.Get(count)?.SetPictureSize(_pictureWidth, _pictureHeight);
-                _collection?.Get(i)?.SetPosition(posX * _placeSizeWidth + 5, posY * _placeSizeHeight + 5);
+                //_collection?.Get(count)?.SetPosition(x, y);
+                //count++;
+                if (count < _collection?.Count)
+                {
+                    try
+                    {
+                        _collection?.Get(count)?.SetPictureSize(_pictureWidth, _pictureHeight);
+                        _collection?.Get(count)?.SetPosition(x, y);
                     }
-            if (posX > 0)
+                    catch (ObjectNotFoundException) { }
-            {
-                posX--;
                 }
-            else
+
-            {
+                count++;
-                posY++;
-                posX = _pictureWidth / _placeSizeWidth - 1;
             }
-            if (posX < 0) { return; }
         }
     }
 }

ProjectBomber/ProjectBomber/CollectionGenericObject/StorageCollection.cs

@@ -1,6 +1,7 @@
 using ProjectAirBomber.Drawnings;
 using ProjectAirBomber.CollectionGenericObject;
 using System.Text;
+using ProjectAirBomber.Exceptions;
 
 namespace ProjectAirBomber.CollectionGenericObject;
 
@@ -99,11 +100,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))
         {
@@ -141,8 +142,6 @@ public class StorageCollection<T>
             }
 
         }
-
-        return true;
     }
 
     /// <summary>
@@ -155,32 +154,27 @@ 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))
         {
-            return false;
+            throw new Exception("Файл не существует");
         }
         using (StreamReader fs = File.OpenText(filename))
         {
             string str = fs.ReadLine();
             if (str == null || str.Length == 0)
             {
-                return false;
+                throw new Exception("В файле нет данных");
             }
             if (!str.StartsWith(_collectionKey))
             {
-                return false;
+                throw new Exception("В файле неверные данные");
             }
             _storages.Clear();
             string strs = "";
             while ((strs = fs.ReadLine()) != null)
             {
-                //по идее этого произойти не должно
-                //if (strs == null)
-                //{
-                //    return false;
-                //}
                 string[] record = strs.Split(_separatorForKeyValue, StringSplitOptions.RemoveEmptyEntries);
                 if (record.Length != 4)
                 {
@@ -190,24 +184,29 @@ public class StorageCollection<T>
                 ICollectionGenericObject<T>? collection = StorageCollection<T>.CreateCollection(collectionType);
                 if (collection == null)
                 {
-                    return false;
+                    throw new Exception("Не удалось создать коллекцию");
                 }
                 collection.MaxCount = Convert.ToInt32(record[2]);
                 string[] set = record[3].Split(_separatorItems, StringSplitOptions.RemoveEmptyEntries);
                 foreach (string elem in set)
                 {
-                    if (elem?.CreateDrawningPlane() is T Plane)
+                    if (elem?.CreateDrawningPlane() is T ship)
                     {
-                        if (collection.Insert(Plane) == -1)
+                        try
                         {
-                            return false;
+                            if (collection.Insert(ship) == -1)
+                            {
+                                throw new Exception("Объект не удалось добавить в коллекцию: " + record[3]);
+                            }
+                        }
+                        catch (CollectionOverflowException ex)
+                        {
+                            throw new Exception("Коллекция переполнена", ex);
                         }
                     }
                 }
                 _storages.Add(record[0], collection);
             }
-            return true;
-
         }
     }
 

ProjectBomber/ProjectBomber/Drawnings/DrawningAirBomber.cs

@@ -19,8 +19,8 @@ public class DrawingAirBomber : DrawningPlane
     /// <param name="weight">Вес</param>
     /// <param name="bodyColor">Основной цвет</param>
     /// <param name="additionalColor">Дополнительный цвет</param>
-    /// <param name="bodyGlass">Признак наличия стёкол</param>
+    /// <param name="bomb">Признак наличия стёкол</param>
-    /// /// <param name="garmoshka">Признак наличия гармошки</param>
+    /// /// <param name="engine">Признак наличия гармошки</param>
 
     public DrawingAirBomber(EntityAirBomber plane) : base(200, 40)
     {
@@ -45,9 +45,13 @@ public class DrawingAirBomber : DrawningPlane
     public override void DrawPlane(Graphics g)
     {
         if (EntityPlane == null || EntityPlane is not EntityAirBomber airBomber || !_startPosX.HasValue || !_startPosY.HasValue)
+        {
             return;
+        }
+
         base.DrawPlane(g);
 
+        Brush AdditionalBrush = new SolidBrush(airBomber.AdditionalColor);
         Pen pen = new(Color.Black);
 
         //наличие топливных баков
@@ -56,8 +60,8 @@ public class DrawingAirBomber : DrawningPlane
             Brush brBrown = new SolidBrush(Color.Brown);
             g.DrawRectangle(pen, _startPosX.Value + 90, _startPosY.Value + 60, 20, 5);
             g.DrawRectangle(pen, _startPosX.Value + 90, _startPosY.Value + 95, 20, 5);
-            g.FillRectangle(brBrown, _startPosX.Value + 90, _startPosY.Value + 60, 20, 5);
+            g.FillRectangle(AdditionalBrush, _startPosX.Value + 90, _startPosY.Value + 60, 20, 5);
-            g.FillRectangle(brBrown, _startPosX.Value + 90, _startPosY.Value + 95, 20, 5);
+            g.FillRectangle(AdditionalBrush, _startPosX.Value + 90, _startPosY.Value + 95, 20, 5);
         }
 
         //наличие дополнительных бомб
@@ -66,8 +70,8 @@ public class DrawingAirBomber : DrawningPlane
             Brush brGreen = new SolidBrush(Color.Green);
             g.DrawRectangle(pen, _startPosX.Value + 55, _startPosY.Value + 35, 20, 8);
             g.DrawRectangle(pen, _startPosX.Value + 55, _startPosY.Value + 115, 20, 8);
-            g.FillRectangle(brGreen, _startPosX.Value + 55, _startPosY.Value + 35, 20, 8);
+            g.FillRectangle(AdditionalBrush, _startPosX.Value + 55, _startPosY.Value + 35, 20, 8);
-            g.FillRectangle(brGreen, _startPosX.Value + 55, _startPosY.Value + 115, 20, 8);
+            g.FillRectangle(AdditionalBrush, _startPosX.Value + 55, _startPosY.Value + 115, 20, 8);
         }
     }
 }

ProjectBomber/ProjectBomber/Drawnings/DrawningPlane.cs

@@ -70,6 +70,8 @@ public class DrawningPlane
         _startPosX = null;
         _startPosY = null;
     }
+
+
     /// <summary>
     /// Конструктор
     /// </summary>
@@ -80,6 +82,7 @@ public class DrawningPlane
     {
         EntityPlane = new EntityPlane(speed, weight, bodyColor);
     }
+
     /// <summary>
     /// Конструктор для наследования
     /// </summary>
@@ -91,6 +94,7 @@ public class DrawningPlane
         _drawningPlaneHeight = drawningPlaneHeight;
     }
 
+
     /// <summary>
     /// Установка границ поля
     /// </summary>
@@ -99,26 +103,23 @@ public class DrawningPlane
     /// <returns>true - границы заданы, false - проверка не пройдена, нельзя 
     public bool SetPictureSize(int width, int height)
     {
-        if (_drawningPlaneWidth <= width && _drawningPlaneHeight <= height)
+        // TODO проверка, что объект "влезает" в размеры поля
+        // если влезает, сохраняем границы и корректируем позицию объекта, если она была уже установлена
+        if (_drawningPlaneWidth <= width || _drawningPlaneHeight <= height)
         {
             _pictureWidth = width;
             _pictureHeight = height;
-            if (_startPosX.HasValue && _startPosY.HasValue)
+            if (_startPosX != null && _startPosY != null)
-            {
                 if (_startPosX + _drawningPlaneWidth > _pictureWidth)
-                {
                     _startPosX = _pictureWidth - _drawningPlaneWidth;
-                }
-
             if (_startPosY + _drawningPlaneHeight > _pictureHeight)
-                {
                 _startPosY = _pictureHeight - _drawningPlaneHeight;
-                }
-            }
             return true;
         }
+        else
             return false;
     }
+
     /// <summary>
     /// Установка позиции
     /// </summary>
@@ -130,14 +131,21 @@ public class DrawningPlane
         {
             return;
         }
-        if (x < 0) x = 0;
+        // TODO если при установке объекта в эти координаты, он будет "выходить" за границы формы
-        else if (x + _drawningPlaneWidth > _pictureWidth) x = _pictureWidth.Value - _drawningPlaneWidth;
+        // то надо изменить координаты, чтобы он оставался в этих границах
+        if (x < 0)
+            x = 0;
+        else if (x + _drawningPlaneWidth > _pictureWidth)
+            x = _pictureWidth.Value - _drawningPlaneWidth;
+        if (y < 0)
+            x = 0;
+        else if (y + _drawningPlaneHeight > _pictureHeight)
+            y = _pictureHeight.Value - _drawningPlaneHeight;
 
-        if (y < 0) y = 0;
-        else if (y + _drawningPlaneHeight > _pictureHeight) y = _pictureHeight.Value - _drawningPlaneHeight;
         _startPosX = x;
         _startPosY = y;
     }
+
     public bool MoveTransport(DirectionType direction)
     {
         if (EntityPlane == null || !_startPosX.HasValue || !_startPosY.HasValue)

ProjectBomber/ProjectBomber/Exceptions/CollectionOverflowException.cs

@@ -0,0 +1,18 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Runtime.Serialization;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace ProjectAirBomber.Exceptions;
+
+[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) { }
+}

ProjectBomber/ProjectBomber/Exceptions/ObjectNotFoundException.cs

@@ -0,0 +1,21 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Runtime.Serialization;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace ProjectAirBomber.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) { }
+}

ProjectBomber/ProjectBomber/Exceptions/PositionOutOfCollectionException.cs

@@ -0,0 +1,21 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Runtime.Serialization;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace ProjectAirBomber.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) { }
+}

ProjectBomber/ProjectBomber/FormPlaneCollection.cs

@@ -11,40 +11,24 @@ using System.Linq;
 using System.Text;
 using System.Threading.Tasks;
 using System.Windows.Forms;
+using Microsoft.Extensions.Logging;
+using ProjectAirBomber.Exceptions;
 
 namespace ProjectAirBomber;
-    public partial class FormPlaneCollection : Form
+public partial class FormPlaneCollection : Form
-    {
+{
     private readonly StorageCollection<DrawningPlane> _storageCollection;
 
-        /// <summary>
-        /// Стратегия перемещения
-        /// </summary>
     private AbstractCompany? _company = null;
 
-        public FormPlaneCollection()
+    private readonly ILogger _logger;
+
+    public FormPlaneCollection(ILogger<FormPlaneCollection> logger)
     {
         InitializeComponent();
         _storageCollection = new();
-        }
+        _logger = logger;
-
+        _logger.LogInformation("Форма загрузилась");
-        private void SetPlane(DrawningPlane plane)
-        {
-            {
-                if (_company == null || plane == null)
-                {
-                    return;
-                }
-                if (_company + plane != -1)
-                {
-                    MessageBox.Show("Объект добавлен");
-                    pictureBox.Image = _company.Show();
-                }
-                else
-                {
-                    MessageBox.Show("Не удалось добавить объект");
-                }
-            }
     }
 
     /// <summary>
@@ -52,7 +36,7 @@ namespace ProjectAirBomber;
     /// </summary>
     /// <param name="sender"></param>
     /// <param name="e"></param>
-        private void comboBoxSelectionCompany_SelectedIndexChanged(object sender, EventArgs e)
+    private void ComboBoxSelectionCompany_SelectedIndexChanged(object sender, EventArgs e)
     {
         panelCompanyTools.Enabled = false;
     }
@@ -65,11 +49,37 @@ namespace ProjectAirBomber;
     private void ButtonAddPlane_Click(object sender, EventArgs e)
     {
         FormCarConfig form = new();
-            // TODO передать метод
-
         form.Show();
         form.AddEvent(SetPlane);
     }
+
+    /// <summary>
+    /// 
+    /// </summary>
+    /// <param name="plane"></param>
+    private void SetPlane(DrawningPlane? plane)
+    {
+        try
+        {
+            if (_company == null || plane == null)
+            {
+                return;
+            }
+            if (_company + plane != -1)
+            {
+                MessageBox.Show("Объект добавлен");
+                pictureBox.Image = _company.Show();
+                _logger.LogInformation("Добавлен объект: " + plane.GetDataForSave());
+            }
+        }
+        catch (ObjectNotFoundException) { }
+        catch (CollectionOverflowException ex)
+        {
+            MessageBox.Show("Не удалось добавить объект");
+            _logger.LogError("Ошибка: {Message}", ex.Message);
+        }
+    }
+
     /// <summary>
     /// кнопка удаления
     /// </summary>
@@ -81,30 +91,43 @@ namespace ProjectAirBomber;
         {
             return;
         }
-            if (MessageBox.Show("Удалить объект?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) != DialogResult.Yes)
+        if (MessageBox.Show("Удалить объект?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No)
         {
             return;
         }
         int pos = Convert.ToInt32(maskedTextBox.Text);
+        try
+        {
             if (_company - pos != null)
             {
                 MessageBox.Show("Объект удален");
                 pictureBox.Image = _company.Show();
+                _logger.LogInformation("Удален объект по позиции " + pos);
             }
-            else
+        }
+        catch (Exception ex)
         {
             MessageBox.Show("Не удалось удалить объект");
+            _logger.LogError("Ошибка: {Message}", ex.Message);
         }
     }
 
+    /// <summary>
+    /// 
+    /// </summary>
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
     private void ButtonGoToCheck_Click(object sender, EventArgs e)
     {
+
         if (_company == null)
         {
             return;
         }
         DrawningPlane? plane = null;
         int counter = 100;
+        try
+        {
             while (plane == null)
             {
                 plane = _company.GetRandomObject();
@@ -114,16 +137,23 @@ namespace ProjectAirBomber;
                     break;
                 }
             }
-            if (plane == null)
-            {
-                return;
-            }
             ProjectAirBomber form = new()
             {
                 SetPlane = plane
             };
             form.ShowDialog();
         }
+        catch (Exception ex)
+        {
+            MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+        }
+    }
+
+    /// <summary>
+    /// 
+    /// </summary>
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
     private void ButtonRefresh_Click(object sender, EventArgs e)
     {
         if (_company == null)
@@ -133,6 +163,9 @@ namespace ProjectAirBomber;
         pictureBox.Image = _company.Show();
     }
 
+    /// <summary>
+    /// 
+    /// </summary>
     private void RefreshListBoxItems()
     {
         listBoxCollection.Items.Clear();
@@ -144,16 +177,23 @@ namespace ProjectAirBomber;
                 listBoxCollection.Items.Add(colName);
             }
         }
-
     }
 
+    /// <summary>
+    /// Добавление компании
+    /// </summary>
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
     private void buttonCollectionAdd_Click(object sender, EventArgs e)
     {
         if (string.IsNullOrEmpty(textBoxCollectionName.Text) || (!radioButtonList.Checked && !radioButtonMassive.Checked))
         {
-                MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error);
+            MessageBox.Show("Не все данные заполнены", "Ошибка",
+            MessageBoxButtons.OK, MessageBoxIcon.Error);
             return;
         }
+        try
+        {
             CollectionType collectionType = CollectionType.None;
             if (radioButtonMassive.Checked)
             {
@@ -163,24 +203,49 @@ namespace ProjectAirBomber;
             {
                 collectionType = CollectionType.List;
             }
-
             _storageCollection.AddCollection(textBoxCollectionName.Text, collectionType);
             RefreshListBoxItems();
+            _logger.LogInformation("Коллекция добавлена " + textBoxCollectionName.Text);
         }
+        catch (Exception ex)
+        {
+            _logger.LogError("Ошибка: {Message}", ex.Message);
+        }
+    }
+
+    /// <summary>
+    /// Удаление компании
+    /// </summary>
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
     private void buttonCollectionDel_Click(object sender, EventArgs e)
     {
-            if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItems == null)
+        if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItem == null)
         {
             MessageBox.Show("Коллекция не выбрана");
             return;
         }
+        try
+        {
             if (MessageBox.Show("Удалить коллекцию?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No)
             {
                 return;
             }
             _storageCollection.DelCollection(listBoxCollection.SelectedItem.ToString());
             RefreshListBoxItems();
+            _logger.LogInformation("Коллекция: " + listBoxCollection.SelectedItem.ToString() + " удалена");
         }
+        catch (Exception ex)
+        {
+            _logger.LogError("Ошибка: {Message}", ex.Message);
+        }
+    }
+
+    /// <summary>
+    /// Создание компании
+    /// </summary>
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
     private void buttonCreateCompany_Click(object sender, EventArgs e)
     {
         if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItem == null)
@@ -188,26 +253,26 @@ namespace ProjectAirBomber;
             MessageBox.Show("Коллекция не выбрана");
             return;
         }
-
+        ICollectionGenericObject<DrawningPlane>? collection =
-            ICollectionGenericObject<DrawningPlane>? collection = _storageCollection[listBoxCollection.SelectedItem.ToString() ?? string.Empty];
+            _storageCollection[listBoxCollection.SelectedItem.ToString() ?? string.Empty];
         if (collection == null)
         {
             MessageBox.Show("Коллекция не проинициализирована");
             return;
         }
-
         switch (comboBoxSelectionCompany.Text)
         {
             case "Ангар":
                 _company = new PlaneHangar(pictureBox.Width, pictureBox.Height, collection);
                 break;
         }
-
         panelCompanyTools.Enabled = true;
         RefreshListBoxItems();
     }
+
+
     /// <summary>
-        /// 
+    /// Обработка кнопки загрузки
     /// </summary>
     /// <param name="sender"></param>
     /// <param name="e"></param>
@@ -215,19 +280,24 @@ namespace ProjectAirBomber;
     {
         if (openFileDialog.ShowDialog() == DialogResult.OK)
         {
-                if (_storageCollection.LoadData(openFileDialog.FileName))
+            try
             {
+                _storageCollection.LoadData(openFileDialog.FileName);
                 MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
                 RefreshListBoxItems();
+                _logger.LogInformation("Загрузка из файла: {filename}", openFileDialog.FileName);
             }
-                else
+            catch (Exception ex)
             {
                 MessageBox.Show("Не загрузилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                _logger.LogError("Ошибка: {Message}", ex.Message);
             }
         }
     }
+
+
     /// <summary>
-        /// 
+    /// Сохранение
     /// </summary>
     /// <param name="sender"></param>
     /// <param name="e"></param>
@@ -235,15 +305,18 @@ namespace ProjectAirBomber;
     {
         if (saveFileDialog.ShowDialog() == DialogResult.OK)
         {
-                if (_storageCollection.SaveData(saveFileDialog.FileName))
+            try
             {
+                _storageCollection.SaveData(saveFileDialog.FileName);
                 MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
+                _logger.LogInformation($"Сохранение наборов в файл {saveFileDialog.FileName}");
             }
-                else
+            catch (Exception ex)
             {
-                    MessageBox.Show("Не сохранилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                MessageBox.Show($"Не сохранилось: {ex.Message}", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
-                }
+                _logger.LogWarning($"Не удалось сохранить наборы с ошибкой: {ex.Message}");
             }
         }
     }
+}
 

ProjectBomber/ProjectBomber/Program.cs

@@ -1,7 +1,13 @@
+using Microsoft.Extensions.Configuration;
+using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.Logging;
+using Serilog;
+
 namespace ProjectAirBomber
 {
     internal static class Program
     {
+
         /// <summary>
         ///  The main entry point for the application.
         /// </summary>
@@ -11,7 +17,27 @@ namespace ProjectAirBomber
             // To customize application configuration such as set high DPI settings or default font,
             // see https://aka.ms/applicationconfiguration.
             ApplicationConfiguration.Initialize();
-            Application.Run(new FormPlaneCollection());
+
+            ServiceCollection services = new();
+            ConfigureServices(services);
+            using ServiceProvider serviceProvider = services.BuildServiceProvider();
+            Application.Run(serviceProvider.GetRequiredService<FormPlaneCollection>());
+
+        }
+        private static void ConfigureServices(ServiceCollection services)
+        {
+            string[] path = Directory.GetCurrentDirectory().Split('\\');
+            string pathNeed = "";
+            for (int i = 0; i < path.Length - 3; i++)
+            {
+                pathNeed += path[i] + "\\";
+            }
+            services.AddSingleton<FormPlaneCollection>()
+            .AddLogging(option =>
+            {
+                option.SetMinimumLevel(LogLevel.Information);
+                option.AddSerilog(new LoggerConfiguration().ReadFrom.Configuration(new ConfigurationBuilder().AddJsonFile($"{pathNeed}serilog.json").Build()).CreateLogger());
+            });
         }
     }
 }

ProjectBomber/ProjectBomber/ProjectAirBomber.csproj

@@ -8,6 +8,19 @@
     <ImplicitUsings>enable</ImplicitUsings>
   </PropertyGroup>
 
+  <ItemGroup>
+    <PackageReference Include="Microsoft.Extensions.Configuration" Version="8.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="8.0.0" />
+    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="8.0.0" />
+    <PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="8.0.1" />
+    <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.1" />
+    <PackageReference Include="NLog.Extensions.Logging" Version="5.3.11" />
+    <PackageReference Include="Serilog" Version="3.1.1" />
+    <PackageReference Include="Serilog.Extensions.Logging" Version="8.0.0" />
+    <PackageReference Include="Serilog.Settings.Configuration" Version="8.0.0" />
+    <PackageReference Include="Serilog.Sinks.File" Version="5.0.0" />
+  </ItemGroup>
+
   <ItemGroup>
     <Compile Update="Properties\Resources.Designer.cs">
       <DesignTime>True</DesignTime>

ProjectBomber/ProjectBomber/serilog.json

@@ -0,0 +1,15 @@
+{
+  "Serilog": {
+    "Using": [ "Serilog.Sinks.File" ],
+    "MinimumLevel": "Debug",
+    "WriteTo": [
+      {
+        "Name": "File",
+        "Args": { "path": "log.log" }
+      }
+    ],
+    "Properties": {
+      "Application": "Sample"
+    }
+  }
+}