Add click if statement

Question

Is there a better way to pass this through the addClick (ideally I don't want this at all and I would want it to auto-pass through)?

public void addClick(object sender, EventArgs e)
{
    if ((string) HttpContext.Current.Session["whichMenu"] == "systemDateFormats")
    {
        WorldViewNet.system.DateFormats dateformats = new WorldViewNet.system.DateFormats();
        dateformats.addClick();
    }
    else if ((string) HttpContext.Current.Session["whichMenu"] == "programmingLabels")
    {
        WorldViewNet.programming.Labels labels = new WorldViewNet.programming.Labels();
        labels.addClick();
    }
    else if ((string) HttpContext.Current.Session["whichMenu"] == "programmingPLUSearch")
    {
        WorldViewNet.programming.PLUSearch pluSearch = new WorldViewNet.programming.PLUSearch();
        pluSearch.addClick();
    }
    else if ((string) HttpContext.Current.Session["whichMenu"] == "programmingServings")
    {
        WorldViewNet.programming.Servings servings = new WorldViewNet.programming.Servings();
        servings.addClick();
    }
    else if ((string) HttpContext.Current.Session["whichMenu"] == "programmingShops")
    {
        WorldViewNet.programming.Shops shops = new WorldViewNet.programming.Shops();
        shops.addClick();
    }
    else if ((string) HttpContext.Current.Session["whichMenu"] == "programmingTextsSearch")
    {
        WorldViewNet.programming.TextsSearch textsSearch = new WorldViewNet.programming.TextsSearch();
        textsSearch.addClick();
    }
    else if ((string) HttpContext.Current.Session["whichMenu"] == "systemTemplates")
    {
        WorldViewNet.system.Templates templates = new WorldViewNet.system.Templates();
        templates.addClick();
    }
}

If anyone has any suggestions, that would help me out I would be grateful.

switch() {} perhaps

Alex K.
– Alex K.

2014-11-21 14:18:06 +00:00
Commented Nov 21, 2014 at 14:18 — Alex K.
– Alex K., Commented Nov 21, 2014 at 14:18
Or a dictionary of strings => actions.

Brian Rasmussen
– Brian Rasmussen

2015-07-31 16:59:59 +00:00
Commented Jul 31, 2015 at 16:59 — Brian Rasmussen
– Brian Rasmussen, Commented Jul 31, 2015 at 16:59

Behzad · Accepted Answer · 2015-07-31 16:55:19Z

I thing below models maybe good for your codes:

public void addClick(object sender, EventArgs e)
{
    object control;
    string opt = (string) HttpContext.Current.Session["whichMenu"];

    switch (opt)
    {
        case "systemDateFormats": control = new WorldViewNet.system.DateFormats();
            break;
        case "programmingLabels": control = new WorldViewNet.programming.Labels();
            break;
        case "programmingPLUSearch": control = new WorldViewNet.programming.PLUSearch();
            break;
        case "programmingServings": control = new WorldViewNet.programming.Servings();
            break;
        case "programmingShops": control = new WorldViewNet.programming.Shops();
            break;
        case "programmingTextsSearch": control = new WorldViewNet.programming.TextsSearch();
            break;
        case "systemTemplates": control = new WorldViewNet.system.Templates();
            break;
        default: new WorldViewNet.system.DefaultType();
    }

    ((dynamic)control).addClick();
}

Collectives™ on Stack Overflow

Add click if statement

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related