Готово

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;
         }
-
+        try
-        DrawningAirCraft? aircraft = null;
-        int counter = 100;
-        while (aircraft == null)
         {
-            aircraft = _company.GetRandomObject();
+            DrawningAirCraft? aircraft = null;
-            counter--;
+            int counter = 100;
-            if (counter <= 0)
+            while (aircraft == null)
             {
-                break;
+                aircraft = _company.GetRandomObject();
+                counter--;
+                if (counter <= 0)
+                {
+                    break;
+                }
             }
+            if (aircraft == null)
+            {
+                return;
+            }
+            FormAirFighter form = new()
+            {
+                SetAirCraft = aircraft
+            };
+            form.ShowDialog();
         }
-
+        catch (ObjectNotFoundException)
-        if (aircraft == null)
         {
-            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;
         }