Why is this consider bad practice? or is it? (ASP.Net)
<p>Would this code be considered bad practice:</p>

<pre><code>&lt;div id="sidebar"&gt;
  &lt;% =DisplayMeetings(12)  %&gt;
&lt;/div&gt;
</code></pre>

<p>This is a snippet of code from the default.aspx of a small web app I have worked on. It works perfectly well, runs very fast, but as always, I am aware of the fact that just because it works, doesn't mean it is OK.</p>

<p>Basically, the DisplayMeetings subroutine outputs a bunch of formatted HTML (an unordered list actually), with no formatting, just the requisite html, and then my CSS performs all the necessary formatting.</p>

<p>The data for generating the list comes from an SQL server database (the parameter controls how many rows to return) and I am using stored procedures and datareader for fast access. This keeps my front-end extraordinary simple and clean, imho, and lets me do all the work in VB or C# in a separate module.</p>

<p>I could of course use a databound repeater (and probably 6 or more other methods) of accomplishing the same thing, but are they any better? Other than loosing the design-time features of VS2010?</p>
