alguem pode me ajudar com essa adição de item numa lista?

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using System.Linq;

public class InventoryController : MonoBehaviour
{
   [SerializeField]
   Item[] arrayInventory;

   public List<Item>_Inventory { get; private set; }

   public void Awake()
   {
      _Inventory = new List<Item>();
      _Inventory = arrayInventory.OrderBy(i => i.Name).ToList();
      MouseSensitive.IsClicked += RemoveItem;
   }

   private void OnDestroy()
   {
      MouseSensitive.IsClicked -= RemoveItem;
   }

   public void AddItem(Item item)
   {
      if (item != null)
         _Inventory.Add (item);
   }

   public void RemoveItem(Item item)
   {
      if (item != null)
         _Inventory.Remove (item);
   }

}


Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class InteragirObjList : MonoBehaviour
{
   public Transform jogador;
   public ScriptableObject ItemDate;
   public InventoryController _Inventory;
   public KeyCode TeclaPegar = KeyCode.P;
   [Range(1,5)]
   public float distanciaMinima = 2;

   void Awake ()
   {
      _Inventory = GetComponent<InventoryController>();
   }

   void Update ()
   {
      if (jogador)
      {
         {
            if (Input.GetKeyDown (TeclaPegar))
            {
               ItemDate += _Inventory.AddItem(item) as ScriptableObject ;
            }
         }
      }
   }
}



dstaroski escreveu:Boa tarde meu caro!

O que exatamente está ocorrendo? qual seria o erro? também está faltando postar uma classe, o Item.

Abraço!

dstaroski escreveu:Boa tarde meu caro!

O que exatamente está ocorrendo? qual seria o erro? também está faltando postar uma classe, o Item.

Abraço!

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using System.Linq;

[CreateAssetMenu (fileName = "New Item", menuName = "Create Item")]
public class Item : ScriptableObject
{
   public GameObject SpawnOBJ;
   public Sprite Icon;
   public string Name;
   public string Description;
   public int ID { get; private set; }
   public int Count { get {
         return
         FindObjectOfType<InventoryController>()._Inventory.FindAll(x => x.ID == this.ID).Count;

         }
   }

   private void OnEnable() =>
   ID = this.GetInstanceID();
}


Código:

ItemDate += _Inventory.AddItem(item) as ScriptableObject;

Código:

public Item ItemDate;

Código:

_Inventory.AddItem(ItemDate);


Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class InteragirObjList : MonoBehaviour
{
  public Transform jogador;
  public Item ItemDate;
  public InventoryController _Inventory;
  public KeyCode TeclaPegar = KeyCode.P;
  [Range(1,5)]
  public float distanciaMinima = 2;

  void Awake ()
  {
      _Inventory = GetComponent<InventoryController>();
  }

  void Update ()
  {
      if (jogador)
      {
        {
            if (Input.GetKeyDown (TeclaPegar))
            {
              _Inventory.AddItem(ItemDate);
            }
        }
      }
  }
}

Pokedlg escreveu:Explicação:

Acredito que você tenha errado na classe InteragirObjList na variavel ItemDate, a qual voce deixou com ScriptableObject (pelo que eu entendi o ItemDate seria o objeto a pegar?) e na hora de adicionar o item na linha 26:

Código:

ItemDate += _Inventory.AddItem(item) as ScriptableObject;

Em relação ao ItemDate você teria que colocar a variável como Item, pois ele é uma Classe derivada do ScriptableObject e não a classe ScriptableObject em si. Tente deixar assim:

Código:

public Item ItemDate;

e na linha 26 voce coloca um "+=" para adicionar a lista? Não é assim que se faz a adição a uma lista. Como no outro script você já criou uma void responsável por fazer esta adição na classe InventoryController, tente deixar a linha 26  assim:

Código:

_Inventory.AddItem(ItemDate);


Script Completo com as modificações:

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class InteragirObjList : MonoBehaviour
{
   public Transform jogador;
   public Item ItemDate;
   public InventoryController _Inventory;
   public KeyCode TeclaPegar = KeyCode.P;
   [Range(1,5)]
   public float distanciaMinima = 2;

   void Awake ()
   {
      _Inventory = GetComponent<InventoryController>();
   }

   void Update ()
   {
      if (jogador)
      {
         {
            if (Input.GetKeyDown (TeclaPegar))
            {
               _Inventory.AddItem(ItemDate);
            }
         }
      }
   }
}

Pokedlg escreveu:Voce nao definiu o ItemDate, ele é nulo, voce não atribui nenhum valor ao ItemDate, voce tem que colocar que ele é o item mais próximo

Pokedlg escreveu:Fica um pouco dificil sem saber as circunstancias, seu jogo é 3d ou 2d? Voce ja fez o sistema para poder pegar o item, pois sem ele nao tem como voce pegar para poder adicioná-lo no inventário. Porque, vejo que voce tem uma variavel representando a distancia minima, mas nao faz uso dela. o ItemDate seria o item mais perto mesmo ou representa outra coisa?

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class InteragirObjList : MonoBehaviour
{
   public Transform jogador;
   public Item ItemDate;
   public InventoryController _Inventory;
   public KeyCode TeclaPegar = KeyCode.P;
   [Range(1,5)]
   public float distanciaMinima = 2;

   void Awake ()
   {
      _Inventory = GetComponent<InventoryController>();
   }

   void Update ()
   {
      float dist = Vector3.Distance(jogador.position, this.gameObject.position);
      if (jogador)
      {
         {
            if (dist <= distanciaMinima && Input.GetKeyDown (TeclaPegar))
            {
               _Inventory.AddItem(ItemDate);
            }
         }
      }
   }
}

Pokedlg escreveu:Tente assim:

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class InteragirObjList : MonoBehaviour
{
   public Transform jogador;
   public Item ItemDate;
   public InventoryController _Inventory;
   public KeyCode TeclaPegar = KeyCode.P;
   [Range(1,5)]
   public float distanciaMinima = 2;

   void Awake ()
   {
      _Inventory = GetComponent<InventoryController>();
   }

   void Update ()
   {
      float dist = Vector3.Distance(jogador.position, this.gameObject.position);
      if (jogador)
      {
         {
            if (dist <= distanciaMinima && Input.GetKeyDown (TeclaPegar))
            {
               _Inventory.AddItem(ItemDate);
            }
         }
      }
   }
}

Mas lembre-se de atribuir um scriptableobject para o itemDate, por exemplo se o objeto pegável é uma espada, atribui a ele o scriptable object da espada

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class InteragirObjList : MonoBehaviour
{
  public Transform jogador;
  public Item ItemDate;
  public InventoryController _Inventory;
  public KeyCode TeclaPegar = KeyCode.P;
  [Range(1,5)]
  public float distanciaMinima = 2;

  void Awake ()
  {
      _Inventory = GetComponent<InventoryController>();
  }

  void Update ()
  {
      float dist = Vector3.Distance(jogador.position, this.gameObject.transform.position);
      if (jogador)
      {
        {
            if (dist <= distanciaMinima && Input.GetKeyDown (TeclaPegar))
            {
              _Inventory.AddItem(ItemDate);
            }
        }
      }
  }
}

Pokedlg escreveu:Opa, erro meu, esqueci do transform:

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class InteragirObjList : MonoBehaviour
{
   public Transform jogador;
   public Item ItemDate;
   public InventoryController _Inventory;
   public KeyCode TeclaPegar = KeyCode.P;
   [Range(1,5)]
   public float distanciaMinima = 2;

   void Awake ()
   {
      _Inventory = GetComponent<InventoryController>();
   }

   void Update ()
   {
      float dist = Vector3.Distance(jogador.position, this.gameObject.transform.position);
      if (jogador)
      {
         {
            if (dist <= distanciaMinima && Input.GetKeyDown (TeclaPegar))
            {
               _Inventory.AddItem(ItemDate);
            }
         }
      }
   }
}

Pokedlg escreveu:Cara é o que eu tinha dito anteriormente, ou é o ItemDate ou o _Inventory é nulo, você atribuiu o scriptableobject na variavel ItemDate de acordo com o item que possui esse script ? o Gameobject que possui esse script contém a classe InventoryController?

Código:

_Inventory = GetComponent<InventoryController>();

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class InteragirObjList : MonoBehaviour
{
  public Transform jogador;
  public Item ItemDate;
  public InventoryController _Inventory;
  public KeyCode TeclaPegar = KeyCode.P;
  [Range(1,5)]
  public float distanciaMinima = 2;

  void Awake ()
  {
      _Inventory = FindObjectOfType<InventoryController>();
  }

  void Update ()
  {
      float dist = Vector3.Distance(jogador.position, this.gameObject.transform.position);
      if (jogador)
      {
        {
            if (dist <= distanciaMinima && Input.GetKeyDown (TeclaPegar))
            {
              _Inventory.AddItem(ItemDate);
            }
        }
      }
  }
}       


Pokedlg escreveu:Entao é isso que está causando o problema, voce usa:

Código:

_Inventory = GetComponent<InventoryController>();

Isso faz com que ele procure o script InventoryController no gameobject que vai o InteragirObjList, tente assim:

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class InteragirObjList : MonoBehaviour
{
   public Transform jogador;
   public Item ItemDate;
   public InventoryController _Inventory;
   public KeyCode TeclaPegar = KeyCode.P;
   [Range(1,5)]
   public float distanciaMinima = 2;

   void Awake ()
   {
      _Inventory = FindObjectOfType<InventoryController>();
   }

   void Update ()
   {
      float dist = Vector3.Distance(jogador.position, this.gameObject.transform.position);
      if (jogador)
      {
         {
            if (dist <= distanciaMinima && Input.GetKeyDown (TeclaPegar))
            {
               _Inventory.AddItem(ItemDate);
            }
         }
      }
   }
}        


Pkneves escreveu:

Pokedlg escreveu:Entao é isso que está causando o problema, voce usa:

Código:

_Inventory = GetComponent<InventoryController>();

Isso faz com que ele procure o script InventoryController no gameobject que vai o InteragirObjList, tente assim:

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class InteragirObjList : MonoBehaviour
{
   public Transform jogador;
   public Item ItemDate;
   public InventoryController _Inventory;
   public KeyCode TeclaPegar = KeyCode.P;
   [Range(1,5)]
   public float distanciaMinima = 2;

   void Awake ()
   {
      _Inventory = FindObjectOfType<InventoryController>();
   }

   void Update ()
   {
      float dist = Vector3.Distance(jogador.position, this.gameObject.transform.position);
      if (jogador)
      {
         {
            if (dist <= distanciaMinima && Input.GetKeyDown (TeclaPegar))
            {
               _Inventory.AddItem(ItemDate);
            }
         }
      }
   }
}        


entendi Assim ele procura o script fora do gameobject... o erro sumiu porem msm assim o ItemDate nao é add ao array, ele na entra na lista

Pokedlg escreveu:Nao entendi muito bem, Nao aparece o item no inventário quando voce pega ele depois de abrir o inventario? é isso? Em que script colocou o instantiate dos itens no inventário?

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class SlotsConfig : MonoBehaviour
{
   public ItemReference _element;

   private List<Item> _inventory;

    void Start()
    {
      _inventory = new List<Item> ();
      _inventory = FindObjectOfType<InventoryController> ()._Inventory;
      InstantiateElements ();
   }

   private void InstantiateElements()                                                                               
   {
      for (int i = 0; i < _inventory.Count; i++)
      {
         if (IsRepeated (i))
            continue;

         (Instantiate(_element, transform) as ItemReference).SetValues(_inventory [i]);
      }
   }

   bool IsRepeated(int i)
   {
      if (i == 0)
         return false;

      return _inventory[i].ID == _inventory[i - 1].ID;
   }
}


Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using System.Linq;

public class InventoryController : MonoBehaviour
{
  
   public Item[] arrayInventory; //deixe-a publica para podermos acessa-la no outro script
  
   public int inventoryLimit; //no inspector defina um valor limite para que funcione

   public List<Item>_Inventory { get; private set; }

   public void Awake()
   {
      _Inventory = new List<Item>();
      arrayInventory = new Item[inventoryLimit];
      _Inventory = arrayInventory.OrderBy(i => i.Name).ToList();
      MouseSensitive.IsClicked += RemoveItem;
   }

   private void OnDestroy()
   {
      MouseSensitive.IsClicked -= RemoveItem;
   }

   public void AddItem(Item item)
   {
      if (item != null)
         _Inventory.Add (item);
   }

   public void RemoveItem(Item item)
   {
      if (item != null)
         _Inventory.Remove (item);
   }

}

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class InteragirObjList : MonoBehaviour
{
   public Transform jogador;
   public Item ItemDate;
   public InventoryController _Inventory;
   public KeyCode TeclaPegar = KeyCode.P;
   [Range(1,5)]
   public float distanciaMinima = 2;

   void Awake ()
   {
      _Inventory = FindObjectOfType<InventoryController>();
   }

   void Update ()
   {
      float dist = Vector3.Distance(jogador.position, this.gameObject.transform.position);
      if (jogador)
      {
         {
            if (dist <= distanciaMinima && Input.GetKeyDown (TeclaPegar))
            {
               for(int i =0; i<_Inventory.arrayInventory.Length; i++)
              {
                  if(_Inventory.arrayInventory[i] == null)
                  {
                      _Inventory.arrayInventory[i] = ItemDate;
                      break;
                  }
            }
               _Inventory.AddItem(ItemDate);
            }
         }
      }
   }
}        

Pokedlg escreveu:Não entendo como isso acontece sendo que você só faz utilização da arrayInventory na void Awake, quando inicia o jogo, quando ainda não pegou nenhum item, depois não vejo ela sendo utilizada mais em nada que possa ser a causa do problema, Mas caso for isso tente adicionar à array um limite que seria o limite de slots para o inventário e então adicione os itens a array com um loop for:

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using System.Linq;

public class InventoryController : MonoBehaviour
{
  
   public Item[] arrayInventory; //deixe-a publica para podermos acessa-la no outro script
  
   public int inventoryLimit; //no inspector defina um valor limite para que funcione

   public List<Item>_Inventory { get; private set; }

   public void Awake()
   {
      _Inventory = new List<Item>();
      arrayInventory = new Item[inventoryLimit];
      _Inventory = arrayInventory.OrderBy(i => i.Name).ToList();
      MouseSensitive.IsClicked += RemoveItem;
   }

   private void OnDestroy()
   {
      MouseSensitive.IsClicked -= RemoveItem;
   }

   public void AddItem(Item item)
   {
      if (item != null)
         _Inventory.Add (item);
   }

   public void RemoveItem(Item item)
   {
      if (item != null)
         _Inventory.Remove (item);
   }

}

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class InteragirObjList : MonoBehaviour
{
   public Transform jogador;
   public Item ItemDate;
   public InventoryController _Inventory;
   public KeyCode TeclaPegar = KeyCode.P;
   [Range(1,5)]
   public float distanciaMinima = 2;

   void Awake ()
   {
      _Inventory = FindObjectOfType<InventoryController>();
   }

   void Update ()
   {
      float dist = Vector3.Distance(jogador.position, this.gameObject.transform.position);
      if (jogador)
      {
         {
            if (dist <= distanciaMinima && Input.GetKeyDown (TeclaPegar))
            {
               for(int i =0; i<_Inventory.arrayInventory.Length; i++)
               {
                   if(_Inventory.arrayInventory[i] == null)
                   {
                       _Inventory.arrayInventory[i] = ItemDate;
                       break;
                   }
            }
               _Inventory.AddItem(ItemDate);
            }
         }
      }
   }
}        

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using System.Linq;

public class InventoryController : MonoBehaviour
{
 
  public Item[] arrayInventory; //deixe-a publica para podermos acessa-la no outro script
 
  public int inventoryLimit; //no inspector defina um valor limite para que funcione

  public List<Item>_Inventory { get; private set; }

  public void Awake()
  {
      _Inventory = new List<Item>();
      arrayInventory = new Item[inventoryLimit];
      MouseSensitive.IsClicked += RemoveItem;
  }

  private void OnDestroy()
  {
      MouseSensitive.IsClicked -= RemoveItem;
  }

  public void AddItem(Item item)
  {
      if (item != null)
        _Inventory.Add (item);
  }

  public void RemoveItem(Item item)
  {
      if (item != null)
        _Inventory.Remove (item);
  }

}

Pokedlg escreveu:Tenta excluir a linha 19, pra ver se resolve este erro.

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using System.Linq;

public class InventoryController : MonoBehaviour
{
  
   public Item[] arrayInventory; //deixe-a publica para podermos acessa-la no outro script
  
   public int inventoryLimit; //no inspector defina um valor limite para que funcione

   public List<Item>_Inventory { get; private set; }

   public void Awake()
   {
      _Inventory = new List<Item>();
      arrayInventory = new Item[inventoryLimit];
      MouseSensitive.IsClicked += RemoveItem;
   }

   private void OnDestroy()
   {
      MouseSensitive.IsClicked -= RemoveItem;
   }

   public void AddItem(Item item)
   {
      if (item != null)
         _Inventory.Add (item);
   }

   public void RemoveItem(Item item)
   {
      if (item != null)
         _Inventory.Remove (item);
   }

}

Pkneves escreveu:

Pokedlg escreveu:Tenta excluir a linha 19, pra ver se resolve este erro.

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using System.Linq;

public class InventoryController : MonoBehaviour
{
  
   public Item[] arrayInventory; //deixe-a publica para podermos acessa-la no outro script
  
   public int inventoryLimit; //no inspector defina um valor limite para que funcione

   public List<Item>_Inventory { get; private set; }

   public void Awake()
   {
      _Inventory = new List<Item>();
      arrayInventory = new Item[inventoryLimit];
      MouseSensitive.IsClicked += RemoveItem;
   }

   private void OnDestroy()
   {
      MouseSensitive.IsClicked -= RemoveItem;
   }

   public void AddItem(Item item)
   {
      if (item != null)
         _Inventory.Add (item);
   }

   public void RemoveItem(Item item)
   {
      if (item != null)
         _Inventory.Remove (item);
   }

}

O erro sumiu, porém o bagu permanece na instância do item no Scroll Viewr...
Hierarquia do seria Canvas > PainelINVENTORIO> Scroll View> Content ... No caso ele instancia  dentro do Content!

Código:

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class SlotsConfig : MonoBehaviour
{
  public ItemReference _element;

  private List<Item> _inventory;

    void Start()
    {
      _inventory = new List<Item> ();
      _inventory = FindObjectOfType<InventoryController> ()._Inventory;
  }

  public static void InstantiateElements()                                                                               
  {
      for (int i = 0; i < _inventory.Count; i++)
      {
        if (IsRepeated (i))
            continue;

        (Instantiate(_element, transform) as ItemReference).SetValues(_inventory [i]);
      }
  }

  bool IsRepeated(int i)
  {
      if (i == 0)
        return false;

      return _inventory[i].ID == _inventory[i - 1].ID;
  }
}

Código:

SlotsConfig.InstantiateElements();