Compare commits

..

3 Commits

7 changed files with 148 additions and 53 deletions

View File

@ -8,6 +8,16 @@
<ImplicitUsings>enable</ImplicitUsings> <ImplicitUsings>enable</ImplicitUsings>
</PropertyGroup> </PropertyGroup>
<ItemGroup>
<PackageReference Include="Microsoft.Extensions.Configuration" Version="8.0.0" />
<PackageReference Include="Microsoft.Extensions.Logging" Version="8.0.0" />
<PackageReference Include="NLog.Extensions.Logging" Version="5.3.7" />
<PackageReference Include="Serilog" Version="3.1.1" />
<PackageReference Include="Serilog.AspNetCore" Version="8.0.0" />
<PackageReference Include="Serilog.Settings.Configuration" Version="8.0.0" />
<PackageReference Include="Serilog.Sinks.Console" Version="5.0.1" />
</ItemGroup>
<ItemGroup> <ItemGroup>
<Compile Update="Properties\Resources.Designer.cs"> <Compile Update="Properties\Resources.Designer.cs">
<DesignTime>True</DesignTime> <DesignTime>True</DesignTime>

View File

@ -0,0 +1,20 @@
{
"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" ],
"Properties": {
"Application": "AirBomber"
}
}
}

View File

@ -10,7 +10,9 @@ using System.Windows.Forms;
using Microsoft.VisualBasic.Logging; using Microsoft.VisualBasic.Logging;
using AirBomber.Exceptions; using AirBomber.Exceptions;
using Serilog; using Serilog;
using Log = Serilog.Log; using _logger = Serilog.Log;
using System.Xml.Linq;
using System.Linq.Expressions;
namespace AirBomber namespace AirBomber
{ {
@ -79,14 +81,17 @@ namespace AirBomber
{ {
return; return;
} }
if (obj + plane > -1) try
{ {
_ = obj + plane;
MessageBox.Show("Объект добавлен"); MessageBox.Show("Объект добавлен");
_logger.Information("Объект добавлен");
pictureBoxCollection.Image = obj.ShowPlanes(); pictureBoxCollection.Image = obj.ShowPlanes();
} }
else catch (Exception ex)
{ {
MessageBox.Show("Не удалось добавить объект"); MessageBox.Show(ex.Message);
_logger.Warning($"Объект не добавлен в набор {listBoxStorages.SelectedItem.ToString()}");
} }
} }
/// <summary> /// <summary>
@ -109,15 +114,30 @@ namespace AirBomber
{ {
return; return;
} }
int pos = Convert.ToInt32(maskedTextBoxNumber.Text); try
if (obj - pos != null)
{ {
MessageBox.Show("Объект удален");
pictureBoxCollection.Image = obj.ShowPlanes(); int pos = Convert.ToInt32(maskedTextBoxNumber.Text);
if (obj - pos != null)
{
MessageBox.Show("Объект удален");
_logger.Information("Объект удален");
pictureBoxCollection.Image = obj.ShowPlanes();
}
else
{
MessageBox.Show("Не удалось удалить объект");
_logger.Warning("Не удалось удалить объект");
}
} }
else catch (PlaneNotFoundException ex)
{ {
MessageBox.Show("Не удалось удалить объект"); MessageBox.Show(ex.Message);
}
catch(Exception ex)
{
MessageBox.Show("Неверный ввод");
_logger.Warning("Неверный ввод");
} }
} }
/// <summary> /// <summary>
@ -147,23 +167,26 @@ namespace AirBomber
{ {
if (string.IsNullOrEmpty(textBoxStorageName.Text)) if (string.IsNullOrEmpty(textBoxStorageName.Text))
{ {
MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error);
MessageBoxButtons.OK, MessageBoxIcon.Error); _logger.Warning("Не все данные заполнены");
return; return;
} }
_storage.AddSet(textBoxStorageName.Text); _storage.AddSet(textBoxStorageName.Text);
ReloadObjects(); ReloadObjects();
_logger.Information($"Добавлен набор: {textBoxStorageName.Text}");
} }
private void buttonDelObject_Click(object sender, EventArgs e) private void buttonDelObject_Click(object sender, EventArgs e)
{ {
string Name = listBoxStorages.SelectedItem.ToString() ?? string.Empty;
if (listBoxStorages.SelectedIndex == -1) if (listBoxStorages.SelectedIndex == -1)
{ {
return; return;
} }
if (MessageBox.Show($"Удалить объект{listBoxStorages.SelectedItem}?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.Yes) if (MessageBox.Show($"Удалить набор {Name}?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.Yes)
{ {
_storage.DelSet(listBoxStorages.SelectedItem.ToString() ?? string.Empty); _storage.DelSet(Name);
ReloadObjects(); ReloadObjects();
_logger.Information($"Удален набор: {Name}");
} }
} }
/// <summary> /// <summary>
@ -175,13 +198,16 @@ namespace AirBomber
{ {
if (saveFileDialog.ShowDialog() == DialogResult.OK) if (saveFileDialog.ShowDialog() == DialogResult.OK)
{ {
if (_storage.SaveData(saveFileDialog.FileName)) try
{ {
_storage.SaveData(saveFileDialog.FileName);
MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
_logger.Information("Сохранение прошло успешно");
} }
else catch (Exception ex)
{ {
MessageBox.Show("Не сохранилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); MessageBox.Show("Не сохранилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
_logger.Warning($"Не сохранилось: {ex.Message}");
} }
} }
} }
@ -195,15 +221,17 @@ namespace AirBomber
// TODO продумать логику DONE // TODO продумать логику DONE
if (openFileDialog.ShowDialog() == DialogResult.OK) if (openFileDialog.ShowDialog() == DialogResult.OK)
{ {
if (_storage.LoadData(openFileDialog.FileName)) try
{ {
_storage.LoadData(openFileDialog.FileName);
MessageBox.Show("Загрузка прошла успешно!", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); MessageBox.Show("Загрузка прошла успешно!", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information);
_logger.Information("Загрузка прошла успешно");
ReloadObjects(); ReloadObjects();
} }
else catch(Exception ex)
{ {
MessageBox.Show("Не загрузилось!", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); MessageBox.Show("Не загрузилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
_logger.Warning($"Не загрузилось: {ex.Message}");
} }
} }
} }

View File

@ -49,13 +49,13 @@ namespace AirBomber
/// <param name="collect"></param> /// <param name="collect"></param>
/// <param name="obj"></param> /// <param name="obj"></param>
/// <returns></returns> /// <returns></returns>
public static int operator +(PlanesGenericCollection<T, U> collect, T? obj) public static bool operator +(PlanesGenericCollection<T, U> collect, T obj)
{ {
if (obj == null) if (obj == null)
{ {
return -1; return false;
} }
return collect?._collection.Insert(obj) ?? -1; return collect._collection.Insert(obj);
} }
/// <summary> /// <summary>
/// Перегрузка оператора вычитания /// Перегрузка оператора вычитания
@ -63,15 +63,15 @@ namespace AirBomber
/// <param name="collect"></param> /// <param name="collect"></param>
/// <param name="pos"></param> /// <param name="pos"></param>
/// <returns></returns> /// <returns></returns>
public static bool operator -(PlanesGenericCollection<T, U> collect, int pos) public static T? operator -(PlanesGenericCollection<T, U> collect, int
pos)
{ {
T? obj = collect._collection[pos]; T? obj = collect._collection[pos];
if (obj != null) if (obj != null)
{ {
collect._collection.Remove(pos); collect._collection.Remove(pos);
return true;
} }
return false; return obj;
} }
/// <summary> /// <summary>
/// Получение объекта IMoveableObject /// Получение объекта IMoveableObject

View File

@ -1,4 +1,5 @@
using System; using AirBomber.Exceptions;
using System;
using System.Collections.Generic; using System.Collections.Generic;
using System.Linq; using System.Linq;
using System.Text; using System.Text;
@ -168,9 +169,20 @@ namespace AirBomber
DrawningAirPlane? plane = elem?.CreateDrawningAirPlane(_separatorForObject, _pictureWidth, _pictureHeight); DrawningAirPlane? plane = elem?.CreateDrawningAirPlane(_separatorForObject, _pictureWidth, _pictureHeight);
if (plane != null) if (plane != null)
{ {
if ((collection + plane) == -1) if (!(collection + plane))
{ {
throw new Exception("Ошибка добавления в коллекцию"); try
{
_ = collection + plane;
}
catch (PlaneNotFoundException e)
{
throw e;
}
catch (StorageOverflowException e)
{
throw e;
}
} }
} }
} }

View File

@ -1,3 +1,8 @@
using Microsoft.Extensions.Configuration;
using Microsoft.Extensions.DependencyInjection;
using Microsoft.Extensions.Logging;
using Serilog;
namespace AirBomber namespace AirBomber
{ {
internal static class Program internal static class Program
@ -11,7 +16,30 @@ namespace AirBomber
// To customize application configuration such as set high DPI settings or default font, // To customize application configuration such as set high DPI settings or default font,
// see https://aka.ms/applicationconfiguration. // see https://aka.ms/applicationconfiguration.
ApplicationConfiguration.Initialize(); ApplicationConfiguration.Initialize();
Application.Run(new FormPlaneCollection()); var services = new ServiceCollection();
ConfigureServices(services);
using (ServiceProvider serviceProvider = services.BuildServiceProvider())
{
Application.Run(serviceProvider.GetRequiredService<FormPlaneCollection>());
}
//Application.Run(new FormPlaneCollection());
}
private static void ConfigureServices(ServiceCollection services)
{
services.AddSingleton<FormPlaneCollection>().AddLogging(option =>
{
string[] path = Directory.GetCurrentDirectory().Split('\\');
string pathNeed = "";
for (int i = 0; i < path.Length - 3; i++)
{
pathNeed += path[i] + "\\";
}
var configuration = new ConfigurationBuilder().SetBasePath(Directory.GetCurrentDirectory()).AddJsonFile(path: $"{pathNeed}AppSettings.json", optional: false, reloadOnChange: true).Build();
var logger = new LoggerConfiguration().ReadFrom.Configuration(configuration).CreateLogger();
option.SetMinimumLevel(LogLevel.Information);
option.AddSerilog(logger);
});
} }
} }
} }

View File

@ -3,6 +3,7 @@ using System.Collections.Generic;
using System.Linq; using System.Linq;
using System.Text; using System.Text;
using System.Threading.Tasks; using System.Threading.Tasks;
using AirBomber.Exceptions;
namespace AirBomber namespace AirBomber
{ {
@ -35,9 +36,8 @@ namespace AirBomber
/// </summary> /// </summary>
/// <param name="plane">Добавляемый самолет</param> /// <param name="plane">Добавляемый самолет</param>
/// <returns></returns> /// <returns></returns>
public int Insert(T plane) public bool Insert(T plane)
{ {
//was TODO
return Insert(plane, 0); return Insert(plane, 0);
} }
/// <summary> /// <summary>
@ -46,19 +46,15 @@ namespace AirBomber
/// <param name="plane">Добавляемый самолет</param> /// <param name="plane">Добавляемый самолет</param>
/// <param name="position">Позиция</param> /// <param name="position">Позиция</param>
/// <returns></returns> /// <returns></returns>
public int Insert(T plane, int position) public bool Insert(T plane, int position)
{ {
// TODO проверка позиции DONE
// TODO проверка, что элемент массива по этой позиции пустой,если нет, то
// проверка, что после вставляемого элемента в массиве есть пустой элемент
// сдвиг всех объектов, находящихся справа от позиции до первого пустого элемента
// TODO вставка по позиции
if (position < 0 || position >= _maxCount) if (position < 0 || position >= _maxCount)
{ throw new StorageOverflowException("Impossible to insert");
if (Count >= _maxCount)
throw new StorageOverflowException(_maxCount); throw new StorageOverflowException(_maxCount);
} _places.Insert(0, plane);
_places.Insert(position, plane); return true;
return position;
} }
/// <summary> /// <summary>
/// Удаление объекта из набора с конкретной позиции /// Удаление объекта из набора с конкретной позиции
@ -67,13 +63,11 @@ namespace AirBomber
/// <returns></returns> /// <returns></returns>
public bool Remove(int position) public bool Remove(int position)
{ {
// TODO проверка позиции DONE if (position >= Count || position < 0)
// TODO удаление объекта из массива, присвоив элементу массива значение null throw new PlaneNotFoundException("Invalid operation");
if (!(position >= 0 && position < Count) || _places[position] == null) if (_places[position] == null)
{
throw new PlaneNotFoundException(position); throw new PlaneNotFoundException(position);
} _places.RemoveAt(position);
_places[position] = null;
return true; return true;
} }
/// <summary> /// <summary>
@ -94,11 +88,14 @@ namespace AirBomber
} }
set set
{ {
// TODO проверка позиции DONE try
// TODO проверка свободных мест в списке DONE {
// TODO вставка в список по позиции DONE Insert(value, position);
if (position < 0 || position >= Count || Count == _maxCount) return; }
_places.Insert(position, value); catch
{
return;
}
} }
} }
/// <summary> /// <summary>