Compare commits

..

No commits in common. "8021e8ec78b6d3cbbd45e9e088b8cb503fa26363" and "557028cc392ede974b525d3a102a79bb2dba4362" have entirely different histories.

5 changed files with 8 additions and 35 deletions

View File

@ -7,10 +7,6 @@ using System.Linq;
using System.Text; using System.Text;
using System.Threading.Tasks; using System.Threading.Tasks;
using System.Windows.Forms; using System.Windows.Forms;
using Microsoft.VisualBasic.Logging;
using AirBomber.Exceptions;
using Serilog;
using Log = Serilog.Log;
namespace AirBomber namespace AirBomber
{ {

View File

@ -5,9 +5,8 @@ using System.Runtime.Serialization;
using System.Text; using System.Text;
using System.Threading.Tasks; using System.Threading.Tasks;
namespace AirBomber.Exceptions namespace AirBomber
{ {
[Serializable]
internal class PlaneNotFoundException: ApplicationException internal class PlaneNotFoundException: ApplicationException
{ {
public PlaneNotFoundException(int i) : base($"Не найден объект по позиции {i}") { } public PlaneNotFoundException(int i) : base($"Не найден объект по позиции {i}") { }

View File

@ -112,7 +112,7 @@ namespace AirBomber
} }
if (data.Length == 0) if (data.Length == 0)
{ {
throw new Exception("Невалидная операция, нет данных для сохранения"); return false;
} }
using StreamWriter sw = new(filename); using StreamWriter sw = new(filename);
@ -128,7 +128,7 @@ namespace AirBomber
{ {
if (!File.Exists(filename)) if (!File.Exists(filename))
{ {
throw new Exception("Файл не найден"); return false;
} }
using (StreamReader sr = new(filename)) using (StreamReader sr = new(filename))
{ {
@ -136,12 +136,12 @@ namespace AirBomber
if (str == null || str.Length == 0) if (str == null || str.Length == 0)
{ {
throw new Exception("Нет данных для загрузки"); return false;
} }
if (!str.StartsWith("PlaneStorage")) if (!str.StartsWith("PlaneStorage"))
{ {
//если нет такой записи, то это не те данные //если нет такой записи, то это не те данные
throw new Exception("Неверный формат данных"); return false;
} }
_planeStorages.Clear(); _planeStorages.Clear();
@ -170,7 +170,7 @@ namespace AirBomber
{ {
if ((collection + plane) == -1) if ((collection + plane) == -1)
{ {
throw new Exception("Ошибка добавления в коллекцию"); return false;
} }
} }
} }

View File

@ -53,10 +53,7 @@ namespace AirBomber
// проверка, что после вставляемого элемента в массиве есть пустой элемент // проверка, что после вставляемого элемента в массиве есть пустой элемент
// сдвиг всех объектов, находящихся справа от позиции до первого пустого элемента // сдвиг всех объектов, находящихся справа от позиции до первого пустого элемента
// TODO вставка по позиции // TODO вставка по позиции
if (position < 0 || position >= _maxCount) if (position < 0 || position >= _maxCount) return -1;
{
throw new StorageOverflowException(_maxCount);
}
_places.Insert(position, plane); _places.Insert(position, plane);
return position; return position;
} }
@ -71,7 +68,7 @@ namespace AirBomber
// TODO удаление объекта из массива, присвоив элементу массива значение null // TODO удаление объекта из массива, присвоив элементу массива значение null
if (!(position >= 0 && position < Count) || _places[position] == null) if (!(position >= 0 && position < Count) || _places[position] == null)
{ {
throw new PlaneNotFoundException(position); return false;
} }
_places[position] = null; _places[position] = null;
return true; return true;

View File

@ -1,19 +0,0 @@
using System;
using System.Collections.Generic;
using System.Linq;
using System.Runtime.Serialization;
using System.Text;
using System.Threading.Tasks;
namespace AirBomber.Exceptions
{
[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 contex) : base(info, contex) { }
}
}