Как четко и быстро параметризовать пустую строку с помощью DBNull.Value - PullRequest
Новая
       35

Как четко и быстро параметризовать пустую строку с помощью DBNull.Value

34 голосов
/ 08 апреля 2010

Я устал писать следующий код: 

/* Commenting out irrelevant parts
public string MiddleName;
public void Save(){
    SqlCommand = new SqlCommand();
    // blah blah...boring INSERT statement with params etc go here. */
    if(MiddleName==null){
        myCmd.Parameters.Add("@MiddleName", DBNull.Value);
    }
    else{
        myCmd.Parameters.Add("@MiddleName", MiddleName);
    }
    /*
    // more boring code to save to DB.
}*/

Итак, я написал это: 

public static object DBNullValueorStringIfNotNull(string value)
{
    object o;
    if (value == null)
    {
        o = DBNull.Value;
    }
    else
    {
        o = value;
    }
    return o;
}

// which would be called like:
myCmd.Parameters.Add("@MiddleName", DBNullValueorStringIfNotNull(MiddleName));

Если это хороший способ сделать это, то что бы вы предложили в качестве имени метода? DBNullValueorStringIfNotNull немного многословно и запутанно.

Я также открыт для путей полного решения этой проблемы. Я бы хотел сделать это: 

myCmd.Parameters.Add("@MiddleName", MiddleName==null ? DBNull.Value : MiddleName);

но это не сработает, потому что «Оператор» ?? не может применяться к операндам типа 'string и' System.DBNull '".

У меня есть C # 3.5 и SQL Server 2005, если это имеет значение.

Ответы [ 8 ]

55 голосов
/ 08 апреля 2010

Приведите любое из ваших значений к object, и оно будет скомпилировано. 

myCmd.Parameters.Add("@MiddleName", MiddleName==null ? (object)DBNull.Value : MiddleName);
17 голосов
/ 07 октября 2014

Вы можете избежать явного приведения к object, используя SqlString.Null вместо DBNull.Value: 

MiddleName ?? SqlString.Null

Существуют соответствующие типы для int, datetime и т. Д. Вот фрагмент кода с еще парой примеров: 

 cmd.Parameters.AddWithValue("@StartDate", StartDate ?? SqlDateTime.Null);
 cmd.Parameters.AddWithValue("@EndDate", EndDate ?? SqlDateTime.Null);
 cmd.Parameters.AddWithValue("@Month", Month ?? SqlInt16.Null);
 cmd.Parameters.AddWithValue("@FormatID", FormatID ?? SqlInt32.Null);
 cmd.Parameters.AddWithValue("@Email", Email ?? SqlString.Null);
 cmd.Parameters.AddWithValue("@ZIP", ZIP ?? SqlBoolean.Null);
11 голосов
/ 09 апреля 2010

Лично это то, что я бы сделал с методом расширения (убедитесь, что это входит в статический класс) 

public static object GetStringOrDBNull(this string obj)
{
    return string.IsNullOrEmpty(obj) ? DBNull.Value : (object) obj
}

Тогда у вас будет 

myCmd.Parameters.Add("@MiddleName", MiddleName.GetStringOrDBNull());
6 голосов
/ 19 марта 2015 
myCmd.Parameters.Add("@MiddleName", MiddleName ?? (object)DBNull.Value);
2 голосов
/ 10 июня 2014

@ Дэвид Спасибо за ваше предложение. Следующий метод отлично работает! 

MiddleName ?? (object)DBNull.Value
1 голос
/ 09 апреля 2010

Я бы предложил использовать обнуляемые свойства вместо открытых полей и метод AddParameter (не знаю, оптимизирован или корректен этот код, просто у меня в голове): 


private string m_MiddleName;

public string MiddleName
{
  get { return m_MiddleName; }
  set { m_MiddleName = value; }
}

.
.
.

public static void AddParameter(SQLCommand cmd, string parameterName, SQLDataType dataType, object value)
{
  SQLParameter param = cmd.Parameters.Add(parameterName, dataType);

  if (value is string) { // include other non-nullable datatypes
    if (value == null) {
      param.value = DBNull.Value;
    } else {
      param.value = value;
    }
  } else { 

    // nullable data types
    // UPDATE: HasValue is for nullable, not object type
    if (value.HasValue) // {{{=====================================================
    {
          param.value = value;
    } else 
    {
          param.value = DBNull.Value;
    }
  }
}

.
.
.
AddParameter(cmd, "@MiddleName", SqlDbType.VarChar, MiddleName);
1 голос
/ 09 апреля 2010

Я бы лучше дал вам два совершенно разных предложения:

  1. Используйте ORM. Существует множество неинтрузивных инструментов ORM.

  2. Напишите свою собственную оболочку для построения команд с более понятным интерфейсом. Что-то вроде: 

    public class MyCommandRunner {
  private SqlCommand cmd;

  public MyCommandRunner(string commandText) {
    cmd = new SqlCommand(commandText);
  }

  public void AddParameter(string name, string value) {
    if (value == null)
     cmd.Parameters.Add(name, DBNull.Value);
    else
      cmd.Parameters.Add(name, value);
  }

  // ... more AddParameter overloads
}

Если вы переименуете свои AddParameter методы в Add, вы можете использовать их очень гладко: 

var cmd = new MyCommand("INSERT ...")
  {
    { "@Param1", null },
    { "@Param2", p2 }
  };
1 голос
/ 08 апреля 2010

Да, мы все хотели бы сделать myCmd.Parameters.Add("@MiddleName", MiddleName ?? DBNull.Value);. Или, что еще лучше, пусть чертов слой SqlClient понимает, что CLR null должен быть сопоставлен с DBNull.Value при добавлении параметра. К сожалению, система типов .Net закрывает первый вариант, а реализация SqlClient закрывает второй.

Я бы пошел с хорошо известным именем функции, например Coalesce или IsNull . Любой разработчик БД в одно мгновение узнает, что он делает, только по названию.

...