C# Lambda：操作 List<T> 中的单个项目

Question

我在做什么

我正在为我的 Unity 2D RPG 游戏开发一些原型系统（库存、设备……）。我遇到了玩家持有的List<Stat> stats 的问题。

我要做什么

如果玩家捡起一件物品，它会直接装备（目前）。该项目可以包含一个List<StatModifier> modifiers，它被添加到PlayerStats。每当装备一个物品时，就会触发以下委托，这是EquipmentSystem 类（单例）的一部分。

public delegate void OnEquipmentChanged(EquipableItem newItem, EquipableItem oldItem);
public OnEquipmentChanged onEquipmentChanged;

PlayerStats 类在创建时订阅此委托。

// Start is called before the first frame update
void Start()
{
    List<StatModifier> empty = new List<StatModifier>();

    // Initialize Stats
    health = new Stat(100, empty, StatType.HP);
    mana = new Stat(200, empty, StatType.MANA);

    // Add stats to list
    stats = new List<Stat>();
    stats.Add(health);
    stats.Add(mana);

    // Subscribe new Function
    EquipmentSystem.instance.onEquipmentChanged += OnEquipmentChanged; 
}

OnEquipmentChanged()-方法然后通过从旧项目中删除 StatModifiers 并从新项目中添加 StatModifiers 来重新计算玩家的统计数据。这些统计信息存储在PlayerStats 类中的private List<Stat> stats 中。

我的问题是什么

OnEquipmentChanged()-方法被正确调用，但它没有按预期工作。仅当StatType 与Stat 的StatType 相同时，才应添加项目的StatModifiers。所以HP应该只加到HP上，Mana只加到Mana上等等。 StatType 是一个公共枚举顺便说一句。但该功能目前将所有StatModifiers 添加到玩家拥有的所有Stats 中。函数如下：

void OnEquipmentChanged(EquipableItem newItem, EquipableItem oldItem)
{
    if (oldItem != null) { //remove modifiers }
    if (newItem != null)
    {
        // add modifiers of new item if new item exists
        foreach (StatModifier item_sm in newItem.GetStatModifiers())
        {                
            int index = stats.FindIndex(stat => stat.GetStatType() == item_sm.GetStatType());

            Debug.Log("Found stat " + stats[index].GetStatType());

            stats[index].AddModifier(item_sm);
            Debug.Log("Added " + item_sm.GetModifierValue() + " " + (StatType)item_sm.GetStatType() + " to " + stats[index].GetStatType());
        }
    }
}

设置断点显示，在第一次迭代之后，StatModifier 已经被添加到 Stats 中。这是为什么呢？

编辑 1：这是用于将 StatModifier 添加到任何 Stat 的（非常简单的）代码。每个Stat 都有自己的StatModifier 私有列表。

public void AddModifier(StatModifier modifier)
{
    if (modifier != null)
    {
        modifiers.Add(modifier);
    }
}

谢谢！

Answer 1

你的问题出在这一行：

stats[index].AddModifier(item_sm);

无论你有什么AddModifier 的实现，它都做错了。

虽然整个代码是不必要的复杂和脆弱，为什么你必须用FindIndex 找到代表不同统计数据的相同对象列表？这就是字典的用途。事实上，这就是单独变量的用途，它会让您的代码更加高效。

另一个真正的大问题是你不能像这样装备物品，首先移除你替换的物品的效果，然后添加新物品。考虑一下这里的操作顺序，任何被删除的flat life stat 都会在你其他装备的life% stat 贡献之前被删除。正确的方法是保留每个修饰符的列表，无论其来源如何，并在需要时将其全部检查，以正确的顺序总结正确的内容，然后使用统计数据。

Answer 2

我会采取不同的方法，我不认为 lambda 是要走的路。这是伪代码我没有你所有的类代码，所以我不能完全为你编写代码，但我会做这样的事情。

foreach (StatModifier item_sm in newItem.GetStatModifiers())
{           
     foreach (Stat stat in stats) {
          if (item_sm.GetStatType() == stat.statType) {
          // remove previous stat from a previous equipped item
          stat.AddModifier(item_sm);
          }
     }
}