From d1e57152312f839d30125596704150a9d52eed69 Mon Sep 17 00:00:00 2001 From: olshab Date: Sun, 26 Nov 2023 00:43:05 +0400 Subject: [PATCH] Add warning logs --- AirBomber/FormEntityCollection.cs | 16 +++++++++++++++- AirBomber/Generics/EntitiesGenericStorage.cs | 11 ++++++----- 2 files changed, 21 insertions(+), 6 deletions(-) diff --git a/AirBomber/FormEntityCollection.cs b/AirBomber/FormEntityCollection.cs index 4b94e50..5135250 100644 --- a/AirBomber/FormEntityCollection.cs +++ b/AirBomber/FormEntityCollection.cs @@ -54,12 +54,15 @@ namespace AirBomber if (obj + (Renderer) != -1) { MessageBox.Show("Объект добавлен"); + _logger.LogInformation("Объект добавлен"); + CollectionPictureBox.Image = obj.ShowEntities(); } else { MessageBox.Show("Не удалось добавить объект"); + _logger.LogWarning("Не удалось добавить объект"); } } @@ -81,11 +84,14 @@ namespace AirBomber if (obj - Pos == true) { MessageBox.Show("Объект удален"); + _logger.LogInformation("Объект удален"); + CollectionPictureBox.Image = obj.ShowEntities(); } else { MessageBox.Show("Не удалось удалить объект"); + _logger.LogWarning("Не удалось удалить объект"); } } catch (EntityNotFoundException ex) @@ -111,6 +117,8 @@ namespace AirBomber if (string.IsNullOrEmpty(SetNameTextBox.Text)) { MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning("Не все данные заполнены"); + return; } @@ -153,11 +161,14 @@ namespace AirBomber try { _storage.SaveData(SaveFileDialog.FileName); + MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); + _logger.LogInformation("Сохранение прошло успешно"); } catch (Exception ex) { MessageBox.Show($"Не сохранилось: {ex.Message}", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning($"Не сохранилось: {ex.Message}"); } } } @@ -169,12 +180,15 @@ namespace AirBomber try { _storage.LoadData(OpenFileDialog.FileName); + MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); + _logger.LogInformation("Загрузка прошла успешно"); } catch (Exception ex) { MessageBox.Show($"Не загрузилось: {ex.Message}", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); - } + _logger.LogWarning($"Не загрузилось: {ex.Message}"); + } } ReloadObjects(); diff --git a/AirBomber/Generics/EntitiesGenericStorage.cs b/AirBomber/Generics/EntitiesGenericStorage.cs index bd9ba7f..6182942 100644 --- a/AirBomber/Generics/EntitiesGenericStorage.cs +++ b/AirBomber/Generics/EntitiesGenericStorage.cs @@ -1,4 +1,5 @@ -using AirBomber.MovementStrategy; +using AirBomber.Exceptions; +using AirBomber.MovementStrategy; using AirBomber.Rendering; using System.Text; @@ -58,7 +59,7 @@ namespace AirBomber.Generics public void SaveData(string FileName) { if (_entityStorages.Count == 0) - throw new Exception("Невалидная операция: нет данных для сохранения"); + throw new InvalidOperationException("Невалидная операция: нет данных для сохранения"); using (StreamWriter writer = new StreamWriter(FileName, false)) { @@ -79,12 +80,12 @@ namespace AirBomber.Generics public void LoadData(string FileName) { if (!File.Exists(FileName)) - throw new Exception("Файл не найден"); + throw new FileNotFoundException("Файл не найден"); using (StreamReader reader = new StreamReader(FileName)) { if (reader.ReadLine() != "BomberStorage") - throw new Exception("Неверный формат данных"); + throw new FormatException("Неверный формат данных"); _entityStorages.Clear(); @@ -106,7 +107,7 @@ namespace AirBomber.Generics if (Renderer != null) { if ((Collection + Renderer) == -1) - throw new Exception("Ошибка добавления в коллекцию"); + throw new StorageOverflowException("Ошибка добавления в коллекцию"); } }