Проверить дублирующееся имя пользователя с параметризованным запросом

Question

Я хочу проверить, нет ли имени пользователя в базе данных.

SqlConnection con = new SqlConnection(ConfigurationManager.AppSettings["DBCOnn"].ToString()); 
try 
{ 
    con.Open(); 

    SqlCommand cmd = new SqlCommand("select client_id from tbl_client where client_name=@cname", con); 
    cmd.CommandType = CommandType.Text; 

    cmd.Parameters.Add("@cname", DbType.String).Value = usernm; 

    int i = cmd.ExecuteNonQuery(); 
    if (i > 0) 
     return true; 
    else 
     return false; 
} 
catch (Exception ex) 
{ 
    throw new Exception("CheckExistingClient:" + ex.Message, ex.InnerException); 
} 
finally 
{ 
    con.Close(); 
} 

Но здесь, i всегда дает в -1

Что является проблемой.?

Answer 1

ExecuteNonQuery возвращает количество затронутых строк и обычно используется для инструкций insert-, update- или delete-statements. Используйте вместо этого ExecuteScalar с COUNT.

using (var con = new SqlConnection(ConfigurationManager.AppSettings["DBCOnn"].ToString())) 
using(var cmd = new SqlCommand("select COUNT(client_id) from tbl_client where client_name=@cname", con)) 
{ 
    cmd.Parameters.Add("@cname", DbType.String).Value = usernm; 
    con.Open(); 
    int i = (int)cmd.ExecuteScalar(); 
    return i > 0; 
} 

Вот родственный ответ на SO: https://stackoverflow.com/a/4269651/284240

Answer 2

ExecuteNonQuery возвращает количество строк, затронутых INSERT, UPDATE или DELETE заявление. Вы используете SELECT.

Мне кажется, что «SELECT FROM Users, где UserName = @ cName» проще, и запустить стандартный Select.

Answer 3

ExecuteNonQuery предназначен для запросов DML. Вы не изменяете и не стробируете. Таким образом, вы получаете результат как -1.

Вы можете использовать select count(client_id) from tbl_client where client_name=@cname

И получить счетчик с cmd.ExecuteScalar