4 Stimmen

Warum wird dies als schlechte Praxis angesehen? oder ist es das? (ASP.Net)

Würde dieser Kodex als schlechte Praxis angesehen werden?

<div id="sidebar">
  <% =DisplayMeetings(12)  %>
</div>

Dies ist ein Codeschnipsel aus der default.aspx einer kleinen Webanwendung, an der ich gearbeitet habe. Es funktioniert perfekt, läuft sehr schnell, aber wie immer bin ich mir der Tatsache bewusst, dass nur weil es funktioniert, es nicht bedeutet, dass es OK ist.

Grundsätzlich gibt das Unterprogramm DisplayMeetings eine Reihe von formatierten HTML (eine ungeordnete Liste tatsächlich), ohne Formatierung, nur die erforderlichen html, und dann meine CSS führt alle notwendigen Formatierung.

Die Daten für die Erstellung der Liste stammen aus einer SQL-Server-Datenbank (der Parameter steuert, wie viele Zeilen zurückgegeben werden sollen), und ich verwende Stored Procedures und Datareader für den schnellen Zugriff. Dies hält mein Front-End außergewöhnlich einfach und sauber, imho, und lässt mich die ganze Arbeit in VB oder C# in einem separaten Modul tun.

Ich könnte natürlich einen databound Repeater (und wahrscheinlich 6 oder mehr andere Methoden) verwenden, um dasselbe zu erreichen, aber sind sie besser? Andere als verlieren die Design-Zeit-Funktionen von VS2010?

CodeJaeger.com

CodeJaeger ist eine Gemeinschaft für Programmierer, die täglich Hilfe erhalten..
Wir haben viele Inhalte, und Sie können auch Ihre eigenen Fragen stellen oder die Fragen anderer Leute lösen.

Powered by:

X