【发布时间】:2015-10-13 21:37:10
【问题描述】:
当我使用下面的方法时,当这个列表为空时,会创建列表并添加一条记录,但在方法之外列表仍然为空。
但是,如果列表已经创建,那么我会看到列表在方法之外增长。
public static void Add(this List<Setting> Settings, object Name, object Value)
{
if(null==Settings)
{
Settings = new List<Setting>();
}
var item = Settings.FirstOrDefault(s => s.Name.ToLower() == Name.ToString().ToLower());
if (null != item)
{
//overwrite the value of an existing item
item.Value = Value.ToString();
}
else
{
//Add the new Setting to the list
Settings.Add(new Setting(Name.ToString(), Value.ToString()));
}
}
【问题讨论】:
-
这对我来说似乎是一种反模式。来自msdn.microsoft.com/en-us/library/bb383977.aspx:“一般来说,我们建议您谨慎地实现扩展方法,并且仅在必要时实现。只要有可能,必须扩展现有类型的客户端代码应该通过创建从现有类型派生的新类型来实现。对于更多信息,请参阅继承(C# 编程指南)。”
-
真正让我印象深刻的是,它是一个试图重新分配自身的对象的扩展方法。如果您的代码试图将
this设置为其他值,这不是一件好事。 -
让这个方法抛出一个空引用对我来说更直观,因为添加一些东西没有意义。
-
我其实同意第一条评论我认为我没有展示的类中的List
字段应该换成一个继承List功能或者封装List的新类. -
另一种简单的看待方式:
Settings不是ref参数。 (因此,这个问题不针对扩展方法,只是在扩展方法中,不能将this参数设为ref参数。)
标签: c# list extension-methods