logging

ProjectCruiser/CollectionGenericObj/AbstractCompany.cs

@@ -17,8 +17,8 @@ public abstract class AbstractCompany
 
     // Коллекция автомобилей
     protected ICollectionGenObj<DrawningBase>? _collection = null;
-    private int GetMaxCount => _pictureWidth * _pictureHeight /
+    private int GetMaxCount => (_pictureWidth / (_placeSizeWidth + 20))
-    (_placeSizeWidth * _placeSizeHeight);
+        * ( _pictureHeight / (_placeSizeHeight + 4));
 
     public AbstractCompany(int picWidth, int picHeight,
         ICollectionGenObj<DrawningBase>? collection)
@@ -51,20 +51,22 @@ public abstract class AbstractCompany
         Bitmap bitmap = new(_pictureWidth, _pictureHeight);
         Graphics graphics = Graphics.FromImage(bitmap);
         DrawBackground(graphics);
-        SetObjectsPosition();
+
-        for (int i = 0; i < (_collection?.Count ?? 0); ++i)
+        SetObjectsPosition(_collection.Count - 1);
+
+        for (int i = 0; i < (_collection?.Count ?? 0); i++)
         {
             DrawningBase? obj = _collection?.GetItem(i);
             obj?.DrawTransport(graphics);
         }
+
         return bitmap;
     }
 
-
     // Вывод заднего фона
     protected abstract void DrawBackground(Graphics g);
 
     // Расстановка объектов
-    protected abstract void SetObjectsPosition();
+    protected abstract void SetObjectsPosition(int border);
 }
 

ProjectCruiser/CollectionGenericObj/ArrayGenObj.cs

@@ -1,4 +1,5 @@
-
+using ProjectCruiser.Exceptions;
+
 namespace ProjectCruiser.CollectionGenericObj;
 
 public class ArrayGenObj<T> : ICollectionGenObj<T>
@@ -9,7 +10,7 @@ public class ArrayGenObj<T> : ICollectionGenObj<T>
 
     // Максимально допустимое число объектов в массиве
     private int _maxCount;
-    public int Count => _collection.Count(s => s != null);
+    public int Count => _collection.Count(s => (s != null));
 
     public int MaxCount
     {
@@ -18,10 +19,10 @@ public class ArrayGenObj<T> : ICollectionGenObj<T>
         {
             if (value > 0)
             {
+                _maxCount = value;
+
                 if (_collection.Length == 0) _collection = new T?[value];
                 else Array.Resize(ref _collection, value);
-
-                _maxCount = value;
             }
         }
     }
@@ -36,22 +37,31 @@ public class ArrayGenObj<T> : ICollectionGenObj<T>
     // methods :
     public T? GetItem(int index)
     {
-        if (index > Count || index < 0)
+        if (index > _maxCount)
-        {
+            throw new CollectionOverflowException(index);
-            return null;
+        if (index < 0)
-        }
+            throw new PositionOutOfCollectionException(index);
+
+        if (_collection[index] == null)
+            throw new ObjectNotFoundException(index);
+
         return _collection[index];
+
+        // CollectionOverflowException
+        // PositionOutOfCollectionException
+        // ObjectNotFoundException
     }
 
     public int Insert(T? item)
     {
-        if (Count >= _maxCount || item == null)
+        if (item == null) throw
-        {
+                new NullReferenceException("> Inserting item is null");
-            return -1;
+
-        }
+        // выход за границы, курируется CollectionOverflowException
+        if (Count >= _maxCount) throw new CollectionOverflowException(Count);
 
         // any empty place -> fill immediately
-        for (int i = 0; i < _collection.Length; i++)
+        for (int i = Count; i < _maxCount; i++)
         {
             if (_collection[i] == null)
             {
@@ -60,50 +70,63 @@ public class ArrayGenObj<T> : ICollectionGenObj<T>
             }
         }
         return Count;
+
+        // NullReferenceException
+        // CollectionOverflowException
     }
 
     public int Insert(T? item, int index)
     {
-        if (index >= _maxCount || Count >= _maxCount ||
+        if (index < 0 || index >= _maxCount) throw new PositionOutOfCollectionException(index);
-            index < 0 || _collection[index] != null
+        if (Count >= _maxCount) throw new CollectionOverflowException(Count);
-            || item == null)
+
-        {
+        if (item == null) throw
-            return -1;
+                new NullReferenceException("> Inserting item (at position) is null");
-        }
 
         if (_collection[index] == null)
         {
             _collection[index] = item;
             return index;
         }
-
+        else
-        int min_diff = 100, firstNullIndex = 100;
-
-        for (int i = 0; i < Count; i++)
         {
-            if (_collection[i] == null
+            int min_diff = 100, firstNullIndex = 100;
-                && min_diff > Math.Abs(index - i))
+
+            for (int i = 0; i < Count; i++)
             {
-                min_diff = Math.Abs(index - i);
+                if (_collection[i] == null && min_diff > Math.Abs(index - i))
-                firstNullIndex = i;
+                {
+                    min_diff = Math.Abs(index - i);
+                    firstNullIndex = i;
+                }
             }
+
+            _collection[firstNullIndex] = item;
+            return firstNullIndex;
         }
 
-        _collection[firstNullIndex] = item;
+        // PositionOutOfCollectionException
-        return firstNullIndex;
+        // CollectionOverflowException
+        // NullReferenceException
     }
 
     public T? Remove(int index)
     {
-        if (index >= Count || index < 0)
+        if (index >= _maxCount || index < 0)
         // on the other positions items don't exist
         {
-            return null;
+            throw new PositionOutOfCollectionException(index);
         }
 
         T? item = _collection[index];
         _collection[index] = null;
+
+        if (item == null) throw new ObjectNotFoundException(index);
+
         return item;
+
+        // PositionOutOfCollectionException
+        // ObjectNotFoundException
     }
 
     public IEnumerable<T?> GetItems()

ProjectCruiser/CollectionGenericObj/ListGenObj.cs

@@ -1,7 +1,4 @@
-using System;
+using ProjectCruiser.Exceptions;
-using System.Collections;
-using System.Collections.Generic;
-using System.Reflection;
 
 namespace ProjectCruiser.CollectionGenericObj;
 
@@ -40,20 +37,24 @@ public class ListGenObj<T> : ICollectionGenObj<T>
 
     public T? GetItem(int position)
     {
-        if (position >= Count || position < 0)
+        if (position > _maxCount)
-        {
+            throw new CollectionOverflowException(position);
-            return null;
+        if (position < 0)
-        }
+            throw new PositionOutOfCollectionException(position);
+
+        if (_collection[position] == null)
+            throw new ObjectNotFoundException(position);
 
         return _collection[position];
     }
 
     public int Insert(T? obj)
     {
-        if (Count >= _maxCount || obj == null)
+        if (obj == null)
-        {
+            throw new NullReferenceException("> Inserting object is null");
-            return -1;
+
-        }
+        // выход за границы, курируется CollectionOverflowException
+        if (Count >= _maxCount) throw new CollectionOverflowException(Count);
 
         _collection.Add(obj);
         return Count;
@@ -61,12 +62,12 @@ public class ListGenObj<T> : ICollectionGenObj<T>
 
     public int Insert(T? obj, int position)
     {
-        if (position >= _maxCount || Count >= _maxCount ||
+        if (position < 0 || position >= _maxCount)
-            position < 0 || _collection[position] != null
+            throw new PositionOutOfCollectionException(position);
-            || obj == null)
+        if (Count >= _maxCount) throw new CollectionOverflowException(Count);
-        {
+
-            return -1;
+        if (obj == null)
-        }
+            throw new NullReferenceException("> Inserting object (at position) is null");
 
         _collection.Insert(position, obj);
         return position;
@@ -74,14 +75,17 @@ public class ListGenObj<T> : ICollectionGenObj<T>
 
     public T? Remove(int position)
     {
-        if (position >= Count || position < 0)
+        if (position >= _maxCount || position < 0)
         // on the other positions items don't exist
         {
-            return null;
+            throw new PositionOutOfCollectionException(position);
         }
 
         T? item = _collection[position];
         _collection.RemoveAt(position);
+
+        if (item == null) throw new ObjectNotFoundException(position);
+
         return item;
     }
 

ProjectCruiser/CollectionGenericObj/ShipSharingService.cs

@@ -41,27 +41,33 @@ public class ShipSharingService : AbstractCompany
         }
     }
 
-    protected override void SetObjectsPosition()
+    protected override void SetObjectsPosition(int border)
     {
         int index_collection = 0;
-        int newX = fromBorder + 6, newY = fromCeiling + 6;
+        int newY = fromCeiling + 4;
 
         if (_collection != null)
         {
-            for (int i = 0; i < MaxInColon; ++i)
+            for (int i = 0; i < MaxInColon; i++)
             {
-                newX = fromBorder + 2;
+                int newX = fromBorder + 2;
-                for (int j = 0; j < MaxInRow; ++j)
+                for (int j = 0; j < MaxInRow; j++)
                 {
-                    if (_collection.GetItem(index_collection) != null)
+                    // TRY / CATCH [?]
+                    _collection.GetItem(index_collection).SetPictureSize(
+                        _pictureWidth, _pictureHeight);
+
+                    _collection.GetItem(index_collection).SetPosition(newX, newY);
+
+                    newX += _placeSizeWidth + between + 2;
+
+                    if (index_collection < border)
                     {
-                        _collection.GetItem(index_collection).SetPictureSize(_pictureWidth, _pictureHeight);
-                        _collection.GetItem(index_collection).SetPosition(newX, newY);
-                        newX += _placeSizeWidth + between + 2;
                         index_collection++;
                     }
+                    else return;
                 }
-                newY += _placeSizeHeight + 2;
+                newY += _placeSizeHeight + 1;
             }
         }
     }

ProjectCruiser/CollectionGenericObj/StorageCollection.cs

@@ -1,5 +1,7 @@
-using System.Text;
+using System.Security.Cryptography;
+using System.Text;
 using ProjectCruiser.DrawningSamples;
+using ProjectCruiser.Exceptions;
 
 namespace ProjectCruiser.CollectionGenericObj;
 
@@ -31,7 +33,7 @@ public class StorageCollection<T>
         if (name == null || _storages.ContainsKey(name)
             || collType == CollectionType.None)
         {
-            return;
+            throw new NullReferenceException("> Not enough information to save");
         }
 
         ICollectionGenObj<T> collection = CreateCollection(collType);
@@ -42,7 +44,7 @@ public class StorageCollection<T>
     public void DelCollection(string name)
     {
         if (_storages.ContainsKey(name)) _storages.Remove(name);
-        return;
+        else throw new NullReferenceException("> No such key in the list");
     }
 
     // Доступ к коллекции ( по ключу-строке - её имени ) - индексатор [!!!]
@@ -55,9 +57,11 @@ public class StorageCollection<T>
     /// <param name="filename">Путь и имя файла</param>
     /// <returns>true - сохранение прошло успешно,
     /// false - ошибка при сохранении данных</returns>
-    public bool SaveData(string filename)
+    public void SaveData(string filename)
     {
-        if (_storages.Count == 0) { return false; }
+        if (_storages.Count == 0)
+            throw new NullReferenceException("> No existing collections to save");
+
         if (File.Exists(filename)) { File.Delete(filename); }
 
         StringBuilder sb = new();
@@ -79,21 +83,6 @@ public class StorageCollection<T>
             foreach (T? item in pair.Value.GetItems())
             {
                 string data = item?.GetDataForSave() ?? string.Empty;
-
-                /*
-                string n = item.GetType().Name;
-                string data = null;
-
-                if (n != null && n == "DrawningCruiser")
-                {
-                    data = ExtentionDrShip.GetDataForSave(item);
-                }
-                else if (n != null && n == "DrawningBase")
-                {
-                    data = ExtentionDrShip.GetDataForSave(item);
-                }
-                */
-
                 if (string.IsNullOrEmpty(data))
                 {
                     continue;
@@ -107,7 +96,6 @@ public class StorageCollection<T>
         using FileStream fs = new(filename, FileMode.Create);
         byte[] info = new UTF8Encoding(true).GetBytes(sb.ToString());
         fs.Write(info, 0, info.Length);
-        return true;
     }
 
     // Создание коллекции по типу
@@ -122,12 +110,9 @@ public class StorageCollection<T>
     }
 
     // Загрузка информации по кораблям в хранилище из файла
-    public bool LoadData(string filename)
+    public void LoadData(string filename)
     {
-        if (!File.Exists(filename))
+        if (!File.Exists(filename)) throw new FileNotFoundException("> No such file");
-        {
-            return false;
-        }
 
         string bufferTextFromFile = "";
 
@@ -145,14 +130,9 @@ public class StorageCollection<T>
             new char[] { '\n', '\r' }, StringSplitOptions.RemoveEmptyEntries);
 
         if (strs == null || strs.Length == 0)
-        {
+            throw new NullReferenceException("> No data to decode");
-            return false;
-        }
         if (!strs[0].Equals(_collectionKey))
-        {
+            throw new InvalidDataException("> Incorrect data");
-            //если нет такой записи, то это не те данные
-            return false;
-        }
 
         string[] companies = new string[strs.Length - 1];
         for (int k = 1; k < strs.Length; k++)
@@ -166,15 +146,13 @@ public class StorageCollection<T>
             string[] record = data.Split(_separatorForKeyValue, StringSplitOptions.RemoveEmptyEntries);
             if (record.Length != 4) // >
                 // key | collType | maxcount | all next inf > 4
-            {
+            { continue; }
-                continue;
+
-            }
             CollectionType collectionType = (CollectionType)Enum.Parse(typeof(CollectionType), record[1]);
             ICollectionGenObj<T>? collection = StorageCollection<T>.CreateCollection(collectionType);
+            
             if (collection == null)
-            {
+                throw new NullReferenceException("[!] Failed to create collection");
-                return false;
-            }
 
             collection.MaxCount = Convert.ToInt32(record[2]);
             string[] set = record[3].Split(_separatorItems,
@@ -184,15 +162,22 @@ public class StorageCollection<T>
             {
                 if (elem?.CreateDrawningCar() is T ship)
                 {
-                    if (collection.Insert(ship) == -1)
+                    try
                     {
-                        return false;
+                        collection.Insert(ship);
+
+                        // throw new IndexOutOfRangeException IF IT WAS Insert(item, pos)
+                        // NullReferenceException >
+                        // CollectionOverflowException >
+                    }
+                    catch (Exception e)
+                    {
+                        throw new Exception(e.Message);
                     }
                 }
             }
             _storages.Add(record[0], collection);
         }
-        return true;
     }
 }
 

ProjectCruiser/Exceptions/CollectionOverflowException.cs

@@ -0,0 +1,18 @@
+using System.Runtime.Serialization;
+namespace ProjectCruiser.Exceptions;
+
+// Класс, описывающий ошибку переполнения коллекции
+[Serializable]
+internal class CollectionOverflowException : ApplicationException
+{
+    public CollectionOverflowException(int count)
+        : base("<> Possible accsess\nof collection is over : " + count) { }
+    public CollectionOverflowException() : base() { }
+    public CollectionOverflowException(string message) : base(message) { }
+    public CollectionOverflowException(string message, Exception exception) :
+    base(message, exception)
+    { }
+    protected CollectionOverflowException(SerializationInfo info,
+    StreamingContext contex) : base(info, contex) { }
+}
+

ProjectCruiser/Exceptions/ObjectNotFoundException.cs

@@ -0,0 +1,16 @@
+using System.Runtime.Serialization;
+namespace ProjectCruiser.Exceptions;
+
+// Класс, описывающий ошибку, что по указанной позиции нет элемента
+[Serializable]
+internal class ObjectNotFoundException : ApplicationException
+{
+    public ObjectNotFoundException(int i)
+        : base("<> Didn't find obj\non this position : " + i) { }
+    public ObjectNotFoundException() : base() { }
+    public ObjectNotFoundException(string message) : base(message) { }
+    public ObjectNotFoundException(string message, Exception exception)
+        : base(message, exception) { }
+    protected ObjectNotFoundException(SerializationInfo info,
+        StreamingContext contex) : base(info, contex) { }
+}

ProjectCruiser/Exceptions/PositionOutOfCollectionException.cs

@@ -0,0 +1,16 @@
+using System.Runtime.Serialization;
+namespace ProjectCruiser.Exceptions;
+
+// Класс, описывающий ошибку выхода за границы коллекции
+[Serializable]
+internal class PositionOutOfCollectionException : ApplicationException
+{
+    public PositionOutOfCollectionException(int i)
+        : base("<> Out of collection\nboarder. Position : " + i) { }
+    public PositionOutOfCollectionException() : base() { }
+    public PositionOutOfCollectionException(string message) : base(message) { }
+    public PositionOutOfCollectionException(string message,
+        Exception exception) : base(message, exception) { }
+    protected PositionOutOfCollectionException(SerializationInfo info,
+        StreamingContext contex) : base(info, contex) { }
+}

ProjectCruiser/Program.cs

@@ -1,17 +1,45 @@
+using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.Logging;
+using Microsoft.Extensions.Configuration;
+using Serilog;
+
 namespace ProjectCruiser
 {
     internal static class Program
     {
         /// <summary>
-        ///  The main entry point for the application.
+        /// The main entry point for the application.
         /// </summary>
         [STAThread]
         static void Main()
         {
             // To customize application configuration such as set high DPI settings or default font,
             // see https://aka.ms/applicationconfiguration.
+
             ApplicationConfiguration.Initialize();
-            Application.Run(new ServiceForm2());
+
+            ServiceCollection services = new();
+            ConfigureServices(services);
+            using ServiceProvider serviceProvider = services.BuildServiceProvider();
+            Application.Run(serviceProvider.GetRequiredService<ServiceForm2>());
+
+        }
+        private static void ConfigureServices(ServiceCollection services)
+        {
+            string[] path = Directory.GetCurrentDirectory().Split('\\');
+            string pathNeed = "";
+            for (int i = 0; i < path.Length - 3; i++)
+            {
+                pathNeed += path[i] + "\\";
+            }
+
+            services.AddSingleton<ServiceForm2>().AddLogging(option =>
+            {
+                option.SetMinimumLevel(LogLevel.Information);
+                option.AddSerilog(new LoggerConfiguration().ReadFrom.Configuration(
+                    new ConfigurationBuilder().AddJsonFile(
+                        $"{pathNeed}serilog.json").Build()).CreateLogger());
+            });
         }
     }
 }

ProjectCruiser/ProjectCruiser.csproj

@@ -8,4 +8,16 @@
     <ImplicitUsings>enable</ImplicitUsings>
   </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="8.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.1" />
+    <PackageReference Include="Serilog" Version="4.0.0" />
+    <PackageReference Include="Serilog.Extensions.Logging" Version="8.0.0" />
+    <PackageReference Include="Serilog.Settings.Configuration" Version="8.0.1" />
+    <PackageReference Include="Serilog.Sinks.File" Version="5.0.0" />
+  </ItemGroup>
+
 </Project>

ProjectCruiser/ServiceForm2.Designer.cs

@@ -221,7 +221,7 @@
             btnDelete.TabIndex = 4;
             btnDelete.Text = "Delete";
             btnDelete.UseVisualStyleBackColor = true;
-            btnDelete.Click += btnRemoveCar_Click;
+            btnDelete.Click += btnRemoveShip_Click;
             // 
             // btnAddCruiser
             // 

ProjectCruiser/ServiceForm2.cs

@@ -1,5 +1,9 @@
 using ProjectCruiser.CollectionGenericObj;
 using ProjectCruiser.DrawningSamples;
+using Microsoft.Extensions.Logging;
+using ProjectCruiser.Exceptions;
+// using NLog.Extensions.Logging;
+
 namespace ProjectCruiser;
 
 public partial class ServiceForm2 : Form
@@ -9,10 +13,16 @@ public partial class ServiceForm2 : Form
 
     private readonly StorageCollection<DrawningBase> _storageCollection;
 
-    public ServiceForm2()
+    // Логер
+    private readonly ILogger _logger;
+
+    // Конструктор > logger
+    public ServiceForm2(ILogger<ServiceForm2> logger)
     {
         InitializeComponent();
         _storageCollection = new();
+        _logger = logger;
+        _logger.LogInformation("> Form is loaded successfully");
     }
 
     // Выбор компании
@@ -21,13 +31,10 @@ public partial class ServiceForm2 : Form
         toolPanel.Enabled = false;
     }
 
-    // Color picker (default : random) <...>
-
     // Добавление корабля
     private void btnAddTransport_Click(object sender, EventArgs e)
     {
         EditorForm3 form3 = new();
-        // TODO передать метод :
         form3.AddEvent(CreateObject);
         form3.Show();
     }
@@ -35,37 +42,53 @@ public partial class ServiceForm2 : Form
     // Создание объекта класса-перемещения
     private void CreateObject(DrawningBase? ship)
     {
-        if (_company == null || ship == null)
+        try
-        {
-            return;
-        }
-        if (_company + ship != -1)
         {
+            if (_company == null || ship == null)
+            {
+                throw new NullReferenceException(" > No existing collections to save");
+            }
+
+            int count = _company + ship;
+
             MessageBox.Show("> Object was added");
             pictureBox.Image = _company.Show();
+
+            _logger.LogInformation("> Adding object succeed {ship} at {count} position", ship, count);
         }
-        else
+        catch (Exception ex)
         {
-            MessageBox.Show("[!] Failed to add object");
+            MessageBox.Show("[!] Failed to add object\n" + ex.Message);
+            _logger.LogError("< Error > : {Message}", ex.Message);
         }
     }
 
     // Удаление объекта
-    private void btnRemoveCar_Click(object sender, EventArgs e)
+    private void btnRemoveShip_Click(object sender, EventArgs e)
     {
         if (string.IsNullOrEmpty(maskedTextBoxPosition.Text)
             || _company == null) return;
 
         if (MessageBox.Show("[*] Remove object: Are you sure?", "Remove",
         MessageBoxButtons.YesNo, MessageBoxIcon.Question) != DialogResult.Yes) return;
+
         int pos = Convert.ToInt32(maskedTextBoxPosition.Text);
 
-        if (_company - Convert.ToInt32(maskedTextBoxPosition.Text) != null)
+        try
         {
-            MessageBox.Show("> Object was removed");
+            if (_company - pos != null)
-            pictureBox.Image = _company.Show();
+            {
+                MessageBox.Show("> Object was removed");
+                pictureBox.Image = _company.Show();
+                _logger.LogInformation("Object at " +
+                    pos + "position was deleted successfully");
+            }
+        }
+        catch (Exception ex)
+        {
+            MessageBox.Show("[!] Failed to remove object");
+            _logger.LogError("< Error > : {Message}", ex.Message);
         }
-        else MessageBox.Show("[!] Failed to remove object");
     }
 
     // Передача объекта в другую форму
@@ -75,27 +98,37 @@ public partial class ServiceForm2 : Form
         {
             return;
         }
-        DrawningBase? car = null;
+        DrawningBase? ship = null;
         int counter = 100;
-        while (car == null)
+        while (ship == null)
         {
-            car = _company.GetRandomObject();
+            try
-            counter--;
-            if (counter <= 0)
             {
-                break;
+                ship = _company.GetRandomObject();
+                counter--;
+
+                if (counter <= 0)
+                {
+                    break;
+                }
             }
+            catch (Exception ex)
+            {
+                Console.WriteLine(ex.Message);
+                return;
+            }
+
         }
-        if (car == null)
+
+        if (ship == null)
         {
             return;
         }
 
-        OceanForm1 form = new() { SetShip = car };
+        OceanForm1 form = new() { SetShip = ship };
         form.ShowDialog();
     }
 
-    // Перерисовка коллекции
     private void btnRefresh_Click(object sender, EventArgs e)
     {
         if (_company == null)
@@ -124,7 +157,17 @@ public partial class ServiceForm2 : Form
             collType = CollectionType.List;
         }
 
-        _storageCollection.AddCollection(maskedTxtBoxCName.Text, collType);
+        try
+        {
+            _storageCollection.AddCollection(maskedTxtBoxCName.Text, collType);
+            _logger.LogInformation("Adding collection succeed : {Name}, {Type}", maskedTxtBoxCName.Text, collType);
+        }
+        catch (NullReferenceException ex)
+        {
+            Console.WriteLine(ex.Message);
+            _logger.LogError("< Error > : {Message}", ex.Message);
+        }
+
         RefreshListBoxItems();
     }
 
@@ -134,13 +177,21 @@ public partial class ServiceForm2 : Form
         {
             MessageBox.Show("Collection was not choosed");
             return;
-        }
+        } if (MessageBox.Show("Are you sure?", "Removing",
-        if (MessageBox.Show("Are you sure?", "Removing", MessageBoxButtons.OK, MessageBoxIcon.Question) != DialogResult.OK)
+            MessageBoxButtons.OK, MessageBoxIcon.Question)
+            != DialogResult.OK) return;
+
+        try
         {
-            return;
+            _storageCollection.DelCollection(listBox.SelectedItem.ToString());
+            RefreshListBoxItems();
+            _logger.LogInformation("Removing collection succeed : {Name}", listBox.SelectedItem.ToString);
+        }
+        catch (NullReferenceException ex)
+        {
+            Console.WriteLine(ex.Message);
+            _logger.LogError("< Error > : {Message}", ex.Message);
         }
-        _storageCollection.DelCollection(listBox.SelectedItem.ToString());
-        RefreshListBoxItems();
     }
 
     private void RefreshListBoxItems()
@@ -177,7 +228,7 @@ public partial class ServiceForm2 : Form
         {
             case "Storage":
                 _company = new ShipSharingService(pictureBox.Width,
-                pictureBox.Height, collection);
+                    pictureBox.Height, collection);
                 break;
         }
 
@@ -190,15 +241,17 @@ public partial class ServiceForm2 : Form
     {
         if (saveFileDialog.ShowDialog() == DialogResult.OK)
         {
-            if (_storageCollection.SaveData(saveFileDialog.FileName))
+            try
             {
+                _storageCollection.SaveData(saveFileDialog.FileName);
                 MessageBox.Show(" < Saved succesfully >",
                 "Result :", MessageBoxButtons.OK, MessageBoxIcon.Information);
+                _logger.LogInformation("Saving to file : {filename}", saveFileDialog.FileName);
             }
-            else
+            catch (Exception ex)
             {
-                MessageBox.Show("< Failed to save >", "Result :",
+                MessageBox.Show(ex.Message, "Result :", MessageBoxButtons.OK, MessageBoxIcon.Error);
-                MessageBoxButtons.OK, MessageBoxIcon.Error);
+                _logger.LogError("< Error > : {Message}", ex.Message);
             }
         }
     }
@@ -208,16 +261,27 @@ public partial class ServiceForm2 : Form
     {
         if (openFileDialog.ShowDialog() == DialogResult.OK)
         {
-            if (_storageCollection.LoadData(openFileDialog.FileName))
+            try
             {
+                _storageCollection.LoadData(openFileDialog.FileName);
+                // LoadData() : Exceptions
+                // FileNotFoundException
+                // NullReferenceException
+                // InvalidDataException
+                // IndexOutOfRangeException
+                // CollectionOverflowException
+
                 MessageBox.Show(" < Loaded succesfully >",
                 "Result :", MessageBoxButtons.OK, MessageBoxIcon.Information);
+                _logger.LogInformation("Loading from file : {Filename}", openFileDialog.FileName);
+
                 RefreshListBoxItems();
             }
-            else
+            catch (Exception ex)
             {
-                MessageBox.Show("< Failed to load >", "Result :",
+                MessageBox.Show("< Failed to load >" + ex.Message,
-                MessageBoxButtons.OK, MessageBoxIcon.Error);
+                    "Result :", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                _logger.LogError("< Error > : {Message}", ex.Message);
             }
         }
     }

ProjectCruiser/serilog.json

@@ -0,0 +1,15 @@
+{
+    "Serilog": {
+        "Using": [ "Serilog.Sinks.File" ],
+        "MinimumLevel": "Debug",
+        "WriteTo": [
+            {
+                "Name": "File",
+                "Args": { "path": "log.log" }
+            }
+        ],
+        "Properties": {
+            "Application": "Sample"
+        }
+    }
+}