سلام و خسته نباشید

دارم سعی می کنم کد هام رو بهینه کنم . البته باید از کلاس برای ارتباط با DBو ... استفاده کنم

اما فعلا وقتش نیست و دارم به صورت زیر عمل می کنم .

در قسمت Public Partial class 2 تا کد زیر رو وارد کردم


        SqlConnection cnn = new SqlConnection(ConfigurationManager.AppSettings["ConnectionString"]);

        SqlCommand cmd = new SqlCommand();


یعنی به این صورت



namespace NoorAutomation

{

    public partial class daroo_tajhizat : Form

    {

        string ConnectionString = ConfigurationManager.AppSettings["ConnectionString"];

        SqlConnection cnn = new SqlConnection(ConfigurationManager.AppSettings["ConnectionString"]);

        SqlCommand cmd = new SqlCommand();

        

        public daroo_tajhizat()

        {

            InitializeComponent();



        }



سپس برای ارتباط با دیتابیس از این کد استفاده کردم



                    cmd.Connection = cnn;

                    cnn.Open();

                    cmd.CommandText = "INSERT INTO tajhiz_usage(patientid,tajid,numberz) VALUES (@patientid,@tajid,@numberz)";

					cmd.Parameters.Clear();

                    cmd.Parameters.AddWithValue("@patientid", PID);

                    cmd.Parameters.AddWithValue("@tajid", 193);

                    cmd.Parameters.AddWithValue("@numberz", textBox284.Text);

                    cmd.ExecuteNonQuery();

                    cnn.Close();


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

بعضی وقت ها یک اررور مبنی بر Connection Was Lost دریافت می کنم !