c#, case, if

ITsolutionWizard asked

on 2017-04-17

4 Comments

1 Solution

127 Views

Last Modified: 2017-04-17

Below codes in c#. how can I use CASE instead?
The goal is to shorten the codes....

Thanks

DropDownList selectedInsuranceProductType = (DropDownList)InsuranceTypesList.FindControl("insuranceProductsList");
            if (selectedInsuranceProductType.SelectedValue == "Health")
            {
                panelHealth.Visible = true;
            }
            else if (selectedInsuranceProductType.SelectedValue == "Life")
            {
                panelLife.Visible = true;
            }
            else if (selectedInsuranceProductType.SelectedValue == "Accident")
            {
                panelAccident.Visible = true;
            }
            else if (selectedInsuranceProductType.SelectedValue == "Dental")
            {
                panelDental.Visible = true;
            }
            else
            {
                panelHealth.Visible = false;
                panelLife.Visible = false;
            }

Open in new window

Comment

Watch Question

View Solution Only

Nitin SontakkeDeveloper

CERTIFIED EXPERT

Commented: 2017-04-17

How about...

    panelHealth.Visible = selectedInsuranceProductType.SelectedValue == "Health";
    panelLife.Visible = selectedInsuranceProductType.SelectedValue == "Life";
    panelAccident.Visible = selectedInsuranceProductType.SelectedValue == "Accident";
    panelDental.Visible = selectedInsuranceProductType.SelectedValue == "Dental";

Open in new window

ITsolutionWizard

Author

Commented: 2017-04-17

missing below
else
{
panelHealth.Visible = false;
panelLife.Visible = false;
}

Nitin SontakkeDeveloper

CERTIFIED EXPERT

Commented: 2017-04-17

Unlock this solution and get a sample of our free trial.

(No credit card required)

UNLOCK SOLUTION

Nitin SontakkeDeveloper

CERTIFIED EXPERT

Commented: 2017-04-17

Furthermore one of the commandments states that "Thou shalt not access object properties often".

If we are to abide by it:

    String insuranceProductType = selectedInsuranceProductType.SelectedValue;
    panelHealth.Visible = insuranceProductType == "Health";
    panelLife.Visible = insuranceProductType == "Life";
    panelAccident.Visible = insuranceProductType == "Accident";
    panelDental.Visible = insuranceProductType == "Dental";

Open in new window

c#, case, if

Premium Content

Premium Content

View Solution Only

Thanks for using Experts Exchange.