Как проверить mysql db для записи с меньшим кодом

Question

Привет, я хочу знать, есть ли более упрощенный метод, чем тот, который используется сейчас, чтобы найти, есть ли запись или нет в mysql db.

public static bool check_db_entry(string query) 
{ 
    using (var conn = new MySqlConnection(DbMethods.constr)) 
    { 
     using (var cmd = new MySqlCommand(query, conn)) 
     { 
      conn.Open(); 
      using (var rdr = cmd.ExecuteReader()) 
      { 
       if (rdr.Read() == false) 
       { 
        return false; 
       } 
       else 
       { 
        return true; 
       } 
      } 
     } 
    } 
} 

Answer 1

Оставляя в стороне теперь тот факт, что методы которые принимают только строки sql, по своей сути небезопасны, это мне плохо. Если у вас будут общедоступные методы, которые принимают произвольные SQL-команды (помните: я сказал «Если»), то, несомненно, у вас есть тот, который возвращает данные напрямую. Вы должны полагаться на это как на свою базу. Вот пример:

private static IEnumerable<IDataRecord> GetDataImpl(string query) 
{ 
    using (var conn = new MySqlConnection(DbMethods.constr)) 
    using (var cmd = new MySqlCommand(query, conn)) 
    { 
     conn.Open(); 

     using (var rdr = cmd.ExecuteReader()) 
     { 
      yield return rdr; 
     } 
    } 
} 

public static bool check_db_entry(string query) 
{ 
    return GetDataImpl(query).Any(); 
} 

Примечание: есть причина, по которой я перечислил первый метод как закрытый. Как написано, это может иметь странные побочные эффекты, если вы сначала не копируете каждый элемент в читателе, а затем возвращаете его на более высокий уровень абстракции в вашей программе. Но вы можете получить общедоступную версию достаточно легко:

public static IEnumerable<T> GetData<T>(string query, Func<IDataRecord,T> copy) 
{ 
    return GetDataImpl(query).Select(copy); 
} 

Answer 2

Принимая сторону ваши вопросы дизайна указали в комментариях на вопрос, в, если вы хотите, чтобы проверить существование записи в базе данных, вы всегда должны запрашивать с COUNT(*): SELECT COUNT(*) FROM yourTable [WHERE theCondition].

Если это все, что вы передаете в вашу функцию, вы можете затем просто с:

public static bool check_db_entry(string query) 
{ 
    using (var conn = new MySqlConnection(DbMethods.constr)) 
    { 
     conn.Open(); 
     using (var cmd = new MySqlCommand(query, conn)) 
     { 
      return (int)cmd.ExecuteScalar() == 1; 
     } 
    } 
} 

И если вы хотите, чтобы оптимизировать его:

public static bool check_db_entry(string query) 
{ 
    using (var conn = new MySqlConnection(DbMethods.constr)) 
    using (var cmd = conn.CreateCommand()) 
    { 
     conn.Open(); 
     cmd.CommandText = query; 

     return (int)cmd.ExecuteScalar() == 1; 
    } 
}