[martin257]

I am trying to write html using c# because i want a dynamic nav menu and i am just not sure that the following solution looks great. I am just wandering if there is a better solution or just any opinions about the code.

if (auth_nav == "admin")
            {
                Label1.Text = "<nav><ul><li><a href='products.aspx'>Add products</a></li><li><a href='login.aspx'>Edit/delete products</a></li></ul></nav>";
}

<asp:Label ID="Label1" runat="server"></asp:Label>

Thoughts?

thanks

[Skydiver]

Consider using the HtmlTextWriter and its descendant classes.
http://msdn.microsof...textwriter.aspx

You didn't mention if you were writing code as WinForms, ASP.NET, or WPF, although the last bit of code you posted hints at ASP.NET. If it is ASP.NET, you can have your own custom control and override the Render() method to output your desired HTML.
http://msdn.microsof...6(v=vs.71).aspx

[martin257]

Skydiver, on 17 December 2012 - 05:57 AM, said:

Consider using the HtmlTextWriter and its descendant classes.
http://msdn.microsof...textwriter.aspx

You didn't mention if you were writing code as WinForms, ASP.NET, or WPF, although the last bit of code you posted hints at ASP.NET. If it is ASP.NET, you can have your own custom control and override the Render() method to output your desired HTML.
http://msdn.microsof...6(v=vs.71).aspx

Yes it is. I will give it a go thanks.

[baavgai]

Your HTML isn't changing, just it's visibility.

When you can, make the markup part of the form do the work for you. e.g.

<asp:PlaceHolder id="AdminNav" runat="server"/>
	<nav>
		<ul>
			<li><a href='products.aspx'>Add products</a></li>
			<li><a href='login.aspx'>Edit/delete products</a></li>
		</ul>
	</nav>
</asp:PlaceHolder>

Then, usually on page load:

AdminNav1.Visible = (auth_nav == "admin");

If you want to use your approach, still prefer placeholder to label.

[JackOfAllTrades]

Moved to ASP.NET

[Nakor]

I generally wouldn't, I like to keep my html separate from my code. You could do as baavgai suggests or you could move the html into a user control and insert that into a placeholder control if the condition is met.