Final

DoubleDeckerBus/DoubleDeckerBus/DoubleDeckerBus.csproj

@@ -9,21 +9,27 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <None Remove="nlog.config" />
+    <None Remove="serilog.json" />
   </ItemGroup>
 
   <ItemGroup>
-    <EmbeddedResource Include="nlog.config">
+    <EmbeddedResource Include="serilog.json">
+      <Generator>SettingsSingleFileGenerator</Generator>
+      <LastGenOutput>serilog.Designer.cs</LastGenOutput>
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </EmbeddedResource>
   </ItemGroup>
 
   <ItemGroup>
+    <PackageReference Include="Microsoft.Extensions.Configuration" Version="7.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Configuration.FileExtensions" Version="7.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="7.0.0" />
     <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="7.0.0" />
     <PackageReference Include="Microsoft.Extensions.Logging" Version="7.0.0" />
     <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="7.0.0" />
-    <PackageReference Include="NLog.Extensions.Logging" Version="5.2.0" />
     <PackageReference Include="Serilog.Extensions.Hosting" Version="5.0.1" />
+    <PackageReference Include="Serilog.Settings.Configuration" Version="3.4.0" />
+    <PackageReference Include="Serilog.Sinks.File" Version="5.0.0" />
   </ItemGroup>
 
   <ItemGroup>
@@ -35,6 +41,11 @@
       <AutoGen>True</AutoGen>
       <DependentUpon>Resources.resx</DependentUpon>
     </Compile>
+    <Compile Update="serilog.Designer.cs">
+      <DesignTimeSharedInput>True</DesignTimeSharedInput>
+      <AutoGen>True</AutoGen>
+      <DependentUpon>serilog.json</DependentUpon>
+    </Compile>
   </ItemGroup>
 
   <ItemGroup>

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} прошла не успешно");
                 }
             }
         }

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();

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 Serilog;
-using System.ServiceProcess;
+using Serilog.Extensions.Logging;
 
 namespace DoubleDeckerBus
 {
@@ -27,8 +28,14 @@ namespace DoubleDeckerBus
         private static void ConfigureServices(ServiceCollection services) {
             services.AddSingleton<FormMapWithSetBuses>().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);
             });
             
         }

DoubleDeckerBus/DoubleDeckerBus/SetBusesGeneric.cs

@@ -28,11 +28,16 @@ namespace DoubleDeckerBus
         }
 
         public int Insert(T bus, int position)
-        {   
+        {
-            // TODO проверка на _maxCount
+            if (BusyPlaces == _maxCount) {
-            if (position < 0 || position >= _maxCount || BusyPlaces == _maxCount) return -1;
+                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;

DoubleDeckerBus/DoubleDeckerBus/nlog.config

@@ -1,15 +0,0 @@
-<?xml version="1.0" encoding="utf-8" ?>
-<configuration>
-  <nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
-        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-        autoReload="true" internalLogLevel="Info">
-
-    <targets>
-      <target xsi:type="File" name="tofile" fileName="buslog-${shortdate}.log" />
-    </targets>
-
-    <rules>
-      <logger name="*" minlevel="Debug" writeTo="tofile" />
-    </rules>
-  </nlog>
-</configuration>

DoubleDeckerBus/DoubleDeckerBus/serilog.Designer.cs

@@ -0,0 +1,26 @@
+//------------------------------------------------------------------------------
+// <auto-generated>
+//     Этот код создан программой.
+//     Исполняемая версия:4.0.30319.42000
+//
+//     Изменения в этом файле могут привести к неправильной работе и будут потеряны в случае
+//     повторной генерации кода.
+// </auto-generated>
+//------------------------------------------------------------------------------
+
+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;
+            }
+        }
+    }
+}

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"
+    }
+  }
+}