From 621db43a11070e54113821422273054a0baf7579 Mon Sep 17 00:00:00 2001
From: artur-kalimullin <144933634+artur-kalimullin@users.noreply.github.com>
Date: Sat, 25 May 2024 10:02:12 +0400
Subject: [PATCH] =?UTF-8?q?=D0=93=D0=BE=D1=82=D0=BE=D0=B2=D0=BE?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../FormAirCraftCollection.cs                 | 61 ++++++++++---------
 1 file changed, 32 insertions(+), 29 deletions(-)

diff --git a/ProjectAirFighter/ProjectAirFighter/FormAirCraftCollection.cs b/ProjectAirFighter/ProjectAirFighter/FormAirCraftCollection.cs
index 11d59fb..2a8613c 100644
--- a/ProjectAirFighter/ProjectAirFighter/FormAirCraftCollection.cs
+++ b/ProjectAirFighter/ProjectAirFighter/FormAirCraftCollection.cs
@@ -2,7 +2,6 @@
 using ProjectAirFighter.CollectionGenericObjects;
 using ProjectAirFighter.Drawnings;
 using ProjectAirFighter.Exceptions;
-using System.Xml.Linq;
 
 namespace ProjectAirFighter;
 
@@ -78,7 +77,7 @@ public partial class FormAirCraftCollection : Form
         catch (CollectionOverflowException ex)
         {
             MessageBox.Show("Не удалось добавить объект");
-            _logger.LogWarning($"Не удалось добавить объект: {ex.Message}");
+            _logger.LogError($"Не удалось добавить объект: {ex.Message}");
         }
         catch (ObjectAlreadyExistsException)
         {
@@ -96,7 +95,7 @@ public partial class FormAirCraftCollection : Form
     {
         if (string.IsNullOrEmpty(maskedTextBoxPosition.Text) || _company == null)
         {
-            _logger.LogWarning("Удаление объекта из несуществующей коллекции");
+            _logger.LogError("Удаление объекта из несуществующей коллекции");
             return;
         }
 
@@ -110,18 +109,18 @@ public partial class FormAirCraftCollection : Form
         {
             object decrementObject = _company - pos;
             MessageBox.Show("Объект удален");
-            _logger.LogInformation($"Удален по позиции {pos}");
+            _logger.LogInformation($"Удален объект по позиции {pos}");
             pictureBox.Image = _company.Show();
         }
-        catch (ObjectNotFoundException ex)
+        catch (ObjectNotFoundException)
         {
             MessageBox.Show("Объект не найден");
-            _logger.LogWarning($"Удаление не найденного объекта в позиции {pos} ");
+            _logger.LogError($"Удаление не найденного объекта в позиции {pos} ");
         }
-        catch (PositionOutOfCollectionException ex)
+        catch (PositionOutOfCollectionException)
         {
             MessageBox.Show("Удаление вне рамках коллекции");
-            _logger.LogWarning($"Удаление объекта за пределами коллекции {pos} ");
+            _logger.LogError($"Удаление объекта за пределами коллекции {pos} ");
         }
     }
 
@@ -136,29 +135,33 @@ public partial class FormAirCraftCollection : Form
         {
             return;
         }
-
-        DrawningAirCraft? aircraft = null;
-        int counter = 100;
-        while (aircraft == null)
+        try
         {
-            aircraft = _company.GetRandomObject();
-            counter--;
-            if (counter <= 0)
+            DrawningAirCraft? aircraft = null;
+            int counter = 100;
+            while (aircraft == null)
             {
-                break;
+                aircraft = _company.GetRandomObject();
+                counter--;
+                if (counter <= 0)
+                {
+                    break;
+                }
             }
+            if (aircraft == null)
+            {
+                return;
+            }
+            FormAirFighter form = new()
+            {
+                SetAirCraft = aircraft
+            };
+            form.ShowDialog();
         }
-
-        if (aircraft == null)
+        catch (ObjectNotFoundException)
         {
-            return;
+            _logger.LogError("Ошибка при передаче объекта на FormAirFighter");
         }
-
-        FormAirFighter form = new()
-        {
-            SetAirCraft = aircraft
-        };
-        form.ShowDialog();
     }
 
     /// <summary>
@@ -186,7 +189,7 @@ public partial class FormAirCraftCollection : Form
         if (string.IsNullOrEmpty(textBoxCollectionName.Text) || (!radioButtonList.Checked && !radioButtonMassive.Checked))
         {
             MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error);
-            _logger.LogWarning("Не заполненная коллекция");
+            _logger.LogError("Не заполненная коллекция");
             return;
         }
 
@@ -215,7 +218,7 @@ public partial class FormAirCraftCollection : Form
         if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItem == null)
         {
             MessageBox.Show("Коллекция не выбрана");
-            _logger.LogWarning("Удаление невыбранной коллекции");
+            _logger.LogError("Удаление невыбранной коллекции");
             return;
         }
         string name = listBoxCollection.SelectedItem.ToString() ?? string.Empty;
@@ -253,7 +256,7 @@ public partial class FormAirCraftCollection : Form
         if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItem == null)
         {
             MessageBox.Show("Коллекция не выбрана");
-            _logger.LogWarning("Создание компании невыбранной коллекции");
+            _logger.LogError("Создание компании невыбранной коллекции");
             return;
         }
 
@@ -261,7 +264,7 @@ public partial class FormAirCraftCollection : Form
         if (collection == null)
         {
             MessageBox.Show("Коллекция не проинициализирована");
-            _logger.LogWarning("Не удалось инициализировать коллекцию");
+            _logger.LogError("Не удалось инициализировать коллекцию");
             return;
         }