[ASP] eindeloze loop

[Verwijderd]

Onderstaand scriptje zorgt er bij mijn website voor dat de kolom P_VISITS wordt gereset naar 0.

Een hidden input met de naam "updater" moet het scriptje in werking zetten.

Nu is mijn probleem dat hij blijft loopen! Hij werkt overigens wel. De waarden worden op nul gezet, maar de pagina wordt niet meer geladen.


Iemand de oplossing?

ASP:

<form method="post" action="betreffendepagina.asp">
<input name="updater" type="hidden" value="1">
<input type="submit" value="Reset">
</form> 

Set rs2 = Server.CreateObject("ADODB.Recordset")
strsql = "SELECT P_VISITS FROM T_PRODUCTS"
set rs2 =  my_Conn.Execute(strSql)

do while not rs2.eof
If Request("updater") <> "" THEN
strSql = "UPDATE T_PRODUCTS SET P_VISITS = 0" 

My_Conn.execute(strSql) 
end if    

rs2.movenext
   loop

[ Voor 55% gewijzigd door Verwijderd op 23-06-2004 13:18 ]

[curry684]

Uhm dat stukje code (dat tussen [code=asp] tags veel beter eruit ziet) slaat, excuse the word, totaal nergens op. Ben je niet een paar where-clauses vergeten? Wat moet het ding uberhaupt doen?

Pas aub even je openingspost aan dus adh van P&W FAQ - De "quickstart"

[whoami]

En dan nog kan het waarschijnlijk met 1 SQL Statement.

[Verwijderd]

In de database staat één kolom in de tabel P_VISITS die het aantal bezoekers per pagina bijhoud. Nu is het de bedoeling dat het aantal gereset kan worden door een reset button.

De button werkt wel, maar de pagina blijft druk en komt niet meer tevoorschijn. Pas nadat de pagina gerefreshed wordt wel.

[Sponge]

Zonder where update ie alles al naar 0, dus je hebt geen loops nodig.

code:

do while not rs2.eof
...
rs2.movenext
 loop

kan volgens mij echt gewoon weg

[whoami]

Gewoon:

code:

UPDATE p_visits SET aantal = 0

That's all. Met jouw code ga je alle records n keer op 0 zetten, waarbij n het aantal records in je DB is.

[ Voor 51% gewijzigd door whoami op 23-06-2004 13:22 ]

[Verwijderd]

Opgelost....

Was aan het sukkelen... maar hier is de oplossing:

ASP:

Set rs2 = Server.CreateObject("ADODB.Recordset")
strsql = "SELECT P_VISITS FROM T_PRODUCTS"
set rs2 =  my_Conn.Execute(strSql)


If Request("updater") <> "" THEN
strSql = "UPDATE T_PRODUCTS SET P_VISITS = 0" 

My_Conn.execute(strSql) 
end if

[whoami]

Waarom heb je die eerste SQL nog nodig ?

[Verwijderd]

Mensen bedankt.

Het zijn 1760 pagina's en dat duurt natuurlijk ff... Maar nu doet ie ut in één keer.

Thanx for the help!

[curry684]

Waarom doe je die SELECT nu nog?

En geeft dat hidden field je nu de security-rechten om die reset uit te voeren?

[Verwijderd]

ASP:

If Request("updater") <> "" THEN
Server.CreateObject("ADODB.Recordset")
strSql = "UPDATE T_PRODUCTS SET P_VISITS = 0" 

My_Conn.execute(strSql) 
end if  

Ik ga maar weer eens op cursus... Iemand een leuk adresje?

[ Voor 13% gewijzigd door Verwijderd op 23-06-2004 13:26 ]

[curry684]

www.loi.nl?

Maar beantwoord m'n 2e vraag ook nog even want ik snap het nut van de 'updater' field nog niet echt

[whoami]

Blijkbaar geef je met dat updater veld aan of je die records wilt resetten of neit ?

[gorgi_19]

whoami schreef op 23 juni 2004 @ 13:30:
Blijkbaar geef je met dat updater veld aan of je die records wilt resetten of neit ?

Veld? Er wordt request gebruikt, dus Querystring, Form en Cookies kunnen allemaal deze waarde bevatten..

[TeeDee]

betreffendepagina.asp?updater=1 aanroepen en je stats worden constant gereset?

[Verwijderd]

code:

<form method="post" action="betreffendepagina.asp">
<input name="updater" type="hidden" value="1">
<input type="submit" value="Reset">
</form>

Hierboven zie je dus de "updater". Dit zou net zo goed "poep" kunnen zijn.

Maar het gaat om het idee... het is inderdaad de waarde die aangeeft of er gereset moet worden... = button.

Duudelijk?

[ Voor 24% gewijzigd door Verwijderd op 23-06-2004 13:35 ]

[NMe]

gorgi_19 schreef op 23 juni 2004 @ 13:34:
[...]

Veld? Er wordt request gebruikt, dus Querystring, Form en Cookies kunnen allemaal deze waarde bevatten..

HTML:

<input name="updater" type="hidden" value="1">

[TeeDee]

Verwijderd schreef op 23 juni 2004 @ 13:35:

code:

<form method="post" action="betreffendepagina.asp">
<input name="updater" type="hidden" value="1">
<input type="submit" value="Reset">
</form>

Hierboven zie je dus de "updater". Dit zou net zo goed "poep" kunnen zijn.

Maar het gaat om het idee... het is inderdaad de waarde die aangeeft of er gereset moet worden... = button.

Duudelijk?

Het is wel duudelijk, alleen lijkt het me niet veilig. Ik bedoel, als je nog meer van dit (imho) ingrijpende soort scripts hebt, wil ik jou niet als opdrachtgever. Nofi

[curry684]

Verwijderd schreef op 23 juni 2004 @ 13:35:
Maar het gaat om het idee... het is inderdaad de waarde die aangeeft of er gereset moet worden... = button.

Duudelijk?

Je kunt toch ook gewoon opvragen welke button er is ingedrukt, dat wordt ook meegePOST namelijk

[gorgi_19]

NMe84 schreef op 23 juni 2004 @ 13:35:
[...]


[...]

Erh... Ik doelde er eigenlijk meer op dat je met
test.asp?updater=1 ook de boel flink kan gaan sarren op vrij eenvoudige wijze.

[Verwijderd]

TeeDee schreef op 23 juni 2004 @ 13:37:
[...]

Het is wel duudelijk, alleen lijkt het me niet veilig. Ik bedoel, als je nog meer van dit (imho) ingrijpende soort scripts hebt, wil ik jou niet als opdrachtgever. Nofi

Dit script draait in de backoffice van de website... dus is er geen reden tot paniek.

[Verwijderd]

Stel dat het wel mogelijk is om test.asp?updater=1 te gebruiken... wat is dan de oplossing?

[Verwijderd]

De waarde is nu geen 1 maar 12314344

Okay?

[sjink]

Request.Form("updater") ipv Request("updater")

[NMe]

Verwijderd schreef op 23 juni 2004 @ 14:03:
[...]


Dit script draait in de backoffice van de website... dus is er geen reden tot paniek.

Wel paniek, niemand is te vertrouwen.

Bovendien hebben we hier ook een edit knopje... 3x achter mekaar posten in een paar minuten tijd is not done en wordt niet geapprecieerd.

[Verwijderd]

Sorry... ik wist niet dat je boos werd... zal er aan denken...

[BaRF]

NMe84 schreef op 23 juni 2004 @ 14:10:
[...]

Wel paniek, niemand is te vertrouwen.

heh lol

[Verwijderd]

Ik vindt overigens de sfeer niet echt goed op GOT...
Je komt als nieuweling hier en wordt gelijk afgezeken alsof je een jochie bent van 20.

[ Voor 4% gewijzigd door Verwijderd op 23-06-2004 14:19 ]

[gorgi_19]

Even dicht, momentje..

[gorgi_19]

En weer open (even de tijd gehad om een berichtje te tikken.. )

Klachten over reacties in een topic kan je kwijt via ; een van de moderators kan dan ingrijpen. Bijsturingen van users of topics in het algemeen is een taak van moderators, alleen wij zien niet altijd alle topics en alle reacties Wel geldt dat het voor veel mensen vervelend leest als je meerdere reacties onder elkaar plaats van dezelfde persoon; vandaar dat we hier liever hebben dat de edit-functie gebruikt wordt om een post aan te passen.

Heb je klachten over de sfeer of over de moderators in het algemeen, dan kan je dit kwijt in een apart topic in Lieve Adjes

---

En nu graag weer ontopic over het eigenlijke probleem

Sowieso geldt wel de regel: vertrouw nooit user input, ook al zit je in een 'beveiligde' omgeving. Immers, ook 'onbewust' kan er dan grote schade worden aangericht of foutmeldingen naar voren worden gebracht. Je mag er vanuit gaan dat er in iedere applicatie fouten nog blijven hangen, dus 'overbodige' fouten laten zitten is in de meeste gevallen niet gewenst.
Voor mensen die trouwens meer interesse hebben in security e.d. http://microsoft.sitestre.../DEV312_files/Default.htm

[ Voor 40% gewijzigd door gorgi_19 op 23-06-2004 14:28 ]