无法在 linq 语句中重新分配值 [重复]

Question

我正在尝试对 int 类型列表中的所有数字求平方。但是，当我运行代码时，列表似乎根本没有改变。

public static List<int> SquareList(List<int> root)
{
    root.ForEach(x => x = x*x);
    return root;
}

x_unit 中的测试类。

[Fact]
public void testSqures()
{
    var input = new List<int> { 1, 2, 3 };
    var expected = new List<int> { 1, 4, 9 };
    var result = Square.SquareList(input);

    Assert.Equal(expected,result);
}

问题foreach 操作符不会改变底层对象吗？

我需要创建一个新列表吗？还是有办法将列表转换到位？

Answer 1

改成

return root.Select(x => x*x).ToList();

这不会改变现有列表，它会创建一个新列表并返回它是首选的。如果您确实想更新原始列表，请更改签名以传入ref，然后更新指向新列表的指针。

public static List<int> SquareList(ref List<int> root)
{
    root = root.Select(x => x*x).ToList();
    return root;
}

或将其与 for 循环一起使用

public static List<int> SquareList(List<int> root)
{
    for(int i = 0; i < root.Count; i++)
        root[i] = root[i]*root[i];
    return root;
}

ForEach 确实会遍历列表，如果您有一个包含 int 的引用类型（类），您可以更改该类型内部的值，但您不能重新分配列表中包含的指针。

Answer 2

如果您想获得集合的新表示，请使用 Select 而不是 ForEach。返回新对象而不是更改它是函数式编程的一种范式。 LINQ 有助于使用这种范例。

Answer 3

来自 MSDN：

不支持修改 Action 委托主体中的基础集合，这会导致未定义的行为。

您应该只使用常规的for 循环来执行此类操作，而不是使用 LINQ 或类似的函数语句

public static List<int> SquareList(List<int> root)
{
    for (int i = 0, n = root.Count; i < n; ++i)
        root[i] = root[i] * root[i];
    return root;
}