gizmo>> det er et system jeg har lavet, virker upåklageligt... men det lader til at det skal køre på en apache server, derfor denne omksrivning til php :)
tdaugaard>> jeg håber du kan tyde mine kragetær\' ;)
cyberfessor:> *G* Jeg vil nødigt sige noget ondt om andre folks koder for det får mig selv til at lyde perfekt (og det er jeg ikke helt! *G*) men den kode der ... sorry, men det ligner noget en nybegynder har lavet ... no offense!
*sidder og kigger den igennem igen* ja, det ser vist lidt rodet ud... jeg har vist ladet bedre koder end det der... for jeg er skam ikke helt nybegynder!
håber du kan se meningen i det, ellers må jeg jo bare smide nogle comments ind og rydde lidt op
cyberfessor:> Ork .. koden er som ikke rodet ... den er skam nem at læse .. men øhh .. synes bare der mangler lidt generel forståelse for standarder/uskrevne standarder *G*
jeg er modtagelig for konstruktiv kritik da jeg koder en hel del og gerne vil ha\', at ikke kun de færdige produkt ser ordenligt ud, men også de bagved liggende koder
websitewithouthttp ... does that ring a bell ? *G* Hvor er variabel prefixes henne ? Whatever happened to the concept of ONE include file ? Du inkludere to filer på samtlige sider (af hvad jeg har set) .. du bruger en funktion med EEN linie i til at returnere et recordset ... need I go on ?
det kom af at det skulle kunne bruges på både mit eget website og så detstyrer.dk, så jeg valgte lige at smide de values i en variabel
ang. de inc, var det bare noget jeg lige havde ved hånden, det er ikke normalt måden jeg laver det på... plejer at bruge een side hvor jeg så inkluderer resten, ala default.asp?page=whatever
jeg kan da ikke se hvad der er galt med den function, den gør jo sit arbejde, finder brugernavnet til det givne id
Kontruktiv kritik, jamen så skal det nok lige skrives på en anden måde ;-)
Well..
1. Dine variabler mangler prefixes. F.eks.:
strWebsiteWithoutHTTP istedet for websitewithouthttp intCounter istedet for f.eks. counter
osv.
2.
<%
end if
%>
_JEG_ plejer at smide det på een linie som:
<%[TAB]End If %> [TAB] erstattes af et tabulator stop.
3. Keywords plejer _JEG_ at skrive i \"korrekt case\", f.eks.:
If expr Then \'... End If
Select Case .. End Select osv.
Det giver et bedre overblik synes jeg.
4. Indrykning. Her bruger de fleste (incl. jeg selv) tabulator stop eller 4 mellemrum.
if expr then \'gør noget else \'noget andet end if
Skriver jeg som:
If expr Then \' gør noget Else \' noget andet End If
5. Inkludering af filer der bruges til header formål (Response.Buffer) skal som bekendt altid foregå FØR noget HTML udskrives (hermed menes blanktegn, tekst osv.)
Selvom du udfører koden conditionally (If\'s) så er det en god idé at placere dem i toppen af dine scripts sådan at du er SIKKER på at kunne bruge dem.
1) præfixes er virkelig noget jeg skal TVINGE mig selv til at bruge, og jeg glemmer det ALTID...
2) de
<%
end if
%>
er noget jeg bruger når jeg skriver koden, for derefter at samle det til <% end if %> til slut
3) det skal jeg huske på
4) hvordan vil du så lave det??
Jeg har aldrig læst en bog om asp, højst et par artikler så alt er selvlært... det gør nok at de grundlæggende ting mangler da det meste er kommet hen af vejen :(
Nu startede jeg med Visual Basic FØR ASP og derfra har jeg min vane med prefixes. Det er en MEGET god ting at huske.
*G* Skriv ALTID koden til at starte med som du vil have den til at se ud til slut. Ellers glemmer man hurtigt noget hvis man sidder med noget stort.
Uhm .. indrykning ? Brug tabulator stops og indryk ét pr. f.eks. If, Select Case, Function, Sub osv. Altså f.eks:
<% Function DoStuff(strString) For intIndex = Len(strString) To 1 Step -1 strDummy = strDummy & Mid(strString, intIndex, 1) Next End Function %>
Bare et simpelt (og totalt ubrugeligt) eksempel ..
Jeg har heller aldrig læst en bog om hverken ASP, VB eller PHP .. og ingen artikler. Har derimod selv skrevet et 35 siders ASP tutorial på DK incl. eksempler, såååe :o)
gizmo-gizmo:> Jeg ku ha svoret der stod ASP i titellinien på min IE ;o)
Synes godt om
Ny brugerNybegynder
Din løsning...
Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.