Этап 3. Форма

Warship/Warship/FormMapWithSetWarships.Designer.cs

@@ -29,77 +29,108 @@
         private void InitializeComponent()
         {
             this.groupBoxTools = new System.Windows.Forms.GroupBox();
-            this.buttonShowOnMap = new System.Windows.Forms.Button();
+            this.groupBoxMaps = new System.Windows.Forms.GroupBox();
-            this.buttonShowStorage = new System.Windows.Forms.Button();
+            this.buttonDeleteMap = new System.Windows.Forms.Button();
-            this.buttonRemoveWarship = new System.Windows.Forms.Button();
+            this.buttonAddMap = new System.Windows.Forms.Button();
-            this.maskedTextBoxPosition = new System.Windows.Forms.MaskedTextBox();
+            this.listBoxMaps = new System.Windows.Forms.ListBox();
-            this.buttonAddWarship = new System.Windows.Forms.Button();
+            this.textBoxMap = new System.Windows.Forms.TextBox();
             this.comboBoxSelectorMap = new System.Windows.Forms.ComboBox();
+            this.buttonAddWarship = new System.Windows.Forms.Button();
+            this.buttonShowOnMap = new System.Windows.Forms.Button();
+            this.buttonRemoveWarship = new System.Windows.Forms.Button();
+            this.buttonShowStorage = new System.Windows.Forms.Button();
+            this.maskedTextBoxPosition = new System.Windows.Forms.MaskedTextBox();
             this.pictureBox1 = new System.Windows.Forms.PictureBox();
             this.pictureBox = new System.Windows.Forms.PictureBox();
             this.groupBoxTools.SuspendLayout();
+            this.groupBoxMaps.SuspendLayout();
             ((System.ComponentModel.ISupportInitialize)(this.pictureBox1)).BeginInit();
             ((System.ComponentModel.ISupportInitialize)(this.pictureBox)).BeginInit();
             this.SuspendLayout();
             // 
             // groupBoxTools
             // 
-            this.groupBoxTools.Controls.Add(this.buttonShowOnMap);
+            this.groupBoxTools.Controls.Add(this.groupBoxMaps);
-            this.groupBoxTools.Controls.Add(this.buttonShowStorage);
-            this.groupBoxTools.Controls.Add(this.buttonRemoveWarship);
-            this.groupBoxTools.Controls.Add(this.maskedTextBoxPosition);
             this.groupBoxTools.Controls.Add(this.buttonAddWarship);
-            this.groupBoxTools.Controls.Add(this.comboBoxSelectorMap);
+            this.groupBoxTools.Controls.Add(this.buttonShowOnMap);
+            this.groupBoxTools.Controls.Add(this.buttonRemoveWarship);
+            this.groupBoxTools.Controls.Add(this.buttonShowStorage);
+            this.groupBoxTools.Controls.Add(this.maskedTextBoxPosition);
             this.groupBoxTools.Controls.Add(this.pictureBox1);
             this.groupBoxTools.Dock = System.Windows.Forms.DockStyle.Right;
-            this.groupBoxTools.Location = new System.Drawing.Point(600, 0);
+            this.groupBoxTools.Location = new System.Drawing.Point(911, 0);
             this.groupBoxTools.Name = "groupBoxTools";
-            this.groupBoxTools.Size = new System.Drawing.Size(200, 450);
+            this.groupBoxTools.Size = new System.Drawing.Size(200, 597);
             this.groupBoxTools.TabIndex = 0;
             this.groupBoxTools.TabStop = false;
             this.groupBoxTools.Text = "Инструменты";
             // 
-            // buttonShowOnMap
+            // groupBoxMaps
             // 
-            this.buttonShowOnMap.Location = new System.Drawing.Point(25, 317);
+            this.groupBoxMaps.Controls.Add(this.buttonDeleteMap);
-            this.buttonShowOnMap.Name = "buttonShowOnMap";
+            this.groupBoxMaps.Controls.Add(this.buttonAddMap);
-            this.buttonShowOnMap.Size = new System.Drawing.Size(163, 23);
+            this.groupBoxMaps.Controls.Add(this.listBoxMaps);
-            this.buttonShowOnMap.TabIndex = 7;
+            this.groupBoxMaps.Controls.Add(this.textBoxMap);
-            this.buttonShowOnMap.Text = "Посмотреть карту";
+            this.groupBoxMaps.Controls.Add(this.comboBoxSelectorMap);
-            this.buttonShowOnMap.UseVisualStyleBackColor = true;
+            this.groupBoxMaps.Location = new System.Drawing.Point(15, 32);
-            this.buttonShowOnMap.Click += new System.EventHandler(this.ButtonShowOnMap_Click);
+            this.groupBoxMaps.Name = "groupBoxMaps";
+            this.groupBoxMaps.Size = new System.Drawing.Size(173, 250);
+            this.groupBoxMaps.TabIndex = 2;
+            this.groupBoxMaps.TabStop = false;
+            this.groupBoxMaps.Text = "Карты";
             // 
-            // buttonShowStorage
+            // buttonDeleteMap
             // 
-            this.buttonShowStorage.Location = new System.Drawing.Point(25, 253);
+            this.buttonDeleteMap.Location = new System.Drawing.Point(6, 214);
-            this.buttonShowStorage.Name = "buttonShowStorage";
+            this.buttonDeleteMap.Name = "buttonDeleteMap";
-            this.buttonShowStorage.Size = new System.Drawing.Size(163, 23);
+            this.buttonDeleteMap.Size = new System.Drawing.Size(152, 23);
-            this.buttonShowStorage.TabIndex = 6;
+            this.buttonDeleteMap.TabIndex = 9;
-            this.buttonShowStorage.Text = "Посмотреть хранилище";
+            this.buttonDeleteMap.Text = "Удалить карту";
-            this.buttonShowStorage.UseVisualStyleBackColor = true;
+            this.buttonDeleteMap.UseVisualStyleBackColor = true;
-            this.buttonShowStorage.Click += new System.EventHandler(this.ButtonShowStorage_Click);
+            this.buttonDeleteMap.Click += new System.EventHandler(this.ButtonDeleteMap_Click);
             // 
-            // buttonRemoveWarship
+            // buttonAddMap
             // 
-            this.buttonRemoveWarship.Location = new System.Drawing.Point(25, 188);
+            this.buttonAddMap.Location = new System.Drawing.Point(3, 77);
-            this.buttonRemoveWarship.Name = "buttonRemoveWarship";
+            this.buttonAddMap.Name = "buttonAddMap";
-            this.buttonRemoveWarship.Size = new System.Drawing.Size(163, 23);
+            this.buttonAddMap.Size = new System.Drawing.Size(155, 31);
-            this.buttonRemoveWarship.TabIndex = 5;
+            this.buttonAddMap.TabIndex = 7;
-            this.buttonRemoveWarship.Text = "Удалить корабль";
+            this.buttonAddMap.Text = "Добавить карту";
-            this.buttonRemoveWarship.UseVisualStyleBackColor = true;
+            this.buttonAddMap.UseVisualStyleBackColor = true;
-            this.buttonRemoveWarship.Click += new System.EventHandler(this.ButtonRemoveWarship_Click);
+            this.buttonAddMap.Click += new System.EventHandler(this.ButtonAddMap_Click);
             // 
-            // maskedTextBoxPosition
+            // listBoxMaps
             // 
-            this.maskedTextBoxPosition.Location = new System.Drawing.Point(25, 123);
+            this.listBoxMaps.FormattingEnabled = true;
-            this.maskedTextBoxPosition.Mask = "00";
+            this.listBoxMaps.ItemHeight = 15;
-            this.maskedTextBoxPosition.Name = "maskedTextBoxPosition";
+            this.listBoxMaps.Location = new System.Drawing.Point(3, 114);
-            this.maskedTextBoxPosition.Size = new System.Drawing.Size(163, 23);
+            this.listBoxMaps.Name = "listBoxMaps";
-            this.maskedTextBoxPosition.TabIndex = 4;
+            this.listBoxMaps.Size = new System.Drawing.Size(158, 94);
+            this.listBoxMaps.TabIndex = 8;
+            this.listBoxMaps.SelectedIndexChanged += new System.EventHandler(this.ListBoxMaps_SelectedIndexChanged);
+            // 
+            // textBoxMap
+            // 
+            this.textBoxMap.Location = new System.Drawing.Point(3, 19);
+            this.textBoxMap.Name = "textBoxMap";
+            this.textBoxMap.Size = new System.Drawing.Size(164, 23);
+            this.textBoxMap.TabIndex = 6;
+            // 
+            // comboBoxSelectorMap
+            // 
+            this.comboBoxSelectorMap.DropDownStyle = System.Windows.Forms.ComboBoxStyle.DropDownList;
+            this.comboBoxSelectorMap.FormattingEnabled = true;
+            this.comboBoxSelectorMap.Items.AddRange(new object[] {
+            "Первая карта",
+            "Вторая карта"});
+            this.comboBoxSelectorMap.Location = new System.Drawing.Point(4, 48);
+            this.comboBoxSelectorMap.Name = "comboBoxSelectorMap";
+            this.comboBoxSelectorMap.Size = new System.Drawing.Size(157, 23);
+            this.comboBoxSelectorMap.TabIndex = 0;
             // 
             // buttonAddWarship
             // 
-            this.buttonAddWarship.Location = new System.Drawing.Point(25, 62);
+            this.buttonAddWarship.Location = new System.Drawing.Point(18, 344);
             this.buttonAddWarship.Name = "buttonAddWarship";
             this.buttonAddWarship.Size = new System.Drawing.Size(163, 23);
             this.buttonAddWarship.TabIndex = 3;
@@ -107,23 +138,49 @@
             this.buttonAddWarship.UseVisualStyleBackColor = true;
             this.buttonAddWarship.Click += new System.EventHandler(this.ButtonAddWarship_Click);
             // 
-            // comboBoxSelectorMap
+            // buttonShowOnMap
             // 
-            this.comboBoxSelectorMap.FormattingEnabled = true;
+            this.buttonShowOnMap.Location = new System.Drawing.Point(15, 509);
-            this.comboBoxSelectorMap.Items.AddRange(new object[] {
+            this.buttonShowOnMap.Name = "buttonShowOnMap";
-            "Первая карта",
+            this.buttonShowOnMap.Size = new System.Drawing.Size(163, 23);
-            "Вторая карта"});
+            this.buttonShowOnMap.TabIndex = 7;
-            this.comboBoxSelectorMap.Location = new System.Drawing.Point(25, 22);
+            this.buttonShowOnMap.Text = "Посмотреть карту";
-            this.comboBoxSelectorMap.Name = "comboBoxSelectorMap";
+            this.buttonShowOnMap.UseVisualStyleBackColor = true;
-            this.comboBoxSelectorMap.Size = new System.Drawing.Size(163, 23);
+            this.buttonShowOnMap.Click += new System.EventHandler(this.ButtonShowOnMap_Click);
-            this.comboBoxSelectorMap.TabIndex = 2;
+            // 
-            this.comboBoxSelectorMap.Click += new System.EventHandler(this.ComboBoxSelectorMap_SelectedIndexChanged);
+            // buttonRemoveWarship
+            // 
+            this.buttonRemoveWarship.Location = new System.Drawing.Point(15, 402);
+            this.buttonRemoveWarship.Name = "buttonRemoveWarship";
+            this.buttonRemoveWarship.Size = new System.Drawing.Size(163, 23);
+            this.buttonRemoveWarship.TabIndex = 5;
+            this.buttonRemoveWarship.Text = "Удалить корабль";
+            this.buttonRemoveWarship.UseVisualStyleBackColor = true;
+            this.buttonRemoveWarship.Click += new System.EventHandler(this.ButtonRemoveWarship_Click);
+            // 
+            // buttonShowStorage
+            // 
+            this.buttonShowStorage.Location = new System.Drawing.Point(18, 480);
+            this.buttonShowStorage.Name = "buttonShowStorage";
+            this.buttonShowStorage.Size = new System.Drawing.Size(163, 23);
+            this.buttonShowStorage.TabIndex = 6;
+            this.buttonShowStorage.Text = "Посмотреть хранилище";
+            this.buttonShowStorage.UseVisualStyleBackColor = true;
+            this.buttonShowStorage.Click += new System.EventHandler(this.ButtonShowStorage_Click);
+            // 
+            // maskedTextBoxPosition
+            // 
+            this.maskedTextBoxPosition.Location = new System.Drawing.Point(19, 373);
+            this.maskedTextBoxPosition.Mask = "00";
+            this.maskedTextBoxPosition.Name = "maskedTextBoxPosition";
+            this.maskedTextBoxPosition.Size = new System.Drawing.Size(163, 23);
+            this.maskedTextBoxPosition.TabIndex = 4;
             // 
             // pictureBox1
             // 
             this.pictureBox1.Location = new System.Drawing.Point(3, 12);
             this.pictureBox1.Name = "pictureBox1";
-            this.pictureBox1.Size = new System.Drawing.Size(197, 438);
+            this.pictureBox1.Size = new System.Drawing.Size(197, 546);
             this.pictureBox1.TabIndex = 0;
             this.pictureBox1.TabStop = false;
             // 
@@ -132,7 +189,7 @@
             this.pictureBox.Dock = System.Windows.Forms.DockStyle.Fill;
             this.pictureBox.Location = new System.Drawing.Point(0, 0);
             this.pictureBox.Name = "pictureBox";
-            this.pictureBox.Size = new System.Drawing.Size(600, 450);
+            this.pictureBox.Size = new System.Drawing.Size(911, 597);
             this.pictureBox.TabIndex = 1;
             this.pictureBox.TabStop = false;
             // 
@@ -140,13 +197,15 @@
             // 
             this.AutoScaleDimensions = new System.Drawing.SizeF(7F, 15F);
             this.AutoScaleMode = System.Windows.Forms.AutoScaleMode.Font;
-            this.ClientSize = new System.Drawing.Size(800, 450);
+            this.ClientSize = new System.Drawing.Size(1111, 597);
             this.Controls.Add(this.pictureBox);
             this.Controls.Add(this.groupBoxTools);
             this.Name = "FormMapWithSetWarships";
             this.Text = "FormMapWithSetWarships";
             this.groupBoxTools.ResumeLayout(false);
             this.groupBoxTools.PerformLayout();
+            this.groupBoxMaps.ResumeLayout(false);
+            this.groupBoxMaps.PerformLayout();
             ((System.ComponentModel.ISupportInitialize)(this.pictureBox1)).EndInit();
             ((System.ComponentModel.ISupportInitialize)(this.pictureBox)).EndInit();
             this.ResumeLayout(false);
@@ -164,5 +223,10 @@
         private Button buttonShowStorage;
         private Button buttonRemoveWarship;
         private MaskedTextBox maskedTextBoxPosition;
+        private GroupBox groupBoxMaps;
+        private TextBox textBoxMap;
+        private Button buttonAddMap;
+        private ListBox listBoxMaps;
+        private Button buttonDeleteMap;
     }
 }

Warship/Warship/FormMapWithSetWarships.cs

@@ -12,40 +12,46 @@ namespace Warship
 {
     public partial class FormMapWithSetWarships : Form
     {
-        private MapWithSetWarshipsGeneric<DrawingObjectWarship, AbstractMap> _mapWarshipsCollectionGeneric;
+
+        private readonly Dictionary<string, AbstractMap> _mapDict = new()
+        {
+            {"Первая карта",new SimpleMap() },
+            {"Вторая карта",new SecondMap() }
+        };
+
+        private readonly MapsCollection _mapsCollection;
 
         public FormMapWithSetWarships()
         {
             InitializeComponent();
+            _mapsCollection = new MapsCollection(pictureBox.Width, pictureBox.Height);
+            comboBoxSelectorMap.Items.Clear();
+            foreach (var elem in _mapDict)
+            {
+                comboBoxSelectorMap.Items.Add(elem.Key);
+            }
         }
 
-        private void ComboBoxSelectorMap_SelectedIndexChanged(object sender, EventArgs e)
+        private void ReloadMaps()
         {
-            AbstractMap map = null;
+            int index = listBoxMaps.SelectedIndex;
-
+            for (int i = 0; i < _mapsCollection.Keys.Count; i++)
-            switch (comboBoxSelectorMap.Text)
             {
-                case "Первая карта":
+                listBoxMaps.Items.Add(_mapsCollection.Keys[i]);
-                    map = new SimpleMap();
-                    break;
-                case "Вторая карта":
-                    map = new SecondMap();
-                    break;
             }
-            if (map != null)
+            if (listBoxMaps.Items.Count > 0 && (index == -1 || index >= listBoxMaps.Items.Count))
             {
-                _mapWarshipsCollectionGeneric = new MapWithSetWarshipsGeneric<DrawingObjectWarship, AbstractMap>(
+                listBoxMaps.SelectedIndex = 0;
-                pictureBox.Width, pictureBox.Height, map);
             }
-            else
+            else if (listBoxMaps.Items.Count > 0 && index > -1 && index < listBoxMaps.Items.Count)
             {
-                _mapWarshipsCollectionGeneric = null;
+                listBoxMaps.SelectedIndex = index;
             }
         }
 
         private void ButtonAddWarship_Click(object sender, EventArgs e)
         {
-            if (_mapWarshipsCollectionGeneric == null)
+            if (listBoxMaps.SelectedIndex == -1)
             {
                 return;
             }
@@ -53,10 +59,10 @@ namespace Warship
             if (form.ShowDialog() == DialogResult.OK)
             {
                 DrawingObjectWarship warship = new(form.SelectedWarship);
-                if (_mapWarshipsCollectionGeneric + warship >= 0)
+                if (_mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty] + warship != -1)
                 {
                     MessageBox.Show("Объект добавлен");
-                    pictureBox.Image = _mapWarshipsCollectionGeneric.ShowSet();
+                    pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet();
                 }
                 else
                 {
@@ -67,6 +73,10 @@ namespace Warship
 
         private void ButtonRemoveWarship_Click(object sender, EventArgs e)
         {
+            if (listBoxMaps.SelectedIndex == -1)
+            {
+                return;
+            }
             if (string.IsNullOrEmpty(maskedTextBoxPosition.Text))
             {
                 return;
@@ -76,10 +86,10 @@ namespace Warship
                 return;
             }
             int pos = Convert.ToInt32(maskedTextBoxPosition.Text);
-            if (_mapWarshipsCollectionGeneric - pos != null)
+            if (_mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty] - pos != null)
             {
                 MessageBox.Show("Объект удален");
-                pictureBox.Image = _mapWarshipsCollectionGeneric.ShowSet();
+                pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet();
             }
             else
             {
@@ -89,25 +99,25 @@ namespace Warship
 
         private void ButtonShowStorage_Click(object sender, EventArgs e)
         {
-            if (_mapWarshipsCollectionGeneric == null)
+            if (listBoxMaps.SelectedIndex == -1)
             {
                 return;
             }
-            pictureBox.Image = _mapWarshipsCollectionGeneric.ShowSet();
+            pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet();
         }
 
         private void ButtonShowOnMap_Click(object sender, EventArgs e)
         {
-            if (_mapWarshipsCollectionGeneric == null)
+            if (listBoxMaps.SelectedIndex == -1)
             {
                 return;
             }
-            pictureBox.Image = _mapWarshipsCollectionGeneric.ShowOnMap();
+            pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowOnMap();
         }
 
         private void ButtonMove_Click(object sender, EventArgs e)
         {
-            if (_mapWarshipsCollectionGeneric == null)
+            if (listBoxMaps.SelectedIndex == -1)
             {
                 return;
             }
@@ -128,7 +138,43 @@ namespace Warship
                     dir = Direction.Right;
                     break;
             }
-            pictureBox.Image = _mapWarshipsCollectionGeneric.MoveObject(dir);
+            pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].MoveObject(dir);
+        }
+
+        private void ButtonAddMap_Click(object sender, EventArgs e)
+        {
+            if (comboBoxSelectorMap.SelectedIndex == -1 || string.IsNullOrEmpty(textBoxMap.Text))
+            {
+                MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                return;
+            }
+            if (!_mapDict.ContainsKey(comboBoxSelectorMap.Text))
+            {
+                MessageBox.Show("Нет такой карты", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                return;
+            }
+            listBoxMaps.Items.Clear();
+            _mapsCollection.AddMap(textBoxMap.Text, _mapDict[comboBoxSelectorMap.Text]);
+            ReloadMaps();
+        }
+
+        private void ListBoxMaps_SelectedIndexChanged(object sender, EventArgs e)
+        {
+            pictureBox.Image = _mapsCollection[listBoxMaps.SelectedItem?.ToString() ?? string.Empty].ShowSet();
+        }
+
+        private void ButtonDeleteMap_Click(object sender, EventArgs e)
+        {
+            if (listBoxMaps.SelectedIndex == -1)
+            {
+                return;
+            }
+            if (MessageBox.Show($"Удалить карту {listBoxMaps.SelectedItem}?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.Yes)
+            {
+                _mapsCollection.DelMap(listBoxMaps.SelectedItem?.ToString() ?? string.Empty);
+                listBoxMaps.Items.Clear();
+                ReloadMaps();
+            }
         }
     }
 }

Warship/Warship/MapWithSetWarshipsGeneric.cs

@@ -49,13 +49,9 @@ namespace Warship
         public Bitmap ShowOnMap()
         {
             Shaking();
-            for (int i = 0; i < _setWarship.Count; i++)
+            foreach (var warship in _setWarship.GetWarships())
             {
-                var warship = _setWarship.Get(i);
+                return _map.CreateMap(_pictureWidth, _pictureHeight, warship);
-                if (warship != null)
-                {
-                    return _map.CreateMap(_pictureWidth, _pictureHeight, warship);
-                }
             }
             return new(_pictureWidth, _pictureHeight);
         }
@@ -74,11 +70,11 @@ namespace Warship
             int j = _setWarship.Count - 1;
             for (int i = 0; i < _setWarship.Count; i++)
             {
-                if (_setWarship.Get(i) == null)
+                if (_setWarship[i] == null)
                 {
                     for (; j > i; j--)
                     {
-                        var warship = _setWarship.Get(j);
+                        var warship = _setWarship[j];
                         if (warship != null)
                         {
                             _setWarship.Insert(warship, i);
@@ -113,14 +109,13 @@ namespace Warship
         {
             int width = _pictureWidth / _placeSizeWidth;
             int height = _pictureHeight / _placeSizeHeight;
+            int i = 0;
 
-            for (int i = 0; i < _setWarship.Count; i++)
+            foreach (var warship in _setWarship.GetWarships())
             {
-                if (_setWarship.Get(i) != null)
+                warship.SetObject(i % width * _placeSizeWidth, (height - 1 - i / width) * _placeSizeHeight, _pictureWidth, _pictureHeight);
-                {
+                warship.DrawingObject(gr);
-                    _setWarship.Get(i).SetObject(i % width * _placeSizeWidth, (height - 1 - i / width) * _placeSizeHeight, _pictureWidth, _pictureHeight);
+                i++;
-                    _setWarship.Get(i)?.DrawingObject(gr);
-                }
             }
         }
     }

Warship/Warship/MapsCollection.cs

@@ -0,0 +1,55 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace Warship
+{
+    internal class MapsCollection
+    {
+        readonly Dictionary<string, MapWithSetWarshipsGeneric<DrawingObjectWarship, AbstractMap>> _mapStorages;
+
+        public List<string> Keys => _mapStorages.Keys.ToList();
+
+        private readonly int _pictureWidth;
+        private readonly int _pictureHeight;
+
+        public MapsCollection(int pictureWidth, int pictureHeight)
+        {
+            _mapStorages = new Dictionary<string, MapWithSetWarshipsGeneric<DrawingObjectWarship, AbstractMap>>();
+            _pictureWidth = pictureWidth;
+            _pictureHeight = pictureHeight;
+        }
+
+        public void AddMap(string name, AbstractMap map)
+        {
+            if (_mapStorages.ContainsKey(name))
+            {
+                MessageBox.Show("Карта уже существует");
+                return;
+            }
+            else
+            {
+                _mapStorages.Add(name, new MapWithSetWarshipsGeneric<DrawingObjectWarship, AbstractMap>(_pictureWidth, _pictureHeight, map));
+            }
+        }
+
+        public void DelMap(string name)
+        {
+
+            _mapStorages.Remove(name);
+
+        }
+
+        public MapWithSetWarshipsGeneric<DrawingObjectWarship, AbstractMap> this[string ind]
+        {
+            get
+            {
+                if (_mapStorages.ContainsKey(ind))
+                    return _mapStorages[ind];
+                return null;
+            }
+        }
+    }
+}

Warship/Warship/SetWarshipsGeneric.cs

@@ -9,69 +9,73 @@ namespace Warship
     internal class SetWarshipsGeneric<T>
        where T : class
     {
-        private readonly T[] _places;
+        private readonly List<T> _places;
 
-        public int Count => _places.Length;
+        public int Count => _places.Count;
+
+        private readonly int _maxCount;
 
         public SetWarshipsGeneric(int count)
         {
-            _places = new T[count];
+            _maxCount = count;
+            _places = new List<T>();
         }
 
         public int Insert(T warship)
         {
-            return Insert(warship, 0);
+            if (_places.Count + 1 >= _maxCount)
+                return -1;
+            _places.Insert(0, warship);
+            return 0;
         }
 
         public int Insert(T warship, int position)
         {
-            int EmptyEl = -1;
+            if (position >= _maxCount || position < 0)
-
-            if (position >= Count || position < 0)
                 return -1;
-
+            if (_places.Count + 1 >= _maxCount)
-            if (_places[position] == null)
+                return -1;
-            {
+            _places.Insert(position, warship);
-                _places[position] = warship;
+            return position;
-                return 1;
-            }
-
-            else if (_places[position] != null)
-            {
-                for (int i = position + 1; i < Count; i++)
-                    if (_places[i] == null)
-                    {
-                        EmptyEl = i;
-                        break;
-                    }
-
-                if (EmptyEl == -1)
-                    return -1;
-
-                for (int i = EmptyEl; i > position; i--)
-                    _places[i] = _places[i - 1];
-            }
-
-            _places[position] = warship;
-            return 1;
         }
 
         public T Remove(int position)
         {
-            if (position >= Count || position < 0 || _places[position] == null)
+            if (position >= _maxCount || position < 0)
                 return null;
 
             T deleted = _places[position];
-            _places[position] = null;
+            _places.RemoveAt(position);
             return deleted;
         }
 
-        public T Get(int position)
+        public T this[int position]
         {
-            if (position >= Count || position < 0)
+            get
-                return null;
+            {
-
+                if (position < 0 || position >= _maxCount)
-            return _places[position];
+                    return null;
+                return _places[position];
+            }
+            set
+            {
+                if (position < 0 || position >= _maxCount)
+                    Insert(value, position);
+            }
+        }
+        public IEnumerable<T> GetWarships()
+        {
+            foreach (var warship in _places)
+            {
+                if (warship != null)
+                {
+                    yield return warship;
+                }
+                else
+                {
+                    yield break;
+                }
+            }
         }
     }
 }