Готовая лаба 6

HoistingCrane/HoistingCrane/CollectionGenericObjects/AbstractCompany.cs

@@ -1,59 +1,82 @@
-using HoistingCrane.Drawning;
+using HoistingCrane.CollectionGenericObjects;
-namespace HoistingCrane.CollectionGenericObjects
+using HoistingCrane.Drawning;
+
+public abstract class AbstractCompany
 {
-    public abstract class AbstractCompany
-    {
     /// <summary>
-        /// Ширина ячейки гаража
+    /// Размер места (ширина)
     /// </summary>
-        protected readonly int _placeSizeWidth = 150;
+    protected readonly int _placeSizeWidth = 180;
+
     /// <summary>
-        /// Высота ячейки гаража
+    /// Размер места (высота)
     /// </summary>
-        protected readonly int _placeSizeHeight = 90;
+    protected readonly int _placeSizeHeight = 100;
+
     /// <summary>
     /// Ширина окна
     /// </summary>
-        protected readonly int pictureWidth;
+    protected readonly int _pictureWidth;
+
     /// <summary>
     /// Высота окна
     /// </summary>
-        protected readonly int pictureHeight;
+    protected readonly int _pictureHeight;
+
     /// <summary>
-        /// Коллекция автомобилей
+    /// Коллекция военных кораблей
     /// </summary>
-        protected ICollectionGenericObjects<DrawningTrackedVehicle>? arr = null;
+    protected ICollectionGenericObjects<DrawningTrackedVehicle>? _collection = null;
+
     /// <summary>
-        /// Максимальное количество гаражей
+    /// Вычисление максимального количества элементов, которые можно разместить в окне
     /// </summary>
-        private int GetMaxCount
+    private int GetMaxCount => _pictureWidth * _pictureHeight / (_placeSizeWidth * _placeSizeHeight);
+
+    /// <summary>
+    /// Конструктор
+    /// </summary>
+    /// <param name="picWidth">Ширина окна</param>
+    /// <param name="picHeight">Высота окна</param>
+    /// <param name="collection">Коллекция военных кораблей</param>
+    public AbstractCompany(int picWidth, int picHeight, ICollectionGenericObjects<DrawningTrackedVehicle> collection)
     {
-            get
+        _pictureWidth = picWidth;
-            {
+        _pictureHeight = picHeight;
-                return (pictureWidth * pictureHeight) / (_placeSizeHeight * _placeSizeWidth);
+        _collection = collection;
-            }
+        _collection.MaxCount = GetMaxCount;
-        }
-        public AbstractCompany(int picWidth, int picHeight, ICollectionGenericObjects<DrawningTrackedVehicle> array)
-        {
-            pictureWidth = picWidth;
-            pictureHeight = picHeight;
-            arr = array;
-            arr.MaxCount = GetMaxCount;
     }
 
-        public static int operator +(AbstractCompany company, DrawningTrackedVehicle car)
+    /// <summary>
+    /// Перегрузка оператора сложения для класса
+    /// </summary>
+    /// <param name="company">Компания</param>
+    /// <param name="warship">Добавляемый объект</param>
+    /// <returns></returns>
+    public static int operator +(AbstractCompany company, DrawningTrackedVehicle crane)
     {
-            return company.arr?.Insert(car) ?? -1;
+        return company._collection.Insert(crane);
     }
+
+    /// <summary>
+    /// Перегрузка оператора удаления для класса
+    /// </summary>
+    /// <param name="company">Компания</param>
+    /// <param name="position">Номер удаляемого объекта</param>
+    /// <returns></returns>
     public static DrawningTrackedVehicle operator -(AbstractCompany company, int position)
     {
-            return company.arr?.Remove(position);
+        return company._collection.Remove(position);
     }
 
+    /// <summary>
+    /// Получение случайного объекта из коллекции
+    /// </summary>
+    /// <returns></returns>
     public DrawningTrackedVehicle? GetRandomObject()
     {
         Random rnd = new();
-            return arr?.Get(rnd.Next(GetMaxCount));
+        return _collection?.Get(rnd.Next(GetMaxCount));
     }
 
     /// <summary>
@@ -62,18 +85,18 @@ namespace HoistingCrane.CollectionGenericObjects
     /// <returns></returns>
     public Bitmap? Show()
     {
-            Bitmap bitmap = new(pictureWidth, pictureHeight);
+        Bitmap bitmap = new(_pictureWidth, _pictureHeight);
         Graphics graphics = Graphics.FromImage(bitmap);
         DrawBackgound(graphics);
-
         SetObjectsPosition();
-            for (int i = 0; i < (arr?.Count ?? 0); i++)
+        for (int i = 0; i < (_collection?.Count ?? 0); ++i)
         {
-                DrawningTrackedVehicle? obj = arr?.Get(i);
+            DrawningTrackedVehicle? obj = _collection?.Get(i);
             obj?.DrawTransport(graphics);
         }
         return bitmap;
     }
+
     /// <summary>
     /// Вывод заднего фона
     /// </summary>
@@ -84,5 +107,4 @@ namespace HoistingCrane.CollectionGenericObjects
     /// Расстановка объектов
     /// </summary>
     protected abstract void SetObjectsPosition();
-    }
 }

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

@@ -8,8 +8,8 @@ namespace HoistingCrane.CollectionGenericObjects
         }
         protected override void DrawBackgound(Graphics g)
         {
-            int width = pictureWidth / _placeSizeWidth;
+            int width = _pictureWidth / _placeSizeWidth;
-            int height = pictureHeight / _placeSizeHeight;
+            int height = _pictureHeight / _placeSizeHeight;
             Pen pen = new(Color.Black, 3);
             for (int i = 0; i < width; i++)
             {
@@ -22,18 +22,18 @@ namespace HoistingCrane.CollectionGenericObjects
         }
         protected override void SetObjectsPosition()
         {
-            int countWidth = pictureWidth / _placeSizeWidth;
+            int countWidth = _pictureWidth / _placeSizeWidth;
-            int countHeight = pictureHeight / _placeSizeHeight;
+            int countHeight = _pictureHeight / _placeSizeHeight;
 
             int currentPosWidth = countWidth - 1;
             int currentPosHeight = countHeight - 1;
 
-            for (int i = 0; i < (arr?.Count ?? 0); i++)
+            for (int i = 0; i < (_collection?.Count ?? 0); i++)
             {
-                if (arr?.Get(i) != null)
+                if (_collection?.Get(i) != null)
                 {
-                    arr?.Get(i)?.SetPictureSize(pictureWidth, pictureHeight);
+                    _collection?.Get(i)?.SetPictureSize(_pictureWidth, _pictureHeight);
-                    arr?.Get(i)?.SetPosition(_placeSizeWidth * currentPosWidth + 25, _placeSizeHeight * currentPosHeight + 15);
+                    _collection?.Get(i)?.SetPosition(_placeSizeWidth * currentPosWidth + 25, _placeSizeHeight * currentPosHeight + 15);
                 }
 
                 if (currentPosWidth > 0)
@@ -47,7 +47,6 @@ namespace HoistingCrane.CollectionGenericObjects
                 {
                     break;
                 }
-
             }
         }
 

HoistingCrane/HoistingCrane/CollectionGenericObjects/ICollectionGenericObjects.cs

@@ -1,49 +1,55 @@
-namespace HoistingCrane.CollectionGenericObjects
+using HoistingCrane.CollectionGenericObjects;
+
+public interface ICollectionGenericObjects<T>
+    where T : class
 {
-    public interface ICollectionGenericObjects<T>
-        where T: class
-    {
     /// <summary>
-        /// Кол-во объектов в коллекции
+    /// Количество объектов в коллекции
     /// </summary>
     int Count { get; }
-        /// /// <summary>
+
+    /// <summary>
     /// Установка максимального количества элементов
     /// </summary>
     int MaxCount { get; set; }
+
     /// <summary>
-        /// Добавление элемента в коллекцию
+    /// Добавление объекта в коллекцию
     /// </summary>
-        /// <param name="obj"></param>
+    /// <param name="obj">Добавляемый объект</param>
-        /// <returns></returns>
+    /// <returns>true - вставка прошла удачно, false - вставка не удалась</returns>
     int Insert(T obj);
+
     /// <summary>
-        /// Добавление элемента в коллекцию на определенную позицию
+    /// Добавление объекта в коллекцию на конкретную позицию
     /// </summary>
-        /// <param name="obj"></param>
+    /// <param name="obj">Добавляемый объект</param>
-        /// <param name="position"></param>
+    /// <param name="position">Позиция</param>
-        /// <returns></returns>
+    /// <returns>true - вставка прошла удачно, false - вставка не удалась</returns>
     int Insert(T obj, int position);
+
     /// <summary>
-        /// Удаление элемента из коллекции по его позиции
+    /// Удаление объекта из коллекции с конкретной позиции
     /// </summary>
-        /// <param name="position"></param>
+    /// <param name="position">Позиция</param>
-        /// <returns></returns>
+    /// <returns>true - удаление прошло удачно, false - удаление не удалось</returns>
     T? Remove(int position);
+
     /// <summary>
-        /// Получение объекта коллекции
+    /// Получение объекта по позиции
     /// </summary>
-        /// <param name="position"></param>
+    /// <param name="position">Позиция</param>
-        /// <returns></returns>
+    /// <returns>Объект</returns>
     T? Get(int position);
+
     /// <summary>
     /// Получение типа коллекции
     /// </summary>
     CollectionType GetCollectionType { get; }
+
     /// <summary>
     /// Получение объектов коллекции по одному
     /// </summary>
-        /// <returns></returns>
+    /// <returns>Поэлементый вывод элементов коллекции</returns>
     IEnumerable<T?> GetItems();
-    }
 }

HoistingCrane/HoistingCrane/CollectionGenericObjects/ListGenericObjects.cs

@@ -2,36 +2,30 @@
 using System.CodeDom.Compiler;
 using System.Windows.Forms.VisualStyles;
 
-namespace HoistingCrane.CollectionGenericObjects
+namespace HoistingCrane.CollectionGenericObjects;
+
+public class ListGenericObjects<T> : ICollectionGenericObjects<T>
+where T : class
 {
-    public class ListGenericObjects<T> : ICollectionGenericObjects<T> where T : class
-    {
     /// <summary>
     /// Список объектов, которые храним
     /// </summary>
-        readonly List<T> list;
+    private readonly List<T?> _collection;
-        /// <summary>
+
-        /// Конструктор
-        /// </summary>
-        public ListGenericObjects()
-        {
-            list = new List<T>();
-        }
     /// <summary>
     /// Максимально допустимое число объектов в списке
     /// </summary>
     private int _maxCount;
-        public int Count
+
-        {
+    public int Count => _collection.Count;
-            get { return list.Count; }
-        }
 
     public int MaxCount
     {
         get
         {
-                return list.Count;
+            return Count;
         }
+
         set
         {
             if (value > 0)
@@ -43,47 +37,68 @@ namespace HoistingCrane.CollectionGenericObjects
 
     public CollectionType GetCollectionType => CollectionType.List;
 
+    /// <summary>
+    /// Конструктор
+    /// </summary>
+    public ListGenericObjects()
+    {
+        _collection = new();
+    }
+
     public T? Get(int position)
     {
-            if (position >= Count || position < 0) return null;
+        // Проверка позиции
-            return list[position];
+        if (position >= Count || position < 0)
+        {
+            return null;
         }
+        return _collection[position];
+    }
+
     public int Insert(T obj)
     {
+        // Проверка, что не превышено максимальное количество элементов
         if (Count == _maxCount)
         {
             return -1;
         }
-            list.Add(obj);
+        _collection.Add(obj);
         return Count;
     }
 
     public int Insert(T obj, int position)
     {
-            if (position < 0 || position >= Count || Count == _maxCount)
+        // Проверка, что не превышено максимальное количество элементов
+        if (Count == _maxCount)
         {
             return -1;
         }
-            list.Insert(position, obj);
+        // Проверка позиции
+        if (position >= Count || position < 0)
+        {
+            return -1;
+        }
+        _collection.Insert(position, obj);
         return position;
     }
+
     public T? Remove(int position)
     {
-            if(position >= 0 && position < list.Count)
+        // Проверка позиции
+        if (position >= Count || position < 0)
         {
-                T? temp = list[position];
-                list.RemoveAt(position);
-                return temp;
-            }
             return null;
         }
+        T? obj = _collection[position];
+        _collection.RemoveAt(position);
+        return obj;
+    }
 
     public IEnumerable<T?> GetItems()
     {
-            for(int i = 0; i < list.Count; i++)
+        for (int i = 0; i < Count; ++i)
         {
-                yield return list[i];
+            yield return _collection[i];
-            }
         }
     }
 }

HoistingCrane/HoistingCrane/CollectionGenericObjects/MassivGenericObjects.cs

@@ -1,35 +1,33 @@
-using System;
+namespace HoistingCrane.CollectionGenericObjects;
 
-namespace HoistingCrane.CollectionGenericObjects
+public class MassiveGenericObjects<T> : ICollectionGenericObjects<T>
+where T : class
 {
-    public class MassivGenericObjects<T> : ICollectionGenericObjects<T> where T : class
+    /// <summary>
-    {
+    /// Массив объектов, которые храним
-        private T?[] arr;
+    /// </summary>
-        public MassivGenericObjects()
+    private T?[] _collection;
-        {
+
-            arr = Array.Empty<T?>();
+    public int Count => _collection.Length;
-        }
+
-        public int Count
-        {
-            get { return arr.Length; } 
-        }
     public int MaxCount
     {
         get
         {
-                return arr.Length;
+            return _collection.Length;
         }
+
         set
         {
             if (value > 0)
             {
-                    if (arr.Length > 0)
+                if (_collection.Length > 0)
                 {
-                        Array.Resize(ref arr, value);
+                    Array.Resize(ref _collection, value);
                 }
                 else
                 {
-                        arr = new T?[value];
+                    _collection = new T?[value];
                 }
             }
         }
@@ -37,69 +35,82 @@ namespace HoistingCrane.CollectionGenericObjects
 
     public CollectionType GetCollectionType => CollectionType.Massive;
 
-        public T? Get(int position)
+    /// <summary>
+    /// Конструктор
+    /// </summary>
+    public MassiveGenericObjects()
     {
-            if (position >= 0 && position < arr.Length)
+        _collection = Array.Empty<T?>();
-            {
-                return arr[position];
-            }
-            return null;
     }
 
-        public IEnumerable<T?> GetItems()
+    public T? Get(int position)
     {
-            for(int i = 0; i < arr.Length; i++)
+        // Проверка позиции
+        if (position < 0 || position >= Count)
         {
-                yield return arr[i];
+            return null;
         }
+        return _collection[position];
     }
 
     public int Insert(T obj)
     {
+        // Вставка в свободное место набора
         return Insert(obj, 0);
     }
 
     public int Insert(T obj, int position)
     {
-
+        // Проверка позиции
         if (position < 0 || position >= Count)
         {
             return -1;
         }
-
+        // Проверка, что элемент массива по этой позиции пустой
-            int copyPos = position - 1;
+        if (_collection[position] == null)
-
-            while (position < Count)
         {
-                if (arr[position] == null)
+            _collection[position] = obj;
-                {
-                    arr[position] = obj;
             return position;
         }
-                position++;
+        //Свободное место после этой позиции
-            }
+        for (int i = position + 1; i < Count; i++)
-            while (copyPos > 0)
         {
-                if (arr[copyPos] == null)
+            if (_collection[i] == null)
             {
-                    arr[copyPos] = obj;
+                _collection[i] = obj;
-                    return copyPos;
+                return i;
+            }
+        }
+        //Свободное место до этой позиции
+        for (int i = position - 1; i >= 0; i--)
+        {
+            if (_collection[i] == null)
+            {
+                _collection[i] = obj;
+                return i;
             }
-                copyPos--;
         }
-
         return -1;
     }
 
     public T? Remove(int position)
     {
-            if (position >= 0 && position < Count)
+        // Проверка позиции и наличия объекта
+        if (position < 0 || position >= Count || _collection[position] == null)
         {
-                T? temp = arr[position];
-                arr[position] = null;
-                return temp;
-            }
             return null;
         }
+        // Удаление объекта из массива
+        T? obj = _collection[position];
+        _collection[position] = null;
+        return obj;
+    }
+
+    public IEnumerable<T?> GetItems()
+    {
+        for (int i = 0; i < _collection.Length; ++i)
+        {
+            yield return _collection[i];
+        }
     }
 }

HoistingCrane/HoistingCrane/CollectionGenericObjects/StorageCollection.cs

@@ -1,64 +1,76 @@
 using HoistingCrane.Drawning;
-using System;
+namespace HoistingCrane.CollectionGenericObjects;
-using System.Configuration;
-using System.Text;
 
-namespace HoistingCrane.CollectionGenericObjects
+public class StorageCollection<T>
+where T : DrawningTrackedVehicle
 {
-    public class StorageCollection<T> where T : DrawningTrackedVehicle
-    {
     /// <summary>
     /// Словарь (хранилище) с коллекциями
     /// </summary>
-        readonly Dictionary<string, ICollectionGenericObjects<T>> dict;
+    readonly Dictionary<string, ICollectionGenericObjects<T>> _storages;
+
     /// <summary>
     /// Возвращение списка названий коллекций
     /// </summary>
-        public List<string> Keys => dict.Keys.ToList();
+    public List<string> Keys => _storages.Keys.ToList();
+
     /// <summary>
     /// Ключевое слово, с которого должен начинаться файл
     /// </summary>
     private readonly string _collectionKey = "CollectionsStorage";
+
     /// <summary>
     /// Разделитель для записи ключа и значения элемента словаря
     /// </summary>
     private readonly string _separatorForKeyValue = "|";
+
     /// <summary>
     /// Разделитель для записей коллекции данных в файл
     /// </summary>
     private readonly string _separatorItems = ";";
+
     /// <summary>
     /// Конструктор
     /// </summary>
     public StorageCollection()
     {
-            dict = new Dictionary<string, ICollectionGenericObjects<T>>();
+        _storages = new Dictionary<string, ICollectionGenericObjects<T>>();
     }
+
     /// <summary>
     /// Добавление коллекции в хранилище
     /// </summary>
     /// <param name="name">Название коллекции</param>
-        /// <param name="collectionType">тип коллекции</param>
+    /// <param name="collectionType">Тип коллекции</param>
     public void AddCollection(string name, CollectionType collectionType)
     {
-            if (dict.ContainsKey(name)) return;
+        // Проверка, что name не пустой и нет в словаре записи с таким ключом
-            if (collectionType == CollectionType.None) return;
+        if (string.IsNullOrEmpty(name) || _storages.ContainsKey(name) || collectionType == CollectionType.None)
-            else if (collectionType == CollectionType.Massive)
+        {
-                dict[name] = new MassivGenericObjects<T>();
+            return;
-            else if (collectionType == CollectionType.List)
-                dict[name] = new ListGenericObjects<T>();
         }
+        if (collectionType == CollectionType.Massive)
+        {
+            _storages[name] = new MassiveGenericObjects<T>();
+        }
+        if (collectionType == CollectionType.List)
+        {
+            _storages[name] = new ListGenericObjects<T>();
+        }
+    }
+
     /// <summary>
     /// Удаление коллекции
     /// </summary>
     /// <param name="name">Название коллекции</param>
     public void DelCollection(string name)
     {
-            if (Keys.Contains(name))
+        if (_storages.ContainsKey(name))
         {
-                dict.Remove(name);
+            _storages.Remove(name);
         }
     }
+
     /// <summary>
     /// Доступ к коллекции
     /// </summary>
@@ -68,19 +80,25 @@ namespace HoistingCrane.CollectionGenericObjects
     {
         get
         {
-                if (dict.ContainsKey(name))
+            if (_storages.ContainsKey(name))
-                    return dict[name];
+            {
+                return _storages[name];
+            }
+            else
+            {
                 return null;
             }
         }
+    }
+
     /// <summary>
-        /// Сохранение информации по автомобилям в хранилище в файл
+    /// Сохранение информации по военным кораблям в хранилище в файл
     /// </summary>
     /// <param name="filename">Путь и имя файла</param>
     /// <returns>true - сохранение прошло успешно, false - ошибка при сохранении данных</returns>
     public bool SaveData(string filename)
     {
-            if (dict.Count == 0)
+        if (_storages.Count == 0)
         {
             return false;
         }
@@ -93,24 +111,20 @@ namespace HoistingCrane.CollectionGenericObjects
         using (StreamWriter writer = new StreamWriter(filename))
         {
             writer.Write(_collectionKey);
-
+            foreach (KeyValuePair<string, ICollectionGenericObjects<T>> value in _storages)
-                foreach (KeyValuePair<string, ICollectionGenericObjects<T>> value in dict)
             {
-                    StringBuilder sb = new();
+                writer.Write(Environment.NewLine);
-                    sb.Append(Environment.NewLine);
-
                 // не сохраняем пустые коллекции
                 if (value.Value.Count == 0)
                 {
                     continue;
                 }
-
+                writer.Write(value.Key);
-                    sb.Append(value.Key);
+                writer.Write(_separatorForKeyValue);
-                    sb.Append(_separatorForKeyValue);
+                writer.Write(value.Value.GetCollectionType);
-                    sb.Append(value.Value.GetCollectionType);
+                writer.Write(_separatorForKeyValue);
-                    sb.Append(_separatorForKeyValue);
+                writer.Write(value.Value.MaxCount);
-                    sb.Append(value.Value.MaxCount);
+                writer.Write(_separatorForKeyValue);
-                    sb.Append(_separatorForKeyValue);
                 foreach (T? item in value.Value.GetItems())
                 {
                     string data = item?.GetDataForSave() ?? string.Empty;
@@ -118,69 +132,75 @@ namespace HoistingCrane.CollectionGenericObjects
                     {
                         continue;
                     }
-                        sb.Append(data);
+                    writer.Write(data);
-                        sb.Append(_separatorItems);
+                    writer.Write(_separatorItems);
                 }
-                    writer.Write(sb);
             }
         }
         return true;
     }
 
     /// <summary>
-        //    /// Загрузка информации по грузовикам в хранилище из файла
+    /// Загрузка информации по военным кораблям в хранилище из файла
-        //    /// </summary>
+    /// </summary>
-        //    /// <param name="filename">Путь и имя файла</param>
+    /// <param name="filename">Путь и имя файла</param>
-        //    /// <returns>true - загрузка прошла успешно, false - ошибка при загрузке данных</returns>
+    /// <returns>true - загрузка прошла успешно, false - ошибка при загрузке данных</returns>
     public bool LoadData(string filename)
     {
         if (!File.Exists(filename))
         {
             return false;
         }
-            using (StreamReader fs = File.OpenText(filename))
+
+        using (StreamReader read = File.OpenText(filename))
         {
-                string str = fs.ReadLine();
+            string str = read.ReadLine();
             if (str == null || str.Length == 0)
             {
                 return false;
             }
+
             if (!str.StartsWith(_collectionKey))
             {
                 return false;
             }
-                dict.Clear();
+
+            _storages.Clear();
             string strs = "";
-                while ((strs = fs.ReadLine()) != null)
+            while ((strs = read.ReadLine()) != null)
             {
                 string[] record = strs.Split(_separatorForKeyValue, StringSplitOptions.RemoveEmptyEntries);
                 if (record.Length != 4)
                 {
                     continue;
                 }
+
                 CollectionType collectionType = (CollectionType)Enum.Parse(typeof(CollectionType), record[1]);
                 ICollectionGenericObjects<T>? collection = StorageCollection<T>.CreateCollection(collectionType);
                 if (collection == null)
                 {
                     return false;
                 }
+
                 collection.MaxCount = Convert.ToInt32(record[2]);
+
                 string[] set = record[3].Split(_separatorItems, StringSplitOptions.RemoveEmptyEntries);
                 foreach (string elem in set)
                 {
-                        if (elem?.CreateDrawningTrackedVehicle() is T truck)
+                    if (elem?.CreateDrawningTrackedVehicle() is T warship)
                     {
-                            if (collection.Insert(truck) == -1)
+                        if (collection.Insert(warship) == -1)
                         {
                             return false;
                         }
                     }
                 }
-                    dict.Add(record[0], collection);
+                _storages.Add(record[0], collection);
+            }
         }
         return true;
     }
-        }
+
     /// <summary>
     /// Создание коллекции по типу
     /// </summary>
@@ -190,11 +210,10 @@ namespace HoistingCrane.CollectionGenericObjects
     {
         return collectionType switch
         {
-                CollectionType.Massive => new MassivGenericObjects<T>(),
+            CollectionType.Massive => new MassiveGenericObjects<T>(),
             CollectionType.List => new ListGenericObjects<T>(),
             _ => null,
         };
     }
-
-    }
 }
+

HoistingCrane/HoistingCrane/FormCarCollection.cs

@@ -1,15 +1,6 @@
 using HoistingCrane.CollectionGenericObjects;
 using HoistingCrane.Drawning;
-using System;
+using Microsoft.Extensions.Logging;
-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
@@ -17,6 +8,10 @@ namespace HoistingCrane
         private AbstractCompany? _company;
 
         private readonly StorageCollection<DrawningTrackedVehicle> _storageCollection;
+        /// <summary>
+        /// Логгер
+        /// </summary>
+        private readonly ILogger logger;
         public FormCarCollection()
         {
             InitializeComponent();
@@ -27,55 +22,15 @@ 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();
-            }
-            else
-            {
-                MessageBox.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)
+            if (_company == null || drawningTrackedVehicle == null) return;
-            {
-                return;
-            }
           
                 if (_company + drawningTrackedVehicle != -1)
                 {
@@ -90,7 +45,6 @@ namespace HoistingCrane
 
         private void buttonDeleteCar_Click(object sender, EventArgs e)
         {
-
             if (string.IsNullOrEmpty(maskedTextBox.Text) || _company == null)
             {
                 return;
@@ -109,6 +63,7 @@ namespace HoistingCrane
                 {
                     MessageBox.Show("Не удалось удалить объект");
                 }
+            
         }
         private void buttonRefresh_Click(object sender, EventArgs e)
         {
@@ -153,8 +108,7 @@ namespace HoistingCrane
         {
             if (string.IsNullOrEmpty(textBoxCollectionName.Text) || (!radioButtonList.Checked && !radioButtonMassive.Checked))
             {
-                MessageBox.Show("Не все данные заполнены", "Ошибка",
+                MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error);
-                MessageBoxButtons.OK, MessageBoxIcon.Error);
                 return;
             }
             CollectionType collectionType = CollectionType.None;
@@ -166,9 +120,7 @@ namespace HoistingCrane
             {
                 collectionType = CollectionType.List;
             }
-            _storageCollection.AddCollection(textBoxCollectionName.Text,
+            _storageCollection.AddCollection(textBoxCollectionName.Text,collectionType);RerfreshListBoxItems();
-            collectionType);
-            RerfreshListBoxItems();
         }
         private void buttonDeleteCollection_Click(object sender, EventArgs e)
         {
@@ -181,9 +133,11 @@ namespace HoistingCrane
                 return;
             _storageCollection.DelCollection(listBoxCollection.SelectedItem.ToString());
             RerfreshListBoxItems();
+            
         }
         private void buttonCreateCompany_Click(object sender, EventArgs e)
         {
+            
             if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItem == null) 
             {
                 MessageBox.Show("Коллекция не выбрана");
@@ -194,7 +148,7 @@ namespace HoistingCrane
             {
                 MessageBox.Show("Коллекция не проинициализирована");
                 return;
-            }
+            };
             switch (comboBoxSelectorCompany.Text)
             {
                 case "Хранилище":
@@ -214,13 +168,15 @@ namespace HoistingCrane
         {
             if (saveFileDialog.ShowDialog() == DialogResult.OK)
             {
-                if (_storageCollection.SaveData(saveFileDialog.FileName))
+                try
                 {
+                    _storageCollection.SaveData(saveFileDialog.FileName);
                     MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
+             
                 }
-                else
+                catch (Exception ex)
                 {
-                    MessageBox.Show("Не сохранилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                    MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
                 }
             }
         }
@@ -234,14 +190,16 @@ namespace HoistingCrane
         {
             if(openFileDialog.ShowDialog() == DialogResult.OK)
             {
-                if (_storageCollection.LoadData(openFileDialog.FileName))
+                try
                 {
-                    MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
+                    _storageCollection.LoadData(openFileDialog.FileName);
                     RerfreshListBoxItems();
+                    MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
+                    
                 }
-                else
+                catch(Exception ex)
                 {
-                    MessageBox.Show("Не сохранилось", "Результат",MessageBoxButtons.OK, MessageBoxIcon.Error);
+                    MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
                 }
             }
         }

HoistingCrane/HoistingCrane/FormCarConfig.cs

@@ -31,6 +31,7 @@ namespace HoistingCrane
             panelColorPurple.MouseDown += panel_MouseDown;
             buttonCancel.Click += (sender, e) => Close();
         }
+        
         /// <summary>
         /// Привязка метода к событию
         /// </summary>

HoistingCrane/HoistingCrane/HoistingCrane.csproj

@@ -12,6 +12,18 @@
     <None Remove="CollectionGenericObjects\MassivGenericObjects.cs~RF2955bcc.TMP" />
   </ItemGroup>
 
+  <ItemGroup>
+    <PackageReference Include="Microsoft.Extensions.Configuration.EnvironmentVariables" Version="8.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="8.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Logging" Version="8.0.0" />
+    <PackageReference Include="Newtonsoft.Json" Version="13.0.3" />
+    <PackageReference Include="NLog.Extensions.Logging" Version="5.3.11" />
+    <PackageReference Include="Serilog" Version="3.1.1" />
+    <PackageReference Include="Serilog.Extensions.Logging" Version="8.0.0" />
+    <PackageReference Include="Serilog.Settings.Configuration" Version="8.0.0" />
+    <PackageReference Include="Serilog.Sinks.File" Version="5.0.0" />
+  </ItemGroup>
+
   <ItemGroup>
     <Compile Update="Properties\Resources.Designer.cs">
       <DesignTime>True</DesignTime>

HoistingCrane/HoistingCrane/Program.cs

@@ -1,7 +1,7 @@
-namespace HoistingCrane
+namespace HoistingCrane;
+
+internal static class Program
 {
-    internal static class Program
-    {
     /// <summary>
     ///  The main entry point for the application.
     /// </summary>
@@ -12,7 +12,5 @@ namespace HoistingCrane
         // see https://aka.ms/applicationconfiguration.
         ApplicationConfiguration.Initialize();
         Application.Run(new FormCarCollection());
-            
-        }
     }
 }

Файл_для_лабораторной_6.txt

@@ -1,3 +0,0 @@
-CollectionsStorage
-массив|Massive|25|EntityTrackedVehicle:100:100:Green;EntityHoistingCrane:100:100:Yellow:Black:False:False;EntityTrackedVehicle:100:100:Gray;
-список|List|2|EntityTrackedVehicle:100:100:Blue;EntityHoistingCrane:100:100:Gray:Black:True:True;

Файл_с_машинами.txt

@@ -1,2 +0,0 @@
-CollectionsStorage
-массив|Massive|24|EntityTrackedVehicle:100:100:Gray;EntityTrackedVehicle:100:100:Red;EntityTrackedVehicle:100:100:Blue;EntityTrackedVehicle:100:100:Yellow;EntityTrackedVehicle:100:100:Green;EntityTrackedVehicle:100:100:Black;EntityTrackedVehicle:100:100:White;EntityTrackedVehicle:100:100:Purple;EntityHoistingCrane:100:100:Green:Black:False:False;EntityHoistingCrane:100:100:Yellow:Gray:False:True;EntityHoistingCrane:100:100:Purple:Black:True:True;EntityTrackedVehicle:100:100:White;EntityHoistingCrane:100:100:White:Black:False:True;EntityTrackedVehicle:100:100:Green;EntityTrackedVehicle:100:100:White;EntityTrackedVehicle:100:100:White;EntityTrackedVehicle:100:100:White;EntityTrackedVehicle:100:100:White;EntityHoistingCrane:100:100:White:Black:False:False;EntityTrackedVehicle:100:100:White;EntityTrackedVehicle:100:100:Yellow;EntityTrackedVehicle:100:100:White;EntityHoistingCrane:100:100:Gray:Black:False:False;EntityTrackedVehicle:100:100:Black;

допка3.txt

@@ -1,3 +0,0 @@
-CollectionsStorage
-массив|Massive
-список|List