ProjectLiner/ProjectLiner/CollectionGenericObjects/ShipSharingService.cs

@@ -35,11 +35,11 @@ public class ShipSharingService : AbstractCompany
                 g.DrawLine(pen, x, y + _placeSizeHeight, x + _placeSizeWidth, y + _placeSizeHeight);
                 g.DrawLine(pen, x + _placeSizeWidth, y + _placeSizeHeight, x + _placeSizeWidth, y);
                 locCoord.Add(new Tuple<int, int>(x, y));
-                x += _placeSizeWidth + 2;
+                x += _placeSizeWidth;
             }
             countInRow = count;
-            x = 1;
-            y += _placeSizeHeight + 5;
+            x = 0;
+            y += _placeSizeHeight;
             countOfRows++;
         }
     }

ProjectLiner/ProjectLiner/CollectionGenericObjects/StorageCollection.cs

@@ -1,9 +1,7 @@
 using ProjectLiner.Drawnings;
-using System.Text;
 
 namespace ProjectLiner.CollectionGenericObjects;
 
-
 /// <summary>
 /// Класс-хранилище коллекций
 /// </summary>
@@ -11,6 +9,16 @@ namespace ProjectLiner.CollectionGenericObjects;
 public class StorageCollection<T>
     where T : DrawningShip
 {
+    /// <summary>
+    /// Словарь (хранилище) с коллекциями
+    /// </summary>
+    private Dictionary<string, ICollectionGenericObjects<T>> _storages;
+
+    /// <summary>
+    /// Возвращение списка названий коллекций
+    /// </summary>
+    public List<string> Keys => _storages.Keys.ToList();
+
     /// <summary>
     /// Ключевое слово, с которого должен начинаться файл
     /// </summary>
@@ -26,15 +34,7 @@ public class StorageCollection<T>
     /// </summary>
     private readonly string _separatorItems = ";";
 
-    /// <summary>
-    /// Словарь (хранилище) с коллекциями
-    /// </summary>
-    readonly Dictionary<string, ICollectionGenericObjects<T>> _storages;
 
-    /// <summary>
-    /// Возвращение списка названий коллекций
-    /// </summary>
-    public List<string> Keys => _storages.Keys.ToList();
 
     /// <summary>
     /// Конструктор
@@ -113,52 +113,32 @@ public class StorageCollection<T>
         {
             return false;
         }
-
         if (File.Exists(filename))
         {
             File.Delete(filename);
         }
+        
 
-        StringBuilder sb = new();
-        sb.Append(_collectionKey);
-
-        foreach (KeyValuePair<string, ICollectionGenericObjects<T>> value in _storages)
+        using (StreamWriter writer = new StreamWriter(filename))
         {
-            sb.Append(Environment.NewLine);
-
-            // не сохраняем пустые коллекции
-            if (value.Value.Count == 0)
+            writer.WriteLine(_collectionKey);
+            foreach (KeyValuePair<string, ICollectionGenericObjects<T>> value in _storages)
             {
-                continue;
-            }
-
-            sb.Append(value.Key);
-            sb.Append(_separatorForKeyValue);
-            sb.Append(value.Value.GetCollectionType);
-            sb.Append(_separatorForKeyValue);
-            sb.Append(value.Value.MaxCount);
-            sb.Append(_separatorForKeyValue);
-
-            foreach (T? item in value.Value.GetItems())
-            {
-                string data = item?.GetDataForSave() ?? string.Empty;
-
-                if (string.IsNullOrEmpty(data))
-                {
-                    continue;
+                writer.Write($"{value.Key}{_separatorForKeyValue}{value.Value.GetCollectionType}{_separatorForKeyValue}{value.Value.MaxCount}{_separatorForKeyValue}");
+                foreach (T? item in value.Value.GetItems())
+                    {
+                    string data = item?.GetDataForSave() ?? string.Empty;
+                    if (!string.IsNullOrEmpty(data))
+                    {
+                        writer.Write(data + _separatorItems);
+                    }
                 }
-
-                sb.Append(data);
-                sb.Append(_separatorItems);
+                writer.WriteLine();
             }
         }
-
-        using FileStream fs = new(filename, FileMode.Create);
-        byte[] info = new UTF8Encoding(true).GetBytes(sb.ToString());
-        fs.Write(info, 0, info.Length);
         return true;
     }
-
+   
     /// <summary>
     /// Загрузка информации по кораблям в хранилище из файла
     /// </summary>
@@ -171,68 +151,50 @@ public class StorageCollection<T>
             return false;
         }
 
-        string bufferTextFromFile = "";
-        using (FileStream fs = new(filename, FileMode.Open))
+        using (StreamReader reader = new StreamReader(filename))
         {
-            byte[] b = new byte[fs.Length];
-            UTF8Encoding temp = new(true);
-
-            while (fs.Read(b, 0, b.Length) > 0)
-            {
-                bufferTextFromFile += temp.GetString(b);
-            }
-        }
-
-        string[] strs = bufferTextFromFile.Split(new char[] { '\n', '\r' },
-        StringSplitOptions.RemoveEmptyEntries);
-
-        if (strs == null || strs.Length == 0)
-        {
-            return false;
-        }
-
-        if (!strs[0].Equals(_collectionKey))
-        {
-            //если нет такой записи, то это не те данные
-            return false;
-        }
-
-        _storages.Clear();
-
-        foreach (string data in strs)
-        {
-            string[] record = data.Split(_separatorForKeyValue, StringSplitOptions.RemoveEmptyEntries);
-
-            if (record.Length != 4)
-            {
-                continue;
-            }
-
-            CollectionType collectionType = (CollectionType)Enum.Parse(typeof(CollectionType), record[1]);
-            ICollectionGenericObjects<T>? collection = StorageCollection<T>.CreateCollection(collectionType);
-
-            if (collection == null)
+            string line = reader.ReadLine();
+            if (line == null || !line.Equals(_collectionKey))
             {
                 return false;
             }
 
-            collection.MaxCount = Convert.ToInt32(record[2]);
+            _storages.Clear();
 
-            string[] set = record[3].Split(_separatorItems, StringSplitOptions.RemoveEmptyEntries);
-            foreach (string elem in set)
+            while (line != null)
             {
-                if (elem?.CreateDrawningShip() is T ship)
+                string[] record = line.Split(_separatorForKeyValue, StringSplitOptions.RemoveEmptyEntries);
+                if (record.Length != 4)
                 {
-                    if (collection.Insert(ship) == -1)
+                    line = reader.ReadLine();
+                    continue;
+                }
+
+                CollectionType collectionType = (CollectionType)Enum.Parse(typeof(CollectionType), record[1]);
+                ICollectionGenericObjects<T>? collection = StorageCollection<T>.CreateCollection(collectionType);
+                if (collection == null)
+                { 
+                    return false; 
+                }
+
+                collection.MaxCount = Convert.ToInt32(record[2]);
+
+                string[] set = record[3].Split(_separatorItems, StringSplitOptions.RemoveEmptyEntries);
+
+                foreach (string elem in set)
+                {
+                    if (elem?.CreateDrawningShip() is T ship)
                     {
-                        return false;
+                        if (collection.Insert(ship) == -1)
+                        {
+                            return false;
+                        }
                     }
                 }
+                _storages.Add(record[0], collection);
+                line = reader.ReadLine();
             }
-
-            _storages.Add(record[0], collection);
         }
-
         return true;
     }