[ASP] Server gaat hangen na deze code *

[Verwijderd]

Beste mensen.. Ik heb een probleem..

Ik heb een poll die de vragen en antwoorden uit een database haalt...
Nu heb ik dus een admin pagina gebouwd. zodat je via een website de vragen en antwoorden kan veranderen.....

Deze code is:

code:

<%
dim cnn,rst
set cnn = Server.CreateObject("ADODB.Connection")
set rst = Server.CreateObject("ADODB.RecordSet")
cnn.Open("Provider=Microsoft.Jet.OLEDB.4.0; Data Source=" & Server.MapPath("weekly_poll.mdb"))
sqltext = "Select * FROM tblPolls"
rst.Open sqltext,cnn,3,3


dim vraag, antwoord1, antwoord2, antwoord3,antwoord4,aantal
vraag = Request.Form("vraag")
antwoord1 = Request.Form("antwoord1")
antwoord2 = Request.Form("antwoord2")
antwoord3 = Request.Form("antwoord3")
antwoord4 = Request.Form("antwoord4")
aantal = Request.Form("aantal")





rst.AddNew
rst("Question") = vraag
rst("Selection_1") = antwoord1
rst("Selection_2") = antwoord2
rst("Selection_3") = antwoord3
rst("Selection_4") = antwoord4

rst.update


set fso = createobject("scripting.filesystemobject")
set act = fso.opentextfile(server.mappath("aantal.txt"))
if aantal = 2 then
Set act = fso.CreateTextFile(server.mappath("aantal.txt"), true)
act.WriteLine("180")
Else IF aantal = 3 then
Set act = fso.CreateTextFile(server.mappath("aantal.txt"), true)
act.WriteLine("230")
Else
Set act = fso.CreateTextFile(server.mappath("aantal.txt"), true)
act.WriteLine("270")


End IF
End IF 


Response.Redirect "adminweekly_poll.asp"

%>

Nou werkt die poll gebeuren foutloos..
Maar als ik mijn lijstje met vragen en antwoorden invul en ik druk op toevoegen (dan gaat ie dus naar deze code) gaat de server hangen... dan kan ik er een hele poos niet meer bij.

Het laadbalkje onder in IE gaat ook heel langzaam...

Als ik na een kwartier er weer bij kan zijn alle weizigingen toch wel in de database beland... en het goede getal komt ook in aantal.txt...

Dus het werkt wel... Maar waarom hangt mijn server een kwartier ?

Please Help...

[ Voor 12% gewijzigd door Verwijderd op 21-10-2004 02:04 ]

[Verwijderd]

Hoe wordt deze code aangeroepen? Is dit een procedure ofzo? Wordt die toevallig meer dan 1 keer aangeroepen? Hoe groot is tblPolls?

En waarom doe je set act = fso.opentextfile(server.mappath("aantal.txt")) als je hem daarna gaat overschrijven?

[alx]

heb zelf nooit serieus naar Vis Basic (.Net) gekeken (of is dit geen VB?), maar los daarvan zou ik rondom enkele statements de tijd naar een bestandje laten schrijven,
zodat je weet welk deel er zo lang duurt.
Dus bv rond statements als regel 3-5, regel 6, regel 10-16, 22-29, 33-42.
En dan evt voor een 2e keer enkele regels isoleren mocht de grote tijdssink nog niet
duidelijk zijn.
Dan weet je in ieder geval wat er 15 min duurt (en of dat inderdaad in dit stuk code
zit). Post dan je resultaat, wellicht dat iemand die deze taal en de exacte
eigenschappen van de gebruikte functies wel kent het weet.

[Verwijderd]

@simulacrum: di's ASP met als taal VBScript, zie http://msdn.microsoft.com...56/html/vtoriVBScript.asp

[Verwijderd]

wat je in ieder geval niet doet is netjes alles weer afsluiten:

ASP:

rst.close
cnn.close
set rst=nothing
set cnn=nothing

Dat is nou niet direct het antwoord op je probleem, maar het blijft wel erg belangrijk

[P_de_B]

Heb je toevallig Norton Antivirus op de Server? Deze kan niet samenwerken met het Scripting.FileSystemobject.

[gorgi_19]

FSO mag trouwens ook wel eens expliciet gesloten worden. En gebruik Server.CreateObject ipv CreateObject

Zie trouwens ook P&W FAQ - ASP ; dit is mede een kwestie van debuggen. Je code uitkleden totdat je weet waar het fout gaat; welk stukje van de code de oorzaak is. Gooi tussendoor eens de tijd tussendoor; weet je welke procedure het langste duurt.

Op het eerste gezicht lijkt het er enorm op dat je resources gebruikt, niet vrijgeeft en daardoor moet wachten tot de resources time-outen (en dat kan lang duren)

Daarnaast [ASP] in de titel gefrot

[ Voor 71% gewijzigd door gorgi_19 op 21-10-2004 08:59 ]

[j_du_pee]

Je doet aan het eind van je script ook een redirect. Is deze redirect niet toevallig naar dezelfde pagina, of een andere pagina die ook weer redirect
Bij een hangende server kijk ik iig altijd eerst naar de mogelijkheid van infinite loops in mijn scripts...

[KurtDB]

1. Zoals eerder gezegd: alle openstaande connecties (naar db of files) altijd expleciet afsluiten.
2. Een "SELECT *" is al niet positief voor de performantie. Gelieve dus de veldnamen aan te geven.
3. Gebruik in ASP de functie getrows om de resultaten van je query in 'n array te krijgen. Hierna kan je je connectie afsluiten, waardoor er weer minder system resources verspeeld worden.
4. Gelieve al je variablen te declareren. (alhoewel dit niet verplicht is in asp, maar ik zie bv een sqltext die niet gedeclareerd wordt)
5. Je gaat inderdaad eerst een txt-file openen en daarna ga je die txt-file ook aanmaken. De logica die daarachter zit ontgaat me echter compleet.
6. Gelieve ook met queries te werken ipv met die rst.Addnew. Een query kan je ten allen tijde gaan debuggen, terwijl je met die addnew eigenlijk niets kan aanpassen. (ik vind 't sowieso een vieze manier van scripten)

[abeker]

KurtDB schreef op 21 oktober 2004 @ 11:03:
6. Gelieve ook met queries te werken ipv met die rst.Addnew. Een query kan je ten allen tijde gaan debuggen, terwijl je met die addnew eigenlijk niets kan aanpassen. (ik vind 't sowieso een vieze manier van scripten)

Vieze manier van scripten.. mwa smaken verschillen blijkbaar

Deze methode van toevoegen van records heeft wel als voordeel dat je na de de toevoeging meteen de identifier van de nieuwe record weet (indien aanwezig uiteraard). Ook voorkom je door deze methode te gebruiken gratis SQL injection, scheelt dus weer code om je data te escapen.

Je punt over debuggen zie ik niet echt.

Oh ja, die SELECT query kan natuurlijk beter door nooit records op halen, want je wilt eigenlijk alleen de structuur van de tabel weten. Als iemand een nog betere methode weet om die structuur te achterhalen, hoor ik het trouwens graag! (ja je kunt natuurlijk ook zelf fields toevoegen, maar dat is minder generiek en slechter onderhoudbaar)

terug naar de code. Alle objecten die je aanmaakt moet je natuurlijk ook weer netjes afbreken, zoals eerder al aangegeven was. Dus ook je files netjes afsluiten. Die redirect zou ook problemen kunnen opleveren als daardoor oneindig veel redirects onstaan. In dat geval merk je dat wel in FireFox, die geeft na een tijdje de foutmelding 'Redirection limit exceeded'.

[ Voor 16% gewijzigd door abeker op 21-10-2004 11:46 ]

[P_de_B]

abeker schreef op 21 oktober 2004 @ 11:41:
[...]

Vieze manier van scripten.. mwa smaken verschillen blijkbaar

Deze methode van toevoegen van records heeft wel als voordeel dat je na de de toevoeging meteen de identifier van de nieuwe record weet (indien aanwezig uiteraard). Ook voorkom je door deze methode te gebruiken gratis SQL injection, scheelt dus weer code om je data te escapen.

Een insert query is sneller, heeft als je het goed doet geen risico op SQL injection, en je kunt net zo goed ook de nieuwe identity weten. Ik zou echt met SQL queries gaan werken en niet met recordsets.

[haroldd]

P_de_B schreef op 22 oktober 2004 @ 07:26:
[...]


Een insert query is sneller, heeft als je het goed doet geen risico op SQL injection, en je kunt net zo goed ook de nieuwe identity weten. Ik zou echt met SQL queries gaan werken en niet met recordsets.

wat is door jou de aanbevolen methode om na insert query de nieuwe identity te achterhalen?

[TeeDee]

code:

select @@identity as blaat from dbo.tabel

zou moeten werken.

[wizzkizz]

haroldd schreef op 22 oktober 2004 @ 12:04:
[...]


wat is door jou de aanbevolen methode om na insert query de nieuwe identity te achterhalen?

SELECT @@IDENTITY FROM TABLE (Access en SQL-Server)

SELECT LAST_INSERT_ID() FROM TABLE (mySQL)

edit:

TeeDee was nét iets sneller

edit:

je kunt het AS-statement gebruiken om er een naampje aan te geven. je kunt het ook ophalen met rso(0)

[ Voor 23% gewijzigd door wizzkizz op 22-10-2004 12:17 ]

[mulder]

Voor SQL-Server is SCOPE_IDENTITY() te prefereren

[haroldd]

ok bedankt, ik ga het proberen

[abeker]

Gokje: Bad_Beaver gebruikt een Access database, dus de genoemde methoden om de identity op te halen werken niet.

Een insert query is idd sneller dan deze methode, maar het zou best kunnen zijn dat door de extra werk die je moet verrichten (escapen van velden) een query langzamer is. Dat is zeker iets dat ik nog ga testen.

[P_de_B]

abeker schreef op 22 oktober 2004 @ 13:16:
Gokje: Bad_Beaver gebruikt een Access database, dus de genoemde methoden om de identity op te halen werken niet.

Jawel hoor, Access2000 ondersteunt @@identity.

[gorgi_19]

Een insert query is idd sneller dan deze methode, maar het zou best kunnen zijn dat door de extra werk die je moet verrichten (escapen van velden) een query langzamer is.

Ook parametrized queries werken in MS Access; je hoeft dan niet eens te escapen. Geen idee alleen of MS Access ook het execution plan cached.

[ Voor 11% gewijzigd door gorgi_19 op 22-10-2004 19:02 ]

[abeker]

aha, ik dacht dat @@identity alleen werkte in MSSQL

Ik heb het een en ander getest, het gebruiken van een insert query is een factor 3 sneller dan een addnew op een recordset. Zodra je ook de identity wilt ophalen, is het gebruik van queries bijna 2x zo snel. Een 'gewone' query is ongeveer even snel als een parametrized query.

Gelukkig is de snelheid van insert-queries niet van belang bij het project waar ik nu aan werk

[Verwijderd]

Wat ik niet begrijp in je code waarom je een textfile naast je database gebruikt...?

Waarom niet een iets ander datamodel waarin je ook het aantal stemmen opslaat?

Volgens mij ben je dan al van je snelheidsproblemen af.