The radiobuttons are placed in a group box (grpBranch)Code://button to display branch name private void btnBranch_Click(object sender, EventArgs e) { //if no branch is selected if (rbtnKnight.Checked == false && rbtnWbldon.Checked == false && rbtnKston.Checked == false) { //show error message MessageBox.Show("Please select Branch!", "Error!"); } else { //if user selects knightsbridge branch if (rbtnKnight.Checked == true) { txtBranch.Text = "Knightsbridge"; btnCost.Focus(); //sets focus in the Cost button when msg is displayed } //if user selects Wimbledon branch if (rbtnWbldon.Checked == true) { txtBranch.Text = "Wimbledon"; //set focus on btnCost once branch name is displayed btnCost.Focus(); } //if user selects Kingston branch if (rbtnKston.Checked == true) { txtBranch.Text = "Kingston"; //set focus on btnCost once Kingston branch is displayed btnCost.Focus(); } } }
Is there a way to make this better i.e making it to a switch statement?
Thanx in advance![]()




Reply With Quote