LINQ 语句在包装到扩展方法中后不再起作用

Question

我需要一种可以获取字符串集合并用另一个字符串替换所有出现的特定字符串的方法。

例如，如果我有一个看起来像这样的 List：

List<string> strings = new List<string> { "a", "b", "delete", "c", "d", "delete" };

并且我想将“delete”替换为“”，我会使用这个 LINQ 语句：

strings = (from s in strings select (s=="delete" ? s=String.Empty : s)).ToList();

它效果很好。但后来我想我应该将其作为扩展方法，因为我以后可能会再次使用它。在这种情况下，我只想编写以下内容：

strings.ReplaceStringInListWithAnother( "delete", String.Empty);

当我的代码编译并且 LINQ 语句在扩展方法内部工作时，当我返回集合时，它会恢复到其原始内容：

public static void ReplaceStringInListWithAnother( this List<string> my_list, string to_replace, string replace_with)
{
    my_list = (from s in my_list select (s==to_replace ? s=replace_with : s)).ToList();
}

所以看起来我只是修改了一个副本列表的...但是当我查看 Pop 的代码时，它类似地修改了集合，但更改仍然存在，所以我的假设是我的方法的参数声明是正确的。

谁能解释我在这里做错了什么？

Answer 1

这里有一个提示：您希望下面的代码做什么？

void SetToTen(int y)
{
    y = 10;
}

int x = 0;
SetToTen(x);

希望您明白上面的 SetToTen 方法没有任何意义，因为它仅更改其自己的局部变量 y 的值，并且对传递值的变量没有影响（为了实现这一点，y 参数必须是 ref int 类型，并且该方法将被调用为 SetToTen(ref x) ）。

请记住，扩展方法实际上只是华丽服装中的静态方法，应该清楚为什么您的 ReplaceStringInListWithAnother 没有执行您期望的操作：它只是设置其本地 < code>my_list 变量为新值，对传递给该方法的原始 List 没有影响。

现在，值得一提的是，这对您不起作用的唯一原因是您的代码通过将变量设置为新对象*来工作。如果您要修改传递给ReplaceStringInListWithAnother的List，一切都会正常工作：

public static void ReplaceStringInListWithAnother( this List<string> my_list, string to_replace, string replace_with)
{
    for (int i = 0; i < my_list.Count; ++i)
    {
        if (my_list[i] == to_replace)
        {
            my_list[i] = replace_with;
        }
    }
}

也是值得一提的是 List 对于该方法来说是一个过于严格的参数类型；您可以为任何实现 IList 的类型实现相同的功能（因此我将 my_list 参数更改为 IList< 类型/代码>）。

---

*再次阅读您的问题，我似乎清楚这是您感到困惑的主要点。您必须意识到的重要一点是，默认情况下，C# 中的所有内容都是按值传递的。对于值类型（定义为 struct 的任何内容 - int、double、DateTime 等等）， 传递的东西是值本身。对于引用类型（定义为类的任何内容），传递的内容是对对象的引用。在后一种情况下，对可变类型对象的引用的所有方法调用实际上都会影响底层对象，因为引用类型的多个变量可以指向同一个对象。但赋值与方法调用不同；如果您将一个引用分配给一个已通过值传递给某个对象的新引用的对象，则您不会对底层对象执行任何操作，因此不会发生原始引用所反映的任何事情。

这是一个非常重要的概念，许多 .NET 开发人员都在努力解决这个问题。但这也是一个在其他地方已经得到彻底解释的话题。如果您需要更多解释，请告诉我，我将尝试找到一个页面链接，使所有这些内容尽可能清晰。

Answer 2

您还没有显示“Pop”的代码，因此很难知道您的意思。您谈论“当我返回集合时”，但您没有返回任何内容 - 该方法具有 void 返回类型。

LINQ 通常不会更改现有集合的内容。通常，您应该从扩展方法返回一个新集合。例如：（

public static IEnumerable<string> ReplaceAll
    (this IEnumerable<string> myList, string toReplace, string replaceWith)
{
    return toReplace.Select(x => x == toReplace ? replaceWith : x);
}

我在这里使它更通用 - 除非确实需要，否则不应该开始具体化列表。）

然后您可以使用以下方式调用它：

strings = strings.ReplaceAll("delete", "").ToList();

... 或更改 string 到 IEnumerable 并使用

strings = strings.ReplaceAll("delete", "");

Answer 3

您编写的 LINQ 语句不会修改集合，它实际上创建一个新集合。

您编写的扩展方法创建了这个新集合，然后丢弃它。该分配是多余的：您正在分配一个本地参数，该参数在分配后立即超出范围。

当您调用该方法时，您还会丢弃其结果而不是将其分配回来。

因此，你应该这样编写方法：

public static List<string> ReplaceStringInListWithAnother(
    this List<string> my_list, string to_replace, string replace_with)
{
    return (from s in my_list select
        (s == to_replace ? replace_with : s)).ToList();
}

和这样的调用：

strings = strings.ReplaceStringInListWithAnother("delete", "");

---

顺便说一句，你可以通过使其通用来使函数更有用：

public static List<T> ReplaceInList<T>(this List<T> my_list,
    T to_replace, T replace_with) where T : IEquatable<T>
{
    return (from s in my_list select
        (s.Equals(to_replace) ? replace_with : s)).ToList();
}

这样你就可以将它用于其他用途，而不仅仅是 strings.此外，您还可以声明它使用 IEnumerable 而不是 List：

public static IEnumerable<T> ReplaceItems<T>(this IEnumerable<T> my_list,
    T to_replace, T replace_with) where T : IEquatable<T>
{
    return from s in my_list select (s.Equals(to_replace) ? replace_with : s);
}

这样您就可以将它用于任何可相等项的集合，而不仅仅是 <代码>列表。请注意，List 实现了 IEnumerable，因此您仍然可以将 List 传递到此函数中。如果您想要输出一个列表，只需在调用此列表后调用 .ToList() 即可。

更新：如果您确实想要替换列表中的元素而不是创建新元素，您仍然可以使用扩展方法来实现这一点，并且它仍然可以是通用的，但你不能使用 Linq，也不能使用 IEnumerable：

public static void ReplaceInList<T>(this List<T> my_list,
    T to_replace, T replace_with) where T : IEquatable<T>
{
    for (int i = 0; i < my_list.Count; i++)
        if (my_list[i].Equals(to_replace))
            my_list[i] = replace_with;
}

这不会返回新列表，而是修改旧列表，因此它有一个 void 返回类型与原始类型相同。