مشکل استفاده از کلاس

[reza97]

سلام دوستان
من در کلاسم این کد رو نوشتم(برای ثبت نام)
try
{
SqlConnection connection = new SqlConnection(constr());
SqlCommand cmd1 = new SqlCommand("select count(*) from TBLusers where email='" + email + "'", connection);
connection.Open();
int i = int.Parse(cmd1.ExecuteScalar().ToString());
if (i == 0)
{


}
else
{
return "این کاربر قبلا ثبت شده است";
}

SqlCommand cmd2 = new SqlCommand("insert into TBLusers (name,family,email,password)values('" + name + "','" + family + "','" + email + "','" + password + "')", connection);
cmd2.ExecuteNonQuery();
connection.Close();
return "";

}
catch
{
return "error";
}

}

و درصفحه مورد نظرم فراخوانی میکنم اما یک مشکلی هست اونم اینه که وقتی ایمیلی در دیتا بیس هست روی لیبلم نشون میده که با این ایمیل قبلا ثبت نام شده
اما وقتی ایمیل تازه وارد میکنم errorرو برمیگزدنونه و جالب اینجاست کد اینسرت رو هم که بر میدارم اوکی رومیده نمیدونم ایراد کار از کجاست.
ممنون میشم راهنمایی بفرمایید
در ضمن کد رو داخل if هم بردم اما جواب نداده

[mohammad973]

اگر اشتباه نکنم باید دستور Select رو به این صورت اصلاح کنی :



SqlCommand cmd2 = new SqlCommand("insert into TBLusers (name,family,email,password)values(@a,@b,@c,@d)", connection);
cmd2.Parameters.AddWithValue("@a",name);
cmd2.Parameters.AddWithValue("@b",family);
cmd2.Parameters.AddWithValue("@c",email);
cmd2.Parameters.AddWithValue("@d",password);



امتحان کن ببین به این مربوط میشه یا نه

[reza97]

اگر اشتباه نکنم باید دستور Select رو به این صورت اصلاح کنی :



SqlCommand cmd2 = new SqlCommand("insert into TBLusers (name,family,email,password)values(@a,@b,@c,@d)", connection);
cmd2.Parameters.AddWithValue("@a",name);
cmd2.Parameters.AddWithValue("@b",family);
cmd2.Parameters.AddWithValue("@c",email);
cmd2.Parameters.AddWithValue("@d",password);



امتحان کن ببین به این مربوط میشه یا نه

جالبه دقیقا این کد رو مینویسم میگه ثبت نام با موفقت انجام شد اما اطلاعات ثبت نمیشه و
cmd2.ExecuteNonQuery();
رو که اضافه میکنم بازم قسمت error رو میده
من که واقعا گیج شده چرا اینجوری میشه

[mirahsani]

به جای استفاده از این کدهای وقت گیر که احتمال اشتباه توش زیاده چرا از استور پروسیجر استفاده نمی کنی؟؟؟؟