使用LINQ按字符串名称选择属性

3

我有一段相当简单的代码:

var list = new List<MyType>();
list = MyItems.Where(x => x.Name.ToLower().Contains(e.Text.ToLower())).ToList();
for (int i = itemOffset; i < endOffset; i++)
{
    combo.Items.Add(new ComboItem(list[i].Name, list[i].Id.ToString()));
}

我希望将此函数改为通用函数,接受任何类型的列表,并接受两个字符串作为属性。(在本例中,我们查看的两个属性是名称和ID。)
private void MakeCombo<T>(Combo combo, IEnumerable<T> lst, string Field1, string Field2)
{
    var list = list = lst.Where(x => x.Field1.ToLower().Contains(searchText.ToLower())).ToList(); //How make this to work???

    for (int i = itemOffset; i < endOffset; i++)
    {
        combo.Items.Add(new ComboItem(list[i].Field1, list[i].Field2.ToString())); //How make this to work???
    }
}

现在,我不明白如何通过字符串名称(Field1和Field2)访问通用列表的属性。


具体是什么出了问题?有任何错误吗?有任何有用的转储文件吗? - Tommaso Belluzzo
1
你正在对列表中的每个项目评估searchText.ToLower()。你应该将其存储到本地变量中,这样你只需要评估一次。 - Mike Parkhill
通常情况下,应尽可能避免像这样做(在这里是可行的)。反射应始终是最后的手段,就像任何将字符串视为代码的其他方法一样。它非常脆弱,阻碍了学习更适当的编码方法(您可能会开始将其用作支撑),创建安全漏洞,并且难以随着时间的推移阅读/理解/维护。 - Servy
4个回答

5

如果您没有创建接口并确保序列中的所有项目实现该接口的能力(如此答案所示),解决这个问题的一种方法是使用委托。 通过传入两个函数,每个函数都以T作为输入,并返回一个stringobject作为输出,您可以模拟该功能:

private void MakeCombo<T>(Combo combo, IEnumerable<T> sequence
    , Func<T, string> field1Selector, Func<T, object> field2Selector)
{
    var foundItems = sequence.Where(item => field1Selector(item).ToLower()
        .Contains(searchText.ToLower()));

    foreach(var item in foundItems)
    {
        combo.Items.Add(new ComboItem(field1Selector(item), field2Selector(item).ToString()));
    }
}

从调用方的角度来看,它可能类似于这样(以您的第一个代码片段为例:
MakeCombo(combo, list, item => item.Name, item => item.Id);

非常感谢您撰写这个答案。这太棒了。 - user194076

2

您将需要使用反射:


typeof(x).GetField("Field1", BindingFlags.Public).GetValue()

你可以对两个对象进行操作,然后比较它们的值。

获取值至少需要一个参数。应该是什么? - user194076
糟糕,你应该传递你想要值的类型的实例。在这种情况下,是x。如果你想走这条路,你真的应该阅读有关反射的内容。http://msdn.microsoft.com/en-us/library/f7ykdhsy(v=vs.110).aspx - Chris

1
如果您能定义一个接口,描述所有可用的字段,并使每个类型都适合实现该接口,则它将如下所示(并且在我看来比通过名称访问字段更好):
public interface IFields
{
    string Prop1 { get; set; }
    string Prop2 { get; set; }
}

public static void DoSmth<T>(List<T> items) where T : IFields
{
    foreach (T t in items)
    {
        Console.WriteLine(t.Prop1);
        Console.WriteLine(t.Prop2);
    }
}

看起来很棒。不幸的是,我无法描述一个接口,因为我对 T 没有真正的控制。有没有不使用接口的方法可以做到这一点? - user194076
@user194076 直接使用 Chris 的答案即可。 - horgh
@user194076 是的,有的。只需传入两个委托,例如 Func<T, string>。这将是文本和值的选择器。在调用方法时,您可以使用lambda,例如 item=>item.Name。这具有100%编译时安全性的优点,不像反射模型那样,您不需要担心是否存在具有给定名称的字段,其可见性等,您可以直接使用该函数。 - Servy
@Servy 你可能想使用类似于此问题中的代码:Property Info有什么优势? - horgh
@KonstantinVasilcov 不需要表达式。这比那更简单。我想我会把它写成答案。 - Servy
我在这里添加了一个答案(https://dev59.com/0W3Xa4cB1Zd3GeqPckGG#14351923),以更详细地解释我的评论。 - Servy

1
private void MakeCombo<T>(Combo combo, IEnumerable<T> list)
{
    Type type = typeof(T);
    List<T> comboList = list.Where(x => ((String)type.GetField("Field1", (BindingFlags.Public | BindingFlags.Instance)).GetValue(x)).ToLower().Contains(m_SearchText.ToLower()));

    for (Int32 i = m_ItemOffset; i < m_OffsetEnd; ++i)
        combo.Items.Add(new ComboItem((String)type.GetField("Field1", (BindingFlags.Public | BindingFlags.Instance)).GetValue(list[i]), (String)type.GetField("Field2", (BindingFlags.Public | BindingFlags.Instance)).GetValue(list[i])));
}

它抱怨说:在 typeof 中找不到类型或命名空间名称“x”(您是否缺少 using 指令或程序集引用?) - user194076
与 typeof(list[i]) 相同 - user194076
你能给我展示一下那些字段声明的代码片段吗? - Tommaso Belluzzo
[Serializable()] public class MyClass { public MyClass() { .... }[Serializable()] public class MyNestedClass { public int Id { get; set; } public string Name { get; set; } public MyNestedClass() { } public MyNestedClass(int val0, string val1) { Id = val0; Name = val1; } }} - user194076
啊,该死...你正在尝试获取属性,而不是字段...你应该获取底层成员...m_Name和m_Id。或者将type.GetField更改为Type.GetProperty。 - Tommaso Belluzzo
显示剩余3条评论

网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接