Выполнение TODO, исправление ошибок

This commit is contained in:
malimova 2023-10-29 22:25:43 +04:00
parent 829029a6dc
commit dfa874b91f
5 changed files with 38 additions and 53 deletions

View File

@ -274,6 +274,5 @@ namespace AirBomber
/// </summary>
public IMoveableObject GetMoveableObject => new
DrawningObjectAirPlane(this);
}
}

View File

@ -45,10 +45,9 @@ namespace AirBomber
/// </summary>
/// <param name="sender"></param>
/// <param name="e"></param>
private void ListBoxObjects_SelectedIndexChanged(object sender, EventArgs e)
private void listBoxObjects_SelectedIndexChanged(object sender, EventArgs e)
{
pictureBoxCollection.Image =
_storage[listBoxStorages.SelectedItem?.ToString()?? string.Empty]?.ShowPlanes();
pictureBoxCollection.Image = _storage[listBoxStorages.SelectedItem?.ToString()?? string.Empty]?.ShowPlanes();
}
/// <summary>
@ -70,7 +69,7 @@ namespace AirBomber
FormAirBomber form = new();
if (form.ShowDialog() == DialogResult.OK)
{
if (obj + form.SelectedPlane != -1)
if (obj + form.SelectedPlane > -1)
{
MessageBox.Show("Объект добавлен");
pictureBoxCollection.Image = obj.ShowPlanes();

View File

@ -103,15 +103,13 @@ namespace AirBomber
Pen pen = new(Color.Black, 3);
for (int i = 0; i < _pictureWidth / _placeSizeWidth; i++)
{
for (int j = 0; j < _pictureHeight / _placeSizeHeight +
1; ++j)
{//линия рамзетки места
for (int j = 0; j < _pictureHeight / _placeSizeHeight + 1; ++j)
{
//линия рамзетки места
g.DrawLine(pen, i * _placeSizeWidth, j *
_placeSizeHeight, i * _placeSizeWidth + _placeSizeWidth / 2, j *
_placeSizeHeight);
_placeSizeHeight, i * _placeSizeWidth + _placeSizeWidth / 2, j * _placeSizeHeight);
}
g.DrawLine(pen, i * _placeSizeWidth, 0, i *
_placeSizeWidth, _pictureHeight / _placeSizeHeight * _placeSizeHeight);
g.DrawLine(pen, i * _placeSizeWidth, 0, i * _placeSizeWidth, _pictureHeight / _placeSizeHeight * _placeSizeHeight);
}
}
/// <summary>
@ -121,18 +119,16 @@ namespace AirBomber
private void DrawObjects(Graphics g)
{
int widthObjCount = _pictureWidth / _placeSizeWidth;
int i = 0;
foreach (var plane in _collection.GetPlanes())
for (int i = 0; i < _collection.Count; i++)
{
T? type = _collection[i];
if (type != null)
{
if (plane != null)
{
int row = i / widthObjCount;
int col = widthObjCount - 1 - (i % widthObjCount);
int row = i / widthObjCount;
int col = widthObjCount - 1 - (i % widthObjCount);
plane.SetPosition(col * _placeSizeWidth, row * _placeSizeHeight);
plane?.DrawPlane(g);
}
type.SetPosition(col * _placeSizeWidth, row * _placeSizeHeight);
type?.DrawPlane(g);
}
}
}

View File

@ -41,7 +41,11 @@ namespace AirBomber
/// <param name="name">Название набора</param>
public void AddSet(string name)
{
// TODO Прописать логику для добавления
// TODO Прописать логику для добавления DONE
if (!_planeStorages.ContainsKey(name))
{
_planeStorages.Add(name, new PlanesGenericCollection<DrawningAirPlane, DrawningObjectAirPlane>(_pictureWidth, _pictureHeight));
}
}
/// <summary>
/// Удаление набора
@ -49,7 +53,11 @@ namespace AirBomber
/// <param name="name">Название набора</param>
public void DelSet(string name)
{
// TODO Прописать логику для удаления
// TODO Прописать логику для удаления DONE
if (_planeStorages.ContainsKey(name))
{
_planeStorages.Remove(name);
}
}
/// <summary>
/// Доступ к набору
@ -61,7 +69,11 @@ namespace AirBomber
{
get
{
// TODO Продумать логику получения набора
// TODO Продумать логику получения набора DONE
if (_planeStorages.ContainsKey(ind))
{
return _planeStorages[ind];
}
return null;
}
}

View File

@ -53,32 +53,9 @@ namespace AirBomber
// проверка, что после вставляемого элемента в массиве есть пустой элемент
// сдвиг всех объектов, находящихся справа от позиции до первого пустого элемента
// TODO вставка по позиции
int NoEmpty = 0, temp = 0;
for (int i = position; i < Count; i++)
{
if (_places[i] != null) NoEmpty++;
}
if (NoEmpty == Count - position) return -1;
if (position < Count && position >= 0)
{
for (int j = position; j < Count; j++)
{
if (_places[j] == null)
{
temp = j;
break;
}
}
for (int i = temp; i > position; i--)
{
_places[i] = _places[i - 1];
}
_places[position] = plane;
return position;
}
return -1;
if (position < 0 || position >= _maxCount) return -1;
_places.Insert(position, plane);
return position;
}
/// <summary>
/// Удаление объекта из набора с конкретной позиции
@ -114,9 +91,11 @@ namespace AirBomber
}
set
{
// TODO проверка позиции
// TODO проверка свободных мест в списке
// TODO вставка в список по позиции
// TODO проверка позиции DONE
// TODO проверка свободных мест в списке DONE
// TODO вставка в список по позиции DONE
if (position < 0 || position >= Count || Count == _maxCount) return;
_places.Insert(position, value);
}
}
/// <summary>