Лабараторная работа #3 2

ProjectWarmlyShip/ProjectWarmlyShip/CollectionGenericObjects/AbstractCompany.cs

@@ -23,7 +23,7 @@ public abstract class AbstractCompany
     /// <summary>
     /// Коллекция судов
     /// </summary>
-    protected ICollectionGenericObjects<DrawningShip>? _collection = null;
+    public ICollectionGenericObjects<DrawningShip>? _collection = null;
     /// <summary>
     /// Вычисление максимального количества элементов, который можно разместить в окне
     /// </summary>
@@ -47,9 +47,9 @@ public abstract class AbstractCompany
     /// <param name="company">Компания</param>
     /// <param name="ship">Добавляемый объект</param>
     /// <returns></returns>
-    public static bool operator +(AbstractCompany company, DrawningShip ship)
+    public static int operator +(AbstractCompany company, DrawningShip ship)
     {
-        return company._collection?.Insert(ship) ?? false;
+        return company._collection.Insert(ship);
     }
     /// <summary>
     /// Перегрузка оператора удаления для класса
@@ -57,9 +57,9 @@ public abstract class AbstractCompany
     /// <param name="company">Компания</param>
     /// <param name="position">Номер удаляемого объекта</param>
     /// <returns></returns>
-    public static bool operator -(AbstractCompany company, int position)
+    public static DrawningShip operator -(AbstractCompany company, int position)
     {
-        return company._collection?.Remove(position) ?? false;
+        return company._collection?.Remove(position);
     }
     /// <summary>
     /// Получение случайного объекта из коллекции

ProjectWarmlyShip/ProjectWarmlyShip/CollectionGenericObjects/ICollectionGenericObjects.cs

@@ -16,20 +16,20 @@ public interface ICollectionGenericObjects<T>
     /// </summary>
     /// <param name="obj">Добавляемый объект</param>
     /// <returns>true - вставка прошла удачно, false - вставка неудалась</returns>
-    bool Insert(T obj);
+    int Insert(T obj);
     /// <summary>
     /// Добавление объекта в коллекцию на конкретную позицию
     /// </summary>
     /// <param name="obj">Добавляемый объект</param>
     /// <param name="position">Позиция</param>
     /// <returns>true - вставка прошла удачно, false - вставка неудалась</returns>
-    bool Insert(T obj, int position);
+    int Insert(T obj, int position);
     /// <summary>
     /// Удаление объекта из коллекции с конкретной позиции
     /// </summary>
     /// <param name="position">Позиция</param>
     /// <returns>true - удаление прошло удачно, false - удаление неудалось</returns>
-    bool Remove(int position);
+    T Remove(int position);
     /// <summary>
     /// Получение объекта по позиции
     /// </summary>

ProjectWarmlyShip/ProjectWarmlyShip/CollectionGenericObjects/MassiveGenericObjects.cs

@@ -22,7 +22,7 @@ public class MassiveGenericObjects<T> : ICollectionGenericObjects<T>
         if (position >= _collection.Length || position < 0) return null; 
         return _collection[position];
     }
-    public bool Insert(T obj)
+    public int Insert(T obj)
     {
         // TODO вставка в свободное место набора
         int index = 0;
@@ -31,13 +31,13 @@ public class MassiveGenericObjects<T> : ICollectionGenericObjects<T>
             if (_collection[index] == null)
             {
                 _collection[index] = obj;
-                return true;
+                return index;
             }
             ++index;
         }
-        return false;
+        return -1;
     }
-    public bool Insert(T obj, int position)
+    public int Insert(T obj, int position)
     {
         // TODO проверка позиции
         // TODO проверка, что элемент массива по этой позиции пустой, если нет, то
@@ -45,10 +45,10 @@ public class MassiveGenericObjects<T> : ICollectionGenericObjects<T>
         // если нет после, ищем до
         // TODO вставка
         if (position >= _collection.Length || position < 0) 
-            return false;
+            return -1;
         if (_collection[position] == null) {
             _collection[position] = obj;
-            return true;
+            return position;
         }
         int index = position + 1;
         while (index < _collection.Length)
@@ -56,7 +56,7 @@ public class MassiveGenericObjects<T> : ICollectionGenericObjects<T>
             if (_collection[index] == null)
             {
                 _collection[index] = obj;
-                return true;
+                return index;
             }
             ++index;
         }
@@ -66,19 +66,20 @@ public class MassiveGenericObjects<T> : ICollectionGenericObjects<T>
             if (_collection[index] == null)
             {
                 _collection[index] = obj;
-                return true;
+                return index;
             }
             --index;
         }
-        return false;
+        return -1;
     }
-    public bool Remove(int position)
+    public T Remove(int position)
     {
         // TODO проверка позиции
         // TODO удаление объекта из массива, присвоив элементу массива значение null
         if (position >= _collection.Length || position < 0)
-            return false;
+            return null;
+        T obj = _collection[position];  
         _collection[position] = null;
-        return true;
+        return obj;
     }
 }

ProjectWarmlyShip/ProjectWarmlyShip/CollectionGenericObjects/ShipPortService.cs

@@ -19,8 +19,7 @@ public class ShipPortService : AbstractCompany
         {
             for (int j = 0; j < height + 1; ++j)
             {
-                g.DrawLine(pen, i * _placeSizeWidth, j * _placeSizeHeight,
+                g.DrawLine(pen, i * _placeSizeWidth, j * _placeSizeHeight, i * _placeSizeWidth + _placeSizeWidth - 5, j * _placeSizeHeight);
-                i * _placeSizeWidth + _placeSizeWidth - 5, j * _placeSizeHeight);
             }
         }
     }
@@ -40,7 +39,6 @@ public class ShipPortService : AbstractCompany
                 _collection.Get(i).SetPictureSize(_pictureWidth, _pictureHeight);
                 _collection.Get(i).SetPosition(_placeSizeWidth * curWidth + 20, curHeight * _placeSizeHeight + 4);
             }
-
             if (curWidth > 0)
                 curWidth--;
             else

ProjectWarmlyShip/ProjectWarmlyShip/FormShipCollection.cs

@@ -42,7 +42,7 @@ public partial class FormShipCollection : Form
             default:
                 return;
         }
-        if (_company + drawningShip)
+        if (_company + drawningShip != -1)
         {
             MessageBox.Show("Объект добавлен");
             pictureBox.Image = _company.Show();
@@ -84,7 +84,7 @@ public partial class FormShipCollection : Form
             return;
         }
         int pos = Convert.ToInt32(maskedTextBox.Text);
-        if (_company - pos)
+        if (_company - pos != null)
         {
             MessageBox.Show("Объект удален");
             pictureBox.Image = _company.Show();