Tank/Tank/DrawiningTankEqutables.cs

@@ -0,0 +1,76 @@
+using System;
+using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using Tank.DrawningObjects;
+using Tank.Entities;
+
+namespace Tank
+{
+    internal class DrawiningTankEqutables : IEqualityComparer<DrawArmoVehicle?>
+    {
+        public bool Equals(DrawArmoVehicle? x, DrawArmoVehicle? y)
+        {
+            if (x == null || x.Tank == null)
+            {
+                throw new ArgumentNullException(nameof(x));
+            }
+            if (y == null || y.Tank == null)
+            {
+                throw new ArgumentNullException(nameof(y));
+            }
+            if (x.GetType().Name != y.GetType().Name)
+            {
+                return false;
+            }
+            if (x.Tank.Speed != y.Tank.Speed)
+            {
+                return false;
+            }
+            if (x.Tank.Weight != y.Tank.Weight)
+            {
+                return false;
+            }
+            if (x.Tank.BodyColor != y.Tank.BodyColor)
+            {
+                return false;
+            }
+            if (x is DrawTank && y is DrawTank)
+            {
+                if (x.Tank as EntityTank == null)
+                {
+                    throw new ArgumentNullException(nameof(x));
+                }
+                if (y.Tank as EntityTank == null)
+                {
+                    throw new ArgumentNullException(nameof(y));
+                }
+                if ((x.Tank as EntityTank).AdditionalColor != (y.Tank as EntityTank).AdditionalColor)
+                {
+                    return false;
+                }
+                if ((x.Tank as EntityTank).BodyKit != (y.Tank as EntityTank).BodyKit)
+                {
+                    return false;
+                }
+                if ((x.Tank as EntityTank).Caterpillar != (y.Tank as EntityTank).Caterpillar)
+                {
+                    return false;
+                }
+                if ((x.Tank as EntityTank).Tower != (y.Tank as EntityTank).Tower)
+                {
+                    return false;
+                }
+
+            }
+            return true;
+        }
+
+        public int GetHashCode([DisallowNull] DrawArmoVehicle obj)
+        {
+            return obj.GetHashCode();
+        }
+    }
+}

Tank/Tank/FormTanksCollections.Designer.cs

@@ -29,6 +29,8 @@
         private void InitializeComponent()
         {
             panel1 = new Panel();
+            SortColorButton = new Button();
+            SortTypeButton = new Button();
             menuStrip1 = new MenuStrip();
             StripMenu = new ToolStripMenuItem();
             SaveItem = new ToolStripMenuItem();
@@ -55,6 +57,8 @@
             // 
             // panel1
             // 
+            panel1.Controls.Add(SortColorButton);
+            panel1.Controls.Add(SortTypeButton);
             panel1.Controls.Add(menuStrip1);
             panel1.Controls.Add(panel2);
             panel1.Controls.Add(UpdateButton);
@@ -63,18 +67,40 @@
             panel1.Controls.Add(InputNum);
             panel1.Controls.Add(label1);
             panel1.Dock = DockStyle.Right;
-            panel1.Location = new Point(649, 0);
+            panel1.Location = new Point(560, 0);
+            panel1.Margin = new Padding(3, 2, 3, 2);
             panel1.Name = "panel1";
-            panel1.Size = new Size(237, 471);
+            panel1.Size = new Size(207, 402);
             panel1.TabIndex = 0;
             // 
+            // SortColorButton
+            // 
+            SortColorButton.Location = new Point(27, 56);
+            SortColorButton.Name = "SortColorButton";
+            SortColorButton.Size = new Size(163, 23);
+            SortColorButton.TabIndex = 10;
+            SortColorButton.Text = "Сортировка по цвету";
+            SortColorButton.UseVisualStyleBackColor = true;
+            SortColorButton.Click += ButtonSortByColor_Click;
+            // 
+            // SortTypeButton
+            // 
+            SortTypeButton.Location = new Point(27, 30);
+            SortTypeButton.Name = "SortTypeButton";
+            SortTypeButton.Size = new Size(163, 23);
+            SortTypeButton.TabIndex = 9;
+            SortTypeButton.Text = "Сортировка по типу";
+            SortTypeButton.UseVisualStyleBackColor = true;
+            SortTypeButton.Click += ButtonSortByType_Click;
+            // 
             // menuStrip1
             // 
             menuStrip1.ImageScalingSize = new Size(20, 20);
             menuStrip1.Items.AddRange(new ToolStripItem[] { StripMenu });
             menuStrip1.Location = new Point(0, 0);
             menuStrip1.Name = "menuStrip1";
-            menuStrip1.Size = new Size(237, 28);
+            menuStrip1.Padding = new Padding(5, 2, 0, 2);
+            menuStrip1.Size = new Size(207, 24);
             menuStrip1.TabIndex = 7;
             menuStrip1.Text = "menuStrip1";
             // 
@@ -82,20 +108,20 @@
             // 
             StripMenu.DropDownItems.AddRange(new ToolStripItem[] { SaveItem, LoadItem });
             StripMenu.Name = "StripMenu";
-            StripMenu.Size = new Size(59, 24);
+            StripMenu.Size = new Size(48, 20);
             StripMenu.Text = "Файл";
             // 
             // SaveItem
             // 
             SaveItem.Name = "SaveItem";
-            SaveItem.Size = new Size(166, 26);
+            SaveItem.Size = new Size(133, 22);
             SaveItem.Text = "Сохранить";
             SaveItem.Click += SaveToolStripMenuItem_Click;
             // 
             // LoadItem
             // 
             LoadItem.Name = "LoadItem";
-            LoadItem.Size = new Size(166, 26);
+            LoadItem.Size = new Size(133, 22);
             LoadItem.Text = "Загрузить";
             LoadItem.Click += LoadToolStripMenuItem_Click;
             // 
@@ -106,16 +132,18 @@
             panel2.Controls.Add(AddCollectButton);
             panel2.Controls.Add(SetTextBox);
             panel2.Controls.Add(label2);
-            panel2.Location = new Point(16, 62);
+            panel2.Location = new Point(14, 105);
+            panel2.Margin = new Padding(3, 2, 3, 2);
             panel2.Name = "panel2";
-            panel2.Size = new Size(214, 220);
+            panel2.Size = new Size(187, 165);
             panel2.TabIndex = 6;
             // 
             // DeleteCollectButton
             // 
-            DeleteCollectButton.Location = new Point(3, 181);
+            DeleteCollectButton.Location = new Point(3, 136);
+            DeleteCollectButton.Margin = new Padding(3, 2, 3, 2);
             DeleteCollectButton.Name = "DeleteCollectButton";
-            DeleteCollectButton.Size = new Size(208, 29);
+            DeleteCollectButton.Size = new Size(182, 22);
             DeleteCollectButton.TabIndex = 4;
             DeleteCollectButton.Text = "Удалить набор";
             DeleteCollectButton.UseVisualStyleBackColor = true;
@@ -124,18 +152,20 @@
             // CollectionListBox
             // 
             CollectionListBox.FormattingEnabled = true;
-            CollectionListBox.ItemHeight = 20;
-            CollectionListBox.Location = new Point(3, 91);
+            CollectionListBox.ItemHeight = 15;
+            CollectionListBox.Location = new Point(3, 68);
+            CollectionListBox.Margin = new Padding(3, 2, 3, 2);
             CollectionListBox.Name = "CollectionListBox";
-            CollectionListBox.Size = new Size(207, 84);
+            CollectionListBox.Size = new Size(182, 64);
             CollectionListBox.TabIndex = 3;
             CollectionListBox.SelectedIndexChanged += ListBoxObjects_SelectedIndexChanged;
             // 
             // AddCollectButton
             // 
-            AddCollectButton.Location = new Point(3, 56);
+            AddCollectButton.Location = new Point(3, 42);
+            AddCollectButton.Margin = new Padding(3, 2, 3, 2);
             AddCollectButton.Name = "AddCollectButton";
-            AddCollectButton.Size = new Size(208, 29);
+            AddCollectButton.Size = new Size(182, 22);
             AddCollectButton.TabIndex = 2;
             AddCollectButton.Text = "Добавить набор";
             AddCollectButton.UseVisualStyleBackColor = true;
@@ -143,9 +173,10 @@
             // 
             // SetTextBox
             // 
-            SetTextBox.Location = new Point(2, 23);
+            SetTextBox.Location = new Point(2, 17);
+            SetTextBox.Margin = new Padding(3, 2, 3, 2);
             SetTextBox.Name = "SetTextBox";
-            SetTextBox.Size = new Size(209, 27);
+            SetTextBox.Size = new Size(183, 23);
             SetTextBox.TabIndex = 1;
             // 
             // label2
@@ -153,15 +184,16 @@
             label2.AutoSize = true;
             label2.Location = new Point(3, 0);
             label2.Name = "label2";
-            label2.Size = new Size(66, 20);
+            label2.Size = new Size(52, 15);
             label2.TabIndex = 0;
             label2.Text = "Наборы";
             // 
             // UpdateButton
             // 
-            UpdateButton.Location = new Point(10, 414);
+            UpdateButton.Location = new Point(9, 369);
+            UpdateButton.Margin = new Padding(3, 2, 3, 2);
             UpdateButton.Name = "UpdateButton";
-            UpdateButton.Size = new Size(219, 37);
+            UpdateButton.Size = new Size(192, 28);
             UpdateButton.TabIndex = 4;
             UpdateButton.Text = "Обновить коллекцию";
             UpdateButton.UseVisualStyleBackColor = true;
@@ -169,9 +201,10 @@
             // 
             // DeleteButton
             // 
-            DeleteButton.Location = new Point(10, 374);
+            DeleteButton.Location = new Point(9, 339);
+            DeleteButton.Margin = new Padding(3, 2, 3, 2);
             DeleteButton.Name = "DeleteButton";
-            DeleteButton.Size = new Size(219, 37);
+            DeleteButton.Size = new Size(192, 28);
             DeleteButton.TabIndex = 3;
             DeleteButton.Text = "Удалить технику";
             DeleteButton.UseVisualStyleBackColor = true;
@@ -179,9 +212,10 @@
             // 
             // AddButton
             // 
-            AddButton.Location = new Point(10, 296);
+            AddButton.Location = new Point(9, 281);
+            AddButton.Margin = new Padding(3, 2, 3, 2);
             AddButton.Name = "AddButton";
-            AddButton.Size = new Size(219, 37);
+            AddButton.Size = new Size(192, 28);
             AddButton.TabIndex = 2;
             AddButton.Text = "Добавить технику";
             AddButton.UseVisualStyleBackColor = true;
@@ -189,17 +223,18 @@
             // 
             // InputNum
             // 
-            InputNum.Location = new Point(10, 337);
+            InputNum.Location = new Point(9, 312);
+            InputNum.Margin = new Padding(3, 2, 3, 2);
             InputNum.Name = "InputNum";
-            InputNum.Size = new Size(219, 27);
+            InputNum.Size = new Size(192, 23);
             InputNum.TabIndex = 1;
             // 
             // label1
             // 
             label1.AutoSize = true;
-            label1.Location = new Point(122, 39);
+            label1.Location = new Point(107, 88);
             label1.Name = "label1";
-            label1.Size = new Size(103, 20);
+            label1.Size = new Size(83, 15);
             label1.TabIndex = 0;
             label1.Text = "Инструменты";
             // 
@@ -207,8 +242,9 @@
             // 
             DrawTank.Dock = DockStyle.Fill;
             DrawTank.Location = new Point(0, 0);
+            DrawTank.Margin = new Padding(3, 2, 3, 2);
             DrawTank.Name = "DrawTank";
-            DrawTank.Size = new Size(649, 471);
+            DrawTank.Size = new Size(560, 402);
             DrawTank.TabIndex = 1;
             DrawTank.TabStop = false;
             // 
@@ -218,11 +254,12 @@
             // 
             // FormTanksCollections
             // 
-            AutoScaleDimensions = new SizeF(8F, 20F);
+            AutoScaleDimensions = new SizeF(7F, 15F);
             AutoScaleMode = AutoScaleMode.Font;
-            ClientSize = new Size(886, 471);
+            ClientSize = new Size(767, 402);
             Controls.Add(DrawTank);
             Controls.Add(panel1);
+            Margin = new Padding(3, 2, 3, 2);
             Name = "FormTanksCollections";
             Text = "CollectionsFrame";
             panel1.ResumeLayout(false);
@@ -256,5 +293,7 @@
         private ToolStripMenuItem LoadItem;
         private OpenFileDialog openFileDialog;
         private SaveFileDialog saveFileDialog;
+        private Button SortColorButton;
+        private Button SortTypeButton;
     }
 }

Tank/Tank/FormTanksCollections.cs

@@ -36,7 +36,7 @@ namespace Tank
             CollectionListBox.Items.Clear();
             foreach (var key in _storage.Keys)
             {
-                CollectionListBox.Items.Add(key);
+                CollectionListBox.Items.Add(key.Name);
             }
             if (CollectionListBox.Items.Count > 0 && (index == -1 || index
             >= CollectionListBox.Items.Count))
@@ -66,8 +66,7 @@ namespace Tank
 
         private void ListBoxObjects_SelectedIndexChanged(object sender, EventArgs e)
         {
-            DrawTank.Image =
-            _storage[CollectionListBox.SelectedItem?.ToString() ?? string.Empty]?.ShowTanks();
+            DrawTank.Image = _storage[CollectionListBox.SelectedItem?.ToString() ?? string.Empty]?.ShowTanks();
         }
 
         private void ButtonDelObject_Click(object sender, EventArgs e)
@@ -116,6 +115,11 @@ namespace Tank
                 MessageBox.Show("Не удалось добавить объект");
                 _logger.LogWarning($"{ex.Message} в наборе {CollectionListBox.SelectedItem.ToString()}");
             }
+            catch (ApplicationException ex)
+            {
+                MessageBox.Show(ex.Message);
+                _logger.LogWarning($"добавление танка неуспешно {ex.Message}");
+            }
         }
 
         private void ButtonAddTank_Click(object sender, EventArgs e)
@@ -226,5 +230,20 @@ namespace Tank
                 }
             }
         }
+
+        private void ButtonSortByType_Click(object sender, EventArgs e) => CompareTank(new TankCompareByType());
+
+        private void ButtonSortByColor_Click(object sender, EventArgs e) => CompareTank(new TankCompareByColor());
+
+        private void CompareTank(IComparer<DrawArmoVehicle?> comparer)
+        {
+            if (CollectionListBox.SelectedIndex == -1)
+                return;
+            var obj = _storage[CollectionListBox.SelectedItem.ToString() ?? string.Empty];
+            if (obj == null)
+                return;
+            obj.Sort(comparer);
+            DrawTank.Image = obj.ShowTanks();
+        }
     }
 }

Tank/Tank/SetGeneric.cs

@@ -17,6 +17,9 @@ namespace Tank
         // Максимальное количество объектов
         private readonly int _maxCount;
 
+        // Сортировка набора объектов
+        public void SortSet(IComparer<T?> comparer) => _places.Sort(comparer);
+
         // Конструктор 
         public SetGeneric(int count)
         {
@@ -25,19 +28,23 @@ namespace Tank
         }
 
         // Добавление объекта в набор
-        public bool Insert(T tank)
+        public bool Insert(T tank, IEqualityComparer<T?>? equal = null)
         {
-            return Insert(tank, 0);
+            return Insert(tank, 0, equal);
         }
 
         // Добавление на конкретную позицию
-        public bool Insert(T tank, int position)
+        public bool Insert(T tank, int position, IEqualityComparer<T?>? equal = null)
         {
             if (position < 0 || position >= _maxCount)
                 throw new TankNotFoundException(position);  // По позиции
 
             if (Count >= _maxCount)
                 throw new TankStorageOverflowException(_maxCount); // Макс количество в коллекции
+
+            if (equal != null && _places.Contains<T>(tank, equal))
+                throw new ApplicationException("уже существует");
+
             _places.Insert(0, tank);
             return true;
         }

Tank/Tank/TankCompareByColor.cs

@@ -0,0 +1,39 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using Tank.DrawningObjects;
+using Tank.Entities;
+
+namespace Tank
+{
+    internal class TankCompareByColor : IComparer<DrawArmoVehicle?>
+    {
+        public int Compare(DrawArmoVehicle? x, DrawArmoVehicle? y)
+        {
+            if (x == null || x.Tank == null)
+            {
+                throw new ArgumentNullException(nameof(x));
+            }
+            if (y == null || y.Tank == null)
+            {
+                throw new ArgumentNullException(nameof(y));
+            }
+            if (x.Tank.BodyColor != y.Tank.BodyColor)
+            {
+                return x.Tank.BodyColor.Name.CompareTo(y.Tank.BodyColor.Name);
+            }
+            if (x.GetType() == y.GetType() && x is DrawTank)
+            {
+                return (x.Tank as EntityTank).AdditionalColor.Name.CompareTo((y.Tank as EntityTank).AdditionalColor.Name);
+            }
+            var speedCompare = x.Tank.Speed.CompareTo(y.Tank.Speed);
+            if (speedCompare != 0)
+            {
+                return speedCompare;
+            }
+            return x.Tank.Weight.CompareTo(y.Tank.Weight);
+        }
+    }
+}

Tank/Tank/TankCompareByType.cs

@@ -0,0 +1,34 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using Tank.DrawningObjects;
+
+namespace Tank
+{
+    internal class TankCompareByType : IComparer<DrawArmoVehicle?>
+    {
+        public int Compare(DrawArmoVehicle? x, DrawArmoVehicle? y)
+        {
+            if (x == null || x.Tank == null)
+            {
+                throw new ArgumentNullException(nameof(x));
+            }
+            if (y == null || y.Tank == null)
+            {
+                throw new ArgumentNullException(nameof(y));
+            }
+            if (x.GetType().Name != y.GetType().Name)
+            {
+                return x.GetType().Name.CompareTo(y.GetType().Name);
+            }
+            var speedCompare = x.Tank.Speed.CompareTo(y.Tank.Speed);
+            if (speedCompare != 0)
+            {
+                return speedCompare;
+            }
+            return x.Tank.Weight.CompareTo(y.Tank.Weight);
+        }
+    }
+}

Tank/Tank/TanksCollectionInfo.cs

@@ -0,0 +1,32 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace Tank
+{
+    internal class TankCollectionInfo : IEquatable<TankCollectionInfo>
+    {
+        public string Name { get; private set; }
+        public string Description { get; private set; }
+        
+        public TankCollectionInfo(string name, string description)
+        {
+            Name = name;
+            Description = description;
+        }
+
+        public bool Equals(TankCollectionInfo? other)
+        {
+            if (other == null || Name == null || other.Name == null) return false;
+            if (Name == other?.Name) return true;
+            throw new NotImplementedException();
+        }
+
+        public override int GetHashCode()
+        {
+            return Name?.GetHashCode() ?? 0;
+        }
+    }
+}

Tank/Tank/TanksGenericCollection.cs

@@ -43,7 +43,7 @@ namespace Tank.Generics
             {
                 return false;
             }
-            return (bool)collect?._collection.Insert(obj);
+            return (bool)collect?._collection.Insert(obj, new DrawiningTankEqutables());
         }
 
         // Перегрузка оператора вычитания
@@ -54,6 +54,9 @@ namespace Tank.Generics
             return obj;
         }
 
+        // Сортировка
+        public void Sort(IComparer<T> comparer) => _collection.SortSet(comparer);
+
         // Получение объекта IMoveableObject
         public U? GetU(int pos)
         {

Tank/Tank/TanksGenericStorage.cs

@@ -7,16 +7,17 @@ using Tank.Generics;
 using Tank.DrawningObjects;
 using Tank.MovementStrategy;
 using Tank.Exceptions;
+using Tank.Entities;
 
 namespace Tank
 {
     internal class TanksGenericStorage
     {
         // Словарь
-        readonly Dictionary<string, TanksGenericCollection<DrawArmoVehicle, DrawingObjectTank>> _tankStorages;
+        readonly Dictionary<TankCollectionInfo, TanksGenericCollection<DrawArmoVehicle, DrawingObjectTank>> _tankStorages;
 
         // Возвращение списка названий наборов
-        public List<string> Keys => _tankStorages.Keys.ToList();
+        public List<TankCollectionInfo> Keys => _tankStorages.Keys.ToList();
         private readonly int _pictureWidth;
         private readonly int _pictureHeight;
 
@@ -27,7 +28,7 @@ namespace Tank
         // Конструктор
         public TanksGenericStorage(int pictureWidth, int pictureHeight)
         {
-            _tankStorages = new Dictionary<string, TanksGenericCollection<DrawArmoVehicle, DrawingObjectTank>>();
+            _tankStorages = new Dictionary<TankCollectionInfo, TanksGenericCollection<DrawArmoVehicle, DrawingObjectTank>>();
             _pictureWidth = pictureWidth;
             _pictureHeight = pictureHeight;
         }
@@ -35,15 +36,17 @@ namespace Tank
         // Добавление набора
         public void AddSet(string name)
         {
-            if (_tankStorages.ContainsKey(name)) return;
-            _tankStorages[name] = new TanksGenericCollection<DrawArmoVehicle, DrawingObjectTank>(_pictureWidth, _pictureHeight);
+            TankCollectionInfo Info = new TankCollectionInfo(name, string.Empty);
+            if (_tankStorages.ContainsKey(Info)) return;
+            _tankStorages[Info] = new TanksGenericCollection<DrawArmoVehicle, DrawingObjectTank>(_pictureWidth, _pictureHeight);
         }
 
         // Удаление набора
         public void DelSet(string name)
         {
-            if (!_tankStorages.ContainsKey(name)) return;
-            _tankStorages.Remove(name);
+            TankCollectionInfo Info = new TankCollectionInfo(name, string.Empty);
+            if (!_tankStorages.ContainsKey(Info)) return;
+            _tankStorages.Remove(Info);
         }
 
         // Доступ к набору
@@ -52,7 +55,8 @@ namespace Tank
         {
             get
             {
-                if (_tankStorages.ContainsKey(ind)) return _tankStorages[ind];
+                TankCollectionInfo Info = new TankCollectionInfo(ind, string.Empty);
+                if (_tankStorages.ContainsKey(Info)) return _tankStorages[Info];
                 return null;
             }
         }
@@ -65,14 +69,14 @@ namespace Tank
             }
 
             StringBuilder data = new();
-            foreach (KeyValuePair<string, TanksGenericCollection<DrawArmoVehicle, DrawingObjectTank>> record in _tankStorages)
+            foreach (KeyValuePair<TankCollectionInfo, TanksGenericCollection<DrawArmoVehicle, DrawingObjectTank>> record in _tankStorages)
             {
                 StringBuilder records = new();
                 foreach (DrawArmoVehicle? elem in record.Value.GetTanks)
                 {
                     records.Append($"{elem?.GetDataForSave(_separatorForObject)}{_separatorRecords}");
                 }
-                data.AppendLine($"{record.Key}{_separatorForKeyValue}{records}");
+                data.AppendLine($"{record.Key.Name}{_separatorForKeyValue}{records}");
 
             }
             if (data.Length == 0)
@@ -135,7 +139,7 @@ namespace Tank
                             }
                         }
                     }
-                    _tankStorages.Add(name, collection);
+                    _tankStorages.Add(new TankCollectionInfo(name, string.Empty), collection);
                 }
             }
         }