добавлены описания + изменены размеры объектов

DoubleDeckerBus/DoubleDeckerBus/Drawnings/DirectionType.cs

@@ -1,8 +1,14 @@
 namespace DoubleDeckerBus.Drawnings;
-
+/// <summary>
+/// Направление перемещения
+/// </summary>
 public enum DirectionType
 {   
+    /// <summary>
+    /// Неизвестное направление
+    /// </summary>
     Unknown = -1,
+
     /// <summary>
     /// вверх
     /// </summary>

DoubleDeckerBus/DoubleDeckerBus/Drawnings/DrawingBus.cs

@@ -7,30 +7,69 @@ using System.Threading.Tasks;
 
 namespace DoubleDeckerBus.Drawnings;
 
+/// <summary>
+/// Класс, отвечающий за прорисовку и перемещение базового объекта-сущности
+/// </summary>
 public class DrawingBus
 {
+    /// <summary>
+    /// Класс-сущность
+    /// </summary>
     public EntityBus? EntityBus { get; protected set; }
 
+    /// <summary>
+    /// Ширина окна
+    /// </summary>
     private int? _pictureWidth;
 
+    /// <summary>
+    /// Высота окна
+    /// </summary>
     private int? _pictureHeight;
 
+    /// <summary>
+    /// Левая координата прорисовки автобуса
+    /// </summary>
     protected int? _startPosX;
 
+    /// <summary>
+    /// Верхняя кооридната прорисовки автобуса
+    /// </summary>
     protected int? _startPosY;
 
-    private readonly int _drawingBusWidth = 105;
+    /// <summary>
+    /// Ширина прорисовки автобуса
+    /// </summary>
+    private readonly int _drawingBusWidth = 100;
 
-    private readonly int _drawingBusHeight = 50;
+    /// <summary>
+    /// Высота прорисовки автобуса
+    /// </summary>
+    private readonly int _drawingBusHeight = 40;
 
+    /// <summary>
+    /// Координата X объекта
+    /// </summary>
     public int? GetPosX => _startPosX;
 
+    /// <summary>
+    /// Координата Y объекта
+    /// </summary>
     public int? GetPosY => _startPosY;
 
+    /// <summary>
+    /// Ширина объекта
+    /// </summary>
     public int GetWidth => _drawingBusWidth;
 
+    /// <summary>
+    /// Высота объекта
+    /// </summary>
     public int GetHeight => _drawingBusHeight;
 
+    /// <summary>
+    /// Пустой конструктор
+    /// </summary>
     private DrawingBus()
     {
         _pictureWidth = null;
@@ -39,32 +78,44 @@ public class DrawingBus
         _startPosY = null;
     }
 
+    /// <summary>
+    /// Конструктор
+    /// </summary>
+    /// <param name="speed">Скорость</param>
+    /// <param name="weight">Вес</param>
+    /// <param name="bodyColor">Основной цвет</param>
     public DrawingBus(int speed, int weight, Color bodyColor) : this()
     {
         EntityBus = new EntityBus(speed, weight, bodyColor);
     }
 
+    /// <summary>
+    /// Конструктор для наследников
+    /// </summary>
+    /// <param name="drawingBusWidth">Ширина прорисовки автобуса</param>
+    /// <param name="drawingBusHeight">Высота прорисовки автобуса</param>
     protected DrawingBus(int drawingBusWidth, int drawingBusHeight) : this()
     {
         _drawingBusWidth = drawingBusWidth;
         _drawingBusHeight = drawingBusHeight;
     }
+
     /// <summary>
-    /// размер окна
+    /// Установка границ поля
     /// </summary>
-    /// <param name="width"></param>
-    /// <param name="hight"></param>
-    /// <returns></returns>
-    public bool SetPictureSize(int width, int hight)
+    /// <param name="width">Ширина поля</param>
+    /// <param name="height">Высота поля</param>
+    /// <returns>true - границы заданы, false - проверка не пройдена, нельзя разместить объект в этих размерах</returns>
+    public bool SetPictureSize(int width, int height)
     {
-        if (_drawingBusWidth > width || _drawingBusHeight > hight)
+        if (_drawingBusWidth > width || _drawingBusHeight > height)
         {
             return false;
         }
 
 
         _pictureWidth = width;
-        _pictureHeight = hight;
+        _pictureHeight = height;
 
         if (_startPosX.HasValue && _startPosX.Value + _drawingBusWidth > _pictureWidth)
         {
@@ -80,10 +131,10 @@ public class DrawingBus
 
     }
     /// <summary>
-    /// установить начальную позицию
+    /// Установка позиции 
     /// </summary>
-    /// <param name="x"></param>
-    /// <param name="y"></param>
+    /// <param name="x">Координата X</param>
+    /// <param name="y">Координа Y</param>
     public void SetPosition(int x, int y)
     {
         if (!_pictureHeight.HasValue || !_pictureWidth.HasValue)
@@ -119,6 +170,11 @@ public class DrawingBus
         }
     }
 
+    /// <summary>
+    /// Изменение направления перемещения
+    /// </summary>
+    /// <param name="direction">Направление</param>
+    /// <returns>true - перемещене выполнено, false - перемещение невозможно</returns>
     public bool MoveTransport(DirectionType direction)
     {
         if (EntityBus == null || !_startPosX.HasValue || !_startPosY.HasValue)
@@ -159,6 +215,10 @@ public class DrawingBus
         }
     }
 
+    /// <summary>
+    /// Прорисовка объекта
+    /// </summary>
+    /// <param name="g"></param>
     public virtual void DrawTrasnport(Graphics g)
     {
         if (EntityBus == null || !_startPosX.HasValue || !_startPosY.HasValue)

DoubleDeckerBus/DoubleDeckerBus/Drawnings/DrawingDoubleDeckerBus.cs

@@ -2,10 +2,19 @@
 
 namespace DoubleDeckerBus.Drawnings;
 /// <summary>
-/// 
+/// Класс, отвечающий за прорисовку и перемещение продвинутого объекта сущности
 /// </summary>
 public class DrawingDoubleDeckerBus : DrawingBus
 {
+    /// <summary>
+    /// Конструктор
+    /// </summary>
+    /// <param name="speed">Скорость</param>
+    /// <param name="weight">Вес</param>
+    /// <param name="bodyColor">Основной цвет</param>
+    /// <param name="additionalColor">Доп цвет</param>
+    /// <param name="secondFloor">Признак наличия второго этажа</param>
+    /// <param name="stripes">Признак наличия полос на кузове</param>
     public DrawingDoubleDeckerBus(int speed, double weight, Color bodyColor, Color additionalColor, bool secondFloor, bool stripes) : base (115,55)
     {
         EntityBus = new EntityDoubleDeckerBus(speed, weight, bodyColor, additionalColor, secondFloor, stripes);

DoubleDeckerBus/DoubleDeckerBus/Entities/EntityBus.cs

@@ -6,7 +6,7 @@ using System.Threading.Tasks;
 
 namespace DoubleDeckerBus.Entities;
 /// <summary>
-/// Класс сущность автобус
+/// Класс сущность "Автобус"
 /// </summary>
 public class EntityBus
 {   
@@ -14,24 +14,28 @@ public class EntityBus
     /// Скорость
     /// </summary>
     public int Speed { get; private set; }
+
     /// <summary>
     /// Вес
     /// </summary>
     public double Weight { get; private set; }
+
     /// <summary>
     /// Основной цвет
     /// </summary>
     public Color BodyColor { get; private set; }
+
     /// <summary>
     /// Расстояние перемещения за раз
     /// </summary>
     public double Step => Speed * 100 / Weight;
+
     /// <summary>
     /// Конструктор базовой сущности
     /// </summary>
-    /// <param name="speed"></param>
-    /// <param name="weight"></param>
-    /// <param name="bodyColor"></param>
+    /// <param name="speed">Скорость</param>
+    /// <param name="weight">Вес автобуса</param>
+    /// <param name="bodyColor">Основной цвет</param>
     public EntityBus(int speed, double weight, Color bodyColor)
     {
         Speed = speed;

DoubleDeckerBus/DoubleDeckerBus/Entities/EntityDoubleDeckerBus.cs

@@ -5,23 +5,26 @@ public class EntityDoubleDeckerBus : EntityBus
     /// Дополнительный цвет
     /// </summary>
     public Color AdditionalColor { get; private set; }
+
     /// <summary>
     /// Признак (опция) наличия второго этажа
     /// </summary>
     public bool SecondFloor { get; private set; }
+
     /// <summary>
     /// Признак (опция) наличия полосок на автобусе
     /// </summary>
     public bool Stripes { get; private set; }
     
     /// <summary>
-    /// Конструктор наследуемого класса
+    /// Конструктор продвинутой сущности
     /// </summary>
-    /// <param name="speed"></param>
-    /// <param name="weight"></param>
-    /// <param name="bodyColor"></param>
-    /// <param name="additionalColor"></param>
-    /// <param name="stripes"></param>
+    /// <param name="speed">Скорость</param>
+    /// <param name="weight">Вес</param>
+    /// <param name="bodyColor">Основной цвет</param>
+    /// <param name="additionalColor">Дополнительный цвет</param>
+    /// <param name="secondFloor">Признак наличия второго этажа</param>
+    /// <param name="stripes">Призна наличия полос на кузове</param>
     public EntityDoubleDeckerBus(int speed, double weight, Color bodyColor, Color additionalColor, bool secondFloor, bool stripes) : base(speed, weight, bodyColor)
     {
         AdditionalColor = additionalColor;

DoubleDeckerBus/DoubleDeckerBus/MovementStrategy/AbstractStrategy.cs

@@ -6,18 +6,42 @@ using System.Threading.Tasks;
 
 namespace DoubleDeckerBus.MovementStrategy;
 
+/// <summary>
+/// Класс-стратегия перемещения объекта
+/// </summary>
 public abstract class AbstractStrategy
 {
+    /// <summary>
+    /// Перемещаемый объект
+    /// </summary>
     private IMoveableObject? _moveableObject;
 
+    /// <summary>
+    /// Статус перемещения
+    /// </summary>
     private StrategyStatus _state = StrategyStatus.NotInit;
 
+    /// <summary>
+    /// Ширина поля
+    /// </summary>
     protected int FieldWidth { get; private set; }
 
+    /// <summary>
+    /// Высота поля
+    /// </summary>
     protected int FieldHeight { get; private set; }
 
+    /// <summary>
+    /// Статус перемещения
+    /// </summary>
     public StrategyStatus GetStatus() { return _state; }
-
+    
+    /// <summary>
+    /// Установка данных 
+    /// </summary>
+    /// <param name="movableObject">Перемещаемый объект</param>
+    /// <param name="width">Ширина поля</param>
+    /// <param name="height">Высота поля</param>
     public void SetData(IMoveableObject movableObject, int width, int height)
     {
         if (movableObject == null)
@@ -31,7 +55,10 @@ public abstract class AbstractStrategy
         FieldWidth = width;
         FieldHeight = height;
     }
-    
+
+    /// <summary>
+    /// Шаг перемещения
+    /// </summary>
     public void MakeStep()
     {
         if (_state != StrategyStatus.InProgress)
@@ -48,16 +75,39 @@ public abstract class AbstractStrategy
         MoveToTarget();
     }
 
+    /// <summary>
+    /// Перемещение влево
+    /// </summary>
+    /// <returns>Результат перемещения (true - удалось переместиться, false - неудача)</returns>
     protected bool MoveLeft() => MoveTo(MovementDirection.Left);
 
+    /// <summary>
+    /// Перемещение вправо
+    /// </summary>
+    /// <returns>Результат перемещения (true - удалось переместиться, false - неудача)</returns>
     protected bool MoveRight() => MoveTo(MovementDirection.Right);
 
+    /// <summary>
+    /// Перемещение вверх
+    /// </summary>
+    /// <returns>Результат перемещения (true - удалось переместиться, false - неудача)</returns>
     protected bool MoveUp() => MoveTo(MovementDirection.Up);
 
+    /// <summary>
+    /// Перемещение вниз
+    /// </summary>
+    /// <returns>Результат перемещения (true - удалось переместиться, false - неудача)</returns>
     protected bool MoveDown() => MoveTo(MovementDirection.Down);
 
+    /// <summary>
+    /// Параметры объекта
+    /// </summary>
     protected ObjectParametrs? GetObjectParaments => _moveableObject?.GetObjectPosition;
 
+    /// <summary>
+    /// Шаг объекта
+    /// </summary>
+    /// <returns></returns>
     protected int? GetStep()
     {
         if (_state != StrategyStatus.InProgress)
@@ -68,10 +118,22 @@ public abstract class AbstractStrategy
         return _moveableObject?.GetStep;
     }
 
+    /// <summary>
+    /// Перемещение к цели
+    /// </summary>
     protected abstract void MoveToTarget();
 
+    /// <summary>
+    /// Достигнута ли цель
+    /// </summary>
+    /// <returns></returns>
     protected abstract bool IsTargetDestination();
 
+    /// <summary>
+    /// Попытка перемещения в требуемом направлении 
+    /// </summary>
+    /// <param name="movementDirection">Направление</param>
+    /// <returns>Результат попытки (true - удалось переместиться, false - неудача)</returns>
     private bool MoveTo(MovementDirection movementDirection)
     {
         if (_state != StrategyStatus.InProgress)
@@ -81,4 +143,4 @@ public abstract class AbstractStrategy
 
         return _moveableObject?.TryMoveObject(movementDirection) ?? false;
     }
-}
+}

DoubleDeckerBus/DoubleDeckerBus/MovementStrategy/IMoveableObject..cs

@@ -6,11 +6,25 @@ using System.Threading.Tasks;
 
 namespace DoubleDeckerBus.MovementStrategy;
 
+/// <summary>
+/// Интерфейс для работы с перемещаемым объектом
+/// </summary>
 public interface IMoveableObject
 {
+    /// <summary>
+    /// Получение координаты объекта
+    /// </summary>
     ObjectParametrs? GetObjectPosition { get; }
 
+    /// <summary>
+    /// Шаг объекта
+    /// </summary>
     int GetStep { get; }
 
+    /// <summary>
+    /// Попытка переместить объект в указанном направлении
+    /// </summary>
+    /// <param name="direction">Направление</param>
+    /// <returns>true - объект перемещен, false - перемещение невозможно</returns>
     bool TryMoveObject(MovementDirection direction);
 }

DoubleDeckerBus/DoubleDeckerBus/MovementStrategy/MoveToCenter.cs

@@ -6,6 +6,9 @@ using System.Threading.Tasks;
 
 namespace DoubleDeckerBus.MovementStrategy;
 
+/// <summary>
+/// Стратегия перемещения объекта в центр экрана
+/// </summary>
 public class MoveToCenter : AbstractStrategy
 {
     protected override bool IsTargetDestination()

DoubleDeckerBus/DoubleDeckerBus/MovementStrategy/MoveableBus.cs

@@ -7,16 +7,25 @@ using System.Threading.Tasks;
 
 namespace DoubleDeckerBus.MovementStrategy;
 
-
+/// <summary>
+/// Класс-реализация IMoveableObject с использованием DrawningBus
+/// </summary>
 public class MoveableBus : IMoveableObject
 {
+    /// <summary>
+    /// Поле-объект класса DrawningBus или его наследника
+    /// </summary>
     private DrawingBus? _bus = null;
 
-
+    /// <summary>
+    /// Конструктор
+    /// </summary>
+    /// <param name="bus">Объект класса DrawningBus</param>
     public MoveableBus(DrawingBus bus)
     {
         _bus = bus;
     }
+
     public ObjectParametrs? GetObjectPosition
     {
         get
@@ -41,6 +50,11 @@ public class MoveableBus : IMoveableObject
         return _bus.MoveTransport(GetDirectionType(direction));
     }
 
+    /// <summary>
+    /// Конвертация из MovementDirection в DirectionType
+    /// </summary>
+    /// <param name="direction">MovementDirection</param>
+    /// <returns>DirectionType</returns>
     private static DirectionType GetDirectionType(MovementDirection direction)
     {
         return direction switch

DoubleDeckerBus/DoubleDeckerBus/MovementStrategy/MovementDirection.cs

@@ -6,6 +6,9 @@ using System.Threading.Tasks;
 
 namespace DoubleDeckerBus.MovementStrategy;
 
+/// <summary>
+/// Направление перемещения 
+/// </summary>
 public enum MovementDirection
 {
     /// <summary>

DoubleDeckerBus/DoubleDeckerBus/MovementStrategy/MovetoBorder.cs

@@ -5,7 +5,9 @@ using System.Text;
 using System.Threading.Tasks;
 
 namespace DoubleDeckerBus.MovementStrategy;
-
+/// <summary>
+/// Стратегия перемещения объекта в правый нижний угол
+/// </summary>
 internal class MovetoBorder : AbstractStrategy
 {
     protected override bool IsTargetDestination()
@@ -15,7 +17,7 @@ internal class MovetoBorder : AbstractStrategy
         {
             return false;
         }
-        return objParams.DownBorder + GetStep() > FieldHeight - 5 && objParams.RightBorder + GetStep() > FieldWidth - 5;
+        return objParams.DownBorder + GetStep() >= FieldHeight && objParams.RightBorder + GetStep() >= FieldWidth;
     }
 
     protected override void MoveToTarget()

DoubleDeckerBus/DoubleDeckerBus/MovementStrategy/ObjectParametrs.cs

@@ -8,26 +8,63 @@ namespace DoubleDeckerBus.MovementStrategy;
 
 public class ObjectParametrs
 {
+    /// <summary>
+    /// Координата X
+    /// </summary>
     private readonly int _x;
 
+    /// <summary>
+    /// Координата Y
+    /// </summary>
     private readonly int _y;
 
+    /// <summary>
+    /// Ширина объекта
+    /// </summary>
     private readonly int _width;
 
+    /// <summary>
+    /// Высота объекта
+    /// </summary>
     private readonly int _height;
 
+    /// <summary>
+    /// Левая граница
+    /// </summary>
     public int LeftBorder => _x;
 
+    /// <summary>
+    /// Верхняя граница
+    /// </summary>
     public int TopBorder => _y;
-
+    
+    /// <summary>
+    /// Правая граница
+    /// </summary>
     public int RightBorder => _x + _width;
 
+    /// <summary>
+    /// Нижняя граница
+    /// </summary>
     public int DownBorder => _y + _height;
 
+    /// <summary>
+    /// Середина обхекта по горизонтали
+    /// </summary>
     public int ObjectMiddleHorizontal => _x + _width / 2;
 
+    /// <summary>
+    /// Середина объекта по вертикали
+    /// </summary>
     public int ObjectMiddleVertical => _y + _height / 2;
 
+    /// <summary>
+    /// Конструктор
+    /// </summary>
+    /// <param name="x">Координата X</param>
+    /// <param name="y">Координата Y</param>
+    /// <param name="width">Ширина объекта</param>
+    /// <param name="height">Высота объекта</param>
     public ObjectParametrs(int x, int y, int width, int height)
     {
         _x = x;

DoubleDeckerBus/DoubleDeckerBus/MovementStrategy/StrategyStatus.cs

@@ -6,16 +6,21 @@ using System.Threading.Tasks;
 
 namespace DoubleDeckerBus.MovementStrategy;
 
+/// <summary>
+/// Статус выполнения операции перемещения
+/// </summary>
 public enum StrategyStatus
 {   
     /// <summary>
     /// Все готово к началу
     /// </summary>
     NotInit,
+
     /// <summary>
     /// Выполняется 
     /// </summary>
     InProgress,
+
     /// <summary>
     /// Завершено
     /// </summary>