Лабораторная работа №7

ProjectMonorail/FormMonorailCollection.cs

@@ -1,4 +1,5 @@
 using Microsoft.Extensions.Logging;
+using ProjectMonorail.Scripts.Exceptions;
 using ProjectMonorail.Scripts.Monorail.CollectionGenericObjects;
 using ProjectMonorail.Scripts.Monorail.Drawnings;
 
@@ -32,6 +33,7 @@ namespace ProjectMonorail
             InitializeComponent();
             _storageCollection = new StorageCollection<DrawingMonorail>();
             _logger = logger;
+            _logger.LogInformation("Форма загрузилась");
         }
 
         /// <summary>
@@ -66,12 +68,13 @@ namespace ProjectMonorail
                 {
                     MessageBox.Show("Объект добавлен");
                     pictureBox.Image = _company.Show();
+                    _logger.LogInformation("Объект добавлен");
                 }
             }
             catch (Exception ex)
             {
                 MessageBox.Show("Не удалось добавить объект");
-                _logger.LogError("Ошибка {Message}", ex.Message);
+                LogException(ex);
             }
         }
 
@@ -89,12 +92,13 @@ namespace ProjectMonorail
                 {
                     MessageBox.Show("Объект удален");
                     pictureBox.Image = _company.Show();
+                    _logger.LogInformation("Объект удален");
                 }
             }
             catch(Exception ex)
             {
                 MessageBox.Show("Не удалось удалить объект");
-                _logger.LogError("Ошибка {Message}", ex.Message);
+                LogException(ex);
             }
         }
 
@@ -153,7 +157,7 @@ namespace ProjectMonorail
                 if (string.IsNullOrEmpty(textBoxCollectionName.Text) || (!radioButtonList.Checked && !radioButtonMassive.Checked))
                 {
                     MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error);
-                    return;
+                    throw new Exception("Не все данные заполнены");
                 }
                 CollectionType collectionType = CollectionType.None;
                 if (radioButtonMassive.Checked)
@@ -164,13 +168,13 @@ namespace ProjectMonorail
                 {
                     collectionType = CollectionType.List;
                 }
-                _storageCollection.AddCollection(textBoxCollectionName.Text,
+                _storageCollection.AddCollection(textBoxCollectionName.Text, collectionType);
-                collectionType);
+                _logger.LogInformation("Коллекция добавлена " + textBoxCollectionName.Text);
                 RerfreshListBoxItems();
             }
             catch (Exception ex)
             {
-                _logger.LogError("Ошибка {Message}", ex.Message);
+                LogException(ex);
             }
         }
 
@@ -181,7 +185,8 @@ namespace ProjectMonorail
         /// <param name="e"></param>
         private void ButtonCollectionDelete_Click(object sender, EventArgs e)
         {
-            try {
+            try
+            {
                 // TODO прописать логику удаления элемента из коллекции
                 // нужно убедиться, что есть выбранная коллекция
                 // спросить у пользователя через MessageBox, что он подтверждает, что хочет удалить запись
@@ -193,11 +198,12 @@ namespace ProjectMonorail
                 }
                 if (MessageBox.Show("Удалить коллекцию?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No) return;
                 _storageCollection.DelCollection(listBoxCollection.SelectedItem.ToString());
+                _logger.LogInformation("Коллекция с название: " + listBoxCollection.SelectedItem.ToString() + " удалена");
                 RerfreshListBoxItems();
             }
             catch (Exception ex)
             {
-                _logger.LogError("Ошибка {Message}", ex.Message);
+                LogException(ex);
             }
         }
 
@@ -213,14 +219,14 @@ namespace ProjectMonorail
                 if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItem == null)
                 {
                     MessageBox.Show("Коллекция не выбрана");
-                    return;
+                    throw new Exception("Коллекция не выбрана");
                 }
                 ICollectionGenericObjects<DrawingMonorail>? collection =
                 _storageCollection[listBoxCollection.SelectedItem.ToString() ?? string.Empty];
                 if (collection == null)
                 {
                     MessageBox.Show("Коллекция не проинициализирована");
-                    return;
+                    throw new Exception("Коллекция не проинициализирована");
                 }
                 switch (comboBoxSelectorCompany.Text)
                 {
@@ -233,7 +239,7 @@ namespace ProjectMonorail
             }
             catch (Exception ex)
             {
-                _logger.LogError("Ошибка {Message}", ex.Message);
+                LogException(ex);
             }
         }
 
@@ -256,7 +262,7 @@ namespace ProjectMonorail
             catch (Exception ex)
             {
                 MessageBox.Show("Не сохранилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
-                _logger.LogError("Ошибка {Message}", ex.Message);
+                LogException(ex);
             }
         }
 
@@ -277,9 +283,37 @@ namespace ProjectMonorail
                     _logger.LogInformation("Загрузка успешна завершена");
                 }
             }
-            catch(Exception ex)
+            catch (Exception ex)
             {
                 MessageBox.Show("Не загрузилось", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                LogException(ex);
+            }
+        }
+
+        private void LogException(Exception ex)
+        {
+            if (ex is CollectionOverflowException)
+            {
+                _logger.LogError("Ошибка {Message}", ((CollectionOverflowException)ex).Message);
+            }
+            else if (ex is FileEmptyException)
+            {
+                _logger.LogError("Ошибка {Message}", ((FileEmptyException)ex).Message);
+            }
+            else if (ex is FileDoesNotExistException)
+            {
+                _logger.LogError("Ошибка {Message}", ((FileDoesNotExistException)ex).Message);
+            }
+            else if (ex is ObjectNotFoundException)
+            {
+                _logger.LogError("Ошибка {Message}", ((ObjectNotFoundException)ex).Message);
+            }
+            else if (ex is PositionOutOfCollectionException)
+            {
+                _logger.LogError("Ошибка {Message}", ((PositionOutOfCollectionException)ex).Message);
+            }
+            else
+            {
                 _logger.LogError("Ошибка {Message}", ex.Message);
             }
         }

ProjectMonorail/ProjectMonorail.csproj

@@ -24,7 +24,7 @@
     </EmbeddedResource>
   </ItemGroup>
   <ItemGroup>
-    <None Update="Configs\nlog.config">
+    <None Update="nlog.config">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>
   </ItemGroup>

ProjectMonorail/Scripts/Exceptions/FileDoesNotExistException.cs

@@ -0,0 +1,16 @@
+using System.Runtime.Serialization;
+
+namespace ProjectMonorail.Scripts.Exceptions
+{
+    /// <summary>
+    /// Класс, описывающий ошибку, что файл пустой
+    /// </summary>
+    [Serializable]
+    public class FileDoesNotExistException : ApplicationException
+    {
+        public FileDoesNotExistException() : base() { }
+        public FileDoesNotExistException(string message) : base(message) { }
+        public FileDoesNotExistException(string message, Exception exception) : base(message, exception) { }
+        protected FileDoesNotExistException(SerializationInfo info, StreamingContext contex) : base(info, contex) { }
+    }
+}

ProjectMonorail/Scripts/Exceptions/FileEmptyException.cs

@@ -0,0 +1,16 @@
+using System.Runtime.Serialization;
+
+namespace ProjectMonorail.Scripts.Exceptions
+{
+    /// <summary>
+    /// Класс, описывающий ошибку, что по указанной позиции нет элемента
+    /// </summary>
+    [Serializable]
+    public class FileEmptyException : ApplicationException
+    {
+        public FileEmptyException() : base() { }
+        public FileEmptyException(string message) : base(message) { }
+        public FileEmptyException(string message, Exception exception) : base(message, exception) { }
+        protected FileEmptyException(SerializationInfo info, StreamingContext contex) : base(info, contex) { }
+    }
+}

ProjectMonorail/Scripts/Monorail/CollectionGenericObjects/AbstractCompany.cs

@@ -36,7 +36,8 @@ namespace ProjectMonorail.Scripts.Monorail.CollectionGenericObjects
         /// <summary>
         /// Вычисление максимального количества элементов, который можно разместить в окне
         /// </summary>
-        private int GetMaxCount => _pictureWidth * _pictureHeight / (_placeSizeWidth * _placeSizeHeight);
+        //private int GetMaxCount => _pictureWidth * _pictureHeight / (_placeSizeWidth * _placeSizeHeight);
+        private int GetMaxCount => 40;
 
         /// <summary>
         /// Конструктор

ProjectMonorail/Scripts/Monorail/CollectionGenericObjects/MassiveGenericObjects.cs

@@ -60,7 +60,7 @@ namespace ProjectMonorail.Scripts.Monorail.CollectionGenericObjects
                 if (InsertingElementCollection(i, obj)) return i; 
             }
 
-            throw new Exception("Превышение лимита Count");
+            throw new CollectionOverflowException("Превышение лимита Count");
         }
 
         public int Insert(T obj, int position)
@@ -83,7 +83,7 @@ namespace ProjectMonorail.Scripts.Monorail.CollectionGenericObjects
                 if (InsertingElementCollection(i, obj)) return i;
             }
 
-            throw new Exception("Нет свободного места для вставки");
+            throw new CollectionOverflowException("Нет свободного места для вставки");
         }
 
         public T Remove(int position)
@@ -91,7 +91,7 @@ namespace ProjectMonorail.Scripts.Monorail.CollectionGenericObjects
             // TODO проверка позиции
             // TODO удаление объекта из массива, присвоив элементу массива значение null
 
-            if (!(position >= 0 && position < Count) || _collection[position] == null) throw new PositionOutOfCollectionException(position);
+            if (!(position >= 0 && position < Count) || _collection[position] == null) throw new ObjectNotFoundException(position);
 
             T obj = _collection[position];
             _collection[position] = null;

ProjectMonorail/Scripts/Monorail/CollectionGenericObjects/StorageCollection.cs

@@ -1,5 +1,4 @@
-using ProjectMonorail.Scripts.Exceptions;
+using ProjectMonorail.Scripts.Monorail.Drawnings;
-using ProjectMonorail.Scripts.Monorail.Drawnings;
 using System.Diagnostics;
 using System.Text;
 
@@ -67,7 +66,7 @@ namespace ProjectMonorail.Scripts.Monorail.CollectionGenericObjects
                         collection = new MassiveGenericObjects<T>();
                         break;
                     default:
-                        throw new Exception("Collection Type is not selected: collectionType = " + collectionType);
+                        return;
                 }
 
                 _storages.Add(name, collection);
@@ -110,30 +109,32 @@ namespace ProjectMonorail.Scripts.Monorail.CollectionGenericObjects
         {
             if (_storages.Count == 0)
             {
-                throw new Exception("В хранилище отсутствуют коллекции для сохранения");
+                throw new Exception("Storage is empty");
             }
-
             if (File.Exists(filename))
             {
                 File.Delete(filename);
             }
-            StringBuilder sb = new();
+            using (StreamWriter writer = new StreamWriter(filename))
-            sb.Append(_collectionKey);
-            foreach (KeyValuePair<string, ICollectionGenericObjects<T>> value in
-            _storages)
             {
-                sb.Append(Environment.NewLine);
+                writer.Write(_collectionKey);
-                // не сохраняем пустые коллекции
+                foreach (KeyValuePair<string, ICollectionGenericObjects<T>> value in _storages)
+                {
+                    StringBuilder stringBuilder = new StringBuilder();
+                    stringBuilder.Append(Environment.NewLine);
+
                     if (value.Value.Count == 0)
                     {
                         continue;
                     }
-                sb.Append(value.Key);
+
-                sb.Append(_separatorForKeyValue);
+                    stringBuilder.Append(value.Key);
-                sb.Append(value.Value.GetCollectionType);
+                    stringBuilder.Append(_separatorForKeyValue);
-                sb.Append(_separatorForKeyValue);
+                    stringBuilder.Append(value.Value.GetCollectionType);
-                sb.Append(value.Value.MaxCount);
+                    stringBuilder.Append(_separatorForKeyValue);
-                sb.Append(_separatorForKeyValue);
+                    stringBuilder.Append(value.Value.MaxCount);
+                    stringBuilder.Append(_separatorForKeyValue);
+
                     foreach (T? item in value.Value.GetItems())
                     {
                         string data = item?.GetDataForSave() ?? string.Empty;
@@ -141,13 +142,12 @@ namespace ProjectMonorail.Scripts.Monorail.CollectionGenericObjects
                         {
                             continue;
                         }
-                    sb.Append(data);
+                        stringBuilder.Append(data);
-                    sb.Append(_separatorItems);
+                        stringBuilder.Append(_separatorItems);
+                    }
+                    writer.Write(stringBuilder);
                 }
             }
-            using FileStream fs = new(filename, FileMode.Create);
-            byte[] info = new UTF8Encoding(true).GetBytes(sb.ToString());
-            fs.Write(info, 0, info.Length);
         }
 
         /// <summary>
@@ -159,67 +159,54 @@ namespace ProjectMonorail.Scripts.Monorail.CollectionGenericObjects
         {
             if (!File.Exists(filename))
             {
-                throw new Exception("Файл не существует");
+                throw new Exceptions.FileDoesNotExistException("Файл не существует " + filename);
             }
 
-            string bufferTextFromFile = "";
+            using (StreamReader streamRader = File.OpenText(filename))
-            using (FileStream fs = new(filename, FileMode.Open))
             {
-                byte[] b = new byte[fs.Length];
+                string inputString = streamRader.ReadLine();
-                UTF8Encoding temp = new(true);
+
-                while (fs.Read(b, 0, b.Length) > 0)
+                if (inputString == null || inputString.Length == 0)
                 {
-                    bufferTextFromFile += temp.GetString(b);
+                    throw new Exceptions.FileEmptyException("Файл пустой " + filename);
-                }
-            }
-            string[] strs = bufferTextFromFile.Split(new char[] { '\n', '\r' }, StringSplitOptions.RemoveEmptyEntries);
-            if (strs == null || strs.Length == 0)
-            {
-                throw new Exception("В файле нет данных");
                 }
 
-            if (!strs[0].Equals(_collectionKey))
+                if (!inputString.StartsWith(_collectionKey))
                 {
-                throw new Exception("В файле неверные данные");
+                    throw new Exceptions.ObjectNotFoundException();
                 }
 
                 _storages.Clear();
-            foreach (string data in strs)
+                string strs = "";
+                while ((strs = streamRader.ReadLine()) != null)
                 {
-                string[] record = data.Split(_separatorForKeyValue,
+                    string[] record = strs.Split(_separatorForKeyValue, StringSplitOptions.RemoveEmptyEntries);
-                StringSplitOptions.RemoveEmptyEntries);
                     if (record.Length != 4)
                     {
                         continue;
                     }
-                CollectionType collectionType =
+                    CollectionType collectionType = (CollectionType)Enum.Parse(typeof(CollectionType), record[1]);
-                (CollectionType)Enum.Parse(typeof(CollectionType), record[1]);
+                    ICollectionGenericObjects<T>? collection = StorageCollection<T>.CreateCollection(collectionType);
-                ICollectionGenericObjects<T>? collection =
-                StorageCollection<T>.CreateCollection(collectionType);
                     if (collection == null)
                     {
-                    throw new Exception("Не удалось определить тип коллекции: " + record[1]);
+                        throw new Exceptions.ObjectNotFoundException();
                     }
                     collection.MaxCount = Convert.ToInt32(record[2]);
-                string[] set = record[3].Split(_separatorItems,
+                    string[] set = record[3].Split(_separatorItems, StringSplitOptions.RemoveEmptyEntries);
-                StringSplitOptions.RemoveEmptyEntries);
                     foreach (string elem in set)
                     {
-                    if (elem?.CreateDrawingMonorail() is T monorail)
+                        if (elem?.CreateDrawingMonorail() is T ship)
                         {
-                        try
+                            if (collection.Insert(ship) == -1)
                             {
-                            if (collection.Insert(monorail) == -1) 
+                                throw new Exceptions.PositionOutOfCollectionException();
-                                throw new Exception("Объект не удалось добавить в коллекцию: " + record[3]);
-                        }
-                        catch (CollectionOverflowException ex) { 
-                            throw new Exception("Коллекция переполнена", ex);
                             }
                         }
                     }
                     _storages.Add(record[0], collection);
                 }
             }
+        }
 
         private static ICollectionGenericObjects<T>? CreateCollection(CollectionType collectionType)
         {

ProjectMonorail/nlog.config

@@ -4,7 +4,7 @@
         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         autoReload="true" internalLogLevel="Info">
   <targets>
-    <target xsi:type="File" name="tofile" fileName="carlog-${shortdate}.log" />
+    <target xsi:type="File" name="tofile" fileName="MonorailLog-${shortdate}.log" />
   </targets>
   <rules>
     <logger name="*" minlevel="Debug" writeTo="tofile" />