C#方法返回List按引用而非值传递的问题及解决需求

问题根源 #

你遇到的问题不是列表的引用传递，而是Definition是引用类型。虽然你调用了ToList()创建了新的列表，但新列表里的每个元素依然指向原始definitions列表中的同一个Definition对象。所以你修改a.Name的时候，实际是修改了原始对象的属性，后续调用GetDefinitions()自然会看到被修改后的结果。

解决方案 #

1. 让GetDefinitions()返回深拷贝的列表 #

修改Context类的GetDefinitions()方法，每次返回时都创建新的Definition对象，彻底隔离原始数据：

public class Context
{
    private static readonly List<Definition> definitions;
    
    static Context()
    {
        definitions = LoadXML("path-to-xml-file.xml");
    }

    public static List<Definition> GetDefinitions()
    {
        // 对每个元素进行拷贝，返回全新的列表和对象
        return definitions.Select(d => new Definition 
        {
            Name = d.Name,
            Alias = d.Alias,
            // 这里要复制Definition的所有其他属性
        }).ToList();
    }
}

这样每次调用GetDefinitions()拿到的都是完全独立的对象，修改它们不会影响原始的静态列表。

2. 使用Select替代ForEach（避免修改原始数据） #

如果不想修改Context类，也可以在调用时通过Select()创建新对象，而非修改原有对象：

var defs = Context.GetDefinitions()
    .Select(d => new Definition 
    {
        Name = d.Alias ?? d.Name,
        Alias = d.Alias,
        // 复制其他需要保留的属性
    })
    .ToList();

这种方式不会改动原始列表里的任何对象，因为你是基于原始对象生成了新实例，修改的是新对象的属性。

3. 把Definition设为不可变类型（推荐） #

从根源杜绝误修改问题，让Definition的属性只读，只能通过构造函数初始化：

public class Definition
{
    public string Name { get; }
    public string Alias { get; }

    public Definition(string name, string alias)
    {
        Name = name;
        Alias = alias;
    }
}

这样你无法修改现有Definition对象的属性，必须通过创建新对象来得到修改后的数据，彻底避免误操作原始数据的情况。

内容的提问来源于stack exchange，提问作者peterpie