[ASP] Er wordt dubbel opgeteld?

[Aakab]

Ok ik sta op het punt om doortedraaien
Ik heb de volgende stukje code:

code:

<!--#include file="include-conn.asp" --><!--#include file ="ayarlar.asp"-->
<%Set Sur = Server.CreateObject("ADODB.Connection")
Sur.Open "DRIVER={Microsoft Access Driver (*.mdb)}; DBQ=" & Server.MapPath(""& dbfolder &"/turkportal.mdb")

idno=Request.QueryString("idno")
Set turkportal = Server.CreateObject("ADODB.Recordset")
sor = "Select * from makale where idno="&idno
turkportal.Open sor,Sur,1,3

turkportal("abc")=turkportal("abc")+1
turkportal.update

Hij leest dus een veld uit het tabelletje makale, ABC..het gegeven uit deze vult hij aan met een waarde van 1, en daarna schrijft hij het weer terug.

Maar inplaats van +1 doet hij plus 2!!! als ik keer 1 doe dan doet hij keer 2..als ik keer 2 doe dan doet hij keer 4.

Ik zie het popupje laden, en daar staat dan bijvoorbeeld eerst 21 maar hij veranderd daarna gelijk weer in 22.

Hij zit niet in een loop ofzo, of iets anders...wie o wie kan me helpen?>

[stp_4]

komt &idno daadwerkelijk ook maar 1 keer voor in je tabel?

[Aakab]

stp_4 schreef op zaterdag 04 december 2004 @ 20:18:
komt &idno daadwerkelijk ook maar 1 keer voor in je tabel?

Ja want het is een autonummering.

Overigens krijgt deze vanuit de vorige pagina deze idno mee(vandaar de get.querystring..)

Pff alleen met deze pagina heb ik het ik draai echt door..

[Verwijderd]

In dit stukje code doe je op zich niets fout. Wat gebeurt er in de include files? Wordt er ergens een redirect in deze pagina gedaan? Kijk eens in je logfiles of deze pagina meer dan eens wordt aangeroepen als je vanaf de pagina waar de idno meegeeft.

HTH

[KurtDB]

Een work-around:

In plaats van met recordsets te werken, kan je ook gewoon 'n update query maken.

code:

sor = "update makale set abc=abc+1 where idno="&Cint(idno)

Die Cint dient om invoercontrole te doen. (wat jij dus alvast niet doet, waardoor je applicatie gevoelig zou kunnen zijn voor vanalle ongure toestanden)

[RobIII]

KurtDB schreef op zondag 05 december 2004 @ 03:38:
Een work-around:

In plaats van met recordsets te werken, kan je ook gewoon 'n update query maken.

code:

sor = "update makale set abc=abc+1 where idno="&Cint(idno)

Die Cint dient om invoercontrole te doen. (wat jij dus alvast niet doet, waardoor je applicatie gevoelig zou kunnen zijn voor vanalle ongure toestanden)

offtopic:
"Controle" zou ik het niet noemen. Ten eerste gaat 'ie klappen (error dus) als je er iets anders in duwt dan een int, en ten tweede kan 'ie maar overweg met een Int (32767 max). Als dat genoeg is, prima. Anders gebruik je CLng. En waarom zou je die niet gebruiken? Kost niks extra

Je hebt overigens natuurlijk wel gelijk door te zeggen dat je ermee voorkomt dat mensen enge dingen met je DB gaan doen

[NMe]

KurtDB schreef op zondag 05 december 2004 @ 03:38:
Een work-around:

In plaats van met recordsets te werken, kan je ook gewoon 'n update query maken.

code:

sor = "update makale set abc=abc+1 where idno="&Cint(idno)

Die Cint dient om invoercontrole te doen. (wat jij dus alvast niet doet, waardoor je applicatie gevoelig zou kunnen zijn voor vanalle ongure toestanden)

Dit is waarschijnlijk niet eens een workaround. Feitelijk doet dit gewoon hetzelfde als in de code uit de topicstart, met als enige verschil dat je de update nu zelf met SQL doet. Ik vermoed dat MrX gelijk heeft: ik denk dat de betreffende code vaker dan één keer aangeroepen wordt, mogelijk door een verkeerde/dubbele include.

[gorgi_19]

RobIII schreef op zondag 05 december 2004 @ 03:53:
[...]

offtopic:
"Controle" zou ik het niet noemen. Ten eerste gaat 'ie klappen (error dus) als je er iets anders in duwt dan een int, en ten tweede kan 'ie maar overweg met een Int (32767 max). Als dat genoeg is, prima. Anders gebruik je CLng. En waarom zou je die niet gebruiken? Kost niks extra

Je hebt overigens natuurlijk wel gelijk door te zeggen dat je ermee voorkomt dat mensen enge dingen met je DB gaan doen

Als dat de enige reden is, is een parametrized query beter

[sopsop]

zet na je update van de recordset eens de volgende regel:

code:

response.end

Kijk dan of de update nog steeds met 2 tegelijk gebeurt. Zo niet, dan wordt de pagina hoogst waarschijnlijk nog een keer geladen door de code na de update.

[Verwijderd]

Heeft het niets met die auto-increment te maken? Dat ie daardoor alles maal 2 doet ofzo?

[MMUilwijk]

Knal eens een response.write in je ASP voordat je er 1 bij optelt en check die waarde even.
En inderdaad, wat staat er in die vage include bestanden van je?

edit:

ik mag toch hopen dat je niet probeert een autonummeringsveld te updaten?

[ Voor 23% gewijzigd door MMUilwijk op 06-12-2004 20:47 ]