diff --git a/DoubleDeckerBus/DoubleDeckerBus/DoubleDeckerBus.csproj b/DoubleDeckerBus/DoubleDeckerBus/DoubleDeckerBus.csproj index b0ba214..4200721 100644 --- a/DoubleDeckerBus/DoubleDeckerBus/DoubleDeckerBus.csproj +++ b/DoubleDeckerBus/DoubleDeckerBus/DoubleDeckerBus.csproj @@ -9,21 +9,27 @@ - + - + + SettingsSingleFileGenerator + serilog.Designer.cs Always + + + - + + @@ -35,6 +41,11 @@ True Resources.resx + + True + True + serilog.json + diff --git a/DoubleDeckerBus/DoubleDeckerBus/FormMapWithSetBuses.cs b/DoubleDeckerBus/DoubleDeckerBus/FormMapWithSetBuses.cs index eec2e1c..779e742 100644 --- a/DoubleDeckerBus/DoubleDeckerBus/FormMapWithSetBuses.cs +++ b/DoubleDeckerBus/DoubleDeckerBus/FormMapWithSetBuses.cs @@ -74,10 +74,12 @@ namespace DoubleDeckerBus { MessageBox.Show("Объект добавлен"); pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); + _logger.LogInformation($"Объект добавлен"); } else { MessageBox.Show("Не удалось добавить объект"); + _logger.LogInformation($"Объект не добаавлен"); } } @@ -97,20 +99,24 @@ namespace DoubleDeckerBus if (_mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty] - pos != null) { MessageBox.Show("Объект удален"); + _logger.LogInformation($"Объект удален"); pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); } else { MessageBox.Show("Не удалось удалить объект"); + _logger.LogInformation($"Объект не удален"); } } catch (BusNotFoundException ex) { MessageBox.Show($"Ошибка удаления: {ex.Message}"); + _logger.LogWarning("Автобус не найден"); } catch (Exception ex) { MessageBox.Show($"Неизветсная шибка: {ex.Message}"); + _logger.LogWarning("Неизвестная ошибка при удалении"); } } @@ -122,6 +128,7 @@ namespace DoubleDeckerBus return; } pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); + _logger.LogInformation($"Отображение хранилища"); } private void ButtonShowOnMap_Click(object sender, EventArgs e) @@ -131,6 +138,7 @@ namespace DoubleDeckerBus return; } pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowOnMap(); + _logger.LogInformation($"Отображение карты"); } private void ButtonMove_Click(object sender, EventArgs e) @@ -158,6 +166,7 @@ namespace DoubleDeckerBus break; } pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].MoveObject(dir); + _logger.LogInformation($"Передвижение {name}"); } private void ButtonAddMap_Click(object sender, EventArgs e) @@ -165,11 +174,13 @@ namespace DoubleDeckerBus if (comboBoxSelectorMap.SelectedIndex == -1 || string.IsNullOrEmpty(textBoxNewMapName.Text)) { MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning($"Не все данные заполнены при добавлени карты"); return; } if (!_mapsDict.ContainsKey(comboBoxSelectorMap.Text)) { MessageBox.Show("Нет такой карты", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning($"Нет такой карты {comboBoxSelectorMap.Text}"); return; } _mapsCollection.AddMap(textBoxNewMapName.Text, _mapsDict[comboBoxSelectorMap.Text]); @@ -180,18 +191,21 @@ namespace DoubleDeckerBus private void listBoxMaps_SelectedIndexChanged(object sender, EventArgs e) { pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); + _logger.LogInformation($"Выбраная карта изменилась {listBoxMaps.SelectedItem?.ToString() ?? string.Empty}"); } private void ButtonDeleteMap_Click(object sender, EventArgs e) { if (listBoxMaps.SelectedIndex == -1) { + _logger.LogWarning("Удаление карты не произошло. Не выбрана карта"); return; } if (MessageBox.Show($"Удалить карту {listBoxMaps.SelectedItem}?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.Yes) { _mapsCollection.DelMap(listBoxMaps.SelectedItem?.ToString() ?? string.Empty); + _logger.LogInformation($"Удалена карта {listBoxMaps.SelectedItem?.ToString() ?? string.Empty}"); ReloadMaps(); } } @@ -204,9 +218,11 @@ namespace DoubleDeckerBus { _mapsCollection.SaveData(saveFileDialog.FileName); MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); + _logger.LogInformation($"Сохранение {openFileDialog.FileName} прошло успешно"); } catch (Exception ex) { MessageBox.Show($"Не сохранилось: {ex.Message}", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning($"Сохранение {openFileDialog.FileName} прошло не успешно"); } } } @@ -218,11 +234,13 @@ namespace DoubleDeckerBus try { _mapsCollection.LoadData(openFileDialog.FileName); MessageBox.Show("Загрузка прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); + _logger.LogInformation($"Загрузка из файла {openFileDialog.FileName} прошла успешна"); ReloadMaps(); } catch (Exception ex) { MessageBox.Show($"Ошибка при загрузке: {ex.Message}", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning($"Загрузка из файла {openFileDialog.FileName} прошла не успешно"); } } } diff --git a/DoubleDeckerBus/DoubleDeckerBus/MapsCollection.cs b/DoubleDeckerBus/DoubleDeckerBus/MapsCollection.cs index c7e70e2..eff74e7 100644 --- a/DoubleDeckerBus/DoubleDeckerBus/MapsCollection.cs +++ b/DoubleDeckerBus/DoubleDeckerBus/MapsCollection.cs @@ -71,7 +71,7 @@ namespace DoubleDeckerBus { if (!File.Exists(filename)) { - throw new Exception("Файл не найден"); + throw new FileNotFoundException("Файл не найден"); } string line; using (StreamReader sw = new(filename)) @@ -79,7 +79,7 @@ namespace DoubleDeckerBus line = sw.ReadLine(); if (line == null || !line.Contains("MapsCollection")) { - throw new Exception("Формат данных в файле не совпадает"); + throw new FileFormatException("Формат данных в файле не совпадает"); } _mapStorages.Clear(); diff --git a/DoubleDeckerBus/DoubleDeckerBus/Program.cs b/DoubleDeckerBus/DoubleDeckerBus/Program.cs index c3de90f..75a901f 100644 --- a/DoubleDeckerBus/DoubleDeckerBus/Program.cs +++ b/DoubleDeckerBus/DoubleDeckerBus/Program.cs @@ -1,7 +1,8 @@ +using Microsoft.Extensions.Configuration; using Microsoft.Extensions.DependencyInjection; using Microsoft.Extensions.Logging; -using NLog.Extensions.Logging; -using System.ServiceProcess; +using Serilog; +using Serilog.Extensions.Logging; namespace DoubleDeckerBus { @@ -27,8 +28,14 @@ namespace DoubleDeckerBus private static void ConfigureServices(ServiceCollection services) { services.AddSingleton().AddLogging(option => { + var configuration = new ConfigurationBuilder().SetBasePath(Directory.GetCurrentDirectory()).AddJsonFile(path: "serilog.json").Build(); + + var logger = new LoggerConfiguration() + .ReadFrom.Configuration(configuration) + .CreateLogger(); + option.SetMinimumLevel(LogLevel.Information); - option.AddNLog("nlog.config"); + option.AddSerilog(logger); }); } diff --git a/DoubleDeckerBus/DoubleDeckerBus/SetBusesGeneric.cs b/DoubleDeckerBus/DoubleDeckerBus/SetBusesGeneric.cs index 00927bc..eed9d3e 100644 --- a/DoubleDeckerBus/DoubleDeckerBus/SetBusesGeneric.cs +++ b/DoubleDeckerBus/DoubleDeckerBus/SetBusesGeneric.cs @@ -28,11 +28,16 @@ namespace DoubleDeckerBus } public int Insert(T bus, int position) - { - // TODO проверка на _maxCount - if (position < 0 || position >= _maxCount || BusyPlaces == _maxCount) return -1; + { + if (BusyPlaces == _maxCount) { + throw new StorageOverflowException(BusyPlaces); + } + + if (position < 0 || position >= _maxCount) + { + throw new BusNotFoundException("Место указано неверно"); + } - // TODO проверка позиции BusyPlaces++; _places.Insert(position, bus); @@ -41,8 +46,10 @@ namespace DoubleDeckerBus public T Remove(int position) { - // TODO проверка позиции - if (position < 0 || position >= _maxCount) return null; + + if (position < 0 || position >= _maxCount) { + throw new BusNotFoundException(position); + } T savedBus = _places[position]; _places.RemoveAt(position); return savedBus; diff --git a/DoubleDeckerBus/DoubleDeckerBus/nlog.config b/DoubleDeckerBus/DoubleDeckerBus/nlog.config deleted file mode 100644 index 3e28e76..0000000 --- a/DoubleDeckerBus/DoubleDeckerBus/nlog.config +++ /dev/null @@ -1,15 +0,0 @@ - - - - - - - - - - - - - \ No newline at end of file diff --git a/DoubleDeckerBus/DoubleDeckerBus/serilog.Designer.cs b/DoubleDeckerBus/DoubleDeckerBus/serilog.Designer.cs new file mode 100644 index 0000000..456cc8b --- /dev/null +++ b/DoubleDeckerBus/DoubleDeckerBus/serilog.Designer.cs @@ -0,0 +1,26 @@ +//------------------------------------------------------------------------------ +// +// Этот код создан программой. +// Исполняемая версия:4.0.30319.42000 +// +// Изменения в этом файле могут привести к неправильной работе и будут потеряны в случае +// повторной генерации кода. +// +//------------------------------------------------------------------------------ + +namespace DoubleDeckerBus { + + + [global::System.Runtime.CompilerServices.CompilerGeneratedAttribute()] + [global::System.CodeDom.Compiler.GeneratedCodeAttribute("Microsoft.VisualStudio.Editors.SettingsDesigner.SettingsSingleFileGenerator", "17.3.0.0")] + internal sealed partial class serilog : global::System.Configuration.ApplicationSettingsBase { + + private static serilog defaultInstance = ((serilog)(global::System.Configuration.ApplicationSettingsBase.Synchronized(new serilog()))); + + public static serilog Default { + get { + return defaultInstance; + } + } + } +} diff --git a/DoubleDeckerBus/DoubleDeckerBus/serilog.json b/DoubleDeckerBus/DoubleDeckerBus/serilog.json new file mode 100644 index 0000000..4c3cf19 --- /dev/null +++ b/DoubleDeckerBus/DoubleDeckerBus/serilog.json @@ -0,0 +1,20 @@ +{ + "Serilog": { + "Using": [ "Serilog.Sinks.File" ], + "MinimumLevel": "Information", + "WriteTo": [ + { + "Name": "File", + "Args": { + "path": "log_.log", + "rollingInterval": "Day", + "outputTemplate": "[{Timestamp:HH:mm:ss.fff}]{Level:u4}: {Message:lj}{NewLine}{Exception}" + } + } + ], + "Enrich": [ "FromLogContext", "WithMachineName", "WithThreadId" ], + "Properties": { + "Application": "DoubleDeckerBus" + } + } +} \ No newline at end of file