diff --git a/AirPlaneWithRadar/AirPlaneWithRadar/AirPlaneWithRadar.csproj b/AirPlaneWithRadar/AirPlaneWithRadar/AirPlaneWithRadar.csproj index 13ee123..4914ae8 100644 --- a/AirPlaneWithRadar/AirPlaneWithRadar/AirPlaneWithRadar.csproj +++ b/AirPlaneWithRadar/AirPlaneWithRadar/AirPlaneWithRadar.csproj @@ -8,6 +8,28 @@ enable + + + + + + + Always + + + + + + + + + + + + + + + True diff --git a/AirPlaneWithRadar/AirPlaneWithRadar/FormMapWithSetPlains.cs b/AirPlaneWithRadar/AirPlaneWithRadar/FormMapWithSetPlains.cs index a2c6cc7..0508b87 100644 --- a/AirPlaneWithRadar/AirPlaneWithRadar/FormMapWithSetPlains.cs +++ b/AirPlaneWithRadar/AirPlaneWithRadar/FormMapWithSetPlains.cs @@ -1,4 +1,5 @@ -using System; +using Microsoft.Extensions.Logging; +using System; using System.Collections.Generic; using System.ComponentModel; using System.Data; @@ -17,10 +18,11 @@ namespace AirPlaneWithRadar { "Простая карта", new SimpleMap()},{"Карта с большими коробками", new UserMap_BigBox()},{"Карта со стенами", new UserMap_Colums()} }; private readonly MapsCollection _mapsCollection; - - public FormMapWithSetPlains() + private readonly ILogger _logger; + public FormMapWithSetPlains(ILogger logger) { InitializeComponent(); + _logger = logger; _mapsCollection = new MapsCollection(pictureBox.Width, pictureBox.Height); comboBoxSelectorMap.Items.Clear(); foreach (var elem in _mapsDict) @@ -62,11 +64,13 @@ namespace AirPlaneWithRadar } _mapsCollection.AddMap(textBoxNewMapName.Text, _mapsDict[comboBoxSelectorMap.Text]); ReloadMaps(); + _logger.LogInformation($"Добавлена карта: {textBoxNewMapName.Text}"); } 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) @@ -78,8 +82,10 @@ namespace AirPlaneWithRadar if (MessageBox.Show($"Удалить карту {listBoxMaps.SelectedItem}?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.Yes) { + string nameMap = listBoxMaps.SelectedItem?.ToString() ?? string.Empty; _mapsCollection.DelMap(listBoxMaps.SelectedItem?.ToString() ?? string.Empty); ReloadMaps(); + _logger.LogInformation($"Удалена карта: {nameMap}"); } } @@ -97,17 +103,35 @@ namespace AirPlaneWithRadar { return; } - if ((_mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty] + new DrawingObjectPlane(dp)) >= 0) + try { - MessageBox.Show("Объект добавлен"); - pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); + if ((_mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty] + new DrawingObjectPlane(dp)) >= 0) + { + MessageBox.Show("Объект добавлен"); + pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); + _logger.LogInformation($"На карту {listBoxMaps.SelectedItem?.ToString() ?? string.Empty} добавлен новый самолет"); + } + else + { + MessageBox.Show("Не удалось добавить объект"); + _logger.LogInformation($"Добавление самолета на карту {listBoxMaps.SelectedItem?.ToString() ?? string.Empty} прошло неудачно"); + } } - else + catch (StorageOverFullException ex) { - MessageBox.Show("Не удалось добавить объект"); + MessageBox.Show($"Ошибка добавления : {ex.Message}"); + _logger.LogWarning($"Ошибка добавления самолета на карту {listBoxMaps.SelectedItem?.ToString() ?? string.Empty} : {ex.Message}"); + } + catch (Exception ex) + { + MessageBox.Show($"Неизвестная ошибка : {ex.Message}"); + _logger.LogWarning($"Неизвестная ошибка при добавлении самолета на карту {listBoxMaps.SelectedItem?.ToString() ?? string.Empty} : {ex.Message}"); } + + } + private void ButtonRemovePlain_Click(object sender, EventArgs e) { if (listBoxMaps.SelectedIndex == -1) @@ -118,18 +142,39 @@ namespace AirPlaneWithRadar { return; } - if (maskedTextBoxPosition.Text == null) + if (maskedTextBoxPosition.Text == null || (maskedTextBoxPosition.Text).Equals("")) + { + MessageBox.Show("Упс, что-то пошло не так"); return; + } int pos = Convert.ToInt32(maskedTextBoxPosition.Text); - if ((_mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty] - (pos - 1)) != null) + try { - MessageBox.Show("Объект удален"); - pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); + if ((_mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty] - (pos - 1)) != null) + { + MessageBox.Show("Объект удален"); + pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); + _logger.LogInformation($"С карты {listBoxMaps.SelectedItem?.ToString() ?? string.Empty} был удален самолет"); + + } + else + { + MessageBox.Show("Не удалось удалить объект"); + _logger.LogInformation($"Удаление на карте{listBoxMaps.SelectedItem?.ToString() ?? string.Empty} прошло неудачно"); + } } - else + catch (PlainNotFoundException ex) { - MessageBox.Show("Не удалось удалить объект"); + MessageBox.Show($"Ошибка удаления : {ex.Message}"); + _logger.LogWarning($"Ошибка удаления самолета на карту {listBoxMaps.SelectedItem?.ToString() ?? string.Empty} : {ex.Message}"); } + catch (Exception ex) + { + MessageBox.Show($"Неизвестная ошибка : {ex.Message}"); + _logger.LogWarning($"Неизвестная ошибка при удалении самолета на карту {listBoxMaps.SelectedItem?.ToString() ?? string.Empty} : {ex.Message}"); + } + + } private void ButtonShowStorage_Click(object sender, EventArgs e) { @@ -138,6 +183,7 @@ namespace AirPlaneWithRadar return; } pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); + _logger.LogInformation($"Переход в гараж карты {listBoxMaps.SelectedItem?.ToString() ?? string.Empty}"); } private void ButtonShowOnMap_Click(object sender, EventArgs e) { @@ -146,6 +192,7 @@ namespace AirPlaneWithRadar return; } pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowOnMap(); + _logger.LogInformation($"Переход на поле карты {listBoxMaps.SelectedItem?.ToString() ?? string.Empty}"); } private void ButtonMove_Click(object sender, EventArgs e) { @@ -173,17 +220,22 @@ namespace AirPlaneWithRadar } pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].MoveObject(dir); } + private void SaveToolStrip_Click(object sender, EventArgs e) { if (saveFileDialog.ShowDialog() == DialogResult.OK) { - if (_mapsCollection.SaveData(saveFileDialog.FileName)) + try { + _mapsCollection.SaveData(saveFileDialog.FileName); MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); + _logger.LogInformation($"Было проведено сохранение в файл : {saveFileDialog.FileName}"); + } - else + catch (Exception ex) { - MessageBox.Show("Не сохранилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + MessageBox.Show($"Не сохранилось {ex.Message} ", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning($"Ошибка при сохранении : {ex.Message}"); } } } @@ -192,17 +244,35 @@ namespace AirPlaneWithRadar { if (openFileDialog.ShowDialog() == DialogResult.OK) { - if (_mapsCollection.LoadData(openFileDialog.FileName) && _mapsCollection.Count != 0) + try { + _mapsCollection.LoadData(openFileDialog.FileName); MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); ReloadMaps(); - pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet(); + _logger.LogInformation($"Была проведена загрузка из файла {openFileDialog.FileName}"); } - else + catch (FileNotFoundException ex) { - MessageBox.Show("Не удалось загрузить", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + MessageBox.Show($"Не удалось загрузить {ex.Message} ", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning($"Ошибка при загрузке из файла {openFileDialog.FileName} : {ex.Message}"); } + catch (NullReferenceException ex) + { + MessageBox.Show($"Не удалось загрузить {ex.Message} ", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning($"Ошибка при загрузке из файла {openFileDialog.FileName} : {ex.Message}"); + } + catch (FormatException ex) + { + MessageBox.Show($"Не удалось загрузить {ex.Message} ", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning($"Ошибка при загрузке из файла {openFileDialog.FileName} : {ex.Message}"); + } + catch (Exception ex) + { + MessageBox.Show($"Не удалось загрузить {ex.Message} ", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogWarning($"Неизвестная ошибка при загрузке из файла {openFileDialog.FileName} : {ex.Message}"); + } + } } diff --git a/AirPlaneWithRadar/AirPlaneWithRadar/Log.txt b/AirPlaneWithRadar/AirPlaneWithRadar/Log.txt new file mode 100644 index 0000000..0274be1 --- /dev/null +++ b/AirPlaneWithRadar/AirPlaneWithRadar/Log.txt @@ -0,0 +1,39 @@ +2022-12-02 19:08:53.919 +03:00 [INF] Переключение на карту 1234 +2022-12-02 19:08:53.934 +03:00 [INF] Была проведена загрузка из файла C:\Users\Вова\Documents\lab7.txt +2022-12-02 19:08:56.950 +03:00 [WRN] Ошибка добавления самолета на карту 1234 : В наборе превышено допустимое количество: 30 +2022-12-02 19:09:04.295 +03:00 [WRN] Неизвестная ошибка при удалении самолета на карту 1234 : Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index') +2022-12-02 19:09:07.323 +03:00 [INF] С карты 1234 был удален самолет +2022-12-02 19:09:13.786 +03:00 [INF] На карту 1234 добавлен новый самолет +2022-12-02 19:09:15.292 +03:00 [INF] Переключение на карту 4321 +2022-12-02 19:09:15.780 +03:00 [INF] Переключение на карту 1234 +2022-12-02 19:09:22.952 +03:00 [INF] Было проведено сохранение в файл : C:\Users\Вова\Documents\123.txt +2022-12-02 19:09:24.254 +03:00 [INF] Переход в гараж карты 1234 +2022-12-02 19:09:25.625 +03:00 [INF] Переход на поле карты 1234 +2022-12-02 19:09:26.998 +03:00 [INF] Переход в гараж карты 1234 +2022-12-02 19:09:30.990 +03:00 [INF] Переключение на карту 1234 +2022-12-02 19:09:30.990 +03:00 [INF] Добавлена карта: 231 +2022-12-02 19:09:32.147 +03:00 [INF] Переключение на карту 231 +2022-12-02 19:09:35.672 +03:00 [INF] На карту 231 добавлен новый самолет +2022-12-02 19:09:45.113 +03:00 [INF] На карту 231 добавлен новый самолет +2022-12-02 19:21:10.651 +03:00 [INF] Переключение на карту 1234 +2022-12-02 19:21:10.666 +03:00 [INF] Была проведена загрузка из файла C:\Users\Вова\Documents\lab7.txt +2022-12-02 19:21:17.227 +03:00 [INF] С карты 1234 был удален самолет +2022-12-02 19:21:23.521 +03:00 [WRN] Неизвестная ошибка при удалении самолета на карту 1234 : Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index') +2022-12-02 19:21:27.938 +03:00 [WRN] Неизвестная ошибка при удалении самолета на карту 1234 : Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index') +2022-12-02 19:23:37.287 +03:00 [WRN] Неизвестная ошибка при удалении самолета на карту 1234 : Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index') +2022-12-02 19:23:43.057 +03:00 [WRN] Неизвестная ошибка при удалении самолета на карту 1234 : Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index') +2022-12-02 19:24:06.386 +03:00 [INF] С карты 1234 был удален самолет +2022-12-02 19:24:14.974 +03:00 [WRN] Неизвестная ошибка при удалении самолета на карту 1234 : Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index') +2022-12-02 19:24:30.815 +03:00 [INF] С карты 1234 был удален самолет +2022-12-02 19:24:45.861 +03:00 [WRN] Неизвестная ошибка при удалении самолета на карту 1234 : Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index') +2022-12-02 19:31:41.675 +03:00 [INF] Переключение на карту 1234 +2022-12-02 19:31:41.694 +03:00 [INF] Была проведена загрузка из файла C:\Users\Вова\Documents\123.txt +2022-12-02 19:31:46.120 +03:00 [WRN] Ошибка при загрузке из файла C:\Users\Вова\Documents\test0.txt : Неверный формат данных в файле +2022-12-02 19:31:49.588 +03:00 [INF] Переключение на карту 1234 +2022-12-02 19:31:49.590 +03:00 [INF] Была проведена загрузка из файла C:\Users\Вова\Documents\123.txt +2022-12-02 19:33:06.889 +03:00 [WRN] Ошибка при загрузке из файла C:\Users\Вова\Documents\test0.txt : Неверный формат данных в файле +2022-12-02 19:35:03.298 +03:00 [INF] Переключение на карту 1234 +2022-12-02 19:35:03.301 +03:00 [INF] Была проведена загрузка из файла C:\Users\Вова\Documents\123.txt +2022-12-02 19:35:15.398 +03:00 [WRN] Неизвестная ошибка при удалении самолета на карту 1234 : Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index') +2022-12-02 19:35:20.416 +03:00 [INF] С карты 1234 был удален самолет +2022-12-02 19:35:24.176 +03:00 [WRN] Неизвестная ошибка при удалении самолета на карту 1234 : Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index') diff --git a/AirPlaneWithRadar/AirPlaneWithRadar/MapsCollection.cs b/AirPlaneWithRadar/AirPlaneWithRadar/MapsCollection.cs index 7c476e0..1a92024 100644 --- a/AirPlaneWithRadar/AirPlaneWithRadar/MapsCollection.cs +++ b/AirPlaneWithRadar/AirPlaneWithRadar/MapsCollection.cs @@ -54,7 +54,7 @@ namespace AirPlaneWithRadar return _mapStorages[ind]; } } - public bool SaveData(string filename) + public void SaveData(string filename) { if (File.Exists(filename)) { @@ -69,22 +69,22 @@ namespace AirPlaneWithRadar } } - return true; + } - public bool LoadData(string filename) + public void LoadData(string filename) { if (!File.Exists(filename)) { - return false; + throw new FileNotFoundException("Файл не найден"); } using (StreamReader sr = new StreamReader(filename)) { if (!sr.ReadLine().Equals("MapsCollection")) { - return false; + throw new FormatException("Неверный формат данных в файле"); } _mapStorages.Clear(); string line = sr.ReadLine(); @@ -109,7 +109,10 @@ namespace AirPlaneWithRadar line = sr.ReadLine(); } } - return true; + if (_mapStorages.Count == 0) + { + throw new NullReferenceException("Пустой Файл"); + } } } diff --git a/AirPlaneWithRadar/AirPlaneWithRadar/PlainNotFoundException.cs b/AirPlaneWithRadar/AirPlaneWithRadar/PlainNotFoundException.cs new file mode 100644 index 0000000..c2b8cf2 --- /dev/null +++ b/AirPlaneWithRadar/AirPlaneWithRadar/PlainNotFoundException.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 AirPlaneWithRadar +{ + [Serializable] + internal class PlainNotFoundException : ApplicationException + { + public PlainNotFoundException(int i) : base($"Не найден объект по позиции {i}") { } + public PlainNotFoundException() : base() { } + public PlainNotFoundException(string message) : base(message) { } + public PlainNotFoundException(string message, Exception exception) : base(message, exception) { } + protected PlainNotFoundException(SerializationInfo info, StreamingContext contex) : base(info, contex) { } + } +} diff --git a/AirPlaneWithRadar/AirPlaneWithRadar/Program.cs b/AirPlaneWithRadar/AirPlaneWithRadar/Program.cs index 8cb6b9c..6b0d9fd 100644 --- a/AirPlaneWithRadar/AirPlaneWithRadar/Program.cs +++ b/AirPlaneWithRadar/AirPlaneWithRadar/Program.cs @@ -1,17 +1,51 @@ +using Microsoft.Extensions.Configuration; +using Microsoft.Extensions.DependencyInjection; +using Microsoft.Extensions.Logging; +using NLog.Extensions.Logging; +using Serilog; +using System; + namespace AirPlaneWithRadar { internal static class Program { + /// /// The main entry point for the application. /// [STAThread] static void Main() { + + // To customize application configuration such as set high DPI settings or default font, // see https://aka.ms/applicationconfiguration. ApplicationConfiguration.Initialize(); - Application.Run(new FormMapWithSetPlains()); + var servives = new ServiceCollection(); + ConfigureServices(servives); + using (ServiceProvider serviceProvider = servives.BuildServiceProvider()) + { + Application.Run(serviceProvider.GetRequiredService()); + } + } + private static void ConfigureServices(ServiceCollection services) + { + + services.AddSingleton() + .AddLogging(option => + { + var configuration = new ConfigurationBuilder() + .AddJsonFile("C:\\Users\\\\source\\repos\\ProjectAirPlaneWithRadar\\AirPlaneWithRadar\\AirPlaneWithRadar\\appsettings.json") + .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/AirPlaneWithRadar/AirPlaneWithRadar/SetPlaneGeneric.cs b/AirPlaneWithRadar/AirPlaneWithRadar/SetPlaneGeneric.cs index 291aa24..bc20ae1 100644 --- a/AirPlaneWithRadar/AirPlaneWithRadar/SetPlaneGeneric.cs +++ b/AirPlaneWithRadar/AirPlaneWithRadar/SetPlaneGeneric.cs @@ -26,6 +26,10 @@ namespace AirPlaneWithRadar } public int Insert(T plain, int position) { + if (_places.Count == _maxCount) + { + throw new StorageOverFullException(_maxCount); + } if (position < 0 || _places.Count < position||position > _maxCount || _places.Count == _maxCount) return -1; else if (plain == null) @@ -36,16 +40,15 @@ namespace AirPlaneWithRadar } public T Remove(int position) { - T mid; - if (position < 0 || _places.Count < position || position > _maxCount) - return null; - else if (_places[position] == null) - return null; - else + if (_places[position] == null || (position < 0 || _places.Count < position || position > _maxCount)) { - mid = _places[position]; - _places.RemoveAt(position); + throw new PlainNotFoundException(position); } + T mid; + + mid = _places[position]; + _places.RemoveAt(position); + return mid; } public T this[int position] diff --git a/AirPlaneWithRadar/AirPlaneWithRadar/StorageOverFullException.cs b/AirPlaneWithRadar/AirPlaneWithRadar/StorageOverFullException.cs new file mode 100644 index 0000000..627eb72 --- /dev/null +++ b/AirPlaneWithRadar/AirPlaneWithRadar/StorageOverFullException.cs @@ -0,0 +1,20 @@ +using System; +using System.Collections.Generic; +using System.Linq; +using System.Runtime.Serialization; +using System.Text; +using System.Threading.Tasks; + +namespace AirPlaneWithRadar +{ + [Serializable] + internal class StorageOverFullException : ApplicationException + { + public StorageOverFullException(int count) : base($"В наборе превышено допустимое количество: {count}") { } + public StorageOverFullException() : base() { } + public StorageOverFullException(string message) : base(message) { } + public StorageOverFullException(string message, Exception exception) : base(message, exception) { } + protected StorageOverFullException(SerializationInfo info, StreamingContext contex) : base(info, contex) { } + + } +} diff --git a/AirPlaneWithRadar/AirPlaneWithRadar/appsettings.json b/AirPlaneWithRadar/AirPlaneWithRadar/appsettings.json new file mode 100644 index 0000000..4c62caa --- /dev/null +++ b/AirPlaneWithRadar/AirPlaneWithRadar/appsettings.json @@ -0,0 +1,15 @@ +{ + "Serilog": { + "Using": [ "Serilog.Sinks.File" ], + "MinimumLevel": "Debug", + "WriteTo": [ + + { + "Name": "File", + "Args": { + "path": "C:\\Users\\Вова\\source\\repos\\ProjectAirPlaneWithRadar\\AirPlaneWithRadar\\AirPlaneWithRadar\\Log.txt" + } + } + ] + } +} \ No newline at end of file