великий done

ProjectDumpTruck/ProjectDumpTruck/CollectionGenericObject/MassiveGenericObjects.cs

@@ -95,6 +95,11 @@ public class MassiveGenericObjects<T> : ICollectionGenericObject<T>
 
     public int Insert(T obj, int position, IEqualityComparer<DrawningTruck?>? comparer = null)
     {
+        if (position < 0 || position >= Count)
+        {
+            throw new PositionOutOfCollectionException(position);
+        }
+
         for (int i = 0; i < Count; i++)
         {
             if (comparer.Equals((_collection[i] as DrawningTruck), (obj as DrawningTruck)))
@@ -103,11 +108,6 @@ public class MassiveGenericObjects<T> : ICollectionGenericObject<T>
             }
         }
 
-        if (position < 0 || position >= Count)
-        {
-            throw new PositionOutOfCollectionException(position);
-        }
-
         if (_collection[position] == null)
         {
             _collection[position] = obj;

ProjectDumpTruck/ProjectDumpTruck/Exceptions/ObjectAlreadyInCollectionException.cs

@@ -3,7 +3,8 @@ using System.Runtime.Serialization;
 
 namespace ProjectDumpTruck.Exceptions;
 
-public class ObjectAlreadyInCollectionException : ApplicationException
+[Serializable]
+internal class ObjectAlreadyInCollectionException : ApplicationException
 {
     public ObjectAlreadyInCollectionException(int i) : base("Такой объект уже есть в коллекции. Позиция " + i) { }
 

ProjectDumpTruck/ProjectDumpTruck/FormTruckCollection.cs

@@ -71,26 +71,36 @@ public partial class FormTruckCollection : Form
     /// </summary>
     /// <param name="truck"></param>
     private void SetTruck(DrawningTruck truck)
-    {
-        try
     {
         if (_company == null || truck == null)
         {
             return;
         }
 
+        try
+        {
+           
             if (_company + truck != -1)
             {
                 MessageBox.Show("Объект добавлен");
                 _logger.LogInformation($"Добавлен объект {truck.GetDataForSave()}");
                 pictureBox.Image = _company.Show();
             }
+            else
+            {
+                MessageBox.Show("Не удалось добавить объект");
+            }
         }
         catch (CollectionOverflowException ex)
         {
             MessageBox.Show(ex.Message);
             _logger.LogWarning($"Ошибка: {ex.Message}");
         }
+        catch (ArgumentException ex)
+        {
+            MessageBox.Show(ex.Message);
+            _logger.LogWarning($"Ошибка: {ex.Message}");
+        }
     }
 
     /// <summary>