2

Stormtrooper/Stormtrooper/CollectionGenericObjects/MassiveGenericObjects.cs

@@ -1,31 +1,33 @@
-using Stormtrooper.Drawnings;
+using Stormtrooper.Exceptions;
+using Stormtrooper.CollectionGenericObjects;
 using Stormtrooper.Exceptions;
-using System.CodeDom;
 
 namespace Stormtrooper.CollectionGenericObjects;
-
 /// <summary>
 /// Параметризованный набор объектов
 /// </summary>
 /// <typeparam name="T">Параметр: ограничение - ссылочный тип</typeparam>
 public class MassiveGenericObjects<T> : ICollectionGenericObjects<T>
-where T : class
+    where T : class
 {
     /// <summary>
     /// Массив объектов, которые храним
     /// </summary>
     private T?[] _collection;
-
     public int Count => _collection.Length;
-
-    public int MaxCount {
+    /// <summary>
+    /// Установка максимального кол-ва объектов
+    /// </summary>
+    public int MaxCount
+    {
         get
         {
             return _collection.Length;
         }
         set
         {
-            if (value > 0) {
+            if (value > 0)
+            {
                 if (_collection.Length > 0)
                 {
                     Array.Resize(ref _collection, value);
@@ -47,6 +49,11 @@ where T : class
     {
         _collection = Array.Empty<T?>();
     }
+    /// <summary>
+    /// Получение объекта по позиции
+    /// </summary>
+    /// <param name="position">Позиция (индекс)</param>
+    /// <returns></returns>
     public T? Get(int position)
     {
         if (position < 0 || position >= _collection.Length)

Stormtrooper/Stormtrooper/CollectionGenericObjects/StorageCollection.cs

@@ -54,20 +54,21 @@ where T : DrawningAircraft
     /// <param name="collectionType">тип коллекции</param>
 	public void AddCollection(string name, CollectionType collectionType)
     {
-        // TODO проверка, что name не пустой и нет в словаре записи с таким ключом
-        // TODO Прописать логику для добавления
-        if (!(collectionType == CollectionType.None) && !_storages.ContainsKey(name))
-        {
-            if (collectionType == CollectionType.List)
+        if (string.IsNullOrEmpty(name) || _storages.ContainsKey(name))
+            return;
+        switch (collectionType)
         {
+            case CollectionType.List:
                 _storages.Add(name, new ListGenericObjects<T>());
-            }
-            else if (collectionType == CollectionType.Massive)
-            {
+                break;
+            case CollectionType.Massive:
                 _storages.Add(name, new MassiveGenericObjects<T>());
+                break;
+            default:
+                break;
         }
     }
-    }
+
     /// <summary>
     /// Удаление коллекции
     /// </summary>
@@ -86,11 +87,8 @@ where T : DrawningAircraft
     {
         get
         {
-            // TODO Продумать логику получения объекта
-            if (_storages.ContainsKey(name))
-            {
-                return _storages[name];
-            }
+            if (_storages.TryGetValue(name, out ICollectionGenericObjects<T>? value))
+                return value;
             return null;
         }
     }
@@ -112,8 +110,6 @@ where T : DrawningAircraft
             File.Delete(filename);
         }
 
-
-
         using FileStream fs = new(filename, FileMode.Create);
         using StreamWriter streamWriter = new StreamWriter(fs);
         streamWriter.Write(_collectionKey);

Stormtrooper/Stormtrooper/Program.cs

@@ -4,10 +4,10 @@ using Microsoft.Extensions.Configuration;
 using Serilog;
 using Stormtrooper;
 
-namespace Battleship
+namespace Stormtrooper;
+
+internal static class Program
 {
-    internal static class Program
-    {
     /// <summary>
     ///  The main entry point for the application.
     /// </summary>
@@ -42,5 +42,4 @@ namespace Battleship
             option.AddSerilog(logger);
         });
     }
-    }
 }

Stormtrooper/Stormtrooper/Stormtrooper.csproj

@@ -9,9 +9,10 @@
   </PropertyGroup>
 
   <ItemGroup>
+    <PackageReference Include="Microsoft.Extensions.Configuration" Version="8.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Configuration.FileExtensions" Version="8.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="8.0.0" />
     <PackageReference Include="Microsoft.Extensions.Logging" Version="7.0.0" />
-    <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.1" />
-    <PackageReference Include="NLog.Extensions.Logging" Version="5.3.10" />
     <PackageReference Include="Serilog" Version="3.1.1" />
     <PackageReference Include="Serilog.AspNetCore" Version="6.0.1" />
     <PackageReference Include="Serilog.Settings.Configuration" Version="8.0.0" />