Лаба 7 готовая.

ProjectBulldozer/ProjectBulldozer/CollectionGenericObjects/AbstractCompany.cs

@@ -41,7 +41,7 @@ public abstract class AbstractCompany
     /// <summary>
     /// Вычисление максимального количества элементов, который можно разместить в окне
     /// </summary>
-    private int GetMaxCount => _pictureWidth * _pictureHeight / (_placeSizeWidth * _placeSizeHeight);
+    private int GetMaxCount => (int)(_pictureWidth / _placeSizeWidth) * (int)(_pictureHeight  / _placeSizeHeight);
 
     /// <summary>
     /// Конструктор

ProjectBulldozer/ProjectBulldozer/CollectionGenericObjects/ListGenericObjects.cs

@@ -1,4 +1,5 @@
-using System;
+using ProjectBulldozer.Exceptions;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;
@@ -53,18 +54,25 @@ public class ListGenericObjects<T> : ICollectoinGenericObjects<T>
     public T? Get(int position)
     {
         // Проверка позиции.
-        if (position < 0 ||  position >= _collection.Count || _collection == null || _collection.Count == 0)
+        if (position < 0 || position >= Count)
         {
-            return null;
+            throw new PositionOutOfCollectionException(position);
         }
+
+        // Выброс ошибки, если выход за границы списка
+        if (_collection[position] == null)
+        {
+            throw new ObjectNotFoundException(position);
+        }
+
         return _collection[position];
     }
     public int Insert(T obj)
     {
-        // Проверка, что не привышено максимальное колличество элементов.
+        // Выброс ошибки, если переполнение
         if (Count == _maxCount)
         {
-            return -1;
+            throw new CollectionOverflowException(Count);
         }
 
         // Вставка в конец набора.
@@ -74,16 +82,16 @@ public class ListGenericObjects<T> : ICollectoinGenericObjects<T>
 
     public int Insert(T obj, int position)
     {
-        // Проверка, что не превышено максимальное колличество элементов.
+        // Выброс ошибки, если переполнение
         if (Count == _maxCount)
         {
-            return -1; 
+            throw new CollectionOverflowException(Count);
         }
 
-        // Проверка позиции.
+        // Выброс ошибки, если выход за границы списка
-        if (position < 0 || position >= _collection.Count || _collection == null || _collection.Count == 0) 
+        if (position >= Count || position < 0)
         {
-            return -1;
+            throw new PositionOutOfCollectionException(position);
         }
 
         // Вставка по позиции.
@@ -94,10 +102,10 @@ public class ListGenericObjects<T> : ICollectoinGenericObjects<T>
 
     public T? Remove(int position)
     {
-        // Проверка позиции.
+        // TODO Выброс ошибки, если выход за границы списка
-        if (position < 0 || position >= _collection.Count || _collection == null || _collection.Count == 0)
+        if (position >= Count || position < 0)
         {
-            return null;
+            throw new PositionOutOfCollectionException(position);
         }
 
         // Удаление объекта из списка.

ProjectBulldozer/ProjectBulldozer/CollectionGenericObjects/MassiveGenericObject.cs

@@ -1,4 +1,5 @@
-using System;
+using ProjectBulldozer.Exceptions;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;
@@ -51,15 +52,20 @@ public class MassiveGenericObject<T> : ICollectoinGenericObjects<T>
     public T? Get(int position)
     {
         //Проверка позиции
-        if ((position >= 0) && (position < Count))
+        // Выброс ошибки, если выход за границы массива
+        if (position < 0 || position >= _collection.Length)
         {
+            throw new PositionOutOfCollectionException(position);
+        }
+
+        // Выброс ошибки, если объект пустой
+        if (position >= _collection.Length && _collection[position] == null)
+        {
+            throw new ObjectNotFoundException(position);
+        }
+
         return _collection[position];
     }
-        else
-        {
-            return null;
-        }
-    }
 
     public int Insert(T obj)
     {
@@ -73,7 +79,8 @@ public class MassiveGenericObject<T> : ICollectoinGenericObjects<T>
             }
         }
 
-        return -1;
+        // Выброс ошибки, если переполнение
+        throw new CollectionOverflowException(Count);
     }
 
     public int Insert(T obj, int position)
@@ -84,51 +91,60 @@ public class MassiveGenericObject<T> : ICollectoinGenericObjects<T>
             return -1;
         }
 
+        // Выброс ошибки, если переполнение
+        if (position < 0 || position >= _collection.Length)
+        {
+            throw new PositionOutOfCollectionException(position);
+        }
+
+
         //Проверка, что элемент массива по этой позиции пустой, если нет, то ищется свободное место после этой
         //позиции и идёт вставка туда, если нет после, ищем до
-        if (_collection[position] != null)
+        if (_collection[position] == null)
         {
-            bool placed = false;
-            for (int i = position + 1; i < Count; i++)
-            {
-                if (_collection[i] == null)
-                {
-                    position = i;
-                    placed = true;
-                    break;
-                }
-            }
-
-            if (placed == false)
-            {
-                for (int i = position - 1; i >= 0; i--)
-                {
-                    if (_collection[i] == null)
-                    {
-                        position = i;
-                        placed = true;
-                        break;
-                    }
-                }
-            }
-
-            if (placed == false)
-            {
-                return -1;
-            }
-        }
-
             //Вставка
             _collection[position] = obj;
             return position;
         }
+        int temp = position + 1;
+        while (temp < Count)
+        {
+            if (_collection[temp] == null)
+            {
+                _collection[temp] = obj;
+                return temp;
+            }
+            ++temp;
+        }
+
+        temp = position - 1;
+        while (temp >= 0)
+        {
+            if (_collection[temp] == null)
+            {
+                _collection[temp] = obj;
+                return temp;
+            }
+            --temp;
+        }
+
+        // Выброс ошибки, если выход за границы массива
+        throw new CollectionOverflowException(Count);
+    }
 
     public T? Remove(int position)
     {
         //Проверка позиции
-        if ((position < 0) || (position >= Count) || (_collection[position] == null))
+        // Выброс ошибки, если выход за границы массива
+        if ((position < 0) || (position >= Count))
         {
-            return null;
+            throw new PositionOutOfCollectionException(position);
+        }
+
+        // Выброс ошибки, если объект пустой
+        if (_collection[position] == null)
+        {
+            throw new ObjectNotFoundException(position);
         }
 
         //Удаление объекта из массива, присвоив элементу массива значение null

ProjectBulldozer/ProjectBulldozer/CollectionGenericObjects/StorageCollection.cs

@@ -4,6 +4,7 @@ using System.Linq;
 using System.Text;
 using System.Threading.Tasks;
 using ProjectBulldozer.Drawnings;
+using ProjectBulldozer.Exceptions;
 
 namespace ProjectBulldozer.CollectionGenericObjects;
 
@@ -105,12 +106,11 @@ public class StorageCollection<T>
     /// Сохранение информации по бульдозерам в хранилище в файла
     /// </summary>
     /// <param name="filename">Путь и имя файла</param>
-    /// <returns>true - сохранение прошло успешно, false - ошибка при сохранении данных</returns>
+    public void SaveData(string filename)
-    public bool SaveData(string filename)
     {
         if (_storages.Count == 0)
         {
-            return false;
+            throw new Exception("В хранилище отсутсвует коллекция для сохранения!");
         }
 
         if (File.Exists(filename)) 
@@ -153,17 +153,18 @@ public class StorageCollection<T>
         using FileStream fileStream = new(filename, FileMode.Create);
         byte[] info = new UTF8Encoding(true).GetBytes(stringBuilder.ToString());
         fileStream.Write(info, 0, info.Length);
-        return true;
     }
 
     /// <summary>
     /// Загрузка информации по бульдозерам в хранилище из файла
     /// </summary>
     /// <param name="filename">Путь и имя файла</param>
-    /// <returns>true- загрузка прошла успешно, false - ошибка при загрузке данных</returns>
+    public void LoadData(string filename)
-    public bool LoadData(string filename)
     {
-        if (!File.Exists(filename)) return false;
+        if (!File.Exists(filename))
+        {
+            throw new Exception("Файл не существует!");
+        }
 
         string bufferTextfromFile = "";
         using (FileStream fileStream = new(filename, FileMode.Open))
@@ -180,13 +181,12 @@ public class StorageCollection<T>
             StringSplitOptions.RemoveEmptyEntries);
         if (strs == null || strs.Length == 0)
         {
-            return false;
+            throw new Exception("В файле нет данных!");
         }
 
         if (!strs[0].Equals(_collectionKey))
         {
-            // если нет такой записи, то это не те данные.
+            throw new Exception("В файле неверные данные!");
-            return false;
         }
 
         _storages.Clear();
@@ -200,12 +200,8 @@ public class StorageCollection<T>
             }
 
             CollectionType collectionType = (CollectionType)Enum.Parse(typeof(CollectionType), record[1]);
-            ICollectoinGenericObjects<T>? collectoin = StorageCollection<T>.CreateCollection(collectionType);
+            ICollectoinGenericObjects<T>? collectoin = StorageCollection<T>.CreateCollection(collectionType) ?? 
-
+                throw new Exception("Не удалось определить тип коллекции: " + record[1]);
-            if (collectoin == null) 
-            {
-                return false;
-            }
 
             collectoin.MaxCount = Convert.ToInt32(record[2]);
 
@@ -214,16 +210,22 @@ public class StorageCollection<T>
             foreach (string elem in set)
             {
                 if (elem?.CreateDrawningDozer() is T dozer)
+                {
+                    try
                     {
                         if (collectoin.Insert(dozer) == -1)
                         {
-                        return false;
+                            throw new Exception("Объект не удалось добавить в коллекцию: " + record[3]);
+                        }
+                    }
+                    catch (CollectionOverflowException exception)
+                    {
+                        throw new Exception("Коллекция переполнена!", exception);
                     }
                 }
             }
             _storages.Add(record[0], collectoin);
         }
-            return true;
     }
 
 

ProjectBulldozer/ProjectBulldozer/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 ProjectBulldozer.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 context) : base(info, context) { }
+}

ProjectBulldozer/ProjectBulldozer/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 ProjectBulldozer.Exceptions;
+
+/// <summary>
+/// Класс, описывающий ошибку, что по указанной позиции нет элемента
+/// </summary>
+[Serializable]
+internal 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) { }
+}
+

ProjectBulldozer/ProjectBulldozer/Exceptions/PositionOutOfCollectionException.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 ProjectBulldozer.Exceptions;
+
+/// <summary>
+/// Класс, описывающий ошибку выхода за границы коллекции
+/// </summary>
+[Serializable]
+internal 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) { }
+}
+

ProjectBulldozer/ProjectBulldozer/FormBulldozerCollection.cs

@@ -1,5 +1,7 @@
-using ProjectBulldozer.CollectionGenericObjects;
+using Microsoft.Extensions.Logging;
+using ProjectBulldozer.CollectionGenericObjects;
 using ProjectBulldozer.Drawnings;
+using ProjectBulldozer.Exceptions;
 using System;
 using System.Collections.Generic;
 using System.ComponentModel;
@@ -10,8 +12,6 @@ using System.Runtime.InteropServices;
 using System.Text;
 using System.Threading.Tasks;
 using System.Windows.Forms;
-using ProjectBulldozer.CollectionGenericObjects;
-using ProjectBulldozer.Drawnings;
 
 namespace ProjectBulldozer;
 
@@ -30,13 +30,16 @@ public partial class FormBulldozerCollection : Form
     /// </summary>
     private AbstractCompany? _company = null;
 
+    private readonly ILogger _logger;
+
     /// <summary>
     /// Конструктор
     /// </summary>
-    public FormBulldozerCollection()
+    public FormBulldozerCollection(ILogger<FormBulldozerCollection> logger)
     {
         InitializeComponent();
         _storageCollection = new();
+        _logger = logger;
     }
 
     /// <summary>
@@ -49,22 +52,6 @@ public partial class FormBulldozerCollection : Form
         panelCompanyTools.Enabled = false;
 
     }
-    /*
-        /// <summary>
-        /// Добавление обычного бульдозера
-        /// </summary>
-        /// <param name="sender"></param>
-        /// <param name="e"></param>
-        private void ButtonAddDozer_Click(object sender, EventArgs e) =>
-            CreateObject(nameof(DrawningDozer));
-
-        /// <summary>
-        /// Добавление крутого бульдозера
-        /// </summary>
-        /// <param name="sender"></param>
-        /// <param name="e"></param>
-        private void ButtonAddBulldozer_Click(object sender, EventArgs e) =>
-            CreateObject(nameof(DrawningBulldozer));*/
 
     /// <summary>
     /// Добавление бульдозера
@@ -84,6 +71,8 @@ public partial class FormBulldozerCollection : Form
     /// </summary>
     /// <param name="dozer"></param>
     private void SetDozer(DrawningDozer? dozer)
+    {
+        try
         {
             if (_company == null || dozer == null)
             {
@@ -94,10 +83,14 @@ public partial class FormBulldozerCollection : Form
             {
                 MessageBox.Show("Объект добавлен.");
                 pictureBox.Image = _company.Show();
+                _logger.LogInformation("Объект добавлен: " + dozer.GetDataForSave());
             }
-        else
+        }
+        catch (CollectionOverflowException exception)
         {
-            MessageBox.Show("Не удалось добавить объект!");
+            MessageBox.Show(exception.Message);
+            _logger.LogError("Ошибка: превышено допустимое коллическтво! {Message}", exception);
+
         }
     }
 
@@ -182,6 +175,7 @@ public partial class FormBulldozerCollection : Form
     {
         if (string.IsNullOrEmpty(maskedTextBox.Text) || _company == null)
         {
+            _logger.LogWarning("Удаление объекта из несуществующей коллекции!");
             return;
         }
 
@@ -191,14 +185,25 @@ public partial class FormBulldozerCollection : Form
         }
 
         int pos = Convert.ToInt32(maskedTextBox.Text);
+
+        try
+        {
             if (_company - pos != null)
             {
                 MessageBox.Show("Объект удалён.");
                 pictureBox.Image = _company.Show();
+                _logger.LogInformation("Объект удалён: " + pos);
             }
-        else
+        }
+        catch (ObjectNotFoundException exception)
         {
-            MessageBox.Show("Не удалось удалить объект...");
+            MessageBox.Show(exception.Message);
+            _logger.LogWarning("Попытка удалить не найденный объект по позиции: " + pos);
+        }
+        catch (PositionOutOfCollectionException)
+        {
+            MessageBox.Show("Объект вне коллекции!");
+            _logger.LogWarning("Попытка удалить объект вне коллекции: " + pos);
         }
     }
 
@@ -216,6 +221,9 @@ public partial class FormBulldozerCollection : Form
 
         DrawningDozer? dozer = null;
         int counter = 100;
+
+        try
+        {
             while (dozer == null)
             {
                 dozer = _company.GetRandomObject();
@@ -237,6 +245,11 @@ public partial class FormBulldozerCollection : Form
             };
             form.ShowDialog();
         }
+        catch (Exception exception)
+        {
+            MessageBox.Show(exception.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+        }
+    }
 
     /// <summary>
     /// Прорисовка коллекции
@@ -250,8 +263,13 @@ public partial class FormBulldozerCollection : Form
             return;
         }
 
+        try
+        {
             pictureBox.Image = _company.Show();
         }
+        catch (PositionOutOfCollectionException) { }
+        catch (Exception) { }
+    }
 
     /// <summary>
     /// Добавление коллекции
@@ -266,6 +284,8 @@ public partial class FormBulldozerCollection : Form
             return;
         }
 
+        try
+        {
             CollectionType collectionType = CollectionType.None;
             if (radioButtonMassive.Checked)
             {
@@ -275,9 +295,15 @@ public partial class FormBulldozerCollection : Form
             {
                 collectionType = CollectionType.List;
             }
-
             _storageCollection.AddCollection(textBoxCollectionName.Text, collectionType);
             RefreshListBoxItems();
+            _logger.LogInformation("Коллекция добавлена " + textBoxCollectionName.Text);
+        }
+        catch (Exception ex)
+        {
+            _logger.LogError("Ошибка: {Message}", ex.Message);
+        }
+
     }
 
 
@@ -295,24 +321,26 @@ public partial class FormBulldozerCollection : Form
             MessageBox.Show("Коллекция не выбрана!");
             return;
         }
-        else
+        try
         {
             // Спросить у пользователя через MessageBox, что он подтверждает, что хочет удалить запись
             if (MessageBox.Show("Удалить коллекцию?", "Удаление...", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No)
             {
-                MessageBox.Show("Коллекция не выбрана!");
                 return;
             }
-            else
-            {
             // Удалить и обновить ListBox
-                _storageCollection.DelCollection(listBoxCollection.SelectedItem.ToString());
+            //_storageCollection.DelCollection(listBoxCollection.SelectedItem.ToString());
+            _logger.LogInformation("Коллекция удалена: " + listBoxCollection.SelectedItem.ToString());
             RefreshListBoxItems();
         }
+        catch (Exception ex)
+        {
+            _logger.LogError("Ошибка: {Message}", ex.Message);
         }
         
 
 
+
     }
 
     /// <summary>
@@ -351,12 +379,16 @@ public partial class FormBulldozerCollection : Form
             return;
         }
 
+        try
+        {
             switch (comboBoxSelectorCompany.Text)
             {
                 case "Хранилище":
                     _company = new Garage(pictureBox.Width, pictureBox.Height, collection);
                     break;
             }
+        }
+        catch (ObjectNotFoundException) { }
 
         panelCompanyTools.Enabled = true;
         RefreshListBoxItems();
@@ -371,15 +403,18 @@ public partial class FormBulldozerCollection : 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 exception)
             {
                 MessageBox.Show("Не сохранено!", "Результат",
                     MessageBoxButtons.OK, MessageBoxIcon.Error);
+                _logger.LogError("Ошибка: {Message}", exception.Message);
             }
         }
     }
@@ -394,17 +429,25 @@ public partial class FormBulldozerCollection : Form
         // Логика
         if (openFileDialog.ShowDialog() == DialogResult.OK)
         {
-            if (_storageCollection.LoadData(openFileDialog.FileName))
+            try
             {
+                _storageCollection.LoadData(openFileDialog.FileName);
                 MessageBox.Show("Загрузка прошла успешно.", "Результат",
                     MessageBoxButtons.OK, MessageBoxIcon.Information);
+                foreach (var collection in _storageCollection.Keys)
+                {
+                    listBoxCollection.Items.Add(collection);
+                }
+                _logger.LogInformation("Загрузка из файла: {filename}", saveFileDialog.FileName);
                 RefreshListBoxItems();
             }
-            else
+            catch (Exception exception)
             {
                 MessageBox.Show("Загрузка не удалась!", "Результат",
                     MessageBoxButtons.OK, MessageBoxIcon.Error);
+                _logger.LogError("Ошибка: {Message}", exception.Message);
             }
         }
+        RefreshListBoxItems();
     }
 }

ProjectBulldozer/ProjectBulldozer/Program.cs

@@ -1,3 +1,8 @@
+using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.Logging;
+using NLog.Extensions.Logging;
+using System;
+
 namespace ProjectBulldozer
 {
     internal static class Program
@@ -10,8 +15,27 @@ namespace ProjectBulldozer
         {
             // To customize application configuration such as set high DPI settings or default font,
             // see https://aka.ms/applicationconfiguration.
+
             ApplicationConfiguration.Initialize();
-            Application.Run(new FormBulldozerCollection());
+
+            ServiceCollection services = new();
+            ConfigureServices(services);
+            using ServiceProvider serviceProvider = services.BuildServiceProvider();
+            Application.Run(serviceProvider.GetRequiredService<FormBulldozerCollection>());
+        }
+
+        /// <summary>
+        /// Êîíôèóðàöèÿ ñåðâèñà DI
+        /// </summary>
+        /// <param name="services"></param>
+        private static void ConfigureServices(ServiceCollection services)
+        {
+            services.AddSingleton<FormBulldozerCollection>()
+                .AddLogging(option =>
+                {
+                    option.SetMinimumLevel(LogLevel.Information);
+                    option.AddNLog("nlog.config");
+                });
         }
     }
 }

ProjectBulldozer/ProjectBulldozer/ProjectBulldozer.csproj

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

ProjectBulldozer/ProjectBulldozer/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>