finish

AirBomber/CollectionGenericObjects/AbstractCompany.cs

@@ -40,7 +40,7 @@ public abstract class AbstractCompany
     /// <summary>
     /// Вычисление максимального количества элементов, который можно разместить в окне
     /// </summary>
-    private int GetMaxCount => _pictureWidth * _pictureHeight / (_placeSizeWidth * _placeSizeHeight);
+    private int GetMaxCount => (_pictureWidth / _placeSizeWidth) * (_pictureHeight / _placeSizeHeight);
 
     /// <summary>
     /// Конструктор
@@ -62,9 +62,9 @@ public abstract class AbstractCompany
     /// <param name="company">Компания</param>
     /// <param name="boat">Добавляемый объект</param>
     /// <returns></returns>
-    public static int operator +(AbstractCompany company, DrawningAirPlane boat)
+    public static int operator +(AbstractCompany company, DrawningAirPlane airPlane)
     {
-        return company._collection?.Insert(boat) ?? -1;
+        return company._collection?.Insert(airPlane) ?? -1;
     }
 
     /// <summary>
@@ -101,8 +101,13 @@ public abstract class AbstractCompany
         SetObjectsPosition();
         for (int i = 0; i < (_collection?.Count ?? 0); ++i)
         {
-            DrawningAirPlane? obj = _collection?.Get(i);
+            try
-            obj?.DrawTransport(graphics);
+            {
+                DrawningAirPlane? obj = _collection?.Get(i);
+                obj?.DrawTransport(graphics);
+            }
+            catch (Exception) { }
+
         }
 
         return bitmap;
@@ -118,4 +123,4 @@ public abstract class AbstractCompany
     /// Расстановка объектов
     /// </summary>
     protected abstract void SetObjectsPosition();
-}
+}

AirBomber/CollectionGenericObjects/ListGenericObjects.cs

@@ -61,6 +61,7 @@ public class ListGenericObjects<T> : ICollectionGenericObjects<T>
             throw new CollectionOverflowException(Count);
         _collection.Insert(position, obj);
         return position;
+
     }
 
     public T Remove(int position)
@@ -78,4 +79,4 @@ public class ListGenericObjects<T> : ICollectionGenericObjects<T>
             yield return _collection[i];
         }
     }
-}
+}

AirBomber/CollectionGenericObjects/MassiveGenericObjects.cs

@@ -119,4 +119,4 @@ where T : class
             yield return _collection[i];
         }
     }
-}
+}

AirBomber/CollectionGenericObjects/StorageCollection.cs

@@ -70,7 +70,7 @@ public class StorageCollection<T>
                 return;
         }
     }
-        
+
     /// <summary>
     /// Удаление коллекции
     /// </summary>
@@ -201,7 +201,6 @@ public class StorageCollection<T>
                         }
                     }
                 }
-
                 _storages.Add(record[0], collection);
             }
         }

AirBomber/FormAirPlaneCollection.cs

@@ -12,337 +12,338 @@ using System.Windows.Forms;
 using Microsoft.Extensions.Logging;
 using AirBomber.Exceptions;
 
-namespace AirBomber
+namespace AirBomber;
+
+public partial class FormAirPlaneCollection : Form
 {
-    public partial class FormAirPlaneCollection : Form
+    /// <summary>
+    /// Хранилище коллекций
+    /// </summary>
+    private readonly StorageCollection<DrawningAirPlane> _storageCollection;
+
+    /// <summary>
+    /// Компания
+    /// </summary>
+    private AbstractCompany? _company = null;
+
+    private readonly ILogger _logger;
+
+    /// <summary>
+    /// Конструктор
+    /// </summary>
+    public FormAirPlaneCollection(ILogger<FormAirPlaneCollection> logger)
     {
-        /// <summary>
+        InitializeComponent();
-        /// Хранилище коллекций
+        _storageCollection = new();
-        /// </summary>
+        _logger = logger;
-        private readonly StorageCollection<DrawningAirPlane> _storageCollection;
+        _logger.LogInformation("Форма загрузилась");
+    }
 
-        /// <summary>
+    /// <summary>
-        /// Компания
+    /// Выбор компании
-        /// </summary>
+    /// </summary>
-        private AbstractCompany? _company;
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
+    private void comboBoxSelectorCompany_SelectedIndexChanged(object sender, EventArgs e)
+    {
+        panelCompanyTools.Enabled = false;
+    }
 
-        private readonly ILogger _logger;
+    /// <summary>
+    /// Создание объекта класса-перемещения
+    /// </summary>
+    /// <param name="type">Тип создаваемого объекта</param>
 
-        /// <summary>
+
-        /// Конструктор
+    /// <summary>
-        /// </summary>
+    /// Добавление самолета
-        public FormAirPlaneCollection(ILogger<FormAirPlaneCollection> logger)
+    /// </summary>
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
+    private void buttonAddAirPlane_Click(object sender, EventArgs e)
+    {
+        FormAirPlaneConfig form = new();
+        //TODO передать метод
+        form.Show();
+        form.AddEvent(SetAirPlane);
+    }
+
+    /// <summary>
+    /// Добавление самолета в коллекцию
+    /// </summary>
+    /// <param name="airplane"></param>
+    private void SetAirPlane(DrawningAirPlane airplane)
+    {
+        try
         {
-            InitializeComponent();
+            if (_company == null || airplane == null)
-            _storageCollection = new();
-        }
-
-        /// <summary>
-        /// Выбор компании
-        /// </summary>
-        /// <param name="sender"></param>
-        /// <param name="e"></param>
-        private void comboBoxSelectorCompany_SelectedIndexChanged(object sender, EventArgs e)
-        {
-            panelCompanyTools.Enabled = false;
-        }
-
-        /// <summary>
-        /// Создание объекта класса-перемещения
-        /// </summary>
-        /// <param name="type">Тип создаваемого объекта</param>
-
-
-        /// <summary>
-        /// Добавление самолета
-        /// </summary>
-        /// <param name="sender"></param>
-        /// <param name="e"></param>
-        private void buttonAddAirPlane_Click(object sender, EventArgs e)
-        {
-            FormAirPlaneConfig form = new();
-            //TODO передать метод
-            form.Show();
-            form.AddEvent(SetAirPlane);
-        }
-
-        /// <summary>
-        /// Добавление самолета в коллекцию
-        /// </summary>
-        /// <param name="airplane"></param>
-        private void SetAirPlane(DrawningAirPlane airplane)
-        {
-            try
-            {
-                if (_company == null || airplane == null)
-                {
-                    return;
-                }
-
-                if (_company + airplane != -1)
-                {
-                    MessageBox.Show("Объект добавлен");
-                    pictureBox.Image = _company.Show();
-                    _logger.LogInformation("Добавлен объект: " + airplane.GetDataForSave());
-                }
-            }
-            catch (ObjectNotFoundException) { }
-            catch (CollectionOverflowException ex)
-            {
-                MessageBox.Show("В коллекции превышено допустимое количество элементов");
-                _logger.LogError("Ошибка: {Message}", ex.Message);
-            }
-
-        }
-
-        /// <summary>
-        /// Получение цвета
-        /// </summary>
-        /// <param name="random">Генератор случайных чисел</param>
-        /// <returns></returns>
-        private static Color GetColor(Random random)
-        {
-            Color color = Color.FromArgb(random.Next(0, 256), random.Next(0, 256), random.Next(0, 256));
-            ColorDialog dialog = new();
-            if (dialog.ShowDialog() == DialogResult.OK)
-            {
-                color = dialog.Color;
-            }
-
-            return color;
-        }
-
-        /// <summary>
-        /// Удаление объекта
-        /// </summary>
-        /// <param name="sender"></param>
-        /// <param name="e"></param>
-        private void ButtonRemoveAirPlane_Click(object sender, EventArgs e)
-        {
-            int pos = Convert.ToInt32(maskedTextBox.Text);
-            try
-            {
-                if (string.IsNullOrEmpty(maskedTextBox.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("Объект удален");
-                }
-            }
-            catch (Exception ex)
-            {
-                MessageBox.Show("Не найден объект по позиции " + pos);
-                _logger.LogError("Ошибка: {Message}", ex.Message);
-            }
-        }
-
-        /// <summary>
-        /// Передача объекта в другую форму
-        /// </summary>
-        /// <param name="sender"></param>
-        /// <param name="e"></param>
-        private void ButtonGoToCheck_Click(object sender, EventArgs e)
-        {
-            if (_company == null)
             {
                 return;
             }
 
-            DrawningAirPlane? airPlane = null;
+            if (_company + airplane != -1)
-            int counter = 100;
+            {
+                MessageBox.Show("Объект добавлен");
+                pictureBox.Image = _company.Show();
+                _logger.LogInformation("Добавлен объект: " + airplane.GetDataForSave());
+            }
+        }
+        catch (ObjectNotFoundException) { }
+        catch (CollectionOverflowException ex)
+        {
+            MessageBox.Show("В коллекции превышено допустимое количество элементов");
+            _logger.LogError("Ошибка: {Message}", ex.Message);
+        }
+
+    }
+
+    /// <summary>
+    /// Получение цвета
+    /// </summary>
+    /// <param name="random">Генератор случайных чисел</param>
+    /// <returns></returns>
+    private static Color GetColor(Random random)
+    {
+        Color color = Color.FromArgb(random.Next(0, 256), random.Next(0, 256), random.Next(0, 256));
+        ColorDialog dialog = new();
+        if (dialog.ShowDialog() == DialogResult.OK)
+        {
+            color = dialog.Color;
+        }
+
+        return color;
+    }
+
+    /// <summary>
+    /// Удаление объекта
+    /// </summary>
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
+    private void ButtonRemoveAirPlane_Click(object sender, EventArgs e)
+    {
+        int pos = Convert.ToInt32(maskedTextBox.Text);
+        try
+        {
+            if (string.IsNullOrEmpty(maskedTextBox.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("Объект удален");
+            }
+        }
+        catch (Exception ex)
+        {
+            MessageBox.Show("Не найден объект по позиции " + pos);
+            _logger.LogError("Ошибка: {Message}", ex.Message);
+        }
+    }
+
+    /// <summary>
+    /// Передача объекта в другую форму
+    /// </summary>
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
+    private void ButtonGoToCheck_Click(object sender, EventArgs e)
+    {
+        if (_company == null)
+        {
+            return;
+        }
+
+        DrawningAirPlane? airPlane = null;
+        int counter = 100;
+        try
+        {
+            while (airPlane == null)
+            {
+                airPlane = _company.GetRandomObject();
+                counter--;
+                if (counter <= 0)
+                {
+                    break;
+                }
+            }
+
+            if (airPlane == null)
+            {
+                return;
+            }
+
+            FormAirBomber form = new FormAirBomber();
+            form.SetAirPlane = airPlane;
+            form.ShowDialog();
+        }
+        catch (Exception ex)
+        {
+            MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+        }
+    }
+
+    /// <summary>
+    /// Перерисовка коллекции
+    /// </summary>
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
+    private void ButtonRefresh_Click(object sender, EventArgs e)
+    {
+        if (_company == null)
+        {
+            return;
+        }
+
+        pictureBox.Image = _company.Show();
+    }
+
+    private void FormAirPlaneCollection_Load(object sender, EventArgs e)
+    {
+
+    }
+
+    private void buttonCollectionAdd_Click(object sender, EventArgs e)
+    {
+        if (string.IsNullOrEmpty(textBoxCollectionName.Text) || (!radioButtonList.Checked && !radioButtonMassive.Checked))
+        {
+            MessageBox.Show("Не все данный заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error);
+            return;
+        }
+
+        try
+        {
+            CollectionType collectionType = CollectionType.None;
+            if (radioButtonMassive.Checked)
+            {
+                collectionType = CollectionType.Massive;
+            }
+            else if (radioButtonList.Checked)
+            {
+                collectionType = CollectionType.List;
+            }
+
+            _storageCollection.AddCollection(textBoxCollectionName.Text, collectionType);
+            RefreshListBoxItems();
+            _logger.LogInformation("Добавлена коллекция:", textBoxCollectionName.Text);
+        }
+        catch (Exception ex)
+        {
+            _logger.LogError("Ошибка: {Message}", ex.Message);
+        }
+    }
+
+    private void RefreshListBoxItems()
+    {
+        listBoxCollection.Items.Clear();
+        for (int i = 0; i < _storageCollection.Keys?.Count; ++i)
+        {
+            string? colName = _storageCollection.Keys?[i];
+            if (!string.IsNullOrEmpty(colName))
+            {
+                listBoxCollection.Items.Add(colName);
+            }
+        }
+    }
+
+    private void buttonCollectionDel_Click(object sender, EventArgs e)
+    {
+        if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItems == null)
+        {
+            MessageBox.Show("Коллекция не выбрана");
+            return;
+        }
+        if (MessageBox.Show("Удалить коллекцию?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No)
+        {
+            return;
+        }
+        _storageCollection.DelCollection(listBoxCollection.SelectedItem.ToString());
+        RefreshListBoxItems();
+    }
+    /// <summary>
+    /// Создание компании
+    /// </summary>
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
+    private void buttonCreateCompany_Click(object sender, EventArgs e)
+    {
+        if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItem == null)
+        {
+            MessageBox.Show("Коллекция не выбрана");
+            return;
+        }
+
+        ICollectionGenericObjects<DrawningAirPlane>? collection = _storageCollection[listBoxCollection.SelectedItem.ToString() ?? string.Empty];
+        if (collection == null)
+        {
+            MessageBox.Show("Коллекция не проинициализирована");
+            return;
+        }
+
+        switch (comboBoxSelectorCompany.Text)
+        {
+            case "Хранилище":
+                _company = new AirPlaneSharingService(pictureBox.Width, pictureBox.Height, collection);
+                break;
+        }
+
+        panelCompanyTools.Enabled = true;
+        RefreshListBoxItems();
+    }
+
+    /// <summary>
+    /// Обработка нажатия "Сохранение"
+    /// </summary>
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
+    private void saveToolStripMenuItem_Click(object sender, EventArgs e)
+    {
+        if (saveFileDialog.ShowDialog() == DialogResult.OK)
+        {
             try
             {
-                while (airPlane == null)
+                _storageCollection.SaveData(saveFileDialog.FileName);
-                {
+                MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
-                    airPlane = _company.GetRandomObject();
+                _logger.LogInformation("Сохранение в файл: {filename}", saveFileDialog.FileName);
-                    counter--;
-                    if (counter <= 0)
-                    {
-                        break;
-                    }
-                }
-
-                if (airPlane == null)
-                {
-                    return;
-                }
-
-                FormAirBomber form = new FormAirBomber();
-                form.SetAirPlane = airPlane;
-                form.ShowDialog();
             }
             catch (Exception ex)
             {
                 MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
-            }
+                _logger.LogError("Ошибка: {Message}", ex.Message);
-        }
-
-        /// <summary>
-        /// Перерисовка коллекции
-        /// </summary>
-        /// <param name="sender"></param>
-        /// <param name="e"></param>
-        private void ButtonRefresh_Click(object sender, EventArgs e)
-        {
-            if (_company == null)
-            {
-                return;
             }
 
-            pictureBox.Image = _company.Show();
-        }
-
-        private void FormAirPlaneCollection_Load(object sender, EventArgs e)
-        {
 
         }
+    }
 
-        private void buttonCollectionAdd_Click(object sender, EventArgs e)
+    /// <summary>
+    /// Обработка кнопки загрузки
+    /// </summary>
+    /// <param name="sender"></param>
+    /// <param name="e"></param>
+    private void loadToolStripMenuItem_Click(object sender, EventArgs e)
+    {
+        if (openFileDialog.ShowDialog() == DialogResult.OK)
         {
-            if (string.IsNullOrEmpty(textBoxCollectionName.Text) || (!radioButtonList.Checked && !radioButtonMassive.Checked))
-            {
-                MessageBox.Show("Не все данный заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error);
-                return;
-            }
-
             try
             {
-                CollectionType collectionType = CollectionType.None;
+                _storageCollection.LoadData(openFileDialog.FileName);
-                if (radioButtonMassive.Checked)
+                MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
-                {
-                    collectionType = CollectionType.Massive;
-                }
-                else if (radioButtonList.Checked)
-                {
-                    collectionType = CollectionType.List;
-                }
-
-                _storageCollection.AddCollection(textBoxCollectionName.Text, collectionType);
                 RefreshListBoxItems();
-                _logger.LogInformation("Добавлена коллекция:", textBoxCollectionName.Text);
+                _logger.LogInformation("Загрузка из файла: {filename}", openFileDialog.FileName);
             }
             catch (Exception ex)
             {
+                MessageBox.Show("Не загрузилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
                 _logger.LogError("Ошибка: {Message}", ex.Message);
             }
         }
-
-        private void RefreshListBoxItems()
-        {
-            listBoxCollection.Items.Clear();
-            for (int i = 0; i < _storageCollection.Keys?.Count; ++i)
-            {
-                string? colName = _storageCollection.Keys?[i];
-                if (!string.IsNullOrEmpty(colName))
-                {
-                    listBoxCollection.Items.Add(colName);
-                }
-            }
-        }
-
-        private void buttonCollectionDel_Click(object sender, EventArgs e)
-        {
-            if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItems == null)
-            {
-                MessageBox.Show("Коллекция не выбрана");
-                return;
-            }
-            if (MessageBox.Show("Удалить коллекцию?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No)
-            {
-                return;
-            }
-            _storageCollection.DelCollection(listBoxCollection.SelectedItem.ToString());
-            RefreshListBoxItems();
-        }
-        /// <summary>
-        /// Создание компании
-        /// </summary>
-        /// <param name="sender"></param>
-        /// <param name="e"></param>
-        private void buttonCreateCompany_Click(object sender, EventArgs e)
-        {
-            if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItem == null)
-            {
-                MessageBox.Show("Коллекция не выбрана");
-                return;
-            }
-
-            ICollectionGenericObjects<DrawningAirPlane>? collection = _storageCollection[listBoxCollection.SelectedItem.ToString() ?? string.Empty];
-            if (collection == null)
-            {
-                MessageBox.Show("Коллекция не проинициализирована");
-                return;
-            }
-
-            switch (comboBoxSelectorCompany.Text)
-            {
-                case "Хранилище":
-                    _company = new AirPlaneSharingService(pictureBox.Width, pictureBox.Height, collection);
-                    break;
-            }
-
-            panelCompanyTools.Enabled = true;
-            RefreshListBoxItems();
-        }
-
-        /// <summary>
-        /// Обработка нажатия "Сохранение"
-        /// </summary>
-        /// <param name="sender"></param>
-        /// <param name="e"></param>
-        private void saveToolStripMenuItem_Click(object sender, EventArgs e)
-        {
-            if (saveFileDialog.ShowDialog() == DialogResult.OK)
-            {
-                try
-                {
-                    _storageCollection.SaveData(saveFileDialog.FileName);
-                    MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
-                    _logger.LogInformation("Сохранение в файл: {filename}", saveFileDialog.FileName);
-                }
-                catch (Exception ex)
-                {
-                    MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
-                    _logger.LogError("Ошибка: {Message}", ex.Message);
-                }
-
-
-            }
-        }
-
-        /// <summary>
-        /// Обработка кнопки загрузки
-        /// </summary>
-        /// <param name="sender"></param>
-        /// <param name="e"></param>
-        private void loadToolStripMenuItem_Click(object sender, EventArgs e)
-        {
-            if (openFileDialog.ShowDialog() == DialogResult.OK)
-            {
-                try
-                {
-                    _storageCollection.LoadData(openFileDialog.FileName);
-                    MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
-                    RefreshListBoxItems();
-                    _logger.LogInformation("Загрузка из файла: {filename}", openFileDialog.FileName);
-                }
-                catch (Exception ex)
-                {
-                    MessageBox.Show("Не загрузилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
-                    _logger.LogError("Ошибка: {Message}", ex.Message);
-                }
-            }
-        }
     }
 }