немного исправила код

This commit is contained in:
Victoria_Isaeva 2024-05-01 21:39:29 +04:00
parent 9deaccb1e1
commit 38f2153806
5 changed files with 16 additions and 29 deletions

View File

@ -37,15 +37,14 @@ public class ListGenericObjects<T> : ICollectionGenericObjects<T>
public T? Get(int position) public T? Get(int position)
{ {
// TODO проверка позиции
if (position >= Count || position < 0) return null; if (position >= Count || position < 0) return null;
return _collection[position]; return _collection[position];
} }
public int Insert(T obj) public int Insert(T obj)
{ {
// TODO проверка, что не превышено максимальное количество элементов
// TODO вставка в конец набора
if (Count + 1 > _maxCount) return -1; if (Count + 1 > _maxCount) return -1;
_collection.Add(obj); _collection.Add(obj);
return Count; return Count;
@ -53,9 +52,7 @@ public class ListGenericObjects<T> : ICollectionGenericObjects<T>
public int Insert(T obj, int position) public int Insert(T obj, int position)
{ {
// TODO проверка, что не превышено максимальное количество элементов
// TODO проверка позиции
// TODO вставка по позиции
if (Count + 1 > _maxCount) return -1; if (Count + 1 > _maxCount) return -1;
if (position < 0 || position > Count) return -1; if (position < 0 || position > Count) return -1;
_collection.Insert(position, obj); _collection.Insert(position, obj);
@ -64,8 +61,7 @@ public class ListGenericObjects<T> : ICollectionGenericObjects<T>
public T? Remove(int position) public T? Remove(int position)
{ {
// TODO проверка позиции
// TODO удаление объекта из списка
if (position < 0 || position > Count) return null; if (position < 0 || position > Count) return null;
T? pos = _collection[position]; T? pos = _collection[position];
_collection.RemoveAt(position); _collection.RemoveAt(position);

View File

@ -48,7 +48,7 @@ internal class MassiveGenericObjects<T> : ICollectionGenericObjects<T>
public T? Get(int position) public T? Get(int position)
{ {
// TODO проверка позиции
if (position >= _collection.Length || position < 0) if (position >= _collection.Length || position < 0)
{ return null; } { return null; }
return _collection[position]; return _collection[position];
@ -56,7 +56,7 @@ internal class MassiveGenericObjects<T> : ICollectionGenericObjects<T>
public int Insert(T obj) public int Insert(T obj)
{ {
// TODO вставка в свободное место набора
int index = 0; int index = 0;
while (index < _collection.Length) while (index < _collection.Length)
{ {
@ -73,11 +73,7 @@ internal class MassiveGenericObjects<T> : ICollectionGenericObjects<T>
public int Insert(T obj, int position) public int Insert(T obj, int position)
{ {
// TODO проверка позиции
// TODO проверка, что элемент массива по этой позиции пустой, если нет, то
// ищется свободное место после этой позиции и идет вставка туда
// если нет после, ищем до
// TODO вставка
if (position >= _collection.Length || position < 0) if (position >= _collection.Length || position < 0)
{ return -1; } { return -1; }
@ -110,8 +106,7 @@ internal class MassiveGenericObjects<T> : ICollectionGenericObjects<T>
public T? Remove(int position) public T? Remove(int position)
{ {
// TODO проверка позиции
// TODO удаление объекта из массива, присвоив элементу массива значение null
if (position >= _collection.Length || position < 0) if (position >= _collection.Length || position < 0)
{ {
return null; return null;

View File

@ -31,8 +31,7 @@ public class StorageCollection<T>
public void AddCollection(string name, CollectionType collectionType) public void AddCollection(string name, CollectionType collectionType)
{ {
// TODO проверка, что name не пустой и нет в словаре записи с таким ключом
// TODO Прописать логику для добавления
if (name == null || _storages.ContainsKey(name)) { return; } if (name == null || _storages.ContainsKey(name)) { return; }
switch (collectionType) switch (collectionType)
@ -51,7 +50,7 @@ public class StorageCollection<T>
public void DelCollection(string name) public void DelCollection(string name)
{ {
// TODO Прописать логику для удаления коллекции
if (_storages.ContainsKey(name)) if (_storages.ContainsKey(name))
_storages.Remove(name); _storages.Remove(name);
} }
@ -61,7 +60,7 @@ public class StorageCollection<T>
{ {
get get
{ {
// TODO Продумать логику получения объекта
if (name == null || !_storages.ContainsKey(name)) { return null; } if (name == null || !_storages.ContainsKey(name)) { return null; }
return _storages[name]; return _storages[name];
} }

View File

@ -44,11 +44,11 @@ public class DrawningBus
EntityBus = new EntityBus(speed, weight, bodyColor); EntityBus = new EntityBus(speed, weight, bodyColor);
} }
//конструктор для наследников
protected DrawningBus(int drawningBusWidth, int drawningBusHeigh) : this() protected DrawningBus(int drawningBusWidth, int drawningBusHeigh) : this()
{ {
drawningBusWidth = drawningBusWidth; //ширина drawningBusWidth = drawningBusWidth;
drawningBusHeigh = drawningBusHeigh; //высота drawningBusHeigh = drawningBusHeigh;
} }
@ -119,7 +119,7 @@ public class DrawningBus
{ {
_startPosX += (int)EntityBus.Step; _startPosX += (int)EntityBus.Step;
} }
//TODO прописать логику сдвига в право
return true; return true;
//вниз //вниз
case DirectionType.Down: case DirectionType.Down:

View File

@ -169,10 +169,7 @@ public partial class FormBusCollection : Form
/// <param name="e"></param> /// <param name="e"></param>
private void buttonCollectionDel_Click(object sender, EventArgs e) private void buttonCollectionDel_Click(object sender, EventArgs e)
{ {
// TODO прописать логику удаления элемента из коллекции
// нужно убедиться, что есть выбранная коллекция
// спросить у пользователя через MessageBox, что он подтверждает, что хочет удалить запись
// удалить и обновить ListBox
if (listBoxCollection.SelectedItem == null) if (listBoxCollection.SelectedItem == null)
{ {