41

Tell me please is this is correct way to check NULL in DataRow if need to return a string

 Convert.ToString(row["Int64_id"] ?? "")

Or should be like check with DBNull.Value.

Need to so much more smaller than

if(row["Int64_id"] != DBNull.Value){...}else if{}
1
  • 1
    row["ColumnName"] != DBNull.Value, also you can try things like this by yourself. One way is to use debuger ! Commented Dec 10, 2014 at 10:05

2 Answers 2

86

Check if the data column is not null with DataRow.IsNull(string columnName)

if (!row.IsNull("Int64_id"))
{
  // here you can use it safety
   long someValue = (long)row["Int64_id"];
}

There are overloads for it using the index of the column or if you have the instance of the DataColumn. If you are sure about the index, use the index version which tends to be faster than the other options.

2
  • returned type string and so much smaller
    – AleksP
    Commented Dec 10, 2014 at 10:04
  • Checking if the value is the right way to do this. :) Commented Dec 10, 2014 at 10:12
24

We have created an extension class that helps in these kinds of situations.

public static class DataRowExtensions
  {
    public static T FieldOrDefault<T>(this DataRow row, string columnName)
    {
      return row.IsNull(columnName) ? default(T) : row.Field<T>(columnName);
    }
  }

You can use is as follows:

int id = dataRow.FieldOrDefault<int>("Id");
1
  • How would you put this on the outside of the foreach?
    – Demodave
    Commented Mar 28, 2019 at 21:52

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.