Hi, I wouldn't say it's sloppy, but it can be inefficient. One of the cardinal rules in ASP development is to avoid context switching as much as possible (switching between asp code blocks and html). Generally, this is better: <% if age > 18 then Response.Write("<A HREF=adult.asp>Adults</A>") end if %>