修改字典中的结构变量

Question

我有一个这样的结构：

public struct MapTile
{
    public int bgAnimation;
    public int bgFrame;
}

但是当我用 foreach 循环它来改变动画帧时，我做不到......

代码如下：

foreach (KeyValuePair<string, MapTile> tile in tilesData)
{
        if (tilesData[tile.Key].bgFrame >= tilesData[tile.Key].bgAnimation)
        {
            tilesData[tile.Key].bgFrame = 0;
        }
        else
        {
            tilesData[tile.Key].bgFrame++;
        }
}

它给了我编译错误：

Error 1 Cannot modify the return value of 'System.Collections.Generic.Dictionary<string,Warudo.MapTile>.this[string]' because it is not a variable
Error 2 Cannot modify the return value of 'System.Collections.Generic.Dictionary<string,Warudo.MapTile>.this[string]' because it is not a variable

为什么我不能更改字典中的结构中的值？

Answer 1

机会结构到类

之前：

public struct MapTile
{
    public int bgAnimation;
    public int bgFrame;
}

之后：

public Class MapTile
{
    public int bgAnimation;
    public int bgFrame;
}

Answer 2

我建议创建一个实用程序类：

public class MutableHolder<T>
{
    public T Value;
    public MutableHolder(T value)
    {
        this.Value = value;
    }
}

然后将新创建的MutableHolder<MapTile> 存储到每个字典槽中，而不是直接存储MapTile。这将允许您轻松更新与任何特定键关联的地图图块，而无需修改字典本身（否则，至少会使 foreach 循环使用的枚举器无效）。

Answer 3

tilesData[tile.Key] 不是存储位置（即，它不是变量）。它是与字典 tilesData 中的键 tile.Key 关联的 MapTile 实例的副本。这就是struct 发生的情况。它们的实例的副本到处传递并返回（这也是为什么可变结构被认为是邪恶的很大一部分）。

你需要做的是：

    MapTile tile = tilesData[tile.Key];
    if (tile.bgFrame >= tile.bgAnimation)
    {
        tile.bgFrame = 0;
    }
    else
    {
        tile.bgFrame++;
    }
    tilesData[tile.Key] = tile;

Answer 4

索引器将返回该值的副本。对该副本进行更改不会对字典中的值做任何事情...编译器阻止您编写错误代码。如果你想修改字典中的值，你需要使用类似的东西：

// Note: copying the contents to start with as you can't modify a collection
// while iterating over it
foreach (KeyValuePair<string, MapTile> pair in tilesData.ToList())
{
    MapTile tile = pair.Value;
    tile.bgFrame = tile.bgFrame >= tile.bgAnimation ? 0 : tile.bgFrame + 1;
    tilesData[pair.Key] = tile;
}

请注意，这也是避免无缘无故进行多次查找，您的原始代码正在这样做。

我个人强烈建议反对以可变结构开始，请注意...

当然，另一种选择是使其成为引用类型，此时您可以使用：

// If MapTile is a reference type...
// No need to copy anything this time; we're not changing the value in the
// dictionary, which is just a reference. Also, we don't care about the
// key this time.
foreach (MapTile tile in tilesData.Values)
{
    tile.bgFrame = tile.bgFrame >= tile.bgAnimation ? 0 : tile.bgFrame + 1;
}