Лабораторная работа №8.2

ProectMilitaryAircraft/ProectMilitaryAircraft/CollectionGenericObjects/AbstractCompany.cs

@@ -59,7 +59,7 @@ public abstract class AbstractCompany
     /// <returns></returns>
     public static bool operator +(AbstractCompany company, DrawningAircraft aircraft)
     {
-        return company._collection?.Insert(aircraft) ?? false;
+        return company._collection?.Insert(aircraft, new DrawningAircraftEqutables()) ?? false;
     }
 
     /// <summary>
@@ -102,6 +102,14 @@ public abstract class AbstractCompany
         return bitmap;
     }
 
+    /// <summary>
+    /// Сортировка
+    /// </summary>
+    /// <param name="comparer">Сравнитель объектов</param>
+    public void Sort(IComparer<DrawningAircraft?> comparer) => _collection?.CollectionSort(comparer);
+
     protected abstract void DrawBackGround(Graphics g);
     protected abstract void SetObjectPosition(Graphics g);
+
+
 }

ProectMilitaryAircraft/ProectMilitaryAircraft/CollectionGenericObjects/Collectioninfo.cs

@@ -0,0 +1,84 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace ProectMilitaryAircraft.CollectionGenericObjects;
+
+
+/// <summary>
+/// Класс, хранящий информацию коллекции
+/// </summary>
+public class Collectioninfo : IEquatable<Collectioninfo>
+{
+    /// <summary>
+	/// Название
+	/// </summary>
+	public string Name { get; private set; }
+
+    /// <summary>
+    /// Тип
+    /// </summary>
+    public CollectionType CollectionType { get; private set; }
+
+    /// <summary>
+    /// Описание
+    /// </summary>
+    public string Description { get; private set; }
+
+    /// <summary>
+    /// Разделитель для записи информации по объекту в файл
+    /// </summary>
+    private static readonly string _separator = "-";
+
+    /// <summary>
+    /// Конструктор
+    /// </summary>
+    /// <param name="name">Название</param>
+    /// <param name="collectionType">Тип</param>
+    /// <param name="description">Описание</param>
+    public Collectioninfo(string name, CollectionType collectionType, string description)
+    {
+        Name = name;
+        CollectionType = collectionType;
+        Description = description;
+    }
+
+    /// <summary>
+    /// Создание объекта из строки
+    /// </summary>
+    /// <param name="data">Строка</param>
+    /// <returns>Объект или null</returns>
+    public static Collectioninfo? GetCollectionInfo(string data)
+    {
+        string[] strs = data.Split(_separator, StringSplitOptions.RemoveEmptyEntries);
+        if (strs.Length < 1 || strs.Length > 3)
+        {
+            return null;
+        }
+
+        return new Collectioninfo(strs[0], (CollectionType)Enum.Parse(typeof(CollectionType), strs[1]), strs.Length > 2 ? strs[2] : string.Empty);
+    }
+
+    public override string ToString()
+    {
+        return Name + _separator + CollectionType + _separator + Description;
+    }
+
+    public bool Equals(Collectioninfo? other)
+    {
+        return Name == other?.Name;
+    }
+
+    public override bool Equals(object? obj)
+    {
+        return Equals(obj as Collectioninfo);
+    }
+
+    public override int GetHashCode()
+    {
+        return Name.GetHashCode();
+    }
+
+}

ProectMilitaryAircraft/ProectMilitaryAircraft/CollectionGenericObjects/ICollectionGenericObjects.cs

@@ -30,7 +30,7 @@ public interface ICollectionGenericObjects<T>
     /// </summary>
     /// <param name="obj">Добавляемый объект</param>
     /// <returns>true - вставка  прошла успешно, false - вставка прошла не успешно</returns>
-    bool Insert(T obj);
+    bool Insert(T obj, IEqualityComparer<T?>? comparer = null);
 
     /// <summary>
     /// Добавление объекта в коллекцию на конкретную позицию
@@ -38,7 +38,7 @@ public interface ICollectionGenericObjects<T>
     /// <param name="obj">Добавляемый объект</param>
     /// <param name="position">Позиция</param>
     /// <returns>true - вставка  прошла успешно, false - вставка прошла не успешно</returns>
-    bool Insert(T obj, int position);
+    bool Insert(T obj, int position, IEqualityComparer<T?>? comparer = null);
 
     /// <summary>
     /// Удаление объекта из коллекции с конкретной позиции
@@ -64,4 +64,10 @@ public interface ICollectionGenericObjects<T>
     /// </summary>
     /// <returns>Поэлементый вывод элементов коллекции</returns>
     IEnumerable<T?> GetItems();
+
+    /// <summary>
+    /// Сортировка коллекции
+    /// </summary>
+    /// <param name="comparer">Сравнитель объектов</param>
+    void CollectionSort(IComparer<T?> comparer);
 }

ProectMilitaryAircraft/ProectMilitaryAircraft/CollectionGenericObjects/ListgenericObjects.cs

@@ -1,9 +1,13 @@
-using System;
+using ProectMilitaryAircraft.Draw;
+using ProectMilitaryAircraft.Exceptions;
+using System;
+using System.Collections;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;
 using System.Threading.Tasks;
 
+
 namespace ProectMilitaryAircraft.CollectionGenericObjects;
 
 
@@ -12,12 +16,12 @@ namespace ProectMilitaryAircraft.CollectionGenericObjects;
 /// </summary>
 /// <typeparam name="T">Параметр : ограничение - ссылочный тип</typeparam>
 public class ListGenericObjects<T> : ICollectionGenericObjects<T>
-    where T : class
+    where T : DrawningAircraft
 {
     /// <summary>
-	/// Список объектов, которые храним
-	/// </summary>
-	private readonly List<T?> _collection;
+    /// Список объектов, которые храним
+    /// </summary>
+    private readonly List<T?> _collection;
 
     /// <summary>
     /// Максимально допустимое число объектов в списке
@@ -52,29 +56,52 @@ public class ListGenericObjects<T> : ICollectionGenericObjects<T>
     public T? Get(int position)
     {
 
-        if (position < 0 || position >= Count) return null;
+        if (position < 0 || position >= Count)
+        {
+            return null;
+        }
+        if (position > _maxCount)
+        {
+            new CollectionOverflowException();
+            return null;
+        }
         return _collection[position];
 
     }
 
-    public bool Insert(T obj)
+    public bool Insert(T obj, IEqualityComparer<T?>? comparer = null)
     {
-        if (Count != _maxCount)
+        if (Count == _maxCount)
         {
-            _collection.Add(obj);
-            return true;
+            throw new CollectionOverflowException(Count);
         }
-        return false;
+
+        if (_collection.Contains(obj, comparer))
+        {
+            MessageBox.Show("Такой объект уже существует");
+            return false;
+        }
+
+        _collection.Add(obj);
+        return true;
     }
 
-    public bool Insert(T obj, int position)
+    public bool Insert(T obj, int position, IEqualityComparer<T?>? comparer = null)
     {
-        if (position > 0 && position <= _maxCount && Count != _maxCount)
+        if (Count == _maxCount)
         {
-            _collection.Insert(position, obj);
-            return true;
+            throw new CollectionOverflowException(Count);
         }
-        return false;
+        if (position < 0 || position >= _maxCount)
+        {
+            throw new PositionOutOfCollectionException(position);
+        }
+        if (_collection.Contains(obj, comparer))
+        {
+            throw new Exception("Такой объект уже существует в коллекции");
+        }
+        _collection.Insert(position, obj);
+        return true;
     }
 
     public bool Remove(int position)
@@ -84,6 +111,10 @@ public class ListGenericObjects<T> : ICollectionGenericObjects<T>
             _collection.RemoveAt(position);
             return true;
         }
+        if (!(position >= 0 && position < Count))
+        {
+            throw new ObjectNotFoundException(position);
+        }
         return false;
     }
 
@@ -94,4 +125,9 @@ public class ListGenericObjects<T> : ICollectionGenericObjects<T>
             yield return _collection[i];
         }
     }
+
+    public void CollectionSort(IComparer<T?> comparer)
+    {
+        _collection.Sort(comparer);
+    }
 }

ProectMilitaryAircraft/ProectMilitaryAircraft/CollectionGenericObjects/MassiveGenericObjects.cs

@@ -1,6 +1,9 @@
 using ProectMilitaryAircraft.Draw;
+using ProectMilitaryAircraft.Exceptions;
 using System;
+using System.Collections;
 using System.Collections.Generic;
+using System.Collections.Immutable;
 using System.Linq;
 using System.Text;
 using System.Threading.Tasks;
@@ -12,31 +15,31 @@ namespace ProectMilitaryAircraft.CollectionGenericObjects
     /// </summary>
     /// <typeparam name="T">Параметр : Ограничение - ссылочный тип</typeparam>
     public class MassiveGenericObjects<T> : ICollectionGenericObjects<T>
-        where T : DrawningAircraft
+        where T : class
     {
         /// <summary>
         /// Массив объектов, которые храним
         /// </summary>
-        private T?[] _collection;
+        private T?[] _collections;
 
-        public int Count => _collection.Length;
+        public int Count => _collections.Length;
 
         public int MaxCount {
             get
             {
-                return _collection.Length;
+                return _collections.Length;
             }
             set
             {
                 if (value > 0)
                 {
-                    if (_collection.Length > 0)
+                    if (_collections.Length > 0)
                     {
-                        Array.Resize(ref _collection, value);
+                        Array.Resize(ref _collections, value);
                     }
                     else
                     {
-                        _collection = new T?[value];
+                        _collections = new T?[value];
                     }
                 }
             }
@@ -49,48 +52,70 @@ namespace ProectMilitaryAircraft.CollectionGenericObjects
         /// </summary>
         public MassiveGenericObjects()
         {
-            _collection = Array.Empty<T?>();
+            _collections = Array.Empty<T?>();
         }
 
         public T? Get(int position)
         {
-            if (_collection[position] != null)
+            if (position < 0 || position >= _collections.Length)
             {
-                return _collection[position];
-            }
-            else
-            {
-                return null;
+                throw new PositionOutOfCollectionException(position);
             }
+            return _collections[position];
+
         }
 
-        public bool Insert(T obj)
+        public bool Insert(T obj, IEqualityComparer<T?>? comparer = null)
         {
-            for (int i = 0; i < _collection.Length; i++)
+            if (_collections.Contains(obj, comparer))
             {
-                if (_collection[i] == null)
+                MessageBox.Show("Объект уже существует");
+                return false;
+            }
+            for (int i = 0; i < _collections.Length; ++i)
+            {
+                if (_collections[i] == null)
                 {
-                    _collection[i] = obj;
+                    _collections[i] = obj;
                     return true;
                 }
             }
             return false;
         }
 
-        public bool Insert(T obj, int position)
+        public bool Insert(T obj, int position, IEqualityComparer<T?>? comparer = null)
         {
-            if (_collection[position] == null)
+            if (_collections[position] == null)
             {
-                _collection[position] = obj;
+                _collections[position] = obj;
                 return true;
             }
-            if (_collection[position] != null)
+            if (position > Count)
             {
-                for (int i = position; i < _collection.Length; i++)
+                throw new CollectionOverflowException(Count);
+            }
+
+            if (!(position >= 0 && position <= Count))
+            {
+                throw new Exception("Неверная позиция для вставки");
+            }
+            if (comparer != null)
+            {
+                if (_collections.Contains(obj, comparer))
                 {
-                    if (_collection[i] == null)
+                    MessageBox.Show("Объект уже существует");
+                    return false;
+                }
+            }
+            if (_collections[position] != null)
+            {
+                
+                for (int i = position; i < _collections.Length; i++)
+                {
+                    
+                    if (_collections[i] == null)
                     {
-                        _collection[i] = obj;
+                        _collections[i] = obj;
                         return true;
                     }
                     break;
@@ -98,9 +123,9 @@ namespace ProectMilitaryAircraft.CollectionGenericObjects
 
                 for (int i = position; i <= 0; i--)
                 {
-                    if (_collection[i] == null)
+                    if (_collections[i] == null)
                     {
-                        _collection[i] = obj;
+                        _collections[i] = obj;
                         return true;
                     }
                     break;
@@ -111,19 +136,34 @@ namespace ProectMilitaryAircraft.CollectionGenericObjects
 
         public bool Remove(int position)
         {
-            if (_collection[position] != null)
+            if (_collections[position] != null)
             {
-                _collection[position] = null;
+                _collections[position] = null;
                 return true;
             }
+
+            if (position >= 0 && position < Count)
+            {
+                throw new ObjectNotFoundException(position);
+            }
             return false;
         }
 
         public IEnumerable<T?> GetItems()
         {
-            for (int i = 0; i < _collection.Length; ++i)
+            for (int i = 0; i < _collections.Length; ++i)
             {
-                yield return _collection[i];
+                yield return _collections[i];
+            }
+        }
+
+        public void CollectionSort(IComparer<T?> comparer)
+        {
+            ;
+            if (_collections != null && _collections.Length > 0)
+            {
+                Array.Sort(_collections, comparer);
+                _collections = _collections.OrderBy(element => element == null).ToArray();
             }
         }
     }

ProectMilitaryAircraft/ProectMilitaryAircraft/CollectionGenericObjects/StorageCollection.cs

@@ -1,4 +1,5 @@
 using ProectMilitaryAircraft.Draw;
+using ProectMilitaryAircraft.Exceptions;
 using ProectMilitaryAircraft.MovementStrategy;
 using System;
 using System.Collections.Generic;
@@ -11,13 +12,13 @@ namespace ProectMilitaryAircraft.CollectionGenericObjects;
 public class StorageCollection<T>
     where T : DrawningAircraft
 {
-    readonly Dictionary<string, ICollectionGenericObjects<T>> _storages;
+    readonly Dictionary<Collectioninfo, ICollectionGenericObjects<T>> _storages;
 
-    public List<string> Keys => _storages.Keys.ToList();
+    public List<Collectioninfo> Keys => _storages.Keys.ToList();
 
     public StorageCollection()
     {
-        _storages = new Dictionary<string, ICollectionGenericObjects<T>>();
+        _storages = new Dictionary<Collectioninfo, ICollectionGenericObjects<T>>();
     }
 
     /// <summary>
@@ -37,38 +38,40 @@ public class StorageCollection<T>
 
     public void AddCollection (string name, CollectionType collectionType)
     {
-        if (name != null && !_storages.ContainsKey(name))
+        if (name != null && !_storages.ContainsKey(new Collectioninfo(name, collectionType, string.Empty)))
         {
 
             if (collectionType == CollectionType.Massive)
             {
-                _storages.Add(name, new MassiveGenericObjects<T>());
+                _storages.Add(new Collectioninfo(name, collectionType,
+                    string.Empty), new MassiveGenericObjects<T>());
             }
             if (collectionType == CollectionType.List)
             {
-                _storages.Add(name, new ListGenericObjects<T>());
+                _storages.Add(new Collectioninfo(name, collectionType,
+                    string.Empty), new ListGenericObjects<T>());
             }
 
         }
     }   
 
-    public void DelCollection (string name) 
+    public void DelCollection (string name, CollectionType collectionType) 
     {
-        if (!_storages.ContainsKey(name))
+        if (!_storages.ContainsKey(new Collectioninfo(name, collectionType, string.Empty)))
         {
             return;
         }
 
-        _storages.Remove(name);
+        _storages.Remove(new Collectioninfo(name, collectionType, string.Empty));
     }
 
-    public ICollectionGenericObjects<T>? this[string name]
+    public ICollectionGenericObjects<T>? this[string name, CollectionType collectionType]
     {
         get
         {
-            if (_storages.ContainsKey(name))
+            if (_storages.ContainsKey(new Collectioninfo(name, collectionType, string.Empty)))
             {
-                return _storages[name];
+                return _storages[new Collectioninfo(name, collectionType, string.Empty)];
             }
             return null;
         }
@@ -78,12 +81,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))
@@ -94,7 +96,7 @@ public class StorageCollection<T>
         StringBuilder sb = new();
 
         sb.Append(_collectionKey);
-        foreach (KeyValuePair<string, ICollectionGenericObjects<T>> value in _storages)
+        foreach (KeyValuePair<Collectioninfo, ICollectionGenericObjects<T>> value in _storages)
         {
             sb.Append(Environment.NewLine);
             // не сохраняем пустые коллекции
@@ -105,9 +107,7 @@ public class StorageCollection<T>
 
             sb.Append(value.Key);
             sb.Append(_separatorForKeyValue);
-            sb.Append(value.Value.GetCollectionType);
-            sb.Append(_separatorForKeyValue);
-            sb.Append(value.Value.Count);
+            sb.Append(value.Value.MaxCount);
             sb.Append(_separatorForKeyValue);
 
             foreach (T? item in value.Value.GetItems())
@@ -126,19 +126,17 @@ public class StorageCollection<T>
         using FileStream fs = new(filename, FileMode.Create);
         byte[] info = new UTF8Encoding(true).GetBytes(sb.ToString());
         fs.Write(info, 0, info.Length);
-        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("Файл не существует");
         }
 
         string bufferTextFromFile = "";
@@ -155,49 +153,53 @@ public class StorageCollection<T>
         string[] strs = bufferTextFromFile.Split(new char[] { '\n', '\r' }, StringSplitOptions.RemoveEmptyEntries);
         if (strs == null || strs.Length == 0)
         {
-            return false;
+            throw new Exception("В файле нет данных");
         }
 
         if (!strs[0].Equals(_collectionKey))
         {
-            //если нет такой записи, то это не те данные
-            return false;
+            throw new Exception("В файле неверные данные");
         }
 
         _storages.Clear();
         foreach (string data in strs)
         {
             string[] record = data.Split(_separatorForKeyValue, StringSplitOptions.RemoveEmptyEntries);
-            if (record.Length != 4)
+            if (record.Length != 3)
             {
                 continue;
             }
 
-            CollectionType collectionType = (CollectionType)Enum.Parse(typeof(CollectionType), record[1]);
-            ICollectionGenericObjects<T>? collection = StorageCollection<T>.CreateCollection(collectionType);
+            Collectioninfo? collectioninfo = Collectioninfo.GetCollectionInfo(record[0]) ??
+                throw new Exception("Не удалось определить информацию коллекции:" + record[0]);
+            ICollectionGenericObjects<T>? collection = StorageCollection<T>.CreateCollection(collectioninfo.CollectionType);
             if (collection == null)
             {
-                return false;
+                throw new Exception("Не удалось определить тип коллекции:" + record[0]);
             }
 
-            collection.MaxCount = Convert.ToInt32(record[2]);
+            collection.MaxCount = Convert.ToInt32(record[1]);
 
-            string[] set = record[3].Split(_separatorItems, StringSplitOptions.RemoveEmptyEntries);
+            string[] set = record[2].Split(_separatorItems, StringSplitOptions.RemoveEmptyEntries);
             foreach (string elem in set)
             {
                 if (elem?.CreateDrawningCar() is T car)
                 {
-                    if (!collection.Insert(car))
+                    try
                     {
-                        return false;
+                        if (!collection.Insert(car))
+                        {
+                            throw new Exception("Объект не удалось добавить в коллекцию: " + record[2]);
+                        }
+                    }
+                    catch (CollectionOverflowException ex)
+                    {
+                        throw new Exception("Коллекция переполнена", ex);
                     }
                 }
             }
-
-            _storages.Add(record[0], collection);
+            _storages.Add(collectioninfo, collection);
         }
-
-        return true;
     }
 
     /// <summary>

ProectMilitaryAircraft/ProectMilitaryAircraft/Draw/DrawningAircraft.cs

@@ -33,17 +33,16 @@ public class DrawningAircraft
     /// Верхняя координа прорисовки самолета
     /// </summary>
     protected int? _startPosY;
-    public DrawningAircraft aircraft;
 
     /// <summary>
     /// Ширина прорисовки самолета
     /// </summary>
-    private readonly int _drawningMilitaryAircraftWidth = 120;
+    private readonly int _drawningAircraftWidth = 120;
 
     /// <summary>
     /// Высота прорисовки самолета
     /// </summary>
-    private readonly int _drawingMilitaryAircraftHeight = 110;
+    private readonly int _drawingAircraftHeight = 110;
 
 
     /// <summary>
@@ -59,12 +58,12 @@ public class DrawningAircraft
     /// <summary>
     /// Ширина объекта
     /// </summary>
-    public int GetWidth => _drawningMilitaryAircraftWidth;
+    public int GetWidth => _drawningAircraftWidth;
 
     /// <summary>
     /// Высота объекта
     /// </summary>
-    public int GetHeight => _drawingMilitaryAircraftHeight;
+    public int GetHeight => _drawingAircraftHeight;
 
     /// <summary>
     /// Пустой конструктор
@@ -86,7 +85,7 @@ public class DrawningAircraft
 
     public DrawningAircraft(int speed, double weight, Color bodyColor, int width, int height) : this()
     {
-        if (width < _drawingMilitaryAircraftHeight || height < _drawningMilitaryAircraftWidth)
+        if (width < _drawingAircraftHeight || height < _drawningAircraftWidth)
         {
             return;
         }
@@ -103,14 +102,14 @@ public class DrawningAircraft
 
     protected DrawningAircraft(int speed, double weight, Color bodyColor, int width, int height, int drawningMilitaryAircraftWidth, int drawingMilitaryAircraftHeight) : this()
     {
-        if (width < _drawingMilitaryAircraftHeight || height < _drawningMilitaryAircraftWidth)
+        if (width < _drawingAircraftHeight || height < _drawningAircraftWidth)
         {
             return;
         }
         _pictureWidth = width; 
         _pictureHeight = height;  
-        _drawningMilitaryAircraftWidth = drawningMilitaryAircraftWidth;
-        _drawingMilitaryAircraftHeight = drawingMilitaryAircraftHeight;
+        _drawningAircraftWidth = drawningMilitaryAircraftWidth;
+        _drawingAircraftHeight = drawingMilitaryAircraftHeight;
         EntityAircraft = new EntityAircraft(speed, weight, bodyColor);
     }
 
@@ -133,8 +132,6 @@ public class DrawningAircraft
     /// <returns>true - границы заданы, false - проверка не пройдена, нельзя разместить объект в этих размерах</returns>
     public bool SetpictureSize(int width, int height)
     {
-        // TODO провека, что объект "влезает" в размеры поля
-        // если влезает, сохраняем границы и корректируем позицию объекта, если она была установлена
         _pictureWidth = width;
         _pictureHeight = height;
         return true;
@@ -187,7 +184,7 @@ public class DrawningAircraft
             //Вправо
             case DirectionType.Right:
 
-                if (_startPosX.Value + _drawningMilitaryAircraftWidth + EntityAircraft.Step < _pictureWidth)
+                if (_startPosX.Value + _drawningAircraftWidth + EntityAircraft.Step < _pictureWidth)
                 {
                     _startPosX += (int)EntityAircraft.Step;
                 }
@@ -197,7 +194,7 @@ public class DrawningAircraft
             //Влево
             case DirectionType.Down:
 
-                if (_startPosY.Value + _drawingMilitaryAircraftHeight + EntityAircraft.Step < _pictureHeight)
+                if (_startPosY.Value + _drawingAircraftHeight + EntityAircraft.Step < _pictureHeight)
                 {
                     _startPosY += (int)EntityAircraft.Step;
                 }

ProectMilitaryAircraft/ProectMilitaryAircraft/Draw/DrawningAircraftCompareByColor.cs

@@ -0,0 +1,41 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace ProectMilitaryAircraft.Draw;
+
+
+/// <summary>
+/// Сравнение по цвету, скорости, весу
+/// </summary>
+public class DrawningAircraftCompareByColor : IComparer<DrawningAircraft?>
+{
+    public int Compare(DrawningAircraft? x, DrawningAircraft? y)
+    {
+        if (x == null || x.EntityAircraft == null)
+        {
+            return -1;
+        }
+
+        if (y == null || y.EntityAircraft == null)
+        {
+            return 1;
+        }
+
+        if (x.EntityAircraft.BodyColor.Name != y.EntityAircraft.BodyColor.Name)
+        {
+            return x.EntityAircraft.BodyColor.Name.CompareTo(y.EntityAircraft.BodyColor.Name);
+        }
+
+        var speedCompare = x.EntityAircraft.Speed.CompareTo(y.EntityAircraft.Speed);
+        
+        if (speedCompare != 0)
+        {
+            return speedCompare;
+        }
+           
+        return x.EntityAircraft.Weight.CompareTo(y.EntityAircraft.Weight);
+    }
+}

ProectMilitaryAircraft/ProectMilitaryAircraft/Draw/DrawningAircraftCompareByType.cs

@@ -0,0 +1,40 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace ProectMilitaryAircraft.Draw;
+
+
+/// <summary>
+/// Сравнение по типу, скорости, весу
+/// </summary>
+public class DrawningAircraftCompareByType : IComparer<DrawningAircraft?>
+{
+    public int Compare(DrawningAircraft? x, DrawningAircraft? y)
+    {
+        if (x == null || x.EntityAircraft == null)
+        {
+            return -1;
+        }
+
+        if (y == null || y.EntityAircraft == null)
+        {
+            return 1;
+        }
+
+        if (x.GetType().Name != y.GetType().Name)
+        {
+            return x.GetType().Name.CompareTo(y.GetType().Name);
+        }
+
+        var speedCompare = x.EntityAircraft.Speed.CompareTo(y.EntityAircraft.Speed);
+        if (speedCompare != 0)
+        {
+            return speedCompare;
+        }
+
+        return x.EntityAircraft.Weight.CompareTo(y.EntityAircraft.Weight);
+    }
+}

ProectMilitaryAircraft/ProectMilitaryAircraft/Draw/DrawningAircraftEqutables.cs

@@ -0,0 +1,63 @@
+using ProectMilitaryAircraft.Entities;
+using System;
+using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace ProectMilitaryAircraft.Draw;
+
+/// <summary>
+/// Реализация сравнения двух объектов класса-прорисовки
+/// </summary>
+public class DrawningAircraftEqutables : IEqualityComparer<DrawningAircraft?>
+{
+    public bool Equals(DrawningAircraft? x, DrawningAircraft? y)
+    {
+        if (x == null || x.EntityAircraft == null)
+        {
+            return false;
+        }
+        if (y == null || y.EntityAircraft == null)
+        {
+            return false;
+        }
+        if (x.GetType().Name != y.GetType().Name)
+        {
+            return false;
+        }
+        if (x.EntityAircraft.Speed != y.EntityAircraft.Speed)
+        {
+            return false;
+        }
+        if (x.EntityAircraft.Weight != y.EntityAircraft.Weight)
+        {
+            return false;
+        }
+        if (x.EntityAircraft.BodyColor != y.EntityAircraft.BodyColor)
+        {
+            return false;
+        }
+        if (x is DrawningMilitaryAircraft && y is DrawningMilitaryAircraft)
+        {
+            EntityMilitaryAircraft EntityX =
+                (EntityMilitaryAircraft)x.EntityAircraft;
+            EntityMilitaryAircraft EntityY =
+                (EntityMilitaryAircraft)y.EntityAircraft;
+            if (EntityX.Pin != EntityY.Pin)
+                return false;
+            if (EntityX.Symbolism != EntityY.Symbolism)
+                return false;
+            if (EntityX.Rokets != EntityY.Rokets)
+                return false;
+            if (EntityX.AdditionalColor != EntityY.AdditionalColor)
+                return false;
+        }
+        return true;
+    }
+    public int GetHashCode([DisallowNull] DrawningAircraft obj)
+    {
+        return obj.GetHashCode();
+    }
+}

ProectMilitaryAircraft/ProectMilitaryAircraft/Exceptions/CollectionOverflowException.cs

@@ -0,0 +1,25 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Runtime.Serialization;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace ProectMilitaryAircraft.Exceptions;
+
+/// <summary>
+/// Класс, Описывающий ошибку переполнения коллекции
+/// </summary>
+[Serializable]
+public 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) { }
+}

ProectMilitaryAircraft/ProectMilitaryAircraft/Exceptions/ObjectNotFoundException.cs

@@ -0,0 +1,26 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Runtime.Serialization;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace ProectMilitaryAircraft.Exceptions;
+
+
+/// <summary>
+/// Класс, описывающий ошибку, что по указаной позиции нет элемента
+/// </summary>
+[Serializable]
+public 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) { }
+}

ProectMilitaryAircraft/ProectMilitaryAircraft/Exceptions/ObjectNotUniqueException.cs

@@ -0,0 +1,15 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace ProectMilitaryAircraft.Exceptions;
+
+/// <summary>
+/// Класс, описывающий ошибку наличия такого же объекта в коллекции
+/// </summary>
+[Serializable]
+internal class ObjectNotUniqueException : ApplicationException
+{
+}

ProectMilitaryAircraft/ProectMilitaryAircraft/Exceptions/PositionOutOfCollectionException.cs

@@ -0,0 +1,25 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Runtime.Serialization;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace ProectMilitaryAircraft.Exceptions;
+
+/// <summary>
+/// Класс, описывающий ошибку выхода за границы коллекции
+/// </summary>
+[Serializable]
+public 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) { }
+}

ProectMilitaryAircraft/ProectMilitaryAircraft/FormAircraftCollection.Designer.cs

@@ -52,6 +52,8 @@
             loadToolStripMenuItem = new ToolStripMenuItem();
             saveFileDialog = new SaveFileDialog();
             openFileDialog = new OpenFileDialog();
+            buttonSortByType = new Button();
+            buttonSortByColor = new Button();
             groupBoxTools.SuspendLayout();
             panelCompanyTools.SuspendLayout();
             panelStorage.SuspendLayout();
@@ -75,6 +77,8 @@
             // 
             // panelCompanyTools
             // 
+            panelCompanyTools.Controls.Add(buttonSortByColor);
+            panelCompanyTools.Controls.Add(buttonSortByType);
             panelCompanyTools.Controls.Add(buttonAddAircraft);
             panelCompanyTools.Controls.Add(buttonRefresh);
             panelCompanyTools.Controls.Add(maskedTextBox);
@@ -100,7 +104,7 @@
             // buttonRefresh
             // 
             buttonRefresh.Anchor = AnchorStyles.Top | AnchorStyles.Left | AnchorStyles.Right;
-            buttonRefresh.Location = new Point(4, 229);
+            buttonRefresh.Location = new Point(4, 176);
             buttonRefresh.Name = "buttonRefresh";
             buttonRefresh.Size = new Size(178, 45);
             buttonRefresh.TabIndex = 5;
@@ -110,7 +114,7 @@
             // 
             // maskedTextBox
             // 
-            maskedTextBox.Location = new Point(4, 98);
+            maskedTextBox.Location = new Point(4, 45);
             maskedTextBox.Mask = "00";
             maskedTextBox.Name = "maskedTextBox";
             maskedTextBox.Size = new Size(178, 23);
@@ -120,7 +124,7 @@
             // buttonGoToCheck
             // 
             buttonGoToCheck.Anchor = AnchorStyles.Top | AnchorStyles.Left | AnchorStyles.Right;
-            buttonGoToCheck.Location = new Point(4, 178);
+            buttonGoToCheck.Location = new Point(4, 125);
             buttonGoToCheck.Name = "buttonGoToCheck";
             buttonGoToCheck.Size = new Size(178, 45);
             buttonGoToCheck.TabIndex = 4;
@@ -131,7 +135,7 @@
             // buttonRemoveAircraft
             // 
             buttonRemoveAircraft.Anchor = AnchorStyles.Top | AnchorStyles.Left | AnchorStyles.Right;
-            buttonRemoveAircraft.Location = new Point(4, 127);
+            buttonRemoveAircraft.Location = new Point(4, 74);
             buttonRemoveAircraft.Name = "buttonRemoveAircraft";
             buttonRemoveAircraft.Size = new Size(178, 45);
             buttonRemoveAircraft.TabIndex = 3;
@@ -292,6 +296,28 @@
             // 
             openFileDialog.Filter = "txt file | *.txt";
             // 
+            // buttonSortByType
+            // 
+            buttonSortByType.Anchor = AnchorStyles.Top | AnchorStyles.Left | AnchorStyles.Right;
+            buttonSortByType.Location = new Point(3, 229);
+            buttonSortByType.Name = "buttonSortByType";
+            buttonSortByType.Size = new Size(85, 45);
+            buttonSortByType.TabIndex = 6;
+            buttonSortByType.Text = "Сортировка по типу";
+            buttonSortByType.UseVisualStyleBackColor = true;
+            buttonSortByType.Click += ButtonSortByType_Click;
+            // 
+            // buttonSortByColor
+            // 
+            buttonSortByColor.Anchor = AnchorStyles.Top | AnchorStyles.Left | AnchorStyles.Right;
+            buttonSortByColor.Location = new Point(94, 229);
+            buttonSortByColor.Name = "buttonSortByColor";
+            buttonSortByColor.Size = new Size(85, 45);
+            buttonSortByColor.TabIndex = 7;
+            buttonSortByColor.Text = "Сортировка по цвету";
+            buttonSortByColor.UseVisualStyleBackColor = true;
+            buttonSortByColor.Click += ButtonSortByColor_Click;
+            // 
             // FormAircraftCollection
             // 
             AutoScaleDimensions = new SizeF(7F, 15F);
@@ -341,5 +367,7 @@
         private ToolStripMenuItem loadToolStripMenuItem;
         private SaveFileDialog saveFileDialog;
         private OpenFileDialog openFileDialog;
+        private Button buttonSortByColor;
+        private Button buttonSortByType;
     }
 }

ProectMilitaryAircraft/ProectMilitaryAircraft/FormAircraftCollection.cs

@@ -1,5 +1,7 @@
-using ProectMilitaryAircraft.CollectionGenericObjects;
+using Microsoft.Extensions.Logging;
+using ProectMilitaryAircraft.CollectionGenericObjects;
 using ProectMilitaryAircraft.Draw;
+using ProectMilitaryAircraft.Exceptions;
 using ProectMilitaryAircraft.MovementStrategy;
 using System;
 using System.Collections.Generic;
@@ -28,14 +30,19 @@ public partial class FormAircraftCollection : Form
     /// </summary>
     private AbstractCompany? _company = null;
 
+    /// <summary>
+	/// Логер
+	/// </summary>
+	private readonly ILogger _logger;
 
     /// <summary>
     /// Конструктор
     /// </summary>
-    public FormAircraftCollection()
+    public FormAircraftCollection(ILogger<FormAircraftCollection> logger)
     {
         InitializeComponent();
         _storageCollection = new();
+        _logger = logger;
     }
     /// <summary>
     /// Выбор компании
@@ -55,7 +62,7 @@ public partial class FormAircraftCollection : Form
     private void ButtonAddAircraft_Click(object sender, EventArgs e)
     {
         if (listBoxCollection.SelectedIndex == -1) return;
-        var obj = _storageCollection[listBoxCollection.SelectedItem?.ToString() ?? string.Empty];
+        var obj = _storageCollection[listBoxCollection.SelectedItem?.ToString() ?? string.Empty, new CollectionType()];
         if (obj == null) return;
 
         FormAircraftConfig form = new();
@@ -74,15 +81,25 @@ public partial class FormAircraftCollection : Form
             return;
         }
 
-        if (_company + aircraft)
+        try
         {
-            MessageBox.Show("Объект добавлен");
-            pictureBox.Image = _company.Show();
-
+            if (_company + aircraft)
+            {
+                MessageBox.Show("Объект добавлен");
+                pictureBox.Image = _company.Show();
+                _logger.LogInformation("Объект добавлен", _company);
+            }
+            else
+            {
+                MessageBox.Show("Не удалось добавить объект");
+                _logger.LogError("Не удалось добавить объект");
+            }
         }
-        else
+        catch (CollectionOverflowException ex)
         {
-            MessageBox.Show("не удалось добавить объект");
+            MessageBox.Show(ex.Message);
+            _logger.LogError("Ошибка: {Message}. Не удалось добавить объект", ex.Message);
+
         }
     }
 
@@ -104,14 +121,20 @@ public partial class FormAircraftCollection : Form
         }
 
         int pos = Convert.ToInt32(maskedTextBox.Text);
-        if (_company - pos)
+        try
         {
-            MessageBox.Show("Объект удален");
-            pictureBox.Image = _company.Show();
+            if (_company - pos)
+            {
+                MessageBox.Show("Объект удален");
+                pictureBox.Image = _company.Show();
+                _logger.LogInformation("Объект удален", _company);
+            }
+
         }
-        else
+        catch (Exception ex)
         {
-            MessageBox.Show("Не удалось удалить объект");
+            MessageBox.Show(ex.Message);
+            _logger.LogError("Ошибка: {Message}", ex.Message);
         }
     }
 
@@ -130,12 +153,22 @@ public partial class FormAircraftCollection : Form
         int counter = 100;
         while (aircraft == null)
         {
-            aircraft = _company.GetRandomObject();
-            counter--;
-            if (counter <= 0)
+            try
             {
-                break;
+                aircraft = _company.GetRandomObject();
+                counter--;
+                if (counter <= 0)
+                {
+                    break;
+                }
+                _logger.LogInformation("Рандомный объект", _company);
             }
+            catch (Exception ex)
+            {
+                MessageBox.Show(ex.Message);
+                _logger.LogError("Ошибка: {Message}", ex.Message);
+            }
+
         }
 
         if (aircraft == null) { return; }
@@ -203,7 +236,7 @@ public partial class FormAircraftCollection : Form
             MessageBoxIcon.Question) == DialogResult.Yes)
         {
             _storageCollection.DelCollection(listBoxCollection.SelectedItem?.ToString()
-            ?? string.Empty);
+            ?? string.Empty, new CollectionType()) ;
             RefreshListBoxItems();
         }
 
@@ -214,7 +247,7 @@ public partial class FormAircraftCollection : Form
         listBoxCollection.Items.Clear();
         for (int i = 0; i < _storageCollection.Keys?.Count; i++)
         {
-            string? colName = _storageCollection.Keys?[i];
+            string? colName = _storageCollection.Keys?[i].Name;
             if (!string.IsNullOrEmpty(colName))
             {
                 listBoxCollection.Items.Add(colName);
@@ -235,7 +268,7 @@ public partial class FormAircraftCollection : Form
             return;
         }
 
-        ICollectionGenericObjects<DrawningAircraft>? collection = _storageCollection[listBoxCollection.SelectedItem.ToString() ?? string.Empty];
+        ICollectionGenericObjects<DrawningAircraft>? collection = _storageCollection[listBoxCollection.SelectedItem.ToString() ?? string.Empty, new CollectionType()];
         if (collection == null)
         {
             MessageBox.Show("Коллкция не проиницилизирована");
@@ -263,13 +296,16 @@ public partial class FormAircraftCollection : Form
     {
         if (saveFileDialog.ShowDialog() == DialogResult.OK)
         {
-            if (_storageCollection.SaveData(saveFileDialog.FileName))
+            try
             {
+                _storageCollection.SaveData(saveFileDialog.FileName);
                 MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
+                _logger.LogInformation("Сохранение в файл: {filename}", saveFileDialog.FileName);
             }
-            else
+            catch (Exception ex)
             {
-                MessageBox.Show("Не сохранилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                _logger.LogError("Ошибка: {Message}", ex.Message);
             }
         }
     }
@@ -283,15 +319,39 @@ public partial class FormAircraftCollection : Form
     {
         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);
+                MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                _logger.LogError("Ошибка: {Message}", ex.Message);
             }
         }
     }
+
+    private void ButtonSortByType_Click(object sender, EventArgs e)
+    {   
+        CompareAircraft(new DrawningAircraftCompareByType());
+    }
+
+    private void ButtonSortByColor_Click(object sender, EventArgs e)
+    {
+        CompareAircraft(new DrawningAircraftCompareByColor());
+    }
+
+    private void CompareAircraft(IComparer<DrawningAircraft?> comparer)
+    {
+        if (_company == null)
+        {
+            return;
+        }
+
+        _company.Sort(comparer);
+        pictureBox.Image = _company.Show();
+    }
 }

ProectMilitaryAircraft/ProectMilitaryAircraft/ProectMilitaryAircraft.csproj

@@ -8,6 +8,11 @@
     <ImplicitUsings>enable</ImplicitUsings>
   </PropertyGroup>
 
+  <ItemGroup>
+    <PackageReference Include="Microsoft.Extensions.Logging" Version="8.0.0" />
+    <PackageReference Include="NLog.Extensions.Logging" Version="5.3.8" />
+  </ItemGroup>
+
   <ItemGroup>
     <Compile Update="Properties\Resources.Designer.cs">
       <DesignTime>True</DesignTime>
@@ -23,4 +28,10 @@
     </EmbeddedResource>
   </ItemGroup>
 
+  <ItemGroup>
+    <None Update="nlog.config">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </None>
+  </ItemGroup>
+
 </Project>

ProectMilitaryAircraft/ProectMilitaryAircraft/Program.cs

@@ -1,3 +1,7 @@
+using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.Logging;
+using NLog.Extensions.Logging;
+
 namespace ProectMilitaryAircraft
 {
     internal static class Program
@@ -10,8 +14,28 @@ namespace ProectMilitaryAircraft
         {
             // To customize application configuration such as set high DPI settings or default font,
             // see https://aka.ms/applicationconfiguration.
+
             ApplicationConfiguration.Initialize();
-            Application.Run(new FormAircraftCollection());
+            ServiceCollection services = new();
+            ConfigureServices(services);
+            using (ServiceProvider serviceProvider = services.BuildServiceProvider())
+            {
+                Application.Run(serviceProvider.GetRequiredService<FormAircraftCollection>());
+            }
+        }
+
+        /// <summary>
+        /// <20><><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD> <20><><EFBFBD><EFBFBD><EFBFBD><EFBFBD><EFBFBD>  DI
+        /// </summary>
+        /// <param name="services"></param>
+        private static void ConfigureServices(ServiceCollection services)
+        {
+            services.AddSingleton<FormAircraftCollection>()
+                .AddLogging(option =>   
+                {
+                    option.SetMinimumLevel(LogLevel.Information);
+                    option.AddNLog("nlog.config");
+                });
         }
     }
 }

ProectMilitaryAircraft/ProectMilitaryAircraft/nlog.config

@@ -0,0 +1,15 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<configuration>
+  <nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
+        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+        autoReload="true" internalLogLevel="Info">
+
+    <targets>
+      <target xsi:type="File" name="tofile" fileName="carlog-${shortdate}.log" />
+    </targets>
+
+    <rules>
+      <logger name="*" minlevel="Debug" writeTo="tofile" />
+    </rules>
+  </nlog>
+</configuration>