Airbus/Airbus/MapWithSetPlanesGeneric.cs

@@ -88,16 +88,13 @@ namespace Airbus
         public Bitmap ShowOnMap()
         {
             Shaking();
-            for (int i = 0; i < _setPlanes.Count; i++)
+            foreach (var plane in _setPlanes.GetPlanes())
             {
-                var plane = _setPlanes.Get(i);
-                if (plane != null)
-                {
-                    return _map.CreateMap(_pictureWidth, _pictureHeight, plane);
-                }
+                return _map.CreateMap(_pictureWidth, _pictureHeight, plane);
             }
             return new(_pictureWidth, _pictureHeight);
         }
+
         /// <summary>
         /// Перемещение объекта по крате
         /// </summary>
@@ -119,11 +116,11 @@ namespace Airbus
             int j = _setPlanes.Count - 1;
             for (int i = 0; i < _setPlanes.Count; i++)
             {
-                if (_setPlanes.Get(i) == null)
+                if (_setPlanes[i] == null)
                 {
                     for (; j > i; j--)
                     {
-                        var plane = _setPlanes.Get(j);
+                        var plane = _setPlanes[j];
                         if (plane != null)
                         {
                             _setPlanes.Insert(plane, i);
@@ -181,19 +178,13 @@ namespace Airbus
         /// <param name="g"></param>
         private void DrawPlanes(Graphics g)
         {
-            int width = _pictureWidth / _placeSizeWidth;
-            int height = _pictureHeight / _placeSizeHeight;
-
-            for (int i = 0; i < _setPlanes.Count; i++)
+            foreach (var plane in _setPlanes.GetPlanes())
             {
                 // TODO установка позиции
-                if(_setPlanes.Get(i) != null)
-                {
-                    _setPlanes.Get(i).SetObject((width - i % width - 1) * _placeSizeWidth,(height - i / width - 1) * _placeSizeHeight, _pictureWidth, _pictureHeight);
-                    _setPlanes.Get(i)?.DrawningObject(g);
-                }
+                plane.DrawningObject(g);
             }
         }
+
     }
 
 }

Airbus/Airbus/SetPlanesGeneric.cs

@@ -10,75 +10,46 @@ namespace Airbus
         where T : class
     {
         /// <summary>
-        /// Массив объектов, которые храним
+        /// Список объектов, которые храним
         /// </summary>
-        private readonly T[] _places;
+        private readonly List<T> _places;
         /// <summary>
-        /// Количество объектов в массиве
+        /// Количество объектов в списке
         /// </summary>
-        public int Count => _places.Length;
+        public int Count => _places.Count;
+        private readonly int _maxCount;
         /// <summary>
         /// Конструктор
         /// </summary>
         /// <param name="count"></param>
         public SetPlanesGeneric(int count)
         {
-            _places = new T[count];
+            _maxCount = count;
+            _places = new List<T>();
         }
         /// <summary>
         /// Добавление объекта в набор
         /// </summary>
-        /// <param name="car">Добавляемый самолёт</param>
+        /// <param name="plane">Добавляемый автомобиль</param>
         /// <returns></returns>
         public int Insert(T plane)
         {
             // TODO вставка в начало набора
-            return Insert(plane, 0); 
+            // TODO проверка на _maxCount
+            return -1;
         }
-
         /// <summary>
         /// Добавление объекта в набор на конкретную позицию
         /// </summary>
-        /// <param name="plane">Добавляемый самолёт</param>
+        /// <param name="plane">Добавляемый автомобиль</param>
         /// <param name="position">Позиция</param>
         /// <returns></returns>
         public int Insert(T plane, int position)
         {
             // TODO проверка позиции
-            // TODO проверка, что элемент массива по этой позиции пустой, если нет, то
-            // проверка, что после вставляемого элемента в массиве есть пустой элемент
-            // сдвиг всех объектов, находящихся справа от позиции до первого пустого элемента
             // TODO вставка по позиции
-
-            if(_places[position] == null)
-            {
-                _places[position] = plane;
-                return position;
-            }
-            else
-            {
-                int pos = -1;
-                bool isFree = false;
-                for (int i = 0; i < Count; i++)
-                {
-                    if (_places[i] == null)
-                    {
-                        isFree = true;
-                        pos = i;
-                        break;
-                    }
-                }
-                if (isFree)
-                {
-                    for (var i = pos; i > position; --i) _places[i] = _places[i - 1];
-                    _places[position] = plane;
-                    return position;
-                }
-                else
-                {
-                    return -1;
-                }
-            }
+            _places[position] = plane;
+            return -1;
         }
         /// <summary>
         /// Удаление объекта из набора с конкретной позиции
@@ -99,15 +70,36 @@ namespace Airbus
         /// </summary>
         /// <param name="position"></param>
         /// <returns></returns>
-        public T Get(int position)
+        public T this[int position]
         {
-            // TODO проверка позиции
-            if (position < 0 || position >= Count)
+            get
             {
-                return null;
+                // TODO проверка позиции
+                return _places[position];
+            }
+            set
+            {
+                // TODO проверка позиции
+                // TODO вставка в список по позиции
+            }
+        }
+        /// <summary>
+        /// Проход по набору до первого пустого
+        /// </summary>
+        /// <returns></returns>
+        public IEnumerable<T> GetPlanes()
+        {
+            foreach (var plane in _places)
+            {
+                if (plane != null)
+                {
+                    yield return plane;
+                }
+                else
+                {
+                    yield break;
+                }
             }
-            return _places[position];
         }
     }
-
 }