From 45443601ee1fc1a24bd54820dcfbbe43c864f038 Mon Sep 17 00:00:00 2001 From: kisame Date: Wed, 5 Jun 2024 05:55:42 +0400 Subject: [PATCH] =?UTF-8?q?=D0=BB=D0=B0=D0=B1=D0=BE=D1=80=D0=B0=D1=82?= =?UTF-8?q?=D0=BE=D1=80=D0=BD=D0=B0=D1=8F=207?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../ListGenericObjects.cs | 12 +- .../MassiveGenericObjects.cs | 16 +- .../StorageCollection.cs | 42 ++--- .../Exceptions/CollectionOverflowException.cs | 17 ++ .../Exceptions/ObjectNotFoundException.cs | 16 ++ .../PositionOutOfCollectionException.cs | 16 ++ .../ProjectAirbus/FormPlaneCollection.cs | 150 ++++++++++++------ ProjectAirbus/ProjectAirbus/Program.cs | 30 +++- .../ProjectAirbus/ProjectAirbus.csproj | 11 ++ ProjectAirbus/ProjectAirbus/nlog.config | 4 + 10 files changed, 229 insertions(+), 85 deletions(-) create mode 100644 ProjectAirbus/ProjectAirbus/Exceptions/CollectionOverflowException.cs create mode 100644 ProjectAirbus/ProjectAirbus/Exceptions/ObjectNotFoundException.cs create mode 100644 ProjectAirbus/ProjectAirbus/Exceptions/PositionOutOfCollectionException.cs create mode 100644 ProjectAirbus/ProjectAirbus/nlog.config diff --git a/ProjectAirbus/ProjectAirbus/CollectionGenericObjects/ListGenericObjects.cs b/ProjectAirbus/ProjectAirbus/CollectionGenericObjects/ListGenericObjects.cs index eb1d335..a301261 100644 --- a/ProjectAirbus/ProjectAirbus/CollectionGenericObjects/ListGenericObjects.cs +++ b/ProjectAirbus/ProjectAirbus/CollectionGenericObjects/ListGenericObjects.cs @@ -1,4 +1,6 @@  +using ProjectAirbus.Exceptions; + namespace ProjectAirbus.CollectionGenericObjects; /// /// Параметризованный набор объектов @@ -49,7 +51,7 @@ public class ListGenericObjects : ICollectionGenericObjects // Проверка позиции if (position >= Count || position < 0) { - return null; + throw new PositionOutOfCollectionException(position); } return _collection[position]; } @@ -59,7 +61,7 @@ public class ListGenericObjects : ICollectionGenericObjects // Проверка, что не превышено максимальное количество элементов if (Count == _maxCount) { - return -1; + throw new CollectionOverflowException(Count); } _collection.Add(obj); return Count; @@ -70,12 +72,12 @@ public class ListGenericObjects : ICollectionGenericObjects // Проверка, что не превышено максимальное количество элементов if (Count == _maxCount) { - return -1; + throw new CollectionOverflowException(Count); } // Проверка позиции if (position >= Count || position < 0) { - return -1; + throw new PositionOutOfCollectionException(position); } _collection.Insert(position, obj); return position; @@ -86,7 +88,7 @@ public class ListGenericObjects : ICollectionGenericObjects // Проверка позиции if (position >= Count || position < 0) { - return null; + throw new PositionOutOfCollectionException(position); } T? obj = _collection[position]; _collection.RemoveAt(position); diff --git a/ProjectAirbus/ProjectAirbus/CollectionGenericObjects/MassiveGenericObjects.cs b/ProjectAirbus/ProjectAirbus/CollectionGenericObjects/MassiveGenericObjects.cs index ef18b96..55e2130 100644 --- a/ProjectAirbus/ProjectAirbus/CollectionGenericObjects/MassiveGenericObjects.cs +++ b/ProjectAirbus/ProjectAirbus/CollectionGenericObjects/MassiveGenericObjects.cs @@ -1,4 +1,6 @@  +using ProjectAirbus.Exceptions; + namespace ProjectAirbus.CollectionGenericObjects; /// @@ -11,7 +13,7 @@ public class MassiveGenericObjects : ICollectionGenericObjects /// /// Массив объектов, которые храним /// - private T?[] _collection; + private T?[] _collection; public int Count => _collection.Length; @@ -54,7 +56,7 @@ public class MassiveGenericObjects : ICollectionGenericObjects return _collection[position]; } - return null; + throw new ObjectNotFoundException(position); } public int Insert(T obj) @@ -67,14 +69,14 @@ public class MassiveGenericObjects : ICollectionGenericObjects return i; } } - return -1; + throw new CollectionOverflowException(Count); } public int Insert(T obj, int position) { if (position < 0 || position >= Count) { - return -1; + throw new PositionOutOfCollectionException(position); } if (_collection[position] == null) { @@ -98,16 +100,16 @@ public class MassiveGenericObjects : ICollectionGenericObjects return i; } } - - return -1; + throw new CollectionOverflowException(Count); } public T Remove(int position) { if (position < 0 || position >= Count) { - return null; + throw new PositionOutOfCollectionException(position); } + if (_collection[position] == null) throw new ObjectNotFoundException(position); T obj = _collection[position]; _collection[position] = null; return obj; diff --git a/ProjectAirbus/ProjectAirbus/CollectionGenericObjects/StorageCollection.cs b/ProjectAirbus/ProjectAirbus/CollectionGenericObjects/StorageCollection.cs index 94b9b8e..a37575e 100644 --- a/ProjectAirbus/ProjectAirbus/CollectionGenericObjects/StorageCollection.cs +++ b/ProjectAirbus/ProjectAirbus/CollectionGenericObjects/StorageCollection.cs @@ -1,4 +1,5 @@ using ProjectAirbus.Drawnings; +using ProjectAirbus.Exceptions; using System.Text; namespace ProjectAirbus.CollectionGenericObjects; @@ -97,12 +98,11 @@ public class StorageCollection /// Сохранение информации хранилища в файл /// /// Путь и имя файла - /// true - сохранение прошло успешно, false - ошибка при сохранении данных - public bool SaveData(string filename) + public void SaveData(string filename) { if (_storages.Count == 0) { - return false; + throw new Exception("В хранилище отсутствуют коллекции для сохранения"); } if (File.Exists(filename)) @@ -138,28 +138,27 @@ public class StorageCollection sb.Clear(); } } - return true; } - /// - /// Загрузка информации по автомобилям в хранилище из файла - /// - /// Путь и имя файла - /// true - загрузка прошла успешно, false - ошибка при загрузке данных - public bool LoadData(string filename) + /// Загрузка информации по автомобилям в хранилище из файла + /// + /// Путь и имя файла + public void LoadData(string filename) { if (!File.Exists(filename)) { - return false; + throw new Exception("Файл не существует"); } using (StreamReader sr = new StreamReader(filename)) { string? str; str = sr.ReadLine(); + if (str == null || str.Length == 0) + throw new Exception("В файле нет данных"); if (str != _collectionKey.ToString()) - return false; + throw new Exception("В файле неверные данные"); _storages.Clear(); while ((str = sr.ReadLine()) != null) { @@ -172,7 +171,7 @@ public class StorageCollection ICollectionGenericObjects? collection = StorageCollection.CreateCollection(collectionType); if (collection == null) { - return false; + throw new Exception("Не удалось создать коллекцию"); } collection.MaxCount = Convert.ToInt32(record[2]); @@ -180,19 +179,26 @@ public class StorageCollection string[] set = record[3].Split(_separatorItems, StringSplitOptions.RemoveEmptyEntries); foreach (string elem in set) { - if (elem?.CreateDrawningPlane() is T plane) + if (elem?.CreateDrawningPlane() is T boat) { - if (collection.Insert(plane) == -1) - return false; + try + { + if (collection.Insert(boat) == -1) + throw new Exception("Объект не удалось добавить в коллекцию: " + record[3]); + } + catch (CollectionOverflowException ex) + { + throw new Exception("Коллекция переполнена", ex); + } } } _storages.Add(record[0], collection); } } - - return true; } + + /// /// Создание коллекции по типу /// diff --git a/ProjectAirbus/ProjectAirbus/Exceptions/CollectionOverflowException.cs b/ProjectAirbus/ProjectAirbus/Exceptions/CollectionOverflowException.cs new file mode 100644 index 0000000..991358d --- /dev/null +++ b/ProjectAirbus/ProjectAirbus/Exceptions/CollectionOverflowException.cs @@ -0,0 +1,17 @@ +using System.Runtime.Serialization; + +namespace ProjectAirbus.Exceptions; + + +/// +/// Класс, описывающий ошибку переполнения коллекции +/// +[Serializable] +internal 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 contex) : base(info, contex) { } +} diff --git a/ProjectAirbus/ProjectAirbus/Exceptions/ObjectNotFoundException.cs b/ProjectAirbus/ProjectAirbus/Exceptions/ObjectNotFoundException.cs new file mode 100644 index 0000000..1a63ca9 --- /dev/null +++ b/ProjectAirbus/ProjectAirbus/Exceptions/ObjectNotFoundException.cs @@ -0,0 +1,16 @@ +using System.Runtime.Serialization; + +namespace ProjectAirbus.Exceptions; + +/// +/// Класс, описывающий ошибку, что по указанной позиции нет элемента +/// +[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) { } +} diff --git a/ProjectAirbus/ProjectAirbus/Exceptions/PositionOutOfCollectionException.cs b/ProjectAirbus/ProjectAirbus/Exceptions/PositionOutOfCollectionException.cs new file mode 100644 index 0000000..baa359a --- /dev/null +++ b/ProjectAirbus/ProjectAirbus/Exceptions/PositionOutOfCollectionException.cs @@ -0,0 +1,16 @@ +using System.Runtime.Serialization; + +namespace ProjectAirbus.Exceptions; + +/// +/// Класс, описывающий ошибку выхода за границы коллекции +/// +[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) { } +} diff --git a/ProjectAirbus/ProjectAirbus/FormPlaneCollection.cs b/ProjectAirbus/ProjectAirbus/FormPlaneCollection.cs index d639f95..5a8caa2 100644 --- a/ProjectAirbus/ProjectAirbus/FormPlaneCollection.cs +++ b/ProjectAirbus/ProjectAirbus/FormPlaneCollection.cs @@ -1,5 +1,7 @@ -using ProjectAirbus.CollectionGenericObjects; +using Microsoft.Extensions.Logging; +using ProjectAirbus.CollectionGenericObjects; using ProjectAirbus.Drawnings; +using ProjectAirbus.Exceptions; namespace ProjectAirbus; @@ -19,13 +21,16 @@ public partial class FormPlaneCollection : Form /// private AbstractCompany? _company = null; + private readonly ILogger _logger; + /// /// Конструктор /// - public FormPlaneCollection() + public FormPlaneCollection(ILogger logger) { InitializeComponent(); _storageCollection = new(); + _logger = logger; } /// @@ -56,18 +61,25 @@ public partial class FormPlaneCollection : Form /// private void SetPlane(DrawningPlane plane) { - if (_company == null || plane == null) + try { - return; + if (_company == null || plane == null) + { + return; + } + + if (_company + plane != -1) + { + MessageBox.Show("Объект добавлен"); + pictureBox.Image = _company.Show(); + _logger.LogInformation("Добавлен объект: " + plane.GetDataForSave()); + } } - if (_company + plane != -1) + catch (ObjectNotFoundException) { } + catch (CollectionOverflowException ex) { - MessageBox.Show("Объект добавлен"); - pictureBox.Image = _company.Show(); - } - else - { - MessageBox.Show("Не удалось добавить объект"); + MessageBox.Show("В коллекции превышено допустимое количество элементов"); + _logger.LogError("Ошибка: {Message}", ex.Message); } } @@ -78,25 +90,31 @@ public partial class FormPlaneCollection : Form /// private void buttonDelPlane_Click(object sender, EventArgs e) { - if (string.IsNullOrEmpty(maskedTextBoxPosition.Text) || _company == null) - { - return; - } - - if (MessageBox.Show("Удалить объект?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) != DialogResult.Yes) - { - return; - } - int pos = Convert.ToInt32(maskedTextBoxPosition.Text); - if (_company - pos != null) + try { - MessageBox.Show("Объект удален"); - pictureBox.Image = _company.Show(); + if (string.IsNullOrEmpty(maskedTextBoxPosition.Text) || _company == null) + { + throw new Exception("Входные данные отсутствуют"); + } + + if (MessageBox.Show("Удалить объект?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) != DialogResult.Yes) + { + return; + } + + + if (_company - pos != null) + { + MessageBox.Show("Объект удален"); + pictureBox.Image = _company.Show(); + _logger.LogInformation("Объект удален"); + } } - else + catch (Exception ex) { - MessageBox.Show("Не удалось удалить объект"); + MessageBox.Show("Не найден объект по позиции " + pos); + _logger.LogError("Ошибка: {Message}", ex.Message); } } @@ -114,24 +132,31 @@ public partial class FormPlaneCollection : Form DrawningPlane? plane = null; int counter = 100; - while (plane == null) + try { - plane = _company.GetRandomObject(); - counter--; - if (counter <= 0) + while (plane == null) { - break; + plane = _company.GetRandomObject(); + counter--; + if (counter <= 0) + { + break; + } } - } - if (plane == null) + if (plane == null) + { + return; + } + + FormAirbus form = new FormAirbus(); + form.SetPlane = plane; + form.ShowDialog(); + } + catch (Exception ex) { - return; + MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); } - - FormAirbus form = new FormAirbus(); - form.SetPlane = plane; - form.ShowDialog(); } /// @@ -158,17 +183,22 @@ public partial class FormPlaneCollection : Form { if (string.IsNullOrEmpty(textBoxCollectionName.Text) || (!radioButtonList.Checked && !radioButtonMassive.Checked)) { - MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error); + MessageBox.Show("Не все данный заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogInformation("Не удалось добавить коллекцию: не все данные заполнены"); return; } - CollectionType collectionType = CollectionType.None; if (radioButtonMassive.Checked) + { collectionType = CollectionType.Massive; + } else if (radioButtonList.Checked) + { collectionType = CollectionType.List; + } _storageCollection.AddCollection(textBoxCollectionName.Text, collectionType); + _logger.LogInformation("Добавлена коллекция типа {type} с названием {name}", collectionType, textBoxCollectionName.Text); RefreshListBoxItems(); } @@ -193,17 +223,26 @@ public partial class FormPlaneCollection : Form private void buttonCollectionDel_Click(object sender, EventArgs e) { - if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItem == null) + if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItems == null) { MessageBox.Show("Коллекция не выбрана"); return; } - if (MessageBox.Show("Удалить коллекцию?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No) + + try { - return; + if (MessageBox.Show("Удалить коллекцию?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No) + { + return; + } + _storageCollection.DelCollection(listBoxCollection.SelectedItem.ToString()); + RefreshListBoxItems(); + _logger.LogInformation("Удалена коллекция: ", listBoxCollection.SelectedItem.ToString()); + } + catch (Exception ex) + { + _logger.LogError("Ошибка: {Message}", ex.Message); } - _storageCollection.DelCollection(listBoxCollection.SelectedItem.ToString() ?? string.Empty); - RefreshListBoxItems(); } /// @@ -226,12 +265,13 @@ public partial class FormPlaneCollection : Form return; } - switch (comboBoxSelectorCompany.Text) { case "Хранилище": _company = new AirbusSharingService(pictureBox.Width, pictureBox.Height, collection); break; + default: + return; } panelCompanyTools.Enabled = true; RefreshListBoxItems(); @@ -246,13 +286,16 @@ public partial class FormPlaneCollection : 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 ex) { - MessageBox.Show("Не сохранилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogError("Ошибка: {Message}", ex.Message); } } } @@ -266,14 +309,17 @@ public partial class FormPlaneCollection : Form { if (openFileDialog.ShowDialog() == DialogResult.OK) { - if (_storageCollection.LoadData(openFileDialog.FileName)) + try { + _storageCollection.LoadData(openFileDialog.FileName); RefreshListBoxItems(); - MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); + MessageBox.Show("Загрузка прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); + _logger.LogInformation("Загрузка из файла: {filename}", openFileDialog.FileName); } - else + catch (Exception ex) { MessageBox.Show("Загрузка не выполнена", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogError("Ошибка: {Message}", ex.Message); } } } diff --git a/ProjectAirbus/ProjectAirbus/Program.cs b/ProjectAirbus/ProjectAirbus/Program.cs index a0ccdfc..b89f5cf 100644 --- a/ProjectAirbus/ProjectAirbus/Program.cs +++ b/ProjectAirbus/ProjectAirbus/Program.cs @@ -1,3 +1,7 @@ +using Microsoft.Extensions.DependencyInjection; +using Microsoft.Extensions.Logging; +using NLog.Extensions.Logging; + namespace ProjectAirbus { internal static class Program @@ -8,10 +12,30 @@ namespace ProjectAirbus [STAThread] static void Main() { - // To customize application configuration such as set high DPI settings or default font, - // see https://aka.ms/applicationconfiguration. ApplicationConfiguration.Initialize(); - Application.Run(new FormPlaneCollection()); + ServiceCollection services = new ServiceCollection(); + ConfigureServices(services); + using (ServiceProvider serviceProvider = services.BuildServiceProvider()) + { + Application.Run(serviceProvider.GetRequiredService()); + } + } + + /// + /// DI + /// + /// + private static void ConfigureServices(ServiceCollection services) + { + services.AddSingleton() + .AddLogging(option => + { + option.SetMinimumLevel(LogLevel.Information); + option.AddNLog("nlog.config"); + }); + + } + } } \ No newline at end of file diff --git a/ProjectAirbus/ProjectAirbus/ProjectAirbus.csproj b/ProjectAirbus/ProjectAirbus/ProjectAirbus.csproj index 244387d..d258f05 100644 --- a/ProjectAirbus/ProjectAirbus/ProjectAirbus.csproj +++ b/ProjectAirbus/ProjectAirbus/ProjectAirbus.csproj @@ -8,6 +8,11 @@ enable + + + + + True @@ -23,4 +28,10 @@ + + + Always + + + \ No newline at end of file diff --git a/ProjectAirbus/ProjectAirbus/nlog.config b/ProjectAirbus/ProjectAirbus/nlog.config new file mode 100644 index 0000000..7f2a901 --- /dev/null +++ b/ProjectAirbus/ProjectAirbus/nlog.config @@ -0,0 +1,4 @@ + + + + \ No newline at end of file -- 2.25.1