Question

Is there a better way to refactor the following method which’s purpose is to make sure the appropriate groupbox is the one displayed apart from the others as if they’re cycled through a list? Is it possible to refactor this all into one solo if statement + “with no elses keywords”?

    // DisplayGroupBox(grpSounds);
    void DisplayGroupBox(GroupBox GroupBoxControls)
    {
        grpSounds.Visible = false;
        grpAlerts.Visible = false;
        grpFilters.Visible = false;

        GroupBox.Visible = true;
    }

Edited: I meant on ways to refactor in one statement to display appropriate groupbox with no else statements. Perhaps a way using the &&, ||, and such operators.

Answer 1

I’d probably just leave the method as is. You can squeeze all of those operations into one line, but the intent of the method won’t be as clear. I think it is fine the way it is, no amount of refactoring can really make it simpler.