Unity Inventory System故障：物品无法叠加且显示为None(Item)

问题描述 #

• 实现的AddItem方法无法叠加已有物品数量，existingItem始终为空，每次添加都会生成新条目
• 编辑器窗口中物品显示为None(Item)，但控制台通过ShowInventory方法能正常输出物品名称与数量

核心问题分析与修复 #

1. 物品数据类错误继承MonoBehaviour #

你的Item类继承了MonoBehaviour，但库存物品属于纯数据模型，不需要作为Unity组件存在。直接用new Item()创建MonoBehaviour实例不符合Unity规范，会导致实例无法被编辑器正确识别（显示None(Item)），甚至引发数据存储异常。

修复方案：将Item改为普通可序列化类，移除MonoBehaviour继承：

[System.Serializable]
public class Item
{
    public string itemName;
    public int itemAmount;
}

同时，场景中的可拾取物品需要单独创建组件挂载数据：

public class ItemPickup : MonoBehaviour
{
    public Item itemData; // 在Inspector中赋值物品数据

    private void Start()
    {
        var spriteRenderer = GetComponent<SpriteRenderer>();
        var textMesh = GetComponentInChildren<TextMeshPro>();
        
        if(spriteRenderer != null) spriteRenderer.sprite = GetSprite(itemData.itemName);
        if(textMesh != null) textMesh.SetText(itemData.itemAmount.ToString());
    }

    public Sprite GetSprite(string itemName)
    {
        SpriteDatas spriteDatas = FindObjectOfType<SpriteDatas>();
        switch (itemName)
        {
            default:
            case "Sword": return spriteDatas.spritesList[0];
            case "HealthPotion": return spriteDatas.spritesList[1];
            case "ManaPotion": return spriteDatas.spritesList[2];
            case "Coin": return spriteDatas.spritesList[3];
            case "MedKit": return spriteDatas.spritesList[4];
        }
    }
}

2. AddItem方法无法匹配已有物品的问题 #

原代码中用new Item()创建的无效MonoBehaviour实例，可能导致Find方法无法正确匹配物品名称。改为纯数据类后，Find逻辑可正常工作，同时需要调整UI更新逻辑，避免重复创建条目。

修复后的AddItem方法：

// 在Inventory类中添加字典，关联物品与UI条目
private Dictionary<Item, RectTransform> itemToUISlot = new Dictionary<Item, RectTransform>();
float itemsSlotCellSize = 110f;
int x = 0;
int y = 0;

public void AddItem(string itemName1, int amount)
{
    Item existingItem = inventory.Find(item => item.itemName == itemName1);
    Debug.Log("existing item: "+ existingItem);

    if (existingItem != null)
    {
        existingItem.itemAmount += amount;
        // 叠加后更新对应UI条目
        UpdateInventoryItemUI(existingItem);
    }
    else
    {
        if (itemInInventoryCounter >= maxItemInInvetory) return;
        
        Item newItem = new Item();
        newItem.itemName = itemName1;
        newItem.itemAmount = amount;
        inventory.Add(newItem);
        itemInInventoryCounter++;
        // 新增物品时创建UI条目
        CreateInventoryItemUI(newItem);
    }
}

3. UI刷新逻辑重构 #

原RefreshInventoryItems每次调用都会实例化新UI Slot，未处理叠加场景，且未与库存数据关联。拆分UI逻辑为创建和更新两个方法：

public void CreateInventoryItemUI(Item item)
{
    RectTransform itemSlotRectTransform = Instantiate(itemSlotTemplate, itemSlotContainer).GetComponent<RectTransform>();
    itemSlotRectTransform.gameObject.SetActive(true);
    itemSlotRectTransform.anchoredPosition = new Vector2(x * itemsSlotCellSize, y * itemsSlotCellSize);
    
    // 设置图标与数量
    Image image = itemSlotRectTransform.Find("SlotItemPicture").GetComponent<Image>();       
    image.sprite = GetSprite(item.itemName);
    TextMeshProUGUI uiText = itemSlotRectTransform.Find("amountText").GetComponent<TextMeshProUGUI>();
    uiText.SetText(item.itemAmount.ToString());
    
    // 记录物品与UI的关联
    itemToUISlot.Add(item, itemSlotRectTransform);
    
    // 更新网格位置
    x++;
    if (x > 3)
    {
        x = 0;
        y--;
    }
}

public void UpdateInventoryItemUI(Item item)
{
    if(itemToUISlot.TryGetValue(item, out var slot))
    {
        TextMeshProUGUI uiText = slot.Find("amountText").GetComponent<TextMeshProUGUI>();
        uiText.SetText(item.itemAmount.ToString());
    }
}

// 统一图标获取方法，移至Inventory类
private Sprite GetSprite(string itemName)
{
    switch (itemName)
    {
        default:
        case "Sword": return item.spriteDatas.spritesList[0];
        case "HealthPotion": return item.spriteDatas.spritesList[1];
        case "ManaPotion": return item.spriteDatas.spritesList[2];
        case "Coin": return item.spriteDatas.spritesList[3];
        case "MedKit": return item.spriteDatas.spritesList[4];
    }
}

4. 修正触发逻辑调用顺序 #

移除原OnTriggerEnter2D中提前调用的RefreshInventoryItems，改为由AddItem统一处理数据与UI更新：

private void OnTriggerEnter2D(Collider2D collision)
{
    if (collision.CompareTag("Item"))
    {     
        ItemPickup itemPickup = collision.gameObject.GetComponent<ItemPickup>();
        if(itemPickup == null) return;

        string itemName = itemPickup.itemData.itemName;
        int itemAmount = itemPickup.itemData.itemAmount;                
        
        if (inventory.itemInInventoryCounter < inventory.maxItemInInvetory)
        {
            inventory.AddItem(itemName, itemAmount);
            Destroy(collision.gameObject);
        }
        else
        {
            Debug.Log("envanter dolu");
        }
    }
}

总结 #

1. 数据与组件分离：库存物品用纯数据类，场景拾取物用单独组件挂载数据
2. 禁止new创建MonoBehaviour实例：避免无效实例导致的编辑器显示异常与数据问题
3. 关联数据与UI：用字典记录物品与UI的对应关系，实现叠加时的UI动态更新
4. 统一逻辑入口：由AddItem方法处理数据修改与UI更新，避免重复创建UI条目

内容的提问来源于stack exchange，提问作者erdem.61