diff --git a/SelfPropelledArtilleryUnit/ArtilleryNotFoundException.cs b/SelfPropelledArtilleryUnit/ArtilleryNotFoundException.cs new file mode 100644 index 0000000..261668b --- /dev/null +++ b/SelfPropelledArtilleryUnit/ArtilleryNotFoundException.cs @@ -0,0 +1,19 @@ +using System; +using System.Collections.Generic; +using System.Linq; +using System.Runtime.Serialization; +using System.Text; +using System.Threading.Tasks; + +namespace Artilleries +{ + [Serializable] + internal class ArtilleryNotFoundException : ApplicationException + { + public ArtilleryNotFoundException(int i) : base($"Не найден объект по позиции {i}") { } + public ArtilleryNotFoundException() : base() { } + public ArtilleryNotFoundException(string message) : base(message) { } + public ArtilleryNotFoundException(string message, Exception exception) : base(message, exception) { } + protected ArtilleryNotFoundException(SerializationInfo info, StreamingContext context) : base(info, context) { } + } +} diff --git a/SelfPropelledArtilleryUnit/FormMapWithSetArtilleries.cs b/SelfPropelledArtilleryUnit/FormMapWithSetArtilleries.cs index f16b53a..bd06995 100644 --- a/SelfPropelledArtilleryUnit/FormMapWithSetArtilleries.cs +++ b/SelfPropelledArtilleryUnit/FormMapWithSetArtilleries.cs @@ -1,4 +1,5 @@ -using System; +using Microsoft.Extensions.Logging; +using System; using System.Collections.Generic; using System.ComponentModel; using System.Data; @@ -21,10 +22,13 @@ namespace Artilleries private readonly MapsCollection _mapsCollection; - public FormMapWithSetArtilleries() + private readonly ILogger _logger; + + public FormMapWithSetArtilleries(ILogger logger) { InitializeComponent(); _mapsCollection = new MapsCollection(pictureBoxArtilleries.Width, pictureBoxArtilleries.Height); + _logger = logger; comboBoxSelectorMap.Items.Clear(); foreach(var element in _mapsDict) { @@ -56,20 +60,24 @@ namespace Artilleries if (comboBoxSelectorMap.SelectedIndex == -1 || string.IsNullOrEmpty(textBoxNewMapName.Text)) { MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning("При добавлении карты {0}", comboBoxSelectorMap.SelectedIndex == -1 ? "не выбрали карту" : "не указали её название"); return; } if (!_mapsDict.ContainsKey(comboBoxSelectorMap.Text)) { MessageBox.Show("Нет такой карты", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning("Нет карты с названием: {0}", textBoxNewMapName.Text); return; } _mapsCollection.AddMap(textBoxNewMapName.Text, _mapsDict[comboBoxSelectorMap.Text]); + _logger.LogInformation("Добавлена карта с названием \"{1}\" типа \"{0}\"", textBoxNewMapName.Text, comboBoxSelectorMap.Text); ReloadMaps(); } private void listBoxMaps_SelectedIndexChanged(object sender, EventArgs e) { pictureBoxArtilleries.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? String.Empty].ShowSet(); + _logger.LogInformation("Переход на карту \"{0}\"", listBoxMaps.SelectedItem?.ToString() ?? String.Empty); } private void buttonDeleteMap_Click(object sender, EventArgs e) @@ -82,6 +90,7 @@ namespace Artilleries if (MessageBox.Show($"Удалить карту {listBoxMaps.SelectedItem}?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.Yes) { _mapsCollection.DelMap(listBoxMaps.SelectedItem?.ToString() ?? string.Empty); + _logger.LogInformation("Удалена карта \"{0}\"", listBoxMaps.SelectedItem?.ToString() ?? string.Empty); ReloadMaps(); } } @@ -95,14 +104,30 @@ namespace Artilleries return; } - if (_mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? String.Empty] + new DrawingObjectArtillery(artillery) != -1) + try { - MessageBox.Show("Объект добавлен"); - } else - { - MessageBox.Show("Не удалось добавить объект"); + if (_mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? String.Empty] + new DrawingObjectArtillery(artillery) != -1) + { + _logger.LogInformation("Добавлен новый объект"); + MessageBox.Show("Объект добавлен"); + } + else + { + _logger.LogWarning("Не удалось добавить объект"); + MessageBox.Show("Не удалось добавить объект"); + } + pictureBoxArtilleries.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? String.Empty].ShowSet(); + } + catch (StorageOverflowException ex) + { + _logger.LogWarning("Ошибка переполнения хранилища: {0}", ex.Message); + MessageBox.Show($"Ошибка переполнения хранилища: {ex.Message}", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + } + catch (Exception ex) + { + _logger.LogWarning("Неизвестная ошибка: {0}", ex.Message); + MessageBox.Show($"Неизвестная ошибка: {ex.Message}"); } - pictureBoxArtilleries.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? String.Empty].ShowSet(); }); formArtilleryConfig.Show(); } @@ -122,14 +147,24 @@ namespace Artilleries return; } int pos = Convert.ToInt32(maskedTextBoxPosition.Text); - if (_mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty] - pos != null) + try { - MessageBox.Show("Объект удален"); - pictureBoxArtilleries.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); + if (_mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty] - pos != null) + { + MessageBox.Show("Объект удален"); + _logger.LogInformation("Удалён объект на позиции {0}", pos); + pictureBoxArtilleries.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); + } + else + { + _logger.LogWarning("Не удалось удалить объект по позиции {0}. Объект равен null", pos); + MessageBox.Show("Не удалось удалить объект"); + } } - else + catch (ArtilleryNotFoundException ex) { - MessageBox.Show("Не удалось удалить объект"); + _logger.LogWarning("Ошибка удаления: {0}", ex.Message); + MessageBox.Show($"Ошибка удаления: {ex.Message}"); } } @@ -182,13 +217,16 @@ namespace Artilleries { if (saveFileDialog.ShowDialog() == DialogResult.OK) { - if (_mapsCollection.SaveData(saveFileDialog.FileName)) + try { + _mapsCollection.SaveData(saveFileDialog.FileName); + _logger.LogInformation("Сохранение в файл \"{0}\"", saveFileDialog.FileName); MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); } - else + catch (Exception ex) { - MessageBox.Show("Не сохранилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning("Не сохранить файл \"{0}\": {1}", saveFileDialog.FileName, ex.Message); + MessageBox.Show($"Не сохранилось: {ex.Message}", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); } } } @@ -197,16 +235,19 @@ namespace Artilleries { if (loadFileDialog.ShowDialog() == DialogResult.OK) { - if (_mapsCollection.LoadData(loadFileDialog.FileName)) + try { + _mapsCollection.LoadData(loadFileDialog.FileName); + _logger.LogInformation("Загрузка из файла \"{0}\"", loadFileDialog.FileName); MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); + ReloadMaps(); + pictureBoxArtilleries.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); } - else + catch (Exception ex) { - MessageBox.Show("Не загрузилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning("Не удалось загрузить файл \"{0}\": {1}", loadFileDialog.FileName, ex.Message); + MessageBox.Show($"Не загрузилось: {ex.Message}", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); } - ReloadMaps(); - pictureBoxArtilleries.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); } } } diff --git a/SelfPropelledArtilleryUnit/MapsCollection.cs b/SelfPropelledArtilleryUnit/MapsCollection.cs index 8e657f8..f0c5483 100644 --- a/SelfPropelledArtilleryUnit/MapsCollection.cs +++ b/SelfPropelledArtilleryUnit/MapsCollection.cs @@ -49,7 +49,7 @@ namespace Artilleries } } - public bool SaveData(string filename) + public void SaveData(string filename) { if (File.Exists(filename)) { @@ -65,15 +65,13 @@ namespace Artilleries sw.WriteLine($"{storage.Key}{separatorDict}{storage.Value.GetData(separatorDict, separatorData)}"); } } - - return true; } - public bool LoadData(string filename) + public void LoadData(string filename) { if (!File.Exists(filename)) { - return false; + throw new FileNotFoundException("Файл не найден"); } using (FileStream fs = new FileStream(filename, FileMode.Open)) @@ -83,7 +81,7 @@ namespace Artilleries if (current_line == null || !current_line.Contains("MapsCollection")) { - return false; + throw new FileFormatException("Неверный формат файла"); } _mapsStorage.Clear(); @@ -105,8 +103,6 @@ namespace Artilleries _mapsStorage.Add(elements[0], new MapWithSetArtilleriesGeneric(_pictureWidth, _pictureHeight, map)); _mapsStorage[elements[0]].LoadData(elements[2].Split(separatorData, StringSplitOptions.RemoveEmptyEntries)); } - - return true; } } } diff --git a/SelfPropelledArtilleryUnit/Program.cs b/SelfPropelledArtilleryUnit/Program.cs index 5f48e30..d1fb3ac 100644 --- a/SelfPropelledArtilleryUnit/Program.cs +++ b/SelfPropelledArtilleryUnit/Program.cs @@ -1,3 +1,8 @@ +using Microsoft.Extensions.Configuration; +using Microsoft.Extensions.DependencyInjection; +using Microsoft.Extensions.Logging; +using Serilog; + namespace Artilleries { internal static class Program @@ -6,7 +11,31 @@ namespace Artilleries static void Main() { ApplicationConfiguration.Initialize(); - Application.Run(new FormMapWithSetArtilleries()); + var services = new ServiceCollection(); + ConfigureServices(services); + using (ServiceProvider serviceProvider = services.BuildServiceProvider()) + { + Application.Run(serviceProvider.GetRequiredService()); + } + } + + private static void ConfigureServices(ServiceCollection services) + { + services.AddSingleton() + .AddLogging(option => + { + var configuration = new ConfigurationBuilder() + .SetBasePath(Directory.GetCurrentDirectory()) + .AddJsonFile(path: "appsettings.json", optional: false, reloadOnChange: true) + .Build(); + + var logger = new LoggerConfiguration() + .ReadFrom.Configuration(configuration) + .CreateLogger(); + + option.SetMinimumLevel(LogLevel.Information); + option.AddSerilog(logger); + }); } } } \ No newline at end of file diff --git a/SelfPropelledArtilleryUnit/SelfPropelledArtilleryUnit.csproj b/SelfPropelledArtilleryUnit/SelfPropelledArtilleryUnit.csproj index b57c89e..f8e8531 100644 --- a/SelfPropelledArtilleryUnit/SelfPropelledArtilleryUnit.csproj +++ b/SelfPropelledArtilleryUnit/SelfPropelledArtilleryUnit.csproj @@ -8,4 +8,24 @@ enable + + + + + + + + + + + + + + + + + Always + + + \ No newline at end of file diff --git a/SelfPropelledArtilleryUnit/SetArtilleriesGeneric.cs b/SelfPropelledArtilleryUnit/SetArtilleriesGeneric.cs index 1f60477..eeab600 100644 --- a/SelfPropelledArtilleryUnit/SetArtilleriesGeneric.cs +++ b/SelfPropelledArtilleryUnit/SetArtilleriesGeneric.cs @@ -26,7 +26,12 @@ namespace Artilleries public int Insert(T artillery, int position) { - if (position < 0 || position > Count || Count == _maxCount) + if (Count == _maxCount) + { + throw new StorageOverflowException(_maxCount); + } + + if (position < 0 || position > Count) { return -1; } @@ -45,6 +50,11 @@ namespace Artilleries var result = _places[position]; + if (result == null) + { + throw new ArtilleryNotFoundException(position); + } + _places.RemoveAt(position); return result; } diff --git a/SelfPropelledArtilleryUnit/StorageOverflowException.cs b/SelfPropelledArtilleryUnit/StorageOverflowException.cs new file mode 100644 index 0000000..1dda785 --- /dev/null +++ b/SelfPropelledArtilleryUnit/StorageOverflowException.cs @@ -0,0 +1,19 @@ +using System; +using System.Collections.Generic; +using System.Linq; +using System.Runtime.Serialization; +using System.Text; +using System.Threading.Tasks; + +namespace Artilleries +{ + [Serializable] + internal class StorageOverflowException : ApplicationException + { + public StorageOverflowException(int count) : base($"В наборе превышено допустимое количество: {count}") { } + public StorageOverflowException() : base() { } + public StorageOverflowException(string message) : base(message) { } + public StorageOverflowException(string message, Exception exception) : base(message, exception) { } + protected StorageOverflowException(SerializationInfo info, StreamingContext context) : base(info, context) { } + } +} \ No newline at end of file diff --git a/SelfPropelledArtilleryUnit/appsettings.json b/SelfPropelledArtilleryUnit/appsettings.json new file mode 100644 index 0000000..fb814f9 --- /dev/null +++ b/SelfPropelledArtilleryUnit/appsettings.json @@ -0,0 +1,17 @@ +{ + "Serilog": { + "Using": [ "Serilog.Sinks.File" ], + "MinimumLevel": "Information", + "WriteTo": [ + { + "Name": "File", + "Args": { + "path": "Logs/log_.log", + "rollingInterval": "Day", + "outputTemplate": "[{Timestamp:HH:mm:ss.fff}]{Level:u4}: {Message:lj}{NewLine}{Exception}" + } + } + ], + "Enrich": [ "FromLogContext", "WithMachineName", "WithThreadId" ] + } +} \ No newline at end of file