2 svar · 169 visningar · startad av origon
Kan jag förbättra denna funktion? jag har för mig att man inte skulle ha Response.Write i en funktion men jag får det inte att fungera om jag tar bort den.
FUNCTION ListView(Nr)
IF RS("Column"&Nr&"") = "1" THEN
Response.Write "<td width=""71"" height=""14""><b>Type</b></td>"
ELSEIF RS("Column"&Nr&"") = "2" THEN
Response.Write "<td width=""84"" height""14""><b>Source</b></td>"
ELSEIF RS("Column"&Nr&"") = "3" THEN
Response.Write "<td width=""91"" height=""14""><b>Quality</b></td>"
ELSEIF RS("Column"&Nr&"") = "4" THEN
Response.Write "<td width=""51""height=""14""><b>CDs</b></td>"
ELSEIF RS("Column"&Nr&"") = "5" THEN
Response.Write "<td width=""49"" height=""14"" nowrap><b>Size</b></td>"
ELSEIF RS("Column"&Nr&"") = "6" THEN
Response.Write "<td width=""93"" height=""14"" nowrap><B>IMDb#</b></td>"
END IF
END FUNCTION
[Redigerat av origon den 08 dec 2001]
En funktion skall enbart returnera ett värde, om den gör response.write så begränsar du möjligheten att använda den i andra sammanhang. Sen så skall man inte använda globala variabler, av samma anledning.
Jag förstår jag inte varför du skulle bryta ut den här koden överhuvudtaget dels då den verkar vara väldigt specifik och dels då är en liten kodmängd.
------------------
essentitia preter non sans multiplicandum
Hej, instämmer med LarsG fullt ut. Den förbättring jag hade gjort, var att bygga om den till select case istället.
mvh Jörgen :)
------------------
Varför bråka när man kan kramas?