C#，ArrayList 属性到列表字段

Question

我有以下 Field:

private IList<Modulo> modulos;

和以下 Property:

    public virtual ArrayList XmlModulos
    {
        get
        {
            if (modulos == null) return new ArrayList();
            var aux = new ArrayList();
            foreach (Modulo m in modulos)
                aux.Add(m);
            return aux;
        }
        set
        {
            modulos = new List<Modulo>();
            foreach (object o in value)
                modulos.Add((Modulo)o);
        }

    }

get 工作完美，因为我有另一个用于不同目的的直接属性。但该集不起作用，因为当我使用此属性时，它不会更新任何内容。调试表明该属性是一个空的 ArrayList（计数为 0）并且该字段保持为 null。知道为什么这不起作用吗？任何帮助将不胜感激。

Answer 1

要理解这里发生的情况，您必须认识到 .NET 属性只是表示两个单独的方法的便捷方式，一个用于获取，另一个用于设置。

也就是说，它们基本上将这些操作映射

ArrayList m = obj.XmlModulos;

obj.XmlModulos = new ArrayList();

到实际上更类似于此的代码：

ArrayList m = obj.get_XmlModulos();

obj.set_XmlModulos(new ArrayList());

好的，考虑到这一点，让我们考虑一下您要做什么（这是基于对 Timwi 的回答）：

obj.XmlModulos.Add(new Modulo());

这真正所做的是：

obj.get_XmlModulos().Add(new Modulo());

考虑当您的 get 访问器创建一个新的 ArrayList 时，您永远无法将 Modulo 添加到您的以这种方式收集。唯一可以做到的方法是：

ArrayList m = obj.XmlModulos;

m.Add(new Modulo());

obj.XmlModulos = m;

但是，这是实现 get 和 set 访问器的一种非常低效的方法。在每个 get 上创建一个新的 ArrayList 并用 modulos 的当前成员填充它的成本很高。枚举传递给 set 的任何值并将每个值复制到 modulos 也是如此。

如果说有什么不同的话，那就是这个类的设计是倒退的（实际上，大多数开发人员现在都会建议完全避开 ArrayList 类型；但如果您坚持的话，那是您的决定）。您在内部存储一个IList（提供类型安全），但在外部暴露一个ArrayList（它不）；这意味着有人可以轻松地尝试向您的列表中添加一些非Modulo 类型的随机对象，并且您会得到一个InvalidCastException。尝试执行此操作的代码实际上根本不应该编译；这就是泛型的全部意义。

好吧，我已经说够了。我有什么建议？只有一个属于此类的集合。确定您实际想要提供什么功能，并仅公开该功能。不要在两个集合之间来回进行这种昂贵的复制。

例如，如果您确实需要一个ArrayList，但您只需要启用客户端代码向其中添加和删除元素，然后按如下方式实现它：

private ArrayList modulos;

public void AddModulo(Modulo m)
{
    modulos.Add(m);
}

public bool RemoveModulo(Modulo m)
{
    return modulos.Remove(m);
}

这样，即使您正在使用内部非泛型 ArrayList（如果您有充分的理由这样做），您仍然可以通过仅允许客户端代码来获得 IList 的有效类型安全性添加和删​​除 Modulo 类型的项目。

这可能与对您实际有意义的解决方案有很大不同。至少希望它能够引导您走向正确的方向。

Answer 2

我无法重现您的问题；我复制了您的代码，然后执行了这一行：

XmlModulos = new ArrayList { new Modulo() };

此后，调试器和 Console.WriteLine 都报告 modulos.Count 和 XmlModulos.Count如1。

因此，错误在于您未向我们展示的其他一些代码。