Изменен тип возвращаемого значения
This commit is contained in:
parent
d13c74aec0
commit
453c7c79c4
@ -121,7 +121,7 @@ namespace AirBomber
|
|||||||
{
|
{
|
||||||
MessageBox.Show(text, caption);
|
MessageBox.Show(text, caption);
|
||||||
}
|
}
|
||||||
else if (!(_mapsCollection[NameMap] + airplane))
|
else if ((_mapsCollection[NameMap] + airplane) == -1)
|
||||||
{
|
{
|
||||||
MessageBox.Show("Не удалось добавить объект");
|
MessageBox.Show("Не удалось добавить объект");
|
||||||
}
|
}
|
||||||
@ -239,17 +239,14 @@ namespace AirBomber
|
|||||||
/// <param name="e"></param>
|
/// <param name="e"></param>
|
||||||
private void ButtonRemoveAirplane_Click(object sender, EventArgs e)
|
private void ButtonRemoveAirplane_Click(object sender, EventArgs e)
|
||||||
{
|
{
|
||||||
if (listBoxMaps.SelectedIndex == -1 || string.IsNullOrEmpty(maskedTextBoxPosition.Text))
|
if (listBoxMaps.SelectedIndex == -1 || string.IsNullOrEmpty(maskedTextBoxPosition.Text)
|
||||||
{
|
|| MessageBox.Show("Удалить объект?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No)
|
||||||
return;
|
|
||||||
}
|
|
||||||
if (MessageBox.Show("Удалить объект?", "Удаление", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.No)
|
|
||||||
{
|
{
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
int pos = Convert.ToInt32(maskedTextBoxPosition.Text);
|
int pos = Convert.ToInt32(maskedTextBoxPosition.Text);
|
||||||
var deletedObject = _mapsCollection[NameMap, pos];
|
var deletedObject = _mapsCollection[NameMap, pos];
|
||||||
if (_mapsCollection[NameMap] - pos)
|
if (_mapsCollection[NameMap] - pos != null)
|
||||||
{
|
{
|
||||||
MessageBox.Show("Объект удален");
|
MessageBox.Show("Объект удален");
|
||||||
_deletedObjects.AddLast(deletedObject);
|
_deletedObjects.AddLast(deletedObject);
|
||||||
|
@ -55,7 +55,7 @@ namespace AirBomber
|
|||||||
/// <param name="map"></param>
|
/// <param name="map"></param>
|
||||||
/// <param name="airplane"></param>
|
/// <param name="airplane"></param>
|
||||||
/// <returns></returns>
|
/// <returns></returns>
|
||||||
public static bool operator +(MapWithSetAirplanesGeneric<T, U> map, T airplane)
|
public static int operator +(MapWithSetAirplanesGeneric<T, U> map, T airplane)
|
||||||
{
|
{
|
||||||
return map.SetAirplanes.Insert(airplane);
|
return map.SetAirplanes.Insert(airplane);
|
||||||
}
|
}
|
||||||
@ -65,7 +65,7 @@ namespace AirBomber
|
|||||||
/// <param name="map"></param>
|
/// <param name="map"></param>
|
||||||
/// <param name="position"></param>
|
/// <param name="position"></param>
|
||||||
/// <returns></returns>
|
/// <returns></returns>
|
||||||
public static bool operator -(MapWithSetAirplanesGeneric<T, U> map, int position)
|
public static T operator -(MapWithSetAirplanesGeneric<T, U> map, int position)
|
||||||
{
|
{
|
||||||
return map.SetAirplanes.Remove(position);
|
return map.SetAirplanes.Remove(position);
|
||||||
}
|
}
|
||||||
|
@ -36,8 +36,8 @@ namespace AirBomber
|
|||||||
/// Добавление объекта в набор
|
/// Добавление объекта в набор
|
||||||
/// </summary>
|
/// </summary>
|
||||||
/// <param name="airplane">Добавляемый самолет</param>
|
/// <param name="airplane">Добавляемый самолет</param>
|
||||||
/// <returns></returns>
|
/// <returns>Возвращает позицию вставленого объекта либо -1, если не получилось его добавить</returns>
|
||||||
public bool Insert(T airplane)
|
public int Insert(T airplane)
|
||||||
{
|
{
|
||||||
return Insert(airplane, 0);
|
return Insert(airplane, 0);
|
||||||
}
|
}
|
||||||
@ -51,27 +51,28 @@ namespace AirBomber
|
|||||||
/// </summary>
|
/// </summary>
|
||||||
/// <param name="airplane">Добавляемый самолет</param>
|
/// <param name="airplane">Добавляемый самолет</param>
|
||||||
/// <param name="position">Позиция</param>
|
/// <param name="position">Позиция</param>
|
||||||
/// <returns></returns>
|
/// <returns>Возвращает позицию вставленого объекта либо -1, если не получилось его добавить</returns>
|
||||||
public bool Insert(T airplane, int position)
|
public int Insert(T airplane, int position)
|
||||||
{
|
{
|
||||||
if (!isCorrectPosition(position))
|
if (!isCorrectPosition(position))
|
||||||
{
|
{
|
||||||
return false;
|
return -1;
|
||||||
}
|
}
|
||||||
_places.Insert(position, airplane);
|
_places.Insert(position, airplane);
|
||||||
return true;
|
return position;
|
||||||
}
|
}
|
||||||
/// <summary>
|
/// <summary>
|
||||||
/// Удаление объекта из набора с конкретной позиции
|
/// Удаление объекта из набора с конкретной позиции
|
||||||
/// </summary>
|
/// </summary>
|
||||||
/// <param name="position"></param>
|
/// <param name="position"></param>
|
||||||
/// <returns></returns>
|
/// <returns>Возвращает удаленный объект, либо null если его не удалось удалить</returns>
|
||||||
public bool Remove(int position)
|
public T Remove(int position)
|
||||||
{
|
{
|
||||||
if (!isCorrectPosition(position) || position >= Count)
|
if (!isCorrectPosition(position) || position >= Count)
|
||||||
return false;
|
return null;
|
||||||
|
var deletedObj = _places[position];
|
||||||
_places.RemoveAt(position);
|
_places.RemoveAt(position);
|
||||||
return true;
|
return deletedObj;
|
||||||
}
|
}
|
||||||
/// <summary>
|
/// <summary>
|
||||||
/// Получение объекта из набора по позиции
|
/// Получение объекта из набора по позиции
|
||||||
|
Loading…
Reference in New Issue
Block a user