[ASP] If statement werkt niet.

[UberTweaker]

Beste Tweakers,

Ik ben bezig met een simpel scriptje om records toe te voegen aan een database. Om te voorkomen dat gegevens in een van de kolommen al bestaan in een ander record, check ik eerst met een zoekopdracht of dat het geval is. Als de recordset >0 is, krijgt men een error te zien dat er een poging wordt gedaan om dubbele gegevens in toe voeren. Als als de recordset niets bevat, wordt het INSERT INTO statement aangeslingerd.

Maar het if statement werkt niet, er zou een end expected moeten komen als ik "end if" weghaal. Maar dat gebeurt niet. het lijkt wel alsof hij de hele if statement niet ziet!

Dit is het scriptje

ASP:

rs_Archief.Open strSQL, adoCon,3,3
i=rs_Archief.RecordCount
response.write("The number of records is: " & i)
if i>0 Then response.write"ERROR!!!!!" else
    'Initialise the strSQL variable with an SQL statement to query the database
    strSQL="INSERT INTO Tbl_Archief(Medium, Nummer--- etc.

Met end if op regel 140 (zie hieronder)krijg ik fout Instructie wordt verwacht, zonder end if op regel 140 krijg ik geen fout.
Zien jullie wat ik fout doe? Of heb ik stront in mijn ogen?
Ik heb aardig wat ASP regels achter de rug en heb dit nog nooit meegemaakt.
Alvast bedankt.
Gehele script:
Niet relevant

[ Voor 66% gewijzigd door NMe op 18-11-2009 16:48 ]

[Nic]

een if..then..else moet ofwel helemaal op 1 regel staan, ofwel allebei op meerdere regels. Maar niet van beide een beetje...

Dus:

code:

if a>0 then
   b=1
else
  b=0
end if

of

code:

if a>0 then b=1 else b=0

[ Voor 14% gewijzigd door Nic op 18-11-2009 16:19 ]

[UberTweaker]

Hey inderdaad, het allebei op een andere regel staan. Nooit geweten dat het zo moest.
Weer wat geleerd

Bedankt voor je reactie!

[Nic]

Trouwens, lees je even in over sql injection. Wat je nu doet is gevaarlijk. Ik weet niet of dit een website is waar mensen ook van buitenaf bij moeten kunnen, maar imho kun je je het beste aanwennen om *altijd* maatregelen te nemen, dan kun je het ook nooit vergeten.

[UberTweaker]

Ok, tnx voor de tip.

De site wordt alleen intern op het intranet benaderd door werknemers. Dus het is niet direct een gevaar, maar wel verstandig om het te voorkomen idd.