How fix lists problem in unity?

Question

[Klim Cheremnyh]

public void RemovePlayerItem()
    {
        List<PlayerListItem> playerListItemToRemove = new List<PlayerListItem>();

        foreach (PlayerListItem playerlistItem in PlayerListItems)
        {
            if(!Manager.GamePlayers.Any(b=> b.ConnectionID == playerlistItem.ConnectionID))
            {
                playerListItemToRemove.Add(playerlistItem);
            }
        }
        if(playerListItemToRemove.Count > 0)
        {
            foreach(PlayerListItem playerlistItemToRemove in playerListItemToRemove)
            {
                GameObject ObjectToRemove = playerListItemToRemove.gameObject;
                PlayerListItems.Remove(playerListItemToRemove);
                Destroy(ObjectToRemove);
                ObjectToRemove = null;
            }
        }
    }

юнити ругается:


вообще не понимаю что такое, туплю уже пару часов, смотрел помогает "using System.Linq;" но мне не помогло...

Comments

[freeExec]

Так над какой строкой тупишь?

[Klim Cheremnyh]

freeExec,

foreach(PlayerListItem playerlistItemToRemove in playerListItemToRemove)

на эту ругается если не ошибаюсь.

[freeExec]

Klim Cheremnyh, и что он говорит?

[Klim Cheremnyh]

freeExec, то что на скриншоте)

Answer 1

[Voland69]

if(playerListItemToRemove.Count > 0)
        {
            // тут у вас каждый элемент итерируемой коллекции - ***list***
            // а вся итерируемая коллекция - ***List***
            foreach(PlayerListItem playerlistItemToRemove in playerListItemToRemove)
            {
                // 1 ошибка - тут вы достаете gameObject у листа айтемов (***List***)
                GameObject ObjectToRemove = playerListItemToRemove.gameObject;
                // 2 ошибка - а тут вы удаляете лист из листа айтемов (***List***)
                PlayerListItems.Remove(playerListItemToRemove);
                Destroy(ObjectToRemove);
                ObjectToRemove = null;
            }
        }

P.S. привыкайте к внятному неймингу - эти тренировки на внимательность Вам ни к чему

Comments

[Klim Cheremnyh]

код не мой, американской ютуберши, переписывал ночью поэтому может что-то упустил. так и не понял как пофиксить)

[Voland69]

Klim Cheremnyh, подозреваю что playerListItemToRemove стоит поменять на playerlistItemToRemove в этих двух строчках, где ошибки. По крайней мере по типам вроде подходит.
Но лучше не переписывать уроки, где List - это много, а list уже один элемент - за такой нейминг переменных в приличном месте коллеги будут бить, и возможно даже ногами.

[Klim Cheremnyh]

Voland69, спасибо, попробую)

[Klim Cheremnyh]

Voland69, помогло, спасибо)