如何重构C＃linq代码？ [关闭]

Question

我创建了一个使用LinqToSql的控制台应用程序。它工作得很好，但我觉得它是一种代码味道。这是我的linq查询：

var personnelInfo = Personnels.Where(p => p.Name.Equals("Steve Jobs"));

现在，我想报告该对象的各个字段。这就是我觉得代码味道：

Console.WriteLine(String.Format("Emp ID = {0}", personnelInfo.Select(p => p.EmpID).FirstOrDefault()));
Console.WriteLine(String.Format("Phone# = ({0}) {1}-{2}", 
    personnelInfo.Select(p => p.Phone).FirstOrDefault().Substring(0,3),
    personnelInfo.Select(p => p.Phone).FirstOrDefault().Substring(3,3),
    personnelInfo.Select(p => p.Phone).FirstOrDefault().Substring(6,4)));
Console.WriteLine(String.Format("City = {0}", personnelInfo.Select(p => p.City).FirstOrDefault()));

这可以重构吗？

Answer 1

您应该只获取一次查询结果：

var firstPerson = personnelInfo.First();

然后，您可以直接访问属性。

您还应将电话号码格式移动到单独的可重复使用功能中。

Answer 2

我会建议一个foreach，因为那里可能还有另一个史蒂夫乔布斯：

foreach(var personel in personelInfo)
{
    Console.WriteLine(String.Format("Emp ID = {0}", personel.EmpID));
    Console.WriteLine(String.Format("Phone# = ({0}) {1}-{2}", 
        personel.Phone.Substring(0,3),
        personel.Phone.Substring(3,3),
        personel.Phone.FirstOrDefault().Substring(6,4)));
    Console.WriteLine(String.Format("City = {0}", personel.City));
}