If statements in aspx files

Question

I have some code that essentially looks like this:

<div>     <% if(Something) { %>         <div id="someUniqueMarkup">             This markup should not be output if Something==true.              <units:MyUserControl runat="server"/>         </div>     <% }     else { %>         <units:MyUserControl runat="server" />     <% } %> </div> 

Depending on Something, one of them is hidden, and that is fine. But if I set break points in the user control, I notice it's being loaded twice (once for each of the controls above) and all it's logic is being run twice. I could of course control this with placeholders or multiviews, but the same thing seems to apply - OnLoad/Page_Load etc is run once for each control that is actually on the page.

EDIT: The reason why im showing/hiding this is because I need to include some markup around the control if Something == true. I could wrap the "unique markup" itself in if-else before and after the control, but that just seems dirty for something that really should be as simple as I've imagined above. The user control itself should be exactly the same in both scenarios, sorry for the confusing property it had.

Is it just me, or is this just a really unintuitive interface? And is it actually possible to not load/execute a user control at all as long as it's on the page?

Answer 1

Since you have two controls on the page it will render them both. The if-check you create, only determines whether it's included in the output. The easiest way to prevent this is to change your code like this:

<div>     <units:MyUserControl runat="server" SomeSetting="<%= Something %>" /> </div> 

EDIT: Answer to edit in the original post:

<div>     <% if(Something) { %>         <div id="someUniqueMarkup">             This markup should not be output if Something==true.              <asp:placeholder id="phItemInDiv" runat="server" />         </div>     <% }     else { %>         <asp:placeholder id="phItemOutsideDiv" runat="server" />     <% } %> </div>    MyUserControl ctrl = (MyUserControl)LoadControl("/pathtousercontrol.ascx") if (something){         phItemInDiv.Controls.Add(ctrl); } else{     phItemOutsideDiv.Controls.Add(ctrl); } 

This way you will only have the user control emitted (and loaded) if Something is true

Answer 2

The best way, in my opinion, is to declare your user control once in the ASPX.

In the code behind, on PageLoad, apply the logic you see fit:

if(something)     MyUserControl.SomeSettings = ... 

If there is a problem in the chronology, do the above logic in PreLoad since it will fire before Page Load of the page and all of its related user controls.

EDIT:

You can put two different IDs on the user controls with Enabled = false. In Page_load, set Enabled to one of them based on the logic you desire.