检查 DataRow 是否包含特定列的最佳实践

Question

目前，当我遍历 DataRow 实例时，我会这样做。

foreach(DataRow row in table)
  return yield new Thingy { Name = row["hazaa"] };

迟早（即早），我会让 table 缺少列 donkey 并且便便会撞到风扇。经过一番广泛的谷歌搜索（大约 30 秒），我发现了以下保护语法。

foreach(DataRow row in table)
  if(row.Table.Columns.Contains("donkey"))
    return yield new Thingy { Name = row["hazaa"] };
  else
    return null;

现在 - 这是最简单的语法吗？！真的吗？我期待有一种方法可以让我获得该字段（如果存在）或 null 否则。或者至少在 row 上直接有一个 Contains 方法。

我错过了什么吗？我会以这种方式在许多领域进行映射，因此代码看起来非常不可读......

Answer 1

由于您的 DataTable 表始终具有相同的列（它们不会因任何行而改变），您只需检查一次列名。

if (table.Columns.Contains("donkey"))
{
    foreach ...
}

Answer 2

你可以创建一个扩展方法让它更干净：

static class DataRowExtensions
{
    public static object GetValue(this DataRow row, string column)
    {
        return row.Table.Columns.Contains(column) ? row[column] : null;
    }
}

现在这样称呼它：

foreach(DataRow row in table)
    return yield new Thingy { Name = row.GetValue("hazaa") };

Answer 3

foreach (DataColumn item in row.Table.Columns)
{
    switch (item.ColumnName)
    {
        case "ID":
            {
                p.ID = Convert.ToInt32(row[item.ColumnName].ToString());
            }
            break;
        case "firstName":
            {
                p.firstName = row[item.ColumnName].ToString();
            }
            break;
        case "lastName":
            {
                p.lastName = row[item.ColumnName].ToString();
            }
            break;

        default:
            break;
    };
}

Answer 4

要基于 Varun K 的答案，请使用泛型类型参数：

public static T GetValue<T>(this DataRow row, string column)
{
    if (!row.Table.Columns.Contains(column))
        return default(T);

    object value = row[ColumnName];
    if (value == DBNull.Value)
        return default(T);
    return (T)value;
}

Answer 5

有时可能存在列名，但行不包含该列的数据；例如，在使用 ReadXML 填充 DataTable 之后。

一个简单、快速和安全的解决方案是使用类型检查：

if(row["columnname"].GetType() != typeof(System.DBNull)){
    //DataRow contains "columname"
}else{
    //a safe scope to set default cell data
}

Answer 6

我真的很喜欢@Varun K 采用的方法。因此，以此为出发点，我只想投入两分钱，以防它对其他人有所帮助。我只是对其进行了改进，使其成为 generic 而不仅仅是使用 object 作为返回类型。

static class Extensions
{
  public static T Get<T>(this DataRow self, string column)
  {
    return self.Table.Columns.Contains(column)
      ? (T)self[column]
      : default(T);
    }
  }
}