[PHP/Mysql] Results uitlezen

[Grum]

NIET zo:

code:

$theResult = mysql_query($theQuery);
if($theRow = mysql_fetch_array($theResult))
{
   do
   {
    // doe wat ruigs
   } while($theRow = mysql_fetch_array($theResult));
}
else
{
   // no rows
}

maar zo:

code:

$theResult = mysql_query($theQuery);
if(mysql_num_rows($theResult))
{
   while($theRow = mysql_fetch_array($theResult))
   {
    // doe wat ruigs
   }
}
else
{
   // no rows
}

Deze post omdat ik dit nu alweer voor de 3e keer voorbij zie komen in de laatste paar dagen.

Ik vraag me af waar de mensen et vandaan halen

[Breuls]

Euhm..

wat is je probleem?

[Grum]

Dat ik anti ranzige/slechte structuren ben.

[Ericston]

Hmm, die eerste herken ik...

Ik geloof dat of van www.php.net of van www.devshed.com komt. Weet het bijna zeker, het kan haast niet anders omdat ik bijna nergens anders kom voor PHP/MySQL gerelateerde dingen.

Kan natuurlijk ook zo zijn dat ik het me fout herinner.

[Breuls]

Op zondag 03 maart 2002 12:44 schreef Grum het volgende:
Dat ik anti ranzige/slechte structuren ben.

Ah.

Ik moet zeggen dat die eerste idd nogal is

[me1299]

Sorry voor het kicken van dit topic. Maar ik zou graag willen weten wat het grote bezwaar is van de bovenste constructie?

Dat voor sommige misschien niet netjes is oke, maar is het ook beduidend trager oid?

[Janoz]

Op dinsdag 23 april 2002 14:50 schreef DeathKnight het volgende:
Sorry voor het kicken van dit topic. Maar ik zou graag willen weten wat het grote bezwaar is van de bovenste constructie?

Dat voor sommige misschien niet netjes is oke, maar is het ook beduidend trager oid?

Je hebt nu 2 posities waarin je hetzelfde aan $theRow toekent. Dat is onoverzichtelijk, foutgevoelig en onoverzichtelijk ..

STel je veranderd het fetch statement of de varname, dan moet je dat nu op twee plaatsten doen ipv 1 wat eigenlijk behoorlijk onlogisch is...

[me1299]

Op dinsdag 23 april 2002 14:55 schreef Janoz het volgende:

[..]

Je hebt nu 2 posities waarin je hetzelfde aan $theRow toekent. Dat is onoverzichtelijk, foutgevoelig en onoverzichtelijk ..

STel je veranderd het fetch statement of de varname, dan moet je dat nu op twee plaatsten doen ipv 1 wat eigenlijk behoorlijk onlogisch is...

Nou je het zegt

Toch moet ik eerlijk bekennen dat ik al voor lange tijd de bovenste constructie gebruik en het nogal wennen wordt om iets anders te gebruiken.

iig bedankt voor de toelichting.

[Grum]

Hij is toch veel logischer ?

Je kijkt of der wel of geen rows zijn, waarom dan niet gewoon de functie die er SPECIAAL voor is gemaakt gebruiken

[steenz]

Ok, de 1e variant is een stuk minder logisch, maar als iedereen maar gewoon topics zou openen over onlogisch programmeren...
Ik weet ook nog wel een paar structuren die mensen vaak gebruiken die niet netjes zijn... Ook posten?

[kaandorp]

Ja graag! Goed innitiatief van Grum. Ben wel blij met dit soort dingen. Hier leer je tenminste wat van.

[bigtree]

Op dinsdag 23 april 2002 15:19 schreef Grum het volgende:
Hij is toch veel logischer ?

Je kijkt of der wel of geen rows zijn, waarom dan niet gewoon de functie die er SPECIAAL voor is gemaakt gebruiken

Omdat die door de hele recordset heen moet lopen. En dat kan nogal inefficient worden bij grotere resultaten. Je wilt niet weten hoeveel records er zijn, maar of er ueberhaubt records in je recordset zitten. Daarom.

[Verwijderd]

en dit dan

code:

if($theResult = mysql_query($theQuery))
{
   while($theRow = mysql_fetch_array($theResult,MYSQL_ASSOC))
   {
    // doe wat nuttigs 
   }
}
else
{
   // no rows
}

werkt ook goed en scheelt weer een functie

[Brakkie]

Op woensdag 24 april 2002 12:01 schreef bigtree het volgende:

[..]

Omdat die door de hele recordset heen moet lopen. En dat kan nogal inefficient worden bij grotere resultaten. Je wilt niet weten hoeveel records er zijn, maar of er ueberhaubt records in je recordset zitten. Daarom.

Klinkt wel logisch dit.

[Janoz]

Op woensdag 24 april 2002 12:01 schreef bigtree het volgende:

[..]

Omdat die door de hele recordset heen moet lopen. En dat kan nogal inefficient worden bij grotere resultaten. Je wilt niet weten hoeveel records er zijn, maar of er ueberhaubt records in je recordset zitten. Daarom.

num_rows hoeft helemaal niet door de hele recordset heen te lopen. Die functie is hier juist speciaal voor bedoeld. Hij leest dit zeg maar uit uit de resultset die de db genereerd. Die heeft dus zelf al geteld hoeveel records erin zitten. De enige manier die sneller is is door gewoon een count query uit te voeren, maar dan kun je niet bij de data natuurlijk.

[Janoz]

Op woensdag 24 april 2002 12:06 schreef woeitje het volgende:
en dit dan

code:

if($theResult = mysql_query($theQuery))
{
   while($theRow = mysql_fetch_array($theResult,MYSQL_ASSOC))
   {
    // doe wat nuttigs 
   }
}
else
{
   // no rows
}

werkt ook goed en scheelt weer een functie

Dit werkt niet zoals het bedoeld is. mysql_query levert alleen false op als de query is mislukt, niet als er geen records worden teruggegeven.

[supersook]

jah, deze foute code heeft mij ook een hoop problemen bezorgd, heb het toen zelf opgelost met mysql_num_rows > 0, maar zo kan dus blijkbaar ook.

[Verwijderd]

Op woensdag 24 april 2002 12:08 schreef Janoz het volgende:
Dit werkt niet zoals het bedoeld is. mysql_query levert alleen false op als de query is mislukt, niet als er geen records worden teruggegeven.

euhm ja..... wat loop ik nou te kletsen sorry was beetje in de war denk ik. Gister beetje teveel zon op me bol gehad, helemaal verbrand

[bigtree]

Op woensdag 24 april 2002 12:07 schreef Janoz het volgende:

[..]

num_rows hoeft helemaal niet door de hele recordset heen te lopen.

Je hebt gelijk; ik was in de war met ADO. Daar weet je pas het aantal resultaten als je bij het laatste record bent, net als bij mysql_unbuffered_query dus.