diff --git a/HoistingCrane/HoistingCrane/FormCarCollection.cs b/HoistingCrane/HoistingCrane/FormCarCollection.cs index acd8ebf..cc3db0d 100644 --- a/HoistingCrane/HoistingCrane/FormCarCollection.cs +++ b/HoistingCrane/HoistingCrane/FormCarCollection.cs @@ -75,23 +75,23 @@ namespace HoistingCrane } private void SetCar(DrawningTrackedVehicle drawningTrackedVehicle) { - + if (_company == null || drawningTrackedVehicle == null) return; try { if (_company + drawningTrackedVehicle != -1) { MessageBox.Show("Объект добавлен"); pictureBox.Image = _company.Show(); - logger.LogInformation("Добавлен объект DrawningTrackedVehicle"); + logger.LogInformation("Добавлен объект {nameObject}", drawningTrackedVehicle.GetType().Name); } else { - throw new Exception(); + throw new Exception("Не удалось добавить объект. Заполнено максимальное количество ячеек"); } } catch (Exception ex) { - logger.LogError("Не удалось добавить объект"); + logger.LogInformation(ex.Message); return; } @@ -99,21 +99,22 @@ namespace HoistingCrane private void buttonDeleteCar_Click(object sender, EventArgs e) { + if (MessageBox.Show("Удалить объект?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No) return; + if (string.IsNullOrEmpty(maskedTextBox.Text) || _company == null) return; try { - if (MessageBox.Show("Удалить объект?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No) throw new Exception(); - if (string.IsNullOrEmpty(maskedTextBox.Text) || _company == null) throw new Exception(); int pos = Convert.ToInt32(maskedTextBox.Text); if ((_company - pos) != null) { MessageBox.Show("Объект удален!"); pictureBox.Image = _company.Show(); - logger.LogInformation("удален объект по позиции: {pos}", pos); + logger.LogInformation("Удален объект по позиции: {pos} ", pos); } + else throw new Exception(); } catch(Exception ex) { - logger.LogError("Не удалось удалить объект по позиции: {pos}", Convert.ToInt32(maskedTextBox.Text)); + logger.LogInformation("Не удалось удалить объект по позиции: {pos}", Convert.ToInt32(maskedTextBox.Text)); return; } } @@ -169,14 +170,14 @@ namespace HoistingCrane if (radioButtonMassive.Checked) { collectionType = CollectionType.Massive; + logger.LogInformation("Создана коллекция типа: {name}", collectionType); } else if (radioButtonList.Checked) { collectionType = CollectionType.List; + logger.LogInformation("Создана коллекция типа: {name}", collectionType); } - _storageCollection.AddCollection(textBoxCollectionName.Text, - collectionType); - RerfreshListBoxItems(); + _storageCollection.AddCollection(textBoxCollectionName.Text,collectionType);RerfreshListBoxItems(); } private void buttonDeleteCollection_Click(object sender, EventArgs e) { @@ -189,6 +190,7 @@ namespace HoistingCrane return; _storageCollection.DelCollection(listBoxCollection.SelectedItem.ToString()); RerfreshListBoxItems(); + logger.LogInformation("Коллекция успешно удалена"); } private void buttonCreateCompany_Click(object sender, EventArgs e) { @@ -212,7 +214,7 @@ namespace HoistingCrane } panelCompanyTool.Enabled = true; RerfreshListBoxItems(); - + logger.LogInformation("Создана компания"); }