[php] While loop geeft max 2 resultaten

[Verwijderd]

Hey mensen,

Ik heb het volgende probleem.
Ik Heb een database connectie en vraag in mn code daar alle resultaten van een klant te geven. De klant selecteer je door op een andere pagina aan te geven van wie je de resultaten wilt zien.
Nu heb ik de volgende code:

code:

$klwk = "SELECT * FROM klantwk WHERE klant = '$klant'";
$res = mysql_query ($klwk);
$row = mysql_fetch_array($res);

//blabla blabla

if ("$row[ma]" == "" && "$row[di]" == "" && "$row[wo]" == "" && "$row[do]" == "" )
{
echo "<table border=\"1\"><tr><td width=\"500\">Geen data geselecteerd! 
<br><a href=\"communicatie.php\">Terug</a></td></tr></table>";
}
else
{  
while ($row = mysql_fetch_object($res)) 
   {
echo "<table border=\"1\">
    <tr>
    <td width=\"100\"></td>
    <td width=\"75\">$row->wk</td>
    <td width=\"75\">$row->ma</td>
    <td width=\"75\">$row->di</td>
    <td width=\"75\">$row->wo</td>
    <td width=\"75\">$row->do</td>
    
  </tr>    
</table>";
}
echo "<br><br><a href=\"communicatie.php\">Terug</a>";
}
?>

Het probleem is dat ie maar max 2 resultaten weer geeft.
Heb al veel gezocht maar kan het probleem niet vinden.

Wie weet waar het fout gaat.

Bettings

[TwoR]

je gebruikt $row = mysql_fetch_array($res); en $row = mysql_fetch_object($res) en die noem je allebei $row. Als je een van beiden nu eens een andere naam geeft wat gebeurt er dan? En je moet er natuurlijk zeker van zijn dat er meer dan twee resultaten zouden moeten zijn!

[ Voor 21% gewijzigd door TwoR op 16-07-2004 10:11 ]

[Jacco Swart]

Hoe ziet de DB eruit? (Met andere woorden zit er wel meer dan 2 tupels in de tabel? )

[Verwijderd]

Ik heb nu van

code:

while ($row = mysql_fetch_object($res))

gemaakt

code:

while ($row = mysql_fetch_array($res)) 
//bla bla
<td width=\"75\">$row[wk]</td>

maar het resultaat is hetzelfde..

Voor een klant waarvan ik zeker weet dat er 4 tupels in de tabel zitten geeft hij als resultaat max 2 resultaten.

De tabel is als volgt

klant
wk
ma
di
wo
do

En ik haal alles op uit de tabel ==>

code:

$klwk = "SELECT * FROM klantwk WHERE klant = '$klant'";
$res = mysql_query ($klwk);
$row = mysql_fetch_array($res);

[ Voor 8% gewijzigd door Verwijderd op 16-07-2004 10:34 ]

[X-Lars]

PHP:

if ("$row[ma]" == "" && "$row[di]" == "" && "$row[wo]" == "" && "$row[do]" == "" )

Dit is ook wel vreemd, die quotes om de $row[]'s, volgens mij horen die daar niet.

Je kunt wellicht ook beter iets doen als volgt (zoals al gezegd):

PHP:

$klwk = "SELECT wk, ma, di, wo, do FROM klantwk WHERE klant = '$klant'";
$res = mysql_query ($klwk);
$row = mysql_fetch_array($res);

if ($row[ma] == "" && $row[di] == "" && $row[wo] == "" && $row[do] == "" )
{
echo "<table border=\"1\"><tr><td width=\"500\">Geen data geselecteerd! 
<br><a href=\"communicatie.php\">Terug</a></td></tr></table>";
}
else
{  
while ($data = mysql_fetch_object($res)) 
   {
echo "<table border=\"1\">
    <tr>
    <td width=\"100\"></td>
    <td width=\"75\">$data->wk</td>
    <td width=\"75\">$data->ma</td>
    <td width=\"75\">$data->di</td>
    <td width=\"75\">$data->wo</td>
    <td width=\"75\">$data->do</td>
    
  </tr>    
</table>";
}
echo "<br><br><a href=\"communicatie.php\">Terug</a>";
}

[Verwijderd]

Het is nu zo dat ie alleen het eerste resultaat niet weer geeft.

Zet je meer resultaten in de DB dan geeft ie alleen de eerste niet weer.

Rarara hoe kan dat?

Dat met die quotes om die code met if($row[ma] bla bla geeft een error maar dnek ook niet dat daar het probleem zit

[ Voor 27% gewijzigd door Verwijderd op 16-07-2004 10:43 ]

[AquilaDus]

Is het niet een beetje vreemd dat je de table sluit in elke iteratie van de whileloop?

[Verwijderd]

Zo werkt t ook niet

code:

echo "<table border=\"1\">";
while ($row = mysql_fetch_array($res)) 
   {
echo "<tr>
    <td width=\"100\"></td>
    <td width=\"75\">$row[wk]</td>
    <td width=\"75\">$row[ma]</td>
    <td width=\"75\">$row[di]</td>
    <td width=\"75\">$row[wo]</td>
    <td width=\"75\">$row[do]</td>
    
  </tr>";    

}
echo "</table>";

geeft nog steeds het eerste resultaat niet

[X-Lars]

Ik geloof dat het te maken heeft met de positie van de pointer in $res. Zet die eens naar het begin voor je begint aan de while-loop.

Edit: of in $row, dat is waarschijnlijker, want dat is een array

[ Voor 22% gewijzigd door X-Lars op 16-07-2004 10:51 ]

[Jacco Swart]

weet niet of het werkt bij dit maar als je nu eens het volgende toevoegd onderaan:

echo "<pre>";
print_r($res);
echo "</pre>";

dan geeft hij keurig weer wat er in het array zit. Zit er dan wel het eerste resultaat in?

[ Voor 22% gewijzigd door Jacco Swart op 16-07-2004 10:52 ]

[Jacco Swart]

oeps verkeerde knopje

[ Voor 93% gewijzigd door Jacco Swart op 16-07-2004 10:51 ]

[Jacco Swart]

-------

[ Voor 98% gewijzigd door Jacco Swart op 16-07-2004 10:52 ]

[Verwijderd]

bij de code

echo "<pre>";
print_r($res);
echo "</pre>";

geeft hij als resultaat:

Resource id #3

Dit doet hij ook als een klant maar 2 tupels in de tabel heeft staan
dus denk dat het niet werkt

[TwoR]

X-Lars schreef op 16 juli 2004 @ 10:49:
Edit: of in $row, dat is waarschijnlijker, want dat is een array

Dat is dus wat ik ook zei maar volgens mij heeft de TS dat nog niet geprobeerd.

[X-Lars]

Jacco Swart schreef op 16 juli 2004 @ 10:50:
weet niet of het werkt bij dit maar als je nu eens het volgende toevoegd onderaan:

echo "<pre>";
print_r($res);
echo "</pre>";

dan geeft hij keurig weer wat er in het array zit. Zit er dan wel het eerste resultaat in?

$res is geen array, maar een resultset

[ Voor 7% gewijzigd door X-Lars op 16-07-2004 11:17 ]

[Verwijderd]

Kom nou mensen... een basic database loopje... dit hoeft toch geen problemen te vormen.

PHP:

$res = mysql_query("SELECT * FROM klantwk WHERE klant = '$klant' ORDER BY wk ASC");
if (mysql_num_rows($res)) {
  echo "<table>";
  while ($row = mysql_fetch_array($res)) {
    echo "<tr><td>$row[wk]</td>";
    if ($row["ma"] || $row["di"] || $row["wo"] || $row["do"]) {
      echo "<td>$row[ma]</td><td>$row[di]</td>";
      echo "<td>$row[wo]</td><td>$row[do]</td>";    
    } else {
      echo "<td colspan='4'>Niets gevonden voor deze week</td>";
    }
    echo "</tr>";
  }
  echo "</table>";
} else {
  echo "Geen rijen gevonden";
}

Als ik de bedoeling van de TS goed begrepen heb.

[ Voor 95% gewijzigd door Verwijderd op 16-07-2004 11:31 ]

[X-Lars]

*iets doms*

En daarnaast is het veel leuker als de TS er zelf uitkomt

[ Voor 34% gewijzigd door X-Lars op 16-07-2004 11:35 ]

[Verwijderd]

Het is me inderdaad gelukt Heb het script als volgt:

code:

$klwk = "SELECT wk,ma,di,wo,do FROM klantwk WHERE klant = '$klant' ORDER BY wk ASC";
$klwk2 = "SELECT ma,di,wo,do FROM klantwk WHERE klant = '$klant'";
$res = mysql_query ($klwk);
$res2 = mysql_query ($klwk2);  
$tab = mysql_fetch_array($res2);

if ("$tab[ma]" == "" && "$tab[di]" == "" && "$tab[wo]" == "" && "$tab[do]" == "" )
{
echo "<table border=\"1\"><tr><td width=\"530\">Geen data geselecteerd! <br></td></tr></table>
<a href=\"communicatie.php\">Terug</a>";
}
else
{ 
echo "<table border=\"1\">";

while ($row = mysql_fetch_array($res)) 
   {
echo "<tr>
    <td width=\"100\"></td>
    <td width=\"75\">$row[wk]</td>
    <td width=\"75\">$row[ma]</td>
    <td width=\"75\">$row[di]</td>
    <td width=\"75\">$row[wo]</td>
    <td width=\"100\">$row[do]</td>
    
  </tr>";    

}

maar dis niet zo mooi als het script wat OneofBorg gaf

tnx voor de reacties.

[X-Lars]

Gebruik toch die van OneofBorg maar (nofi), want dit is wel heel erg inefficiënt.

[ Voor 4% gewijzigd door X-Lars op 16-07-2004 11:53 ]

[NMe]

Verwijderd schreef op 16 juli 2004 @ 11:47:

PHP:

if ("$tab[ma]" == "" && "$tab[di]" == "" && "$tab[wo]" == "" && "$tab[do]" == "" )

Iemand heeft je er in dit topic al eerder op gewezen, maar je lijkt het niet nodig te vinden dit te veranderen. Variabelen horen niet tussen quotes als ze niet in een grotere string staan, en bovendien moeten indexes van een array juist wel weer tussen quotes. Dat dit bij jou geen errors oplevert komt alleen maar omdat je niet dit boven je code hebt staan:

PHP:

error_reporting(E_ALL);

Maak van die regel dus liever netjes dit:

PHP:

if ($tab["ma"] == "" && $tab["di"] == "" && $tab["wo"] == "" && $tab["do"] == "" )

Of nog beter:

PHP:

if (empty($tab["ma"]) && empty($tab["di"]) && empty($tab["wo"]) && empty($tab["do"]))

X-Lars schreef op 16 juli 2004 @ 11:53:
Gebruik toch die van OneofBorg maar (nofi), want dit is wel heel erg inefficiënt.

OneofBorg laat ook 2 kleine steekjes vallen. Ten eerste dus dat de indexen van de arrays niet tussen quotes staan, bijvoorbeeld op regel 7.

En mysql_num_rows() (regel 2) returnt feitelijk (bijna) nooit false, dus is het fout om dit te gebruiken. Ik zelf had gekozen voor deze regel:

PHP:

if ((mysql_num_rows($res) === false) || (mysql_num_rows($res) === 0)) {

[ Voor 49% gewijzigd door NMe op 16-07-2004 12:03 ]

[crisp]

NMe84 schreef op 16 juli 2004 @ 11:56:
[...]
En mysql_num_rows() (regel 2) returnt feitelijk (bijna) nooit false, dus is het fout om dit te gebruiken. Ik zelf had gekozen voor deze regel:

PHP:

if ((mysql_num_rows($res) === false) || (mysql_num_rows($res) === 0)) {

In een conditional statement vind ik het niet fout hoor, 0 evalueert immers gewoon naar false...

[Verwijderd]

NMe84, nee hoor.

Als je al in een string zit, hoef je geen quotes meer te gebruiken om je associatieve array te indexeren.

PHP:

$boe["schrik"] = "Aaaah!";

echo "Je riep $boe[schrik], schijterd!";

Dit is gewoon ondersteunde syntax, en levert geen fouten op, ook niet met error reporting aan.

In mijn string was het dus terecht gebruik. Bovendien is de originele code van Bettings, :-

PHP:

"$row[ma]" == ""

Dus niet erg logisch, maar ook niet fout. Dit werkt gewoon zoals het zou moeten werken.

Voor de geïnteresseerden, multidimensionale arrays kun je op deze manier niet in een string gebruiken. Het volgende :-

PHP:

echo "Je bent een $jij[karakter][beschrijving]!";

werkt bijvoorbeeld niet.

[crisp]

Verwijderd schreef op 16 juli 2004 @ 12:44:
NMe84, nee hoor.

Als je al in een string zit, hoef je geen quotes meer te gebruiken om je associatieve array te indexeren.

PHP:

$boe["schrik"] = "Aaaah!";

echo "Je riep $boe[schrik], schijterd!";

Dit is gewoon ondersteunde syntax, en levert geen fouten op, ook niet met error reporting aan.

Ik weet zeker dat dit in vroegere versies (iig in 4.3.1) wel degelijk een warning opleverde, en waarschijnlijk zal dit in PHP5 ook wel een warning opleveren:

PHP:

<?php

error_reporting(E_STRICT);

$boe['schrik'] = 'Aaaah!'; 

echo "Je riep $boe[schrik], schijterd!";

?>

[Verwijderd]

Aangezien een andere thread die op dit onderwerp leek over te stappen gesloten is, wil ik deze even nieuw leven inblazen.

Ik heb 't zelf even getest, en bij mij doettie dat dus niet.


Exhibit A:

PHP:

  error_reporting(E_ALL);

  $zaad["banaan"] = "vies";

  echo "Wat is dat $zaad[banaan]!";

Levert in 4.3.4 nog niet het minste notice'je op.

Verwijderd schreef op 25 juli 2004 @ 21:00:
Oftewel: in PHP4 is het ondersteunde syntax, in PHP5 toegestane syntax, en ik gebruikt het want ik vind het er zo een stuk overzichtelijker uitzien dan met een kwak aanhalingstekens erdoor verweven.

[ Voor 10% gewijzigd door Verwijderd op 26-07-2004 09:16 ]

[RwD]

Verwijderd schreef op 16 juli 2004 @ 12:44:
...

PHP:

echo "Je bent een $jij[karakter][beschrijving]!";

werkt bijvoorbeeld niet.

Weer een reden om de { en } te gebruiken

PHP:

<?
$jij[karakter][beschrijving] = 'zenuwpees';
echo "Je bent een {$jij[karakter][beschrijving]}!";
?>

Werkt dus weer wel

[ Voor 19% gewijzigd door RwD op 26-07-2004 09:59 ]

[Verwijderd]

RwD schreef op 26 juli 2004 @ 09:58:
[...]
Weer een reden om de { en } te gebruiken

PHP:

<?
$jij[karakter][beschrijving] = 'zenuwpees';
echo "Je bent een {$jij[karakter][beschrijving]}!";
?>

Werkt dus weer wel

Ah, dat zou kunnen ja. En

PHP:

  echo "Ben je echt een {$jij[karakter]}[beschrijving]?";

... zou dat werken?

Overigens heb je natuurlijk gezondigd tegen de aanhalingstekens bij de assignment. Maar ik zal 't je vergeven want je heb waarschijnlijk gecopy/paste .

[ Voor 6% gewijzigd door Verwijderd op 26-07-2004 10:12 ]

[Verwijderd]

PHP:

<?php
    error_reporting(E_ALL);
    $jij['karakter']['beschrijving'] = "Zenuwpees";
    echo "Jij bent een {$jij[karakter]}[beschrijving]!<br />"; //Output: Jij bent een Array[Beschrijving]
    echo "Jij bent een {$jij[karakter][beschrijving]}!<br />"; //Output: Jij bent een Zenuwpees! (Met notices)
    echo "Jij bent een {$jij['karakter']['beschrijving']}!"; //Output: Jij bent een Zenuwpees!
?>

zonder single quotes krijg je wel mooie notices:
Notice: Use of undefined constant karakter - assumed 'karakter'
Notice: Use of undefined constant beschrijving - assumed 'beschrijving'
Dus het gebruik van single quotes lijkt me wel wat netter

[drm]

Als iedereen nou gewoon voortaan printf zou gebruiken, zouden we deze discussie niet elk halfjaar hoeven houden en hebben we voortaan allemaal leesbare en minder error-prone code.

[RwD]

En dan constant mn variablen door mekaar gaan halen, en onzinoutput genereren.

Ik weet wel hoe dat gaat hoor als je een beetje moe bent van een bijna voorbije werkdag