如何更好地写这个？

Question

让我们看看这段代码：

IList<IHouseAnnouncement> list = new List<IHouseAnnouncement>();
var table = adapter.GetData(); //get data from repository object -> DataTable

if (table.Rows.Count >= 1)
{
    for (int i = 0; i < table.Rows.Count; i++)
    {
        var anno = new HouseAnnouncement();
        anno.Area = float.Parse(table.Rows[i][table.areaColumn].ToString());
        anno.City = table.Rows[i][table.cityColumn].ToString();
        list.Add(anno);
    }
  }
  return list;

是否有更好的方法以更少的代码和更好的方式编写此代码（必须是:-)）？也许使用 lambda （但让我知道如何）？

提前致谢！

Answer 1

仅供参考，您永远不会将新的 HouseAnnouncement 添加到列表中，并且您的循环永远不会针对 last 行执行，但我假设这些是示例而不是在您的实际代码中。

你可以这样做：

return adapter.GetData().Rows.Cast<DataRow>().Select(row =>
    new HouseAnnouncement()
    {
        Area = Convert.ToSingle(row["powierzchnia"]),
        City = (string)row["miasto"],
    }).ToList();

我通常追求可读性而不是简洁性，但我觉得这是非常可读的。

请注意，虽然您仍然可以缓存 DataTable 并在 lambda 中使用 table.powierzchniaColumn，但我消除了这一点，这样您就不会使用不必要的闭包（闭包给 lambda 的内部实现带来了相当大的复杂性，所以如果可能的话我会避免使用它们）。

如果保持列引用原样对您来说很重要，那么您可以这样做：

using (var table = adapter.GetData())
{
    return table.Rows.Cast<DataRow>().Select(row =>
        new HouseAnnouncement()
        {
            Area = Convert.ToSingle(row[table.powierzchniaColumn]),
            City = (string)row[table.miastoColumn],
        }).ToList();
}

这会增加编译器生成的实际 IL 的复杂性，但应该可以满足您的要求。

Answer 2

你可以在 Linq 中做这样的事情：

var table = adapter.GetData();
var q = from row in table.Rows.Cast<DataRow>()
        select new HouseAnnouncement() 
           { Area = float.Parse(row[table.areaColumn].ToString()),
             City = row[table.cityColumn].ToString()
           };
return q.ToList();

Answer 3

你的“if 语句”是不必要的。您的“for 循环”已经处理了这种情况。

另外，当表行数为 1 时，“for 循环”将不会执行。这似乎是一个错误，而且不是设计使然，但我可能是错的。如果你想解决这个问题，只需去掉“-1”：

for (int i = 0; i < table.Rows.Count; i++)

Answer 4

好吧，一方面，您似乎遇到了差一错误：

for (int i = 0; i < table.Rows.Count - 1; i++)
{
}

如果您的表有三行，则当 i 小于 3 - 1 或 2 时，这将运行，这意味着它将针对第 0 行和第 1 行运行，但不会针对第 2 行运行。这可能不是您想要的。

Answer 5

没有比一个 for 循环和没有 if 语句更简单的了：

var table = adapter.GetData(); //get data from repository object -> DataTable
IList<IHouseAnnouncement> list = new List<IHouseAnnouncement>(table.Rows.Count);

for (int i = 0; i < list.Length; i++)
{
   list[i] = new HouseAnnouncement();
   list[i].Area = float.Parse(table.Rows[i][table.areaColumn].ToString());
   list[i].City = table.Rows[i][table.cityColumn].ToString();
}

return list;

它比 linq 版本需要更多的字符，但程序员的大脑解析速度更快。 :)

Answer 6

对我来说，可读性比代码简洁更好——只要性能不是牺牲品。另外，我相信任何以后需要维护代码的人也会欣赏它。
即使当我维护自己的代码时，我也不想在几个月后查看它并思考“我到底想完成什么”

Answer 7

我可能会做这样的事情：

var table = adapter.GetData(); //get data from repository object -> DataTable

return table.Rows.Take(table.Rows.Count-1).Select(row => new HouseAnnouncement() {
    Area = float.Parse(row[table.powierzchniaColumn].ToString()),
    City = row[table.miastoColumn].ToString()
}).ToList();