راهنمایی در روش صحیح کد نویسی

[MahmoodM30]

سلام دوستان میخواستم ببینیم چطوری میتونم این کد رو جوری بنویسم که این همه تو در تو نباشه من اینجا از if استفاده کردم برای چک کردن داده ها ولی اینطوری طولانی میشه

دوستان روشی دارن که کوتاه تر باشه و همه این عملیات انجام بشه؟

if (UserNameText.Text != "" & PasswordText.Text != "" & RPasswordText.Text != "")
{
if (PasswordText.Text == RPasswordText.Text)
{
try
{
con.Open();
SqlCommand cmd = new SqlCommand("Insert INTO Acount (UserName,Password) Values (@UserName,@Password)", con);
cmd.Parameters.AddWithValue("@UserName", UserNameText.Text);
cmd.Parameters.AddWithValue("@Password", PasswordText.Text);
cmd.ExecuteNonQuery();
con.Close();
MessageBox.Show("اطلاعات با موفقیت ثبت شد", "پیغام", MessageBoxButtons.OK, MessageBoxIcon.Asterisk);
UserNameText.Text = "";
PasswordText.Text = "";
RPasswordText.Text = "";
Acount_Load(sender, e);
}
catch
{
MessageBox.Show("خطا در ثبت اطلاعات", "پیغام", MessageBoxButtons.OK, MessageBoxIcon.Error);
}

}
else { MessageBox.Show("پسورد های وارد شده یکسان نمیباشد", "پیغام", MessageBoxButtons.OK, MessageBoxIcon.Warning); }
}
else
{
MessageBox.Show("لطفا تمامی فیلد ها را پر کنید", "پیغام", MessageBoxButtons.OK, MessageBoxIcon.Warning);
}

[hamid_hr]

if (UserNameText.Text == "" || PasswordText.Text == "" || RPasswordText.Text == ""){
MessageBox.Show("لطفا تمامی فیلد ها را پر کنید", "پیغام", MessageBoxButtons.OK, MessageBoxIcon.Warning);
return;
}

if (PasswordText.Text == RPasswordText.Text){
MessageBox.Show("پسورد های وارد شده یکسان نمیباشد", "پیغام", MessageBoxButtons.OK, MessageBoxIcon.Warning);
return;
}

try
{
con.Open();
SqlCommand cmd = new SqlCommand("Insert INTO Acount (UserName,Password) Values (@UserName,@Password)", con);
cmd.Parameters.AddWithValue("@UserName", UserNameText.Text);
cmd.Parameters.AddWithValue("@Password", PasswordText.Text);
cmd.ExecuteNonQuery();
con.Close();
MessageBox.Show("اطلاعات با موفقیت ثبت شد", "پیغام", MessageBoxButtons.OK, MessageBoxIcon.Asterisk);
UserNameText.Text = "";
PasswordText.Text = "";
RPasswordText.Text = "";
Acount_Load(sender, e);
}
catch
{
MessageBox.Show("خطا در ثبت اطلاعات", "پیغام", MessageBoxButtons.OK, MessageBoxIcon.Error);
}

[Abbas Naghdi]

شما اول تایید اطلاعات رو چک کن بعد بیا ثبت کن اصلا این روش بهینه نسیت هر چند اکثر برنامه نویسان مبتدی از این روش استفاده میکنند

اول از همه باید نرم افزارت کلاس بندی داشته باشه و متد هایی برای ثبت و چک کردن و تایید اطلاعات داشته باشی ...
اطلاعات رو پاس بدی به متد ها و بعد از تاییید صحت اطلاعات عملیات انجام بشه و ...
یه کار دیگه میتونی انجام بدی اینه که کد هاتو جدا کنی و در صورت امکان از If و ... استفاده نکنی
من روش اول رو پیشنهاد میکنم .
این کد اصلا اصولی نست ...

[MahmoodM30]

میشه در مورد روش اولتون مثالی بزنید یا بیشتر راهنمایی کنید تا خودم روش کار کنم ممنون ازتون