Question

我正在使用LINQ和HtmlAgilitypack从HTML创建数据表。以下内容获取html表头并构造数据表列：

var nodes = htmlDoc.DocumentNode.SelectNodes("//table[@class='ps-sellers-table crts']/tr");
             nodes[0].Elements("th")
            .Skip(0)
            .Select(th => th.InnerText
            .Trim())
            .ToList()
            .ForEach(header => dt.Columns.Add(header));

到目前为止，除了我需要一些定制之外，它的工作完美。

选择要添加的列。
并指定列类型。

这个select语句将完成上述两件事：

switch (header)
            {
                case ("id") : dt.Columns.Add(header,typeof(int)); break;
                case ("name") : dt.Columns.Add(header,typeof(string)); break;
                case ("price") : dt.Columns.Add(header,typeof(decimal)); break;
                case ("shipping") : dt.Columns.Add(header,typeof(decimal)); break;
                default : //skip the column and dont add it
            }

然而我对LINQ和C＃真的很新，我想在第一个片段中的foreach中实现上面的switch case，我知道我应该使用ternary operator，但我不确定sytax。

Answer 1

我将开关包装到一个方法中，然后在select语句中调用该方法。

我的版本看起来像：

var nodes = htmlDoc.DocumentNode.SelectNodes("//table[@class='ps-sellers-table crts']/tr");
         nodes[0].Elements("th")
        .Select(th => th.InnerText.Trim());

foreach(var node in nodes)
    AddColumnToDataTable(node, dt);

请注意，不需要Skip(0)，仅使用ToList()调用List<T>.ForEach会降低可读性并增加开销。我个人更喜欢使用普通的foreach。

话虽这么说，您的方法可以重构为使用Dictionary<string, Type>代替。如果你把它添加到你的班级：

Dictionary<string, Type> typeLookup;
// In your constructor:

public YourClass()
{
    typeLookup.Add("id", typeof(int));
    typeLookup.Add("name", typeof(string));
    typeLookup.Add("price", typeof(decimal));
    typeLookup.Add("shipping", typeof(decimal));
}

然后您可以将您的方法编写为：

void AddColumnToDataTable(string columnName, DataTable table)
{
    table.Columns.Add(columnName, typeLookup[columnName]);
}

Answer 2

您可以将开关添加到foreach，不知道为什么必须使用三元运算符

.ToList().ForEach(header =>
{
    switch (header)
    {
       case ("id"): dt.Columns.Add(header, typeof(int)); break;
       case ("name"): dt.Columns.Add(header, typeof(string)); break;
       case ("price"): dt.Columns.Add(header, typeof(decimal)); break;
       case ("shipping"): dt.Columns.Add(header, typeof(decimal)); break;
       default: break; //skip the column and dont add it
    }
});

C＃LINQ三元运算符作为foreach中的switchcase

2 个答案: