diff --git a/laba 0/laba 0/CollectionGenericObjects/AbstractCompany.cs b/laba 0/laba 0/CollectionGenericObjects/AbstractCompany.cs index d045b1d..fc69662 100644 --- a/laba 0/laba 0/CollectionGenericObjects/AbstractCompany.cs +++ b/laba 0/laba 0/CollectionGenericObjects/AbstractCompany.cs @@ -40,7 +40,7 @@ public abstract class AbstractCompany /// /// Вычисление максимального количества элементов, который можно разместить в окне /// - private int GetMaxCount => _pictureWidth * _pictureHeight / (_placeSizeWidth * _placeSizeHeight); + private int GetMaxCount => (_pictureWidth * _pictureHeight / (_placeSizeWidth * _placeSizeHeight)) - 5; /// /// Конструктор diff --git a/laba 0/laba 0/CollectionGenericObjects/BoatHarbor.cs b/laba 0/laba 0/CollectionGenericObjects/BoatHarbor.cs index 8d38026..7908088 100644 --- a/laba 0/laba 0/CollectionGenericObjects/BoatHarbor.cs +++ b/laba 0/laba 0/CollectionGenericObjects/BoatHarbor.cs @@ -38,7 +38,7 @@ public class BoatHarbor : AbstractCompany if (_collection.Get(i) != null) { _collection?.Get(i)?.SetPictureSize(_pictureWidth, _pictureHeight); - _collection?.Get(i)?.SetPosition(posX * _placeSizeWidth +200, posY * _placeSizeHeight + 15); + _collection?.Get(i)?.SetPosition(posX * _placeSizeWidth +200, posY * _placeSizeHeight + 30); } posX++; diff --git a/laba 0/laba 0/CollectionGenericObjects/ICollectionGenericObjects.cs b/laba 0/laba 0/CollectionGenericObjects/ICollectionGenericObjects.cs index d675b62..9ef4425 100644 --- a/laba 0/laba 0/CollectionGenericObjects/ICollectionGenericObjects.cs +++ b/laba 0/laba 0/CollectionGenericObjects/ICollectionGenericObjects.cs @@ -1,10 +1,4 @@ -using System; -using System.Collections.Generic; -using System.Linq; -using System.Text; -using System.Threading.Tasks; - -namespace MotorBoat.CollectionGenericObjects; +namespace MotorBoat.CollectionGenericObjects; /// /// Интерфейс описания действий для набора хранимых объектов @@ -27,7 +21,7 @@ public interface ICollectionGenericObjects /// Добавление объекта в коллекцию /// /// Добавляемый объект - /// true - вставка прошла удачно, false - вставка неудалась + /// true - вставка прошла удачно, false - вставка не удалась int Insert(T obj); /// @@ -35,7 +29,7 @@ public interface ICollectionGenericObjects /// /// Добавляемый объект /// Позиция - /// true - вставка прошла удачно, false - вставка неудалась + /// true - вставка прошла удачно, false - вставка не удалась int Insert(T obj, int position); /// @@ -43,7 +37,7 @@ public interface ICollectionGenericObjects /// /// Позиция /// true - удаление прошло удачно, false - удаление не удалось - T Remove(int position); + T? Remove(int position); /// /// Получение объекта по позиции diff --git a/laba 0/laba 0/CollectionGenericObjects/ListGenericObjects.cs b/laba 0/laba 0/CollectionGenericObjects/ListGenericObjects.cs index f7f7dee..d5e775c 100644 --- a/laba 0/laba 0/CollectionGenericObjects/ListGenericObjects.cs +++ b/laba 0/laba 0/CollectionGenericObjects/ListGenericObjects.cs @@ -1,4 +1,6 @@ -namespace MotorBoat.CollectionGenericObjects; +using MotorBoat.Exceptions; + +namespace MotorBoat.CollectionGenericObjects; /// /// Параметризованный набор объекта @@ -12,8 +14,6 @@ public class ListGenericObjects : ICollectionGenericObjects /// private readonly List _collection; - public CollectionType GetCollectionType => CollectionType.List; - /// /// Максимально допустимое число объектов в списке /// @@ -27,6 +27,7 @@ public class ListGenericObjects : ICollectionGenericObjects { return Count; } + set { if (value > 0) @@ -36,6 +37,8 @@ public class ListGenericObjects : ICollectionGenericObjects } } + public CollectionType GetCollectionType => CollectionType.List; + /// /// Конструктор /// @@ -46,34 +49,36 @@ public class ListGenericObjects : ICollectionGenericObjects public T? Get(int position) { - // TODO проверка позиции - if (position >= 0 && position < Count) + // Проверка позиции + if (position >= Count || position < 0) { - return _collection[position]; - } - else - { - return null; + throw new PositionOutOfCollectionException(position); } + return _collection[position]; } public int Insert(T obj) { - // TODO проверка, что не превышено максимальное количество элементов - // TODO вставка в конец набора - if (Count == _maxCount) { return -1; } + // Проверка, что не превышено максимальное количество элементов + if (Count == _maxCount) + { + throw new CollectionOverflowException(Count); + } _collection.Add(obj); return Count; } public int Insert(T obj, int position) { - // TODO проверка, что не превышено максимальное количество элементов - // TODO проверка позиции - // TODO вставка по позиции - if (position < 0 || position >= Count || Count == _maxCount) + // Проверка, что не превышено максимальное количество элементов + if (Count == _maxCount) { - return -1; + throw new CollectionOverflowException(Count); + } + // Проверка позиции + if (position >= Count || position < 0) + { + throw new PositionOutOfCollectionException(position); } _collection.Insert(position, obj); return position; @@ -81,9 +86,11 @@ public class ListGenericObjects : ICollectionGenericObjects public T? Remove(int position) { - // TODO проверка позиции - // TODO удаление объекта из списка - if (position >= Count || position < 0) return null; + // Проверка позиции + if (position >= Count || position < 0) + { + throw new PositionOutOfCollectionException(position); + } T? obj = _collection[position]; _collection.RemoveAt(position); return obj; diff --git a/laba 0/laba 0/CollectionGenericObjects/MassiveGenericObjects.cs b/laba 0/laba 0/CollectionGenericObjects/MassiveGenericObjects.cs index ca75819..dd6bed4 100644 --- a/laba 0/laba 0/CollectionGenericObjects/MassiveGenericObjects.cs +++ b/laba 0/laba 0/CollectionGenericObjects/MassiveGenericObjects.cs @@ -1,11 +1,12 @@ -using System; -using System.Collections.Generic; -using System.Linq; -using System.Text; -using System.Threading.Tasks; + +using MotorBoat.Exceptions; namespace MotorBoat.CollectionGenericObjects; +/// +/// Параметризованный набор объектов +/// +/// Параметр: ограничение - ссылочный тип public class MassiveGenericObjects : ICollectionGenericObjects where T : class { @@ -22,6 +23,7 @@ public class MassiveGenericObjects : ICollectionGenericObjects { return _collection.Length; } + set { if (value > 0) @@ -50,35 +52,32 @@ public class MassiveGenericObjects : ICollectionGenericObjects public T? Get(int position) { - if (position < 0 || position >= _collection.Length) return null; + // Проверка позиции + if (position >= _collection.Length || position < 0) + { return null; } return _collection[position]; } public int Insert(T obj) { - for (int i = 0; i < Count; i++) - { - if (_collection[i] == null) - { - _collection[i] = obj; - return i; - } - } - return -1; + // Вставка в свободное место набора + return Insert(obj, 0); } public int Insert(T obj, int position) { + // Проверка позиции if (position < 0 || position >= Count) { - return -1; + throw new PositionOutOfCollectionException(position); } + // Проверка, что элемент массива по этой позиции пустой if (_collection[position] == null) { _collection[position] = obj; return position; } - + //Свободное место после этой позиции for (int i = position + 1; i < Count; i++) { if (_collection[i] == null) @@ -87,6 +86,7 @@ public class MassiveGenericObjects : ICollectionGenericObjects return i; } } + //Свободное место до этой позиции for (int i = position - 1; i >= 0; i--) { if (_collection[i] == null) @@ -95,16 +95,21 @@ public class MassiveGenericObjects : ICollectionGenericObjects return i; } } - - return -1; + throw new CollectionOverflowException(Count); } public T? Remove(int position) { + // Проверка позиции и наличия объекта if (position < 0 || position >= Count) { - return null; + throw new PositionOutOfCollectionException(position); } + if (_collection[position] == null) + { + throw new ObjectNotFoundException(position); + } + // Удаление объекта из массива T? obj = _collection[position]; _collection[position] = null; return obj; diff --git a/laba 0/laba 0/CollectionGenericObjects/StorageCollection.cs b/laba 0/laba 0/CollectionGenericObjects/StorageCollection.cs index 9dde66f..90922c6 100644 --- a/laba 0/laba 0/CollectionGenericObjects/StorageCollection.cs +++ b/laba 0/laba 0/CollectionGenericObjects/StorageCollection.cs @@ -1,4 +1,6 @@ using MotorBoat.Drawning; +using MotorBoat.Exceptions; +using System.Text; namespace MotorBoat.CollectionGenericObjects; @@ -19,6 +21,21 @@ public class StorageCollection /// public List Keys => _storages.Keys.ToList(); + /// + /// Ключевое слово, с которого должен начинаться файл + /// + private readonly string _collectionKey = "CollectionsStorage"; + + /// + /// Разделитель для записи ключа и значения элемента словаря + /// + private readonly string _separatorForKeyValue = "|"; + + /// + /// Разделитель для записей коллекции данных в файл + /// + private readonly string _separatorItems = ";"; + /// /// Конструктор /// @@ -34,22 +51,18 @@ public class StorageCollection /// тип коллекции public void AddCollection(string name, CollectionType collectionType) { - // TODO проверка, что name не пустой и нет в словаре записи с таким ключом - // TODO Прописать логику для добавления - if (string.IsNullOrEmpty(name) || _storages.ContainsKey(name)) + // Проверка, что name не пустой и нет в словаре записи с таким ключом + if (string.IsNullOrEmpty(name) || _storages.ContainsKey(name) || collectionType == CollectionType.None) { return; } - switch (collectionType) + if (collectionType == CollectionType.Massive) { - case CollectionType.Massive: - _storages[name] = new MassiveGenericObjects(); - break; - case CollectionType.List: - _storages[name] = new ListGenericObjects(); - break; - default: - return; + _storages[name] = new MassiveGenericObjects(); + } + if (collectionType == CollectionType.List) + { + _storages[name] = new ListGenericObjects(); } } @@ -59,7 +72,6 @@ public class StorageCollection /// Название коллекции public void DelCollection(string name) { - // TODO Прописать логику для удаления коллекции if (_storages.ContainsKey(name)) { _storages.Remove(name); @@ -75,37 +87,26 @@ public class StorageCollection { get { - // TODO Продумать логику получения объекта if (_storages.ContainsKey(name)) { return _storages[name]; } - return null; + else + { + return null; + } } } - /// - /// Ключевое слово, с которого должен начинаться файл - /// - private readonly string _collectionKey = "CollectionsStorage"; - /// - /// Разделитель для записи ключа и значения элемента словаря - /// - private readonly string _separatorForKeyValue = "|"; - /// - /// Разделитель для записей коллекции данных в файл - /// - private readonly string _separatorItems = ";"; /// /// Сохранение информации по катеру в хранилище в файл /// /// Путь и имя файла - /// true - сохранение прошло успешно, false - ошибка при сохранении данных - public bool SaveData(string filename) + public void SaveData(string filename) { if (_storages.Count == 0) { - return false; + throw new ArgumentException("В хранилище отсутствуют коллекции для сохранения"); } if (File.Exists(filename)) @@ -124,14 +125,12 @@ public class StorageCollection { continue; } - writer.Write(value.Key); writer.Write(_separatorForKeyValue); writer.Write(value.Value.GetCollectionType); writer.Write(_separatorForKeyValue); writer.Write(value.Value.MaxCount); writer.Write(_separatorForKeyValue); - foreach (T? item in value.Value.GetItems()) { string data = item?.GetDataForSave() ?? string.Empty; @@ -139,42 +138,41 @@ public class StorageCollection { continue; } - writer.Write(data); writer.Write(_separatorItems); } } } - return true; } /// /// Загрузка информации по автомобилям в хранилище из файла /// /// Путь и имя файла - /// true - загрузка прошла успешно, false - ошибка при загрузке данных - public bool LoadData(string filename) + public void LoadData(string filename) { if (!File.Exists(filename)) { - return false; + throw new FileNotFoundException("Файл не существует"); } - using (StreamReader fs = File.OpenText(filename)) + + using (StreamReader read = File.OpenText(filename)) { - string str = fs.ReadLine(); + string str = read.ReadLine(); + if (str == null || str.Length == 0) { - return false; + throw new FormatException("В файле нет данных"); } if (!str.StartsWith(_collectionKey)) { - return false; + throw new FormatException("В файле неверные данные"); } - + _storages.Clear(); string strs = ""; - while ((strs = fs.ReadLine()) != null) + while ((strs = read.ReadLine()) != null) { string[] record = strs.Split(_separatorForKeyValue, StringSplitOptions.RemoveEmptyEntries); if (record.Length != 4) @@ -186,25 +184,31 @@ public class StorageCollection ICollectionGenericObjects? collection = StorageCollection.CreateCollection(collectionType); if (collection == null) { - return false; + throw new InvalidOperationException("Не удалось создать коллекцию"); } collection.MaxCount = Convert.ToInt32(record[2]); + string[] set = record[3].Split(_separatorItems, StringSplitOptions.RemoveEmptyEntries); foreach (string elem in set) { - if (elem?.CreateDrawningB() is T ship) + if (elem?.CreateDrawningB() is T B) { - if (collection.Insert(ship) == -1) + try { - return false; + if (collection.Insert(B) == -1) + { + throw new InvalidOperationException("Объект не удалось добавить в коллекцию: " + record[3]); + } + } + catch (CollectionOverflowException ex) + { + throw new CollectionOverflowException("Коллекция переполнена", ex); } } } - _storages.Add(record[0], collection); } - return true; } } diff --git a/laba 0/laba 0/Exceptions/CollectionOverflowException.cs b/laba 0/laba 0/Exceptions/CollectionOverflowException.cs new file mode 100644 index 0000000..91c6417 --- /dev/null +++ b/laba 0/laba 0/Exceptions/CollectionOverflowException.cs @@ -0,0 +1,16 @@ +using System.Runtime.Serialization; + +namespace MotorBoat.Exceptions; + +/// +/// Класс, описывающий ошибку переполнения коллекции +/// +[Serializable] +internal class CollectionOverflowException : ApplicationException +{ + public CollectionOverflowException(int count) : base("В коллекции превышено допустимое количество: " + 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) { } +} \ No newline at end of file diff --git a/laba 0/laba 0/Exceptions/ObjectNotFoundException.cs b/laba 0/laba 0/Exceptions/ObjectNotFoundException.cs new file mode 100644 index 0000000..7f10ea1 --- /dev/null +++ b/laba 0/laba 0/Exceptions/ObjectNotFoundException.cs @@ -0,0 +1,16 @@ +using System.Runtime.Serialization; + +namespace MotorBoat.Exceptions; + +/// +/// Класс, описывающий ошибку, что по указанной позиции нет элемента +/// +[Serializable] +internal class ObjectNotFoundException : ApplicationException +{ + public ObjectNotFoundException(int i) : base("Не найден объект по позиции " + 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) { } +} \ No newline at end of file diff --git a/laba 0/laba 0/Exceptions/PositionOutOfCollectionException.cs b/laba 0/laba 0/Exceptions/PositionOutOfCollectionException.cs new file mode 100644 index 0000000..0b83b2d --- /dev/null +++ b/laba 0/laba 0/Exceptions/PositionOutOfCollectionException.cs @@ -0,0 +1,16 @@ +using System.Runtime.Serialization; + +namespace MotorBoat.Exceptions; + +/// +/// Класс, описывающий ошибку выхода за границы коллекции +/// +[Serializable] +internal class PositionOutOfCollectionException : ApplicationException +{ + public PositionOutOfCollectionException(int i) : base("Выход за границы коллекции. Позиция " + 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) { } +} \ No newline at end of file diff --git a/laba 0/laba 0/FormBoatCollection.Designer.cs b/laba 0/laba 0/FormBoatCollection.Designer.cs index 385e959..3a6fe36 100644 --- a/laba 0/laba 0/FormBoatCollection.Designer.cs +++ b/laba 0/laba 0/FormBoatCollection.Designer.cs @@ -68,7 +68,7 @@ groupBoxTools.Dock = DockStyle.Right; groupBoxTools.Location = new Point(489, 24); groupBoxTools.Name = "groupBoxTools"; - groupBoxTools.Size = new Size(194, 497); + groupBoxTools.Size = new Size(194, 440); groupBoxTools.TabIndex = 0; groupBoxTools.TabStop = false; groupBoxTools.Text = "Инструменты"; @@ -82,15 +82,15 @@ panelCompanyTools.Controls.Add(buttonGoToCheck); panelCompanyTools.Dock = DockStyle.Bottom; panelCompanyTools.Enabled = false; - panelCompanyTools.Location = new Point(3, 278); + panelCompanyTools.Location = new Point(3, 265); panelCompanyTools.Name = "panelCompanyTools"; - panelCompanyTools.Size = new Size(188, 216); + panelCompanyTools.Size = new Size(188, 172); panelCompanyTools.TabIndex = 9; // // buttonAddBoat // buttonAddBoat.Anchor = AnchorStyles.Top | AnchorStyles.Left | AnchorStyles.Right; - buttonAddBoat.Location = new Point(6, 3); + buttonAddBoat.Location = new Point(3, 3); buttonAddBoat.Name = "buttonAddBoat"; buttonAddBoat.Size = new Size(179, 38); buttonAddBoat.TabIndex = 1; @@ -100,7 +100,7 @@ // // maskedTextBox // - maskedTextBox.Location = new Point(6, 91); + maskedTextBox.Location = new Point(3, 47); maskedTextBox.Mask = "00"; maskedTextBox.Name = "maskedTextBox"; maskedTextBox.Size = new Size(173, 23); @@ -110,45 +110,45 @@ // buttonRefresh // buttonRefresh.Anchor = AnchorStyles.Top | AnchorStyles.Left | AnchorStyles.Right; - buttonRefresh.Location = new Point(6, 186); + buttonRefresh.Location = new Point(6, 142); buttonRefresh.Name = "buttonRefresh"; buttonRefresh.Size = new Size(179, 27); buttonRefresh.TabIndex = 6; buttonRefresh.Text = "Обновить "; buttonRefresh.UseVisualStyleBackColor = true; - buttonRefresh.Click += buttonRefresh_Click; + buttonRefresh.Click += ButtonRefresh_Click; // // buttonRemoveBoat // buttonRemoveBoat.Anchor = AnchorStyles.Top | AnchorStyles.Left | AnchorStyles.Right; - buttonRemoveBoat.Location = new Point(6, 120); + buttonRemoveBoat.Location = new Point(3, 76); buttonRemoveBoat.Name = "buttonRemoveBoat"; buttonRemoveBoat.Size = new Size(179, 27); buttonRemoveBoat.TabIndex = 4; buttonRemoveBoat.Text = "Удалить катер\r\n "; buttonRemoveBoat.UseVisualStyleBackColor = true; - buttonRemoveBoat.Click += buttonRemoveBoat_Click; + buttonRemoveBoat.Click += ButtonRemoveBoat_Click; // // buttonGoToCheck // buttonGoToCheck.Anchor = AnchorStyles.Top | AnchorStyles.Left | AnchorStyles.Right; - buttonGoToCheck.Location = new Point(6, 153); + buttonGoToCheck.Location = new Point(3, 109); buttonGoToCheck.Name = "buttonGoToCheck"; buttonGoToCheck.Size = new Size(179, 27); buttonGoToCheck.TabIndex = 5; buttonGoToCheck.Text = "Передать на тесты "; buttonGoToCheck.UseVisualStyleBackColor = true; - buttonGoToCheck.Click += buttonGoToCheck_Click; + buttonGoToCheck.Click += ButtonGoToCheck_Click; // // buttonCreateCompany // - buttonCreateCompany.Location = new Point(3, 252); + buttonCreateCompany.Location = new Point(3, 242); buttonCreateCompany.Name = "buttonCreateCompany"; buttonCreateCompany.Size = new Size(188, 23); buttonCreateCompany.TabIndex = 8; buttonCreateCompany.Text = "Создать компанию"; buttonCreateCompany.UseVisualStyleBackColor = true; - buttonCreateCompany.Click += buttonCreateCompany_Click; + buttonCreateCompany.Click += ButtonCreateCompany_Click; // // panelStorage // @@ -162,42 +162,42 @@ panelStorage.Dock = DockStyle.Top; panelStorage.Location = new Point(3, 19); panelStorage.Name = "panelStorage"; - panelStorage.Size = new Size(188, 206); + panelStorage.Size = new Size(188, 190); panelStorage.TabIndex = 7; // // buttonCollectionDel // - buttonCollectionDel.Location = new Point(3, 180); + buttonCollectionDel.Location = new Point(3, 165); buttonCollectionDel.Name = "buttonCollectionDel"; buttonCollectionDel.Size = new Size(182, 23); buttonCollectionDel.TabIndex = 6; buttonCollectionDel.Text = "Удалить коллекцию"; buttonCollectionDel.UseVisualStyleBackColor = true; - buttonCollectionDel.Click += buttonCollectionDel_Click; + buttonCollectionDel.Click += ButtonCollectionDel_Click; // // listBoxCollection // listBoxCollection.FormattingEnabled = true; listBoxCollection.ItemHeight = 15; - listBoxCollection.Location = new Point(3, 110); + listBoxCollection.Location = new Point(3, 96); listBoxCollection.Name = "listBoxCollection"; listBoxCollection.Size = new Size(182, 64); listBoxCollection.TabIndex = 5; // // buttonCollectionAdd // - buttonCollectionAdd.Location = new Point(3, 81); + buttonCollectionAdd.Location = new Point(3, 70); buttonCollectionAdd.Name = "buttonCollectionAdd"; buttonCollectionAdd.Size = new Size(182, 23); buttonCollectionAdd.TabIndex = 4; buttonCollectionAdd.Text = "Добавить коллекцию"; buttonCollectionAdd.UseVisualStyleBackColor = true; - buttonCollectionAdd.Click += buttonCollectionAdd_Click; + buttonCollectionAdd.Click += ButtonCollectionAdd_Click; // // radioButtonList // radioButtonList.AutoSize = true; - radioButtonList.Location = new Point(97, 56); + radioButtonList.Location = new Point(97, 50); radioButtonList.Name = "radioButtonList"; radioButtonList.Size = new Size(67, 19); radioButtonList.TabIndex = 3; @@ -208,7 +208,7 @@ // radioButtonMassive // radioButtonMassive.AutoSize = true; - radioButtonMassive.Location = new Point(17, 56); + radioButtonMassive.Location = new Point(13, 49); radioButtonMassive.Name = "radioButtonMassive"; radioButtonMassive.Size = new Size(69, 19); radioButtonMassive.TabIndex = 2; @@ -218,7 +218,7 @@ // // textBoxCollectionName // - textBoxCollectionName.Location = new Point(3, 27); + textBoxCollectionName.Location = new Point(3, 22); textBoxCollectionName.Name = "textBoxCollectionName"; textBoxCollectionName.Size = new Size(182, 23); textBoxCollectionName.TabIndex = 1; @@ -226,11 +226,12 @@ // labelCollectionName // labelCollectionName.AutoSize = true; - labelCollectionName.Location = new Point(29, 9); + labelCollectionName.Location = new Point(29, 5); labelCollectionName.Name = "labelCollectionName"; labelCollectionName.Size = new Size(135, 15); labelCollectionName.TabIndex = 0; labelCollectionName.Text = "Название коллекции:"; + labelCollectionName.Click += labelCollectionName_Click; // // comboBoxSelectorCompany // @@ -238,18 +239,18 @@ comboBoxSelectorCompany.DropDownStyle = ComboBoxStyle.DropDownList; comboBoxSelectorCompany.FormattingEnabled = true; comboBoxSelectorCompany.Items.AddRange(new object[] { "Хранилище" }); - comboBoxSelectorCompany.Location = new Point(3, 228); + comboBoxSelectorCompany.Location = new Point(3, 214); comboBoxSelectorCompany.Name = "comboBoxSelectorCompany"; comboBoxSelectorCompany.Size = new Size(188, 23); comboBoxSelectorCompany.TabIndex = 0; - comboBoxSelectorCompany.SelectedIndexChanged += comboBoxSelectorCompany_SelectedIndexChanged; + comboBoxSelectorCompany.SelectedIndexChanged += ComboBoxSelectorCompany_SelectedIndexChanged; // // pictureBox // pictureBox.Dock = DockStyle.Fill; pictureBox.Location = new Point(0, 24); pictureBox.Name = "pictureBox"; - pictureBox.Size = new Size(489, 497); + pictureBox.Size = new Size(489, 440); pictureBox.TabIndex = 1; pictureBox.TabStop = false; // @@ -298,7 +299,7 @@ // AutoScaleDimensions = new SizeF(7F, 15F); AutoScaleMode = AutoScaleMode.Font; - ClientSize = new Size(683, 521); + ClientSize = new Size(683, 464); Controls.Add(pictureBox); Controls.Add(groupBoxTools); Controls.Add(menuStrip); diff --git a/laba 0/laba 0/FormBoatCollection.cs b/laba 0/laba 0/FormBoatCollection.cs index 293a423..cf0adcb 100644 --- a/laba 0/laba 0/FormBoatCollection.cs +++ b/laba 0/laba 0/FormBoatCollection.cs @@ -1,14 +1,7 @@ -using System; -using System.Collections.Generic; -using System.ComponentModel; -using System.Data; -using System.Drawing; -using System.Linq; -using System.Text; -using System.Threading.Tasks; -using System.Windows.Forms; +using Microsoft.Extensions.Logging; using MotorBoat.CollectionGenericObjects; using MotorBoat.Drawning; +using MotorBoat.Exceptions; namespace MotorBoat; @@ -27,80 +20,124 @@ public partial class FormBoatCollection : Form /// private AbstractCompany? _company = null; + /// + /// Логгер + /// + private readonly ILogger _logger; + /// /// Конструктор /// - public FormBoatCollection() + public FormBoatCollection(ILogger logger) { InitializeComponent(); _storageCollection = new(); + _logger = logger; } /// - /// Выбор компании - /// - /// - /// - private void comboBoxSelectorCompany_SelectedIndexChanged(object sender, EventArgs e) + /// Выбор компании + /// + /// + /// + private void ComboBoxSelectorCompany_SelectedIndexChanged(object sender, EventArgs e) { panelCompanyTools.Enabled = false; } + /// + /// Добавление лодки + /// + /// + /// private void buttonAddBoat_Click(object sender, EventArgs e) { FormBoatConfig form = new(); - // TODO передать метод form.Show(); form.AddEvent(SetBoat); } /// - /// Добавление катера в коллекцию + /// Добавление лодки в коллекцию /// /// - private void SetBoat(DrawningB? b) + private void SetBoat(DrawningB b) { - if (_company == null || b == null) + try + { + if (_company == null || b == null) + { + return; + } + if (_company + b != -1) + { + MessageBox.Show("Объект добавлен"); + pictureBox.Image = _company.Show(); + _logger.LogInformation("Добавлен объект: " + b.GetDataForSave()); + } + } + catch (CollectionOverflowException ex) + { + MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogError("Ошибка: {Message}", ex.Message); + } + } + + /// + /// Удаление объекта + /// + /// + /// + private void ButtonRemoveBoat_Click(object sender, EventArgs e) + { + if (string.IsNullOrEmpty(maskedTextBox.Text) || _company == null) { return; } - if (_company + b != -1) + if (MessageBox.Show("Удалить объект?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) != DialogResult.Yes) { - MessageBox.Show("Объект добавлен"); - pictureBox.Image = _company.Show(); + return; } - else + + try { - MessageBox.Show("Не удалось добавить объект"); + int pos = Convert.ToInt32(maskedTextBox.Text); + if (_company - pos != null) + { + MessageBox.Show("Объект удален"); + pictureBox.Image = _company.Show(); + _logger.LogInformation("Удаление объекта по индексу " + pos); + } + else + { + MessageBox.Show("Не удалось удалить объект"); + _logger.LogInformation("Не удалось удалить объект из коллекции по индексу " + pos); + } + } + catch (ObjectNotFoundException ex) + { + MessageBox.Show(ex.Message); + _logger.LogError("Ошибка: {Message}", ex.Message); + } + catch (PositionOutOfCollectionException ex) + { + MessageBox.Show(ex.Message); + _logger.LogError("Ошибка: {Message}", ex.Message); } } - private void buttonRemoveBoat_Click(object sender, EventArgs e) - { - if (_company == null || string.IsNullOrEmpty(maskedTextBox.Text)) return; - - if (MessageBox.Show("Удалить объект?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No) return; - - int pos = Convert.ToInt32(maskedTextBox.Text); - if (_company - pos != null) - { - MessageBox.Show("Объект удалён"); - pictureBox.Image = _company.Show(); - } - else - { - MessageBox.Show("Не удалось удалить объект"); - } - } - - private void buttonGoToCheck_Click(object sender, EventArgs e) + /// + /// Передача объекта в другую форму + /// + /// + /// + private void ButtonGoToCheck_Click(object sender, EventArgs e) { if (_company == null) { return; } - DrawningB? B = null; int counter = 100; while (B == null) @@ -112,12 +149,10 @@ public partial class FormBoatCollection : Form break; } } - if (B == null) { return; } - FormBoat form = new() { SetB = B @@ -125,10 +160,17 @@ public partial class FormBoatCollection : Form form.ShowDialog(); } - private void buttonRefresh_Click(object sender, EventArgs e) + /// + /// Перерисовка коллекции + /// + /// + /// + private void ButtonRefresh_Click(object sender, EventArgs e) { - if (_company == null) return; - + if (_company == null) + { + return; + } pictureBox.Image = _company.Show(); } @@ -137,14 +179,14 @@ public partial class FormBoatCollection : Form /// /// /// - private void buttonCollectionAdd_Click(object sender, EventArgs e) + private void ButtonCollectionAdd_Click(object sender, EventArgs e) { if (string.IsNullOrEmpty(textBoxCollectionName.Text) || (!radioButtonList.Checked && !radioButtonMassive.Checked)) { MessageBox.Show("Не все данные заполнены", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogInformation("Не удалось добавить коллекцию: не все данные заполнены"); return; } - CollectionType collectionType = CollectionType.None; if (radioButtonMassive.Checked) { @@ -154,15 +196,16 @@ public partial class FormBoatCollection : Form { collectionType = CollectionType.List; } - _storageCollection.AddCollection(textBoxCollectionName.Text, collectionType); RefreshListBoxItems(); + _logger.LogInformation("Добавлена коллекция типа " + collectionType + " с названием " + + textBoxCollectionName.Text); } /// - /// Обновление списка в listBoxCollection - /// - private void RefreshListBoxItems() + /// Обновление списка в listBoxCollection + /// + private void RefreshListBoxItems() { listBoxCollection.Items.Clear(); for (int i = 0; i < _storageCollection.Keys?.Count; ++i) @@ -180,18 +223,19 @@ public partial class FormBoatCollection : Form /// /// /// - private void buttonCollectionDel_Click(object sender, EventArgs e) + private void ButtonCollectionDel_Click(object sender, EventArgs e) { - if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItems == null) + if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItem == null) { MessageBox.Show("Коллекция не выбрана"); return; } - if (MessageBox.Show("Удалить коллекцию?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No) + if (MessageBox.Show("Удалить коллекцию?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) != DialogResult.Yes) { return; } _storageCollection.DelCollection(listBoxCollection.SelectedItem.ToString()); + _logger.LogInformation("Удаление коллекции с названием {name}", listBoxCollection.SelectedItem.ToString()); RefreshListBoxItems(); } @@ -200,28 +244,25 @@ public partial class FormBoatCollection : Form /// /// /// - private void buttonCreateCompany_Click(object sender, EventArgs e) + private void ButtonCreateCompany_Click(object sender, EventArgs e) { if (listBoxCollection.SelectedIndex < 0 || listBoxCollection.SelectedItem == null) { MessageBox.Show("Коллекция не выбрана"); return; } - ICollectionGenericObjects? collection = _storageCollection[listBoxCollection.SelectedItem.ToString() ?? string.Empty]; if (collection == null) { MessageBox.Show("Коллекция не проинициализирована"); return; } - switch (comboBoxSelectorCompany.Text) { case "Хранилище": _company = new BoatHarbor(pictureBox.Width, pictureBox.Height, collection); break; } - panelCompanyTools.Enabled = true; RefreshListBoxItems(); } @@ -235,40 +276,47 @@ public partial class FormBoatCollection : Form { if (saveFileDialog.ShowDialog() == DialogResult.OK) { - if (_storageCollection.SaveData(saveFileDialog.FileName)) + try { - MessageBox.Show("Сохранение прошло успешно", - "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); + _storageCollection.SaveData(saveFileDialog.FileName); + MessageBox.Show("Сохранение прошло успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); + _logger.LogInformation("Сохранение в файл: {filename}", saveFileDialog.FileName); + } - else + catch (Exception ex) { - MessageBox.Show("Не сохранилось", "Результат", - MessageBoxButtons.OK, MessageBoxIcon.Error); + MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogError("Ошибка: {Message}", ex.Message); } } } /// - /// Обработка нажатия "Загрузка" + /// Обработка кнопки загрузки /// /// /// private void LoadToolStripMenuItem_Click(object sender, EventArgs e) { - //TODO продумать логику if (openFileDialog.ShowDialog() == DialogResult.OK) { - if (_storageCollection.LoadData(openFileDialog.FileName)) + try { - MessageBox.Show("Загрузка прошла успешно", - "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); + _storageCollection.LoadData(openFileDialog.FileName); + MessageBox.Show("Загрузка прошла успешно", "Результат", MessageBoxButtons.OK, MessageBoxIcon.Information); RefreshListBoxItems(); + _logger.LogInformation("Загрузка из файла: {filename}", openFileDialog.FileName); } - else + catch (Exception ex) { - MessageBox.Show("Не сохранилось", "Результат", - MessageBoxButtons.OK, MessageBoxIcon.Error); + MessageBox.Show(ex.Message, "Результат", MessageBoxButtons.OK, MessageBoxIcon.Error); + _logger.LogError("Ошибка: {Message}", ex.Message); } } } + + private void labelCollectionName_Click(object sender, EventArgs e) + { + + } } diff --git a/laba 0/laba 0/FormBoatCollection.resx b/laba 0/laba 0/FormBoatCollection.resx index 151c7f3..b44b220 100644 --- a/laba 0/laba 0/FormBoatCollection.resx +++ b/laba 0/laba 0/FormBoatCollection.resx @@ -126,4 +126,7 @@ 270, 17 + + 25 + \ No newline at end of file diff --git a/laba 0/laba 0/MotorBoat.csproj b/laba 0/laba 0/MotorBoat.csproj index 1a83870..6ad1d74 100644 --- a/laba 0/laba 0/MotorBoat.csproj +++ b/laba 0/laba 0/MotorBoat.csproj @@ -9,6 +9,15 @@ enable + + + + + + + + + True @@ -24,4 +33,10 @@ + + + Always + + + \ No newline at end of file diff --git a/laba 0/laba 0/Program.cs b/laba 0/laba 0/Program.cs index 8145421..9a0c6c5 100644 --- a/laba 0/laba 0/Program.cs +++ b/laba 0/laba 0/Program.cs @@ -1,4 +1,10 @@ -namespace MotorBoat +using Microsoft.Extensions.Configuration; +using Microsoft.Extensions.DependencyInjection; +using Microsoft.Extensions.Logging; +using MotorBoat; +using Serilog; + +namespace ProjectCatamaran { internal static class Program { @@ -11,7 +17,34 @@ namespace MotorBoat // To customize application configuration such as set high DPI settings or default font, // see https://aka.ms/applicationconfiguration. ApplicationConfiguration.Initialize(); - Application.Run(new FormBoatCollection()); + ServiceCollection services = new(); + ConfigureServices(services); + using ServiceProvider serviceProvider = services.BuildServiceProvider(); + Application.Run(serviceProvider.GetRequiredService()); + } + + /// + /// DI + /// + /// + 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() + .AddLogging(option => + { + option.SetMinimumLevel(LogLevel.Information); + option.AddSerilog(new LoggerConfiguration() + .ReadFrom.Configuration(new ConfigurationBuilder() + .AddJsonFile($"{pathNeed}serilog.json") + .Build()) + .CreateLogger()); + }); } } } \ No newline at end of file diff --git a/laba 0/laba 0/serilog.json b/laba 0/laba 0/serilog.json new file mode 100644 index 0000000..a68507b --- /dev/null +++ b/laba 0/laba 0/serilog.json @@ -0,0 +1,16 @@ +{ + "Serilog": { + "Using": [ "Serilog.Sinks.File" ], + "MinimumLevel": "Debug", + "WriteTo": [ + { + "Name": "File", + "Args": { "path": "log.log" } + } + ], + + "Properties": { + "Application": "Sample" + } + } +} \ No newline at end of file