Убрал лишние юзинги, сделал исключения на операции (добавить, удалить, получить) и сделал исключения на загрузку и выгрузку файлов

HoistingCrane/HoistingCrane/CollectionGenericObjects/AbstractCompany.cs

@@ -1,5 +1,4 @@
 using HoistingCrane.Drawning;
-using System;
 namespace HoistingCrane.CollectionGenericObjects
 {
     public abstract class AbstractCompany

HoistingCrane/HoistingCrane/CollectionGenericObjects/CollectionType.cs

@@ -1,5 +1,4 @@
-using System;
+namespace HoistingCrane.CollectionGenericObjects
-namespace HoistingCrane.CollectionGenericObjects
 {
     public enum CollectionType
     {

HoistingCrane/HoistingCrane/CollectionGenericObjects/Garage.cs

@@ -1,6 +1,5 @@
 using HoistingCrane.Drawning;
-using System;
+using HoistingCrane.Exceptions;
-using System.Collections.Specialized;
 
 namespace HoistingCrane.CollectionGenericObjects
 {
@@ -35,10 +34,13 @@ namespace HoistingCrane.CollectionGenericObjects
             for (int i = 0; i < (arr?.Count ?? 0); i++)
             {
                 if (arr?.Get(i) != null)
+                {
+                    try
                     {
                         arr?.Get(i)?.SetPictureSize(pictureWidth, pictureHeight);
                         arr?.Get(i)?.SetPosition(_placeSizeWidth * currentPosWidth + 25, _placeSizeHeight * currentPosHeight + 15);
-                    
+                    }
+                    catch (ObjectNotFoundException) { }
                 }
               
                 if (currentPosWidth > 0)

HoistingCrane/HoistingCrane/CollectionGenericObjects/ICollectionGenericObjects.cs

@@ -1,5 +1,4 @@
-using System;
+namespace HoistingCrane.CollectionGenericObjects
-namespace HoistingCrane.CollectionGenericObjects
 {
     public interface ICollectionGenericObjects<T>
         where T: class

HoistingCrane/HoistingCrane/CollectionGenericObjects/ListGenericObjects.cs

@@ -1,7 +1,4 @@
 using HoistingCrane.Exceptions;
-using System;
-using System.CodeDom.Compiler;
-
 namespace HoistingCrane.CollectionGenericObjects
 {
     public class ListGenericObjects<T> : ICollectionGenericObjects<T> where T : class
@@ -45,73 +42,35 @@ namespace HoistingCrane.CollectionGenericObjects
 
         public T? Get(int position)
         {
-            try
+            if (position >= Count || position < 0) throw new PositionOutOfCollectionException(position);
-            {
-                if (position >= Count || position < 0)
-                {
-                    throw new PositionOutOfCollectionException(position);
-                }
             return list[position];
         }
-            catch (PositionOutOfCollectionException ex)
-            {
-                MessageBox.Show(ex.Message);
-                return null;
-            }
-        }
 
         public int Insert(T obj)
         {
-            try
+            if (Count == _maxCount)
             {
-                if (Count == _maxCount) throw new CollectionOverflowException(Count);
+                throw new CollectionOverflowException(Count);
+            }
             list.Add(obj);
             return Count;
         }
-            catch(CollectionOverflowException ex)
-            {
-                MessageBox.Show(ex.Message);
-                return -1;
-            }
-        }
 
         public int Insert(T obj, int position)
         {
-            try
+            if (position < 0 || position >= Count) throw new PositionOutOfCollectionException(position);
-            {
-                if (position < 0 || position >= _maxCount) throw new PositionOutOfCollectionException(position);
             if (Count == _maxCount) throw new CollectionOverflowException(Count);
             list.Insert(position, obj);
             return position;
         }
-            catch (CollectionOverflowException ex)
-            {
-                MessageBox.Show(ex.Message);
-                return -1;
-            }
-            catch (PositionOutOfCollectionException ex)
-            {
-                MessageBox.Show(ex.Message);
-                return -1;
-            }
-            
-        }
 
         public T? Remove(int position)
         {
-            try
+            if (position < 0 || position >= list.Count) throw new PositionOutOfCollectionException(position);
-            {
-                if (position < 0 || position >= Count) throw new PositionOutOfCollectionException(position);
             T? temp = list[position];
             list.RemoveAt(position);
             return temp;
         }
-            catch (PositionOutOfCollectionException ex)
-            {
-                MessageBox.Show(ex.Message);
-                return null;
-            }
-        }
 
         public IEnumerable<T?> GetItems()
         {

HoistingCrane/HoistingCrane/CollectionGenericObjects/MassivGenericObjects.cs

@@ -1,6 +1,4 @@
 using HoistingCrane.Exceptions;
-using System;
-
 namespace HoistingCrane.CollectionGenericObjects
 {
     public class MassivGenericObjects<T> : ICollectionGenericObjects<T> where T : class
@@ -39,97 +37,64 @@ namespace HoistingCrane.CollectionGenericObjects
         public CollectionType GetCollectionType => CollectionType.Massive;
 
         public T? Get(int position)
-        {
-            try
         {
             if (position < 0 || position >= Count) throw new PositionOutOfCollectionException(position);
-                //if (arr[position] == null) throw new ObjectNotFoundException(position);
             return arr[position];
         }
-            catch (PositionOutOfCollectionException ex)
-            {
-                MessageBox.Show(ex.Message);
-                return null;
-            }
-            //catch (ObjectNotFoundException ex)
-            //{
-            //    MessageBox.Show(ex.Message);
-            //    return null;
-            //}
-        }
-
-        public IEnumerable<T?> GetItems()
-        {
-            for(int i = 0; i < arr.Length; i++)
-            {
-                yield return arr[i];
-            }
-        }
 
         public int Insert(T obj)
         {
-            try
+            int countObjectNotNull = 0;
+            for(int i = 0; i < Count; i++)
             {
-                if (arr.Count(x => x != null) == MaxCount) throw new CollectionOverflowException(Count);
+                if (arr[i] != null) countObjectNotNull += 1;
+            }
+            if(countObjectNotNull == MaxCount) throw new CollectionOverflowException(Count);
             return Insert(obj, 0);
         }
-            catch(CollectionOverflowException ex)
-            {
-                MessageBox.Show(ex.Message);
-                return -1;
-            }
-        }
-
         public int Insert(T obj, int position)
         {
-            try
+            if (position < 0 || position >= Count)
             {
-                if (position < 0 || position > Count) throw new PositionOutOfCollectionException(position);
+                throw new PositionOutOfCollectionException(position);
+            }
+            int copyPos = position - 1;
 
+            while (position < Count)
+            {
                 if (arr[position] == null)
                 {
                     arr[position] = obj;
                     return position;
                 }
-                else
+                position++;
+            }
+            while (copyPos > 0)
             {
-                    if (Insert(obj, position + 1) != -1)
+                if (arr[copyPos] == null)
                 {
-                        return position;
+                    arr[copyPos] = obj;
+                    return copyPos;
                 }
-                    if (Insert(obj, position - 1) != -1)
+                copyPos--;
-                    {
-                        return position;
-                    }
-                    return -1;
-                }
-            }
-            catch (PositionOutOfCollectionException ex)
-            {
-                MessageBox.Show(ex.Message);
-                return -1;
             }
+            throw new CollectionOverflowException(Count);
         }
 
         public T? Remove(int position)
         {
-            try
+            if (position < 0 || position >= Count) throw new PositionOutOfCollectionException(position);
-            {
-                if (position < 0 || position >= MaxCount) throw new PositionOutOfCollectionException(position);
             if (arr[position] == null) throw new ObjectNotFoundException(position);
             T? temp = arr[position];
             arr[position] = null;
             return temp;
         }
-            catch (ObjectNotFoundException ex)
+
+        public IEnumerable<T?> GetItems()
         {
-                MessageBox.Show(ex.Message);
+            for (int i = 0; i < arr.Length; i++)
-                return null;
-            }
-            catch (PositionOutOfCollectionException ex)
             {
-                MessageBox.Show(ex.Message);
+                yield return arr[i];
-                return null;
             }
         }
     }

HoistingCrane/HoistingCrane/CollectionGenericObjects/StorageCollection.cs

@@ -1,8 +1,6 @@
 using HoistingCrane.Drawning;
 using HoistingCrane.Exceptions;
-using System;
 using System.Text;
-
 namespace HoistingCrane.CollectionGenericObjects
 {
     public class StorageCollection<T> where T : DrawningTrackedVehicle
@@ -84,7 +82,7 @@ namespace HoistingCrane.CollectionGenericObjects
         {
             if (dict.Count == 0)
             {
-                throw new Exception("В хранилище отсутствуют коллекции для сохранения");
+                throw new InvalidOperationException("В хранилище отсутствуют коллекции для сохранения");
             }
 
             if (File.Exists(filename))
@@ -95,6 +93,7 @@ namespace HoistingCrane.CollectionGenericObjects
             using (StreamWriter writer = new StreamWriter(filename))
             {
                 writer.Write(_collectionKey);
+
                 foreach (KeyValuePair<string, ICollectionGenericObjects<T>> value in dict)
                 {
                     StringBuilder sb = new();
@@ -105,7 +104,6 @@ namespace HoistingCrane.CollectionGenericObjects
                     {
                         continue;
                     }
-
                     sb.Append(value.Key);
                     sb.Append(_separatorForKeyValue);
                     sb.Append(value.Value.GetCollectionType);
@@ -124,31 +122,30 @@ namespace HoistingCrane.CollectionGenericObjects
                     }
                     writer.Write(sb);
                 }
-
             }
         }
 
         /// <summary>
-        //    /// Загрузка информации по грузовикам в хранилище из файла
+        /// Загрузка информации по грузовикам в хранилище из файла
-        //    /// </summary>
+        /// </summary>
-        //    /// <param name="filename">Путь и имя файла</param>
+        /// <param name="filename"></param>
-        //    /// <returns>true - загрузка прошла успешно, false - ошибка при загрузке данных</returns>
+        /// <exception cref="Exception"></exception>
         public void LoadData(string filename)
         {
             if (!File.Exists(filename))
             {
-                throw new Exception("Файл не существует");
+                throw new FileNotFoundException("Файл не существует");
             }
             using (StreamReader fs = File.OpenText(filename))
             {
                 string str = fs.ReadLine();
                 if (str == null || str.Length == 0)
                 {
-                    throw new Exception("В файле не присутствуют данные");
+                    throw new InvalidOperationException("В файле не присутствуют данные");
                 }
                 if (!str.StartsWith(_collectionKey))
                 {
-                    throw new Exception("В файле неверные данные");
+                    throw new FormatException("В файле неверные данные");
                 }
                 dict.Clear();
                 string strs = "";
@@ -163,7 +160,7 @@ namespace HoistingCrane.CollectionGenericObjects
                     ICollectionGenericObjects<T>? collection = StorageCollection<T>.CreateCollection(collectionType);
                     if (collection == null)
                     {
-                        throw new Exception("Не удалось создать коллекцию");
+                        throw new InvalidOperationException("Не удалось определить тип коллекции:" + record[1]);
                     }
                     collection.MaxCount = Convert.ToInt32(record[2]);
                     string[] set = record[3].Split(_separatorItems, StringSplitOptions.RemoveEmptyEntries);
@@ -175,12 +172,12 @@ namespace HoistingCrane.CollectionGenericObjects
                             {
                                 if (collection.Insert(crane) == -1)
                                 {
-                                    throw new Exception("Объект не удалось добавить в коллекцию: " + record[3]);
+                                    throw new InvalidOperationException("Объект не удалось добавить в коллекцию: " + record[3]);
                                 }
                             }
                             catch(CollectionOverflowException ex)
                             {
-                                throw new Exception("Коллекция переполнена");
+                                throw new CollectionOverflowException("Коллекция переполнена", ex);
                             }
                         }
                     }

HoistingCrane/HoistingCrane/Exceptions/CollectionOverflowException.cs

@@ -1,10 +1,4 @@
-using System;
+using System.Runtime.Serialization;
-using System.Collections.Generic;
-using System.Linq;
-using System.Runtime.Serialization;
-using System.Text;
-using System.Threading.Tasks;
-
 namespace HoistingCrane.Exceptions
 {
     [Serializable]

HoistingCrane/HoistingCrane/Exceptions/ObjectNotFoundException.cs

@@ -1,10 +1,4 @@
-using System;
+using System.Runtime.Serialization;
-using System.Collections.Generic;
-using System.Linq;
-using System.Runtime.Serialization;
-using System.Text;
-using System.Threading.Tasks;
-
 namespace HoistingCrane.Exceptions
 {
     [Serializable]

HoistingCrane/HoistingCrane/Exceptions/PositionOutOfCollectionException.cs

@@ -1,10 +1,4 @@
-using System;
+using System.Runtime.Serialization;
-using System.Collections.Generic;
-using System.Linq;
-using System.Runtime.Serialization;
-using System.Text;
-using System.Threading.Tasks;
-
 namespace HoistingCrane.Exceptions
 {
     [Serializable]

HoistingCrane/HoistingCrane/FormCarCollection.cs

@@ -1,17 +1,7 @@
 using HoistingCrane.CollectionGenericObjects;
 using HoistingCrane.Drawning;
-using HoistingCrane.Entities;
+using HoistingCrane.Exceptions;
 using Microsoft.Extensions.Logging;
-using System;
-using System.Collections.Generic;
-using System.ComponentModel;
-using System.Data;
-using System.Drawing;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
-using System.Windows.Forms;
-
 namespace HoistingCrane
 {
     public partial class FormCarCollection : Form
@@ -34,46 +24,13 @@ namespace HoistingCrane
         {
             panelCompanyTool.Enabled = false;
         }
-        private void CreateObject(string type)
-        {
-            DrawningTrackedVehicle drawning;
-            if (_company == null) return;
-            Random rand = new();
-            switch (type)
-            {
-                case nameof(DrawningHoistingCrane):
-                    drawning = new DrawningHoistingCrane(rand.Next(100, 300), rand.Next(1000, 3000), GetColor(rand), GetColor(rand), true, true);
-                    break;
-
-                case nameof(DrawningTrackedVehicle):
-                    drawning = new DrawningTrackedVehicle(rand.Next(100, 300), rand.Next(1000, 3000), GetColor(rand));
-                    break;
-                default:
-                    return;
-            }
-            if ((_company + drawning) != -1)
-            {
-                MessageBox.Show("Объект добавлен");
-                pictureBox.Image = _company.Show();
-            }
-        }
-        private static Color GetColor(Random random)
-        {
-            Color color = Color.FromArgb(random.Next(0, 256), random.Next(0, 256), random.Next(0, 256));
-            ColorDialog dialog = new();
-            if (dialog.ShowDialog() == DialogResult.OK)
-            {
-                color = dialog.Color;
-            }
-            return color;
-        }
         private void buttonCreateHoistingCrane_Click(object sender, EventArgs e)
         {
             FormCarConfig form = new();
             form.Show();
-            form.AddEvent(SetCar);
+            form.AddEvent(SetCrane);
         }
-        private void SetCar(DrawningTrackedVehicle drawningTrackedVehicle)
+        private void SetCrane(DrawningTrackedVehicle drawningTrackedVehicle)
         {
             if (_company == null || drawningTrackedVehicle == null) return;
             try
@@ -86,37 +43,53 @@ namespace HoistingCrane
                 }
                 else
                 {
-                    throw new Exception("Не удалось добавить объект. Заполнено максимальное количество ячеек");
+                    MessageBox.Show("Не удалось добавить объект");
+                    logger.LogInformation("Не удалось добавить корабль {ship} в коллекцию", drawningTrackedVehicle.GetType().Name);
                 }
             }
-            catch (Exception ex)
+            catch (CollectionOverflowException ex)
             {
-                logger.LogInformation(ex.Message);
+                MessageBox.Show("Ошибка переполнения коллекции");
-                return;
+                logger.LogError("Ошибка: {Message}", ex.Message);
             }
             
         }
 
         private void buttonDeleteCar_Click(object sender, EventArgs e)
         {
-            //if (MessageBox.Show("Удалить объект?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No) return;
+            if (string.IsNullOrEmpty(maskedTextBox.Text) || _company == null)
-            //if (string.IsNullOrEmpty(maskedTextBox.Text) || _company == null) return;
+            {
-            //try
+                return;
-            //{
+            }
-            //    int pos = Convert.ToInt32(maskedTextBox.Text);
+            if (MessageBox.Show("Удалить объект?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No)
-            //    if ((_company - pos) != null)
+            {
-            //    {
+                return;
-            //        MessageBox.Show("Объект удален!");
+            }
-            //        pictureBox.Image = _company.Show();
+            try
-            //        logger.LogInformation("Удален объект по позиции: {pos} ", pos);
+            {
-            //    }
+                int pos = Convert.ToInt32(maskedTextBox.Text);
-            //    else throw new Exception();
+                if ((_company - pos) != null)
-            //}
+                {
-            //catch(Exception ex)
+                    MessageBox.Show("Объект удален!");
-            //{
+                    pictureBox.Image = _company.Show();
-            //    logger.LogInformation("Не удалось удалить объект по позиции: {pos}", Convert.ToInt32(maskedTextBox.Text));
+                    logger.LogInformation("Удаление авто по индексу {pos}", pos);
-            //    return;
+                }
-            //}
+                else
+                {
+                    MessageBox.Show("Не удалось удалить объект");
+                    logger.LogInformation("Не удалось удалить авто из коллекции по индексу {pos}", pos);
+                }
+            }
+            catch (ObjectNotFoundException ex)
+            {
+                MessageBox.Show("Ошибка: отсутствует объект");
+                logger.LogError("Ошибка: {Message}", ex.Message);
+            }
+            catch (PositionOutOfCollectionException ex)
+            {
+                MessageBox.Show("Ошибка: неправильная позиция");
+                logger.LogError("Ошибка: {Message}", ex.Message);
+            }
         }
         private void buttonRefresh_Click(object sender, EventArgs e)
         {
@@ -162,20 +135,20 @@ namespace HoistingCrane
         {
             if (string.IsNullOrEmpty(textBoxCollectionName.Text) || (!radioButtonList.Checked && !radioButtonMassive.Checked))
             {
-                MessageBox.Show("Не все данные заполнены", "Ошибка",
+                MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error);
-                MessageBoxButtons.OK, MessageBoxIcon.Error);
+                logger.LogInformation("Не удалось добавить коллекцию: не все данные заполнены");
                 return;
             }
             CollectionType collectionType = CollectionType.None;
             if (radioButtonMassive.Checked)
             {
                 collectionType = CollectionType.Massive;
-                logger.LogInformation("Создана коллекция типа: {name}", collectionType);
+                logger.LogInformation("Создана коллекция типа: {name}, название: {name}", collectionType, textBoxCollectionName.Text);
             }
             else if (radioButtonList.Checked)
             {
                 collectionType = CollectionType.List;
-                logger.LogInformation("Создана коллекция типа: {name}", collectionType);
+                logger.LogInformation("Создана коллекция типа: {name}, название: {name}", collectionType, textBoxCollectionName.Text);
             }
             _storageCollection.AddCollection(textBoxCollectionName.Text,collectionType);RerfreshListBoxItems();
         }
@@ -190,7 +163,7 @@ namespace HoistingCrane
                 return;
             _storageCollection.DelCollection(listBoxCollection.SelectedItem.ToString());
             RerfreshListBoxItems();
-            logger.LogInformation("Коллекция успешно удалена");
+            logger.LogInformation("Коллекция {name} успешно удалена", listBoxCollection.SelectedItem.ToString());
         }
         private void buttonCreateCompany_Click(object sender, EventArgs e)
         {
@@ -233,7 +206,7 @@ namespace HoistingCrane
                     MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
                     logger.LogInformation("Сохранение в файл: {filename}", saveFileDialog.FileName);
                 }
-                catch (FileNotFoundException ex)
+                catch (Exception ex)
                 {
                     MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
                     logger.LogError("Ошибка: {Message}", ex.Message);
@@ -253,11 +226,11 @@ namespace HoistingCrane
                 try
                 {
                     _storageCollection.LoadData(openFileDialog.FileName);
-                    MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
                     RerfreshListBoxItems();
+                    MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
                     logger.LogInformation("Загрузка в файл: {filename}", saveFileDialog.FileName);
                 }
-                catch(FileNotFoundException ex)
+                catch(Exception ex)
                 {
                     MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
                     logger.LogError("Ошибка: {Message}", ex.Message);