Как найти в List несколько объектов без цикла?

Question

[p4p]

Есть такой пул объектов

public List<Enemy> enemyPool = new List<Enemy>();

В нем допустим 30 объектов

public class Enemy
{
    public int Id { get; set; }
    public int ParentRoomId { get; set; }
    public GameObject GameObj { get; set; }
}

Есть цикл

foreach (Enemy enemy in enemyPool)
            {
                if (amount > 0)
                {
                    if (enemy.ParentRoomId == 0)
                    {
                        enemy.ParentRoomId = roomId;
                        enemy.GameObj.transform.position = spawnPosition.position;
                        enemy.GameObj.transform.rotation = spawnPosition.rotation;
                        enemy.GameObj.SetActive(true);
                        amount--;
                    }
                }
                
            }

Мне надо найти объекты у которых enemy.ParentRoomId == 0. Цикл foreach штрудирует весь список. Есть ли способ найти определенное количество без цикла? И не просто найти, а совершить над ними некий процесс при помощи науки.

Answer 1

[John_Nash]

break или goto для выхода из foreach

Answer 2

[eRKa]

Подозреваю, что Вы хотите что-то подобное

enemyPool
	.Where(e => e.ParentRoomId == 0)
	.Take(amount)
	.ToList()
	.ForEach(e => 
	{
		e.ParentRoomId = roomId;
		e.GameObj.transform.position = spawnPosition.position;
		e.GameObj.transform.rotation = spawnPosition.rotation;
		e.GameObj.SetActive(true);
	});

Если уж совсем хотите с оптимизировать, то можно так

count = 0;
for(vat i = 0; i < enemyPool.Count; i++)
{
	if(enemyPool[i].ParentRoomId  == 0)
	{
		count ++;
		if(count >= amount)
			break;
		enemyPool[i].ParentRoomId = roomId;
		enemyPool[i].GameObj.transform.position = spawnPosition.position;
		enemyPool[i].GameObj.transform.rotation = spawnPosition.rotation;
		enemyPool[i].GameObj.SetActive(true);
	}
}

Comments

[p4p]

Первое кажется оно, а второй вариант все равно ведь прогонит цикл на все количество объектов. Завтра буду замерять скорость первого варианта.

[eRKa]

p4p, второй не прогонит, вот это не даст

if(count >= amount)
      break;

Если счетчик будет больше amount, то будет выход из цикла.

[GavriKos]

ТАк и первый вариант прогонит цикл по всем объектам. Просто он спрятан в Where.

[eRKa]

GavriKos, ну первый вариант для IQueryable, второй для локальной коллекции.

[p4p]

Это все равно получается медленнее чем цикл

foreach (Enemy enemy in enemyPool)
            {
                //print("foreach");

                if (amount > 0)
                {
                    if (enemy.ParentRoomId == 0)
                    {
                        // id комнаты родителя
                        enemy.ParentRoomId = roomId;
                        // позиция для спавна
                        enemy.GameObj.transform.position = spawnPosition.position;
                        enemy.GameObj.transform.rotation = spawnPosition.rotation;
                        enemy.AICombat = enemy.GameObj.GetComponent<AICombat>();
                        enemy.AICombat.parentRoomId = roomId;

                        // активация объекта
                        enemy.GameObj.SetActive(true);
                        amount--;
                        
                    }
                }
                if (amount == 0) break;
            }

Answer 3

[GavriKos]

Не сильно понятно что вам нужно и главное почему.
Цикл так или иначе будет. Слово "найти" уже подразумевает перебор вариантов. Цикл может быть скрыт (Where из LINQ). Он может пройти не по всему массиву, а только до нахождения нужного количества (просто выходим из цикла когда нашли все что нужно.

Без цикла можно разве что заранее сложить нужные объекты куда то. Но по моему это не то что вам нужно...
Можно конечно отсортировать массив по parentRoomId и взять первые 3 элемента - но сортировка тоже цикл.

Comments

[p4p]

Ну вот смотрите, если я использую Dictionary а не лист я могу использовать for и пройтись только 2 раза. Но этот вариант получается медленнее из за поиска в Dictionary

for (int i = 0; i < amount; i++)
            {
                Enemy enemy = enemyPool2.FirstOrDefault(x => x.Value.ParentRoomId == 0).Value;

                if (enemy != null)
                {
                    enemy.ParentRoomId = roomId;
                    // позиция для спавна
                    enemy.GameObj.transform.position = spawnPosition.position;
                    enemy.GameObj.transform.rotation = spawnPosition.rotation;

                    // активация объекта
                    enemy.GameObj.SetActive(true);
                }

                
            }

[GavriKos]

p4p, вы можете использовать for и для листа. А вот приведенный вами код ужасен. Потому что содержит два цикла - FirstOrDefault внутри - цекл - да еще и какое то копирование.
Чем вас foreach не устроил - вы так и не ответили.

[eRKa]

p4p, Этот код с фором у Вас вообще не рабочий. Он всегда будет находить одного и того же первого врага в списке.

[p4p]

GavriKos, я просто не хотел все перебирать.. Думал найду способ получить 2 объекта с заданным Id без цикла.

[GavriKos]

p4p, из листа - нет. Используйте другую структуру данных.
Но чет мне кажется это преждевременная оптимизация.

Answer 4

[Maxim Isaev]

Возможно не прав, но у List есть такой метод, называется contains

Comments

[Алексей 〒.]

Что тоже подразумевает перебор значений

[GavriKos]

Каким чертом Contains поможет найти инстанс объекта? Тем более несколько инстансов.

Answer 5

[Griboks]

Нельзя. В любом случае потребуется цикл. Количество итераций можно сократить, если список сортирован. Но зачем? Можно сократить длину кода. Для этого надо использовать обёртку цикла, например linq.