Вопрос по инвентарю

[vovaU]

Делаю инвентарь создал скрипт для предметов ItemObj в нем находятся параметры предмета: ID, спрайт и количество единиц добавляемых в стак. В инвентаре создаю лист

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

public List<itemObj> inventory_item = new List<itemObj>();

далее пускаю луч в объекты которые есть на сцене проверяю наличие скрипта ItemObj и нажата ли кнопка Е . Далее код проверки на наличие итема в листе и действия, если объект уже есть в листе то к его количеству добавляем N количество единиц, если нету его в листе то добавляем этот объект в лист. Вот код

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

RaycastHit hit;
       
        if (Physics.Raycast(ray, out hit, 3.0f))
        {
            if (Input.GetKeyDown(KeyCode.E) && hit.collider.gameObject.GetComponent<itemObj>())
            {
                        if (inventory_item.Count > 0)
                        {
                            int counti = inventory_item.Count;
                    for (int k = 0; k < counti; k++)
                    {

                        if (inventory_item[k].Id == hit.collider.gameObject.GetComponent<itemObj>().Id)
                        {
                            Debug.Log("id = id");
                            inventory_item[k].Stack_Count += hit.collider.gameObject.GetComponent<itemObj>().Stack_Count;
                            Destroy(hit.collider.gameObject);
                            break;
                        }
                        else
                        {
                            Debug.Log("id != id");
                            inventory_item.Add(hit.collider.gameObject.GetComponent<itemObj>());
                            Destroy(hit.collider.gameObject);
                            break;
                        }

                    }

                }
                          else
                          {
                           
                            inventory_item.Add(hit.collider.gameObject.GetComponent<itemObj>());
                            Destroy(hit.collider.gameObject);
                          }
            }
        }

Проблема в том что мой код делает проверку только для первого объекта, всех следующих если они есть то не видит, и в место того что бы добавить к уже существующему количество единиц, добавляет как новый объект. Но циклом я перебираю элементы листа от 0 до inventory_item.Count.

[Friend123]

В чем проблема расставить брейкпоинты и отдебажить код? Явно где-то в коде ошибка

[Anonymyx]

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

inventory_item.Add(hit.collider.gameObject.GetComponent<itemObj>());
Destroy(hit.collider.gameObject);
 

Вы добавляете объект в лист, и сразу уничтожаете его. В листе его не будет.

[vovaU]

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

inventory_item.Add(hit.collider.gameObject.GetComponent<itemObj>());
Destroy(hit.collider.gameObject);
 

Вы добавляете объект в лист, и сразу уничтожаете его. В листе его не будет.

Я добавляю скрипт который прикреплен к объекту. А как тогда правильно сделать добавление элементов с сцены их удаление после. Я в начале добавлял из другого листа в котором хранились все предметы, это не годится.

[Anonymyx]

Самое верное решение-просто запомнить ID объекта, и удалить. У вас же есть какая-то "база данных", массив (в идеале словарь <ItemObj/ID>, в котором хранятся все объекты, раз у вас есть параметр ID. Вот по ID и будете взаимодействовать.
А еще можно не уничтожать, а просто 'выключить', gameObject.SetActive(false);

[vovaU]

Самое верное решение-просто запомнить ID объекта, и удалить. У вас же есть какая-то "база данных", массив (в идеале словарь <ItemObj/ID>, в котором хранятся все объекты, раз у вас есть параметр ID. Вот по ID и будете взаимодействовать.
А еще можно не уничтожать, а просто 'выключить', gameObject.SetActive(false);

Я наверное нашел свою ошибку, у меня там цикл перебирает все объекты и если

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

inventory_item[k].Id == hit.collider.gameObject.GetComponent<itemObj>().Id

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

Синтаксис:

Синтаксис: [ Показать ]

Используется csharp

for (int k = 0; k < inventory_item.Count; k++)
                    {
                       
                        if (inventory_item[k].Id != hit.collider.gameObject.GetComponent<itemObj>().Id)
                        {
                            Debug.Log("k = " + k);
                            inventory_item.Add(hit.collider.GetComponent<itemObj>());
                            Cell_id.Add(hit.collider.GetComponent<itemObj>().Id);
                            Destroy(hit.collider.gameObject);
                            break;
                        }


                    }

Но он также сравнивается не со всеми объектами листа, а только с первым.Получается что, цикл запустился и сравнивает первый элемент с элементом Х, они не совпадают и цикл остановился, а у меня второй элемент такой же самый как Х и если бы сравнился со вторым то условие в if не исполнилось. Как тогда правильно перебрать все элементы и сравнить с элементом Х к примеру, и если нет совпадений то выполнить следующие действия.

[seaman]

Уберите break из нужной ветки