[/14] voor alle mysql'ende PHP'ers

[Grum]

AAAAARGGGGGGGGGGHHHHHHHHH

ik word er nu echt GEK van ... alle phpers opgelet aub

een constructie als:

PHP:

<?
$result = mysql_query('...');
if($array = mysql_fetch_array($result))
{
   do
   {
       print_r($array);
   } while($array = mysql_fetch_array($result));
}
else
{
   print "nix";
}
?>

is ZOOOOOOOO ontiegelijk krom/scheef/fout/mislukt .. naast onleesbaar

doet et aub zo:

PHP:

<?
$result = mysql_query('...');
if(mysql_num_rows($result))
{
   while($array = mysql_fetch_array($result))
   {
       print_r($array);
   }
}
else
{
   print "nix";
}
?>

dit uit reactie op ongeveer 10 threads die ik vandaag heb gezien die het op deze kromme manier deden ...

plz leer er wat van

TIP (na voorzetje van Janoz)
Gebruik GEEN 'SELECT * FROM table ....' in je scripts dat staat erorm zielig en het is enorm sloom als je een grote table hebt.

Gebruik 'SELECT veld1[,veldx] FROM table ....' in plaats ervan en zoals wel duidelijk zal zijn .. haal alleen op wat je ECHT nodig hebt

PS dit geld natuurlijk niet als je aan het testen bent of het zelf in de mysql prompt invoert ofzo


^{[edit]
1/ jakkes .. copy/paste error, de ';' na de while in mijn stukje kan natuurlijk optyfe (tnx Arien )
2/ zielige foutjes gevonden/gesloopt (tnx P&W'ers )
[/edit]}

[prototype]

Als je toch bezig bent, leg ze dan ook meteen uit hoe je een database abstractie laag kan toepassen

[D2k]

Op zondag 30 september 2001 21:40 schreef Grum_ het volgende <verkort op verzoek

* D2k rent binnen met kladblok in de hand

*noteert* voor in de faq

[Grum]

Op zondag 30 september 2001 21:46 schreef D2k het volgende:

[..]

* D2k rent binnen met kladblok in de hand

*noteert* voor in de faq

quote es niet 2 paginas text mee wil je

[Woef]

volgens mij was mijn thread de druppel die de emmer liet overlopen

[Verwijderd]

Grum_: Doet et aub zo:

PHP:

<?
$result = mysql_query('...');
if(mysql_num_rows($result))
{
   while(mysql_fetch_array($result));
   {
       print_r($array);
   }
}
else
{
   print "nix";
}
?>

Wat doet die punt-komma aan het eind van while (...) ?
Wat doe je met het array dat je krijgt van mysql_fetch_array() ?


Zelf doe ik het zo:

code:

$result = mysql_query('...');
if (!mysql_num_rows($result)) {
    print "nix";
}
while ($array = mysql_fetch_array($result)) {
    print_r($array);
}

[Rense Klinkenberg]

Op zondag 30 september 2001 23:23 schreef Arien het volgende:
Zelf doe ik het zo:

code:

$result = mysql_query('...');
if (!mysql_num_rows($result)) {
    print "nix";
}
while ($array = mysql_fetch_array($result)) {
    print_r($array);
}

Nu loop je ook alle records af als je al weet dat die er nit zijn.

code:

$result = mysql_query('...');
if (!mysql_num_rows($result)) {
    print "nix";
} else {
    while ($array = mysql_fetch_array($result)) {
      print_r($array);
    }
}

Lijkt me een stuk handiger / efficienter

[Verwijderd]

freak007: Nu loop je ook alle records af als je al weet dat die er niet zijn.

Nee, zo werkt while niet.

[Grum]

gniffel ... ; aant einde van de while was een copy/paste fout (uit een do{}while(); loop .. vandaar de ; :PPPP

wat jij doet is toch niet efficienter Arien ?

het is toch het efficientste om een 'if' naar de 'true' kant te laten springen ? (hehe .. iig in c/c++ wel )

en als jij met een dusdanige constructie dingen uit de database wilt halen kan je er wel vanuit gaan dat er wat in staat anders had je et net zo goed NIET kunnen doen

[edit]
ps Arien .. jij bent toch een perl-fanaat ?
do you like obfrusted code ?
[topic=221760/8/25] (post nummer 6 van boven )

[pimlie]

Weet je waar ik gek van word? Van die mensen die de { en } bij elk statement op een nieuwe line zetten. Dat is pas overbodig, doe het toch asjeblief gewoon zo:

PHP:

<?
$result = mysql_query('...');
if(mysql_num_rows($result)) {   
  while(mysql_fetch_array($result))   {       
    print_r($array);   
  }
}else{
  print "nix";
}
?>

[Verwijderd]

Grum_: Aan het einde van de while was een copy/paste fout (uit een do{} while(); loop .. vandaar de ; :PPPP

Je hebt nog een probleempje met je mysql_fetch_array() ... (Je wijst het resultaat nergens aan toe.)

Wat jij doet is toch niet efficienter Arien?

Nee, dat klopt. Je doet een vergelijking meer in (zeer waarschijnlijk) een klein percentage van de gevallen. (In ruil voor minder geneste code die je al snel krijgt in PHP bij gebrek aan betere oplossingen.)

Jij bent toch een perl-fanaat ? Do you like obfrusted code ?

Hij is leuk.

pimlie: Weet je waar ik gek van word? Van die mensen die de { en } bij elk statement op een nieuwe line zetten. Dat is pas overbodig, doe het toch asjeblief gewoon zo: [code]

Dat heet programmeerstijl en dat heeft met voorkeuren enzo te maken.

[pimlie]

Dat heet programmeerstijl en dat heeft met voorkeuren enzo te maken.

En dat is dus precies wat ook het gebruik van de do..while() loop is. Newbiez worden vaak door verwezen naar de manual en wat schetst de verbazing, daar staat juist de do..while() loop uitgelegd!hier dus?

Ben ik nou de enige die dit een beetje een nutteloos topic vind?

[Verwijderd]

pimlie: Newbiez worden vaak door verwezen naar de manual en wat schetst de verbazing, daar staat juist de do..while() loop uitgelegd! Hier dus.

Ja, het is een beetje een open deur intrappen. (Het gaat trouwens nu ondertussen om een while() { ... } loop.)

Ben ik nou de enige die dit een beetje een nutteloos topic vind?

Ach... er zijn nuttigere geweest. OTOH, mensen blijven de meest brakke code produceren dus ik kan me voorstellen dat anderen daar wat aan willen doen... (Al zal het zo goed als geen effect hebben, maar dat is weer een ander verhaal.)

[Lurge]

PHP:

<?
while(mysql_fetch_array($result))
?>

Moet naar mijns inziens worden:

PHP:

<?
while($array = mysql_fetch_array($result))
?>

[Verwijderd]

Dat staat er nu drie keer dus nou moet het toch gaan opvallen.

[Grum]

meer kan ik der niet over zegge

ow .. mja ..

poog je et goed te doen doe je et zelf fout

[Janoz]

Trouwens .. Kan er ook ergens neergezet worden dat SELECT * niet voor mag komen in een script? Dat gebruik je alleen als je lui bent in de console, of als je niet weet welke kolomen er in de db staan....

[Grum]

done

[dusty]

Op zondag 30 september 2001 21:40 schreef Grum_ het volgende:
AAAAARGGGGGGGGGGHHHHHHHHH

ik word er nu echt GEK van ... alle phpers opgelet aub

een constructie als:

PHP:

<?
$result = mysql_query('...');
if($array = mysql_fetch_array($result))
{
   do
   {
       print_r($array);
   } while($array = mysql_fetch_array($result));
}
else
{
   print "nix";
}
?>

is ZOOOOOOOO ontiegelijk krom/scheef/fout/mislukt .. naast onleesbaar

Toch maar eens een reply van mij, aangezien iedereen het als zoete koek slikt:

De methode is niet ontiegelijk krom, niet scheef, niet fout en het is ook niet mislukt.

Wat het wel is hangt af van je doelstelling. Betekent dus dat de mensen die jij het heb zien gebruiken het krom, scheef, fout en mislukt gebruiken. Betekent dus niet dat de Methode fout is.

Programmeer technisch wordt het verschillend verwerkt en kan je er dus verschillende dingen mee doen.

[dirkpostma]

Op maandag 01 oktober 2001 07:01 schreef pimlie het volgende:
Weet je waar ik gek van word? Van die mensen die de { en } bij elk statement op een nieuwe line zetten. Dat is pas overbodig, doe het toch asjeblief gewoon zo:

PHP:

<?
$result = mysql_query('...');
if(mysql_num_rows($result)) {   
  while(mysql_fetch_array($result))   {       
    print_r($array);   
  }
}else{
  print "nix";
}
?>

Is volgens mij een kwestie van smaak... Ik zelf doe wel altijd

PHP:

<?
  if (een of ander iets)
  {
    doe iets
  };
?>

maar een klein vraagje:
waarom niet:

PHP:

<?
$result = mysql_query('...');
if(mysql_num_rows($result)) {   
  while(mysql_fetch_array($result)) {       
    print_r($array); }}
else{
  print "nix"; }
?>

Ben je nog minder regels kwijt...

[D2k]

Op maandag 01 oktober 2001 07:01 schreef pimlie het volgende:
Weet je waar ik gek van word? Van die mensen die de { en } bij elk statement op een nieuwe line zetten. Dat is pas overbodig, doe het toch asjeblief gewoon zo:

hier ga ik mezelf ff quoten uit de faq

Let op niet iedereen is het eens met het bovenstaande.
Zie ook hier.
Die discussie is geweest. Consequentie is belangrijker dan een de keuze maken tussen die paar manieren. Zie ook hier.
Ik pretendeer niet gelijk te hebben maar weet wel uit ervaring, dat hier vaak fouten in ontstaan.

dus je hangt of het 1 aan of het ander