Database+kode-tjek

Test http://www.eksperten.dk/spm/38703

Test http://www.eksperten.dk/spm/38703

Der er en fejl i din forespørgsel FS=alle, så den ville jeg ikke bruge.

Lav en ny forespørgsel med tabellerne sommerhus og postnr. Medtag alle felter fra sommerhus og kun feltet by fra tabellen postnr.

Ret så linien i sql-sætningen i asp-koden til den nye forespørgsel.

Det vil også gå hurtigere at hente siden, da den ikke skal søge efter så mange data.

jalle>>> Det kan jeg ikke rigtig få til at virke, desuden skal inventar vel også med? Skal den så ikke også med i forespørgelsen..

Jeg har kan hente en opdateret zip her:

http://213.237.25.21/files/sommerhus.zip

Jeg har lagt en postnummertabel ind i databasen med samtidige danske postnr. Jeg har også ændre lidt på dine relationer.

Håber at du kan bruge dette.

Jalle>>> Så skulle der ikke være brug for andre optimeringer?
Så skal jeg i gang med et større indskrivnings-arbejde. Det ville for at side det på godt thybosk være \"træls\", at finde ud af at man skulle til at ændre hele lortet fordi man have lavet en lille fejl, formegentlig i databasen.

Der er noget der hedder:

rs.activeconnection = cn     
rs.cursortype = adopenforwardonly   
rs.locktype = adlockreadonly


Skal det så sættes ind her eller hvad, kender du noget til det?:
<%

Set Conn = Server.CreateObject(\"ADODB.Connection\")
Conn.Open \"DRIVER={Microsoft Access Driver (*.mdb)}; DBQ=\" & server.MapPath(\"sommerhuse.mdb\")

SQLstmt = \"SELECT * FROM Q_Sommerhus WHERE husid = \" & Request(\"husid\")

Conn.Execute (SQLstmt)

Set rs = conn.Execute(SQLstmt)
rs.activeconnection = cn     
rs.cursortype = adopenforwardonly   
rs.locktype = adlockreadonly

    adresse = rs(\"adresse\")
    Postnr = rs(\"Postnr\")
    by = rs(\"by\")
    billede = rs(\"Billede\")
    beskrivelse = rs(\"Beskrivelse\")
       
%>

Det skulle gøre databasen hurtigere.

Til det første spm.ville jeg lave prisen om, således at den kommer til at se sådan ud:

tabel uge
id | autonummering
ugenr | tal

tabel pris
id | autonummering
husid | tal  (nøgle)
ugeid | tal (nøgle)
pris(sæson) | valuta

Det er altid godt at have et felt med autonummering selvom man ikke skal bruge det til noget. På denne måde kan du sikre du dig, at de samme uger ikke kommer fremkommer flere gange på det samme hus. Du kan også frasortere uger, hvor sommerhusene ikke skal udlejes i.

Spm 2.

adopenforwardonly: En markør der kun kan scrolle fremad.

adlockreadonly: Stanard. Data kan kun læses, men ikke skrives.

Kilde: \"Active Server Pages\"

Jeg tror ikke at det har nogen inflydelse på om du tager det med.

Okay