[PHP] probleem met foreach

[ilovetechno]

Ik heb een script dat mijn foto's weergeeft. Nu heb ik redelijk wat foto's en wil dus meerderepagina te hebben. Dit is gelukt. Echter gaat het hier niet goed. Wat doe ik fout? op iedere pagina heb ik dezelfde output..
Code waar het foutgaat:

PHP:

for($i=0; $i<$aantal2; $i++){ 
      while($data=mysql_fetch_array($sql2)){ 

        foreach ($data as $item){
            echo $item; echo "<BR>";}}
}

Volledige code

PHP:

<? 
require"verbinding2.php"; 
$pagina =isset($_GET["pagina"]);
if(!$pagina){ 
    $pagina=1; 
} 


$sql=mysql_query("SELECT filename FROM fotos")or die("Het selecteren van alle berichten is niet gelukt.Mysql retourneerde: ".mysql_error()); 
$aantal=mysql_num_rows($sql); 

$tot=10;   
$van=($pagina*$tot)-$tot; 


$sql2=mysql_query("SELECT filename FROM fotos LIMIT $van,$tot")or die("Het selecteren van de berichten is niet gelukt. Mysql retourneerde: ".mysql_error()); 
$aantal2=mysql_num_rows($sql2); 
?>
<link rel="stylesheet" href='css/hoverbox.css' type="text/css" media="screen, projection"  />
<link rel="stylesheet" href='css/ie_fixes.css' type="text/css" media="screen, projection" />
<div class="linkermenu">
 <div class="hd"><div class="c"></div></div>
 <div class="bd">
  <div class="c">
   <div class="s" align="center">
   <h1>Supersized</h1>
<?php
echo '<ul class="hoverbox">';
    $count = 0;
    
for($i=0; $i<$aantal2; $i++){ 
      while($data=mysql_fetch_array($sql2)){ 

        foreach ($data as $item){
            echo $item; echo "<BR>";}}
} 
?>

 </ul>
  </li>
  <br><br><br><br><br>
  <?php
  
$aantal3=ceil($aantal/$tot); 

if($aantal3<>'1'){ 
    if($pagina<>'1'){ 
    $pagina2=$pagina-1; 
    print("<A href=\"index.php?p=modules/photo/weergeven&pagina=$pagina2\">vorige</A>&nbsp;"); 
    } 
    for($j=0; $j<$aantal3; $j++){ 
    $k=$j+1; 
    print("[<A href=\"index.php?p=modules/photo/weergeven&pagina=$k\">$k</A>] "); 
    } 
    if($pagina<>$aantal3){ 
    $pagina3=$pagina+1; 
    print("<A href=\"index.php?p=modules/photo/weergeven&pagina=$pagina3\">volgende</A>"); 
    } 
} 
?> 
  
  
  
  
   </div>
  </div>
 </div>
 <div class="ft"><div class="c"></div></div>
</div>
<br>

[Osiris]

't Hele idee van die while-loop bij mysql_fetch_array is juist om alle rows uit te lezen, je bent dus dubbelop bezig met je mysql_num_rows en de for-loop die je erbij gemaakt hebt.

[killercow]

idd,

gewoon met je while loop door de records lopen, je hebt dan toch een array van alle velden? (data bevat dan alle veldjes), $data['filename'] je filename.

En je css class names zijn echt Super niet duidelijk (maar das een andere probleem), class names werken handiger als je ze omschrijvend maakt naar de content die ze beslaan. (dus niet een kleur of positie, maar gewoon content, header, copyright)

[ilovetechno]

killercow schreef op donderdag 12 oktober 2006 @ 20:12:
idd,

gewoon met je while loop door de records lopen, je hebt dan toch een array van alle velden? (data bevat dan alle veldjes), $data['filename'] je filename.

En je css class names zijn echt Super niet duidelijk (maar das een andere probleem), class names werken handiger als je ze omschrijvend maakt naar de content die ze beslaan. (dus niet een kleur of positie, maar gewoon content, header, copyright)

De CCS classes zijn inderdaad wat onduidelijk. Dit ben ik met je eens. Hier kijk ik later nog naar.

wat betreft de code.. Ik heb deze gewijzigd in:

PHP:

for($i=0; $i<$aantal2; $i++){ 
      while($data=mysql_fetch_array($sql2)){ 

        
            echo $data['filename']; echo "<BR>";}
} 
?>

Ik krijg helaas dezelfde output op het scherm.. Als ik naar pagina 2 ga zie ik wederom dezelfde resultaten als op pagina.

[user109731]

Volgens mij stel je $pagina niet in? Moet zoiets zijn:

PHP:

$pagina = isset($_GET["pagina"]) ? $_GET['pagina'] : 1;

In deze code:

PHP:

$pagina =isset($_GET["pagina"]);
if(!$pagina){ 
    $pagina=1; 
}

Zal $pagina 1 of true zijn. Waarschijnlijk word die true naar een 1 omgezet, zodat je idd altijd op pagina 1 zit.

[ Voor 48% gewijzigd door user109731 op 12-10-2006 20:29 ]

[Technicality]

Ik doe het altijd zo:

PHP:

$sql2="SELECT filename FROM fotos LIMIT $van,$tot";
$result=mysql_query($sql2);
while ($row= mysql_fetch_array($result));
 {
  echo $data['filename']; 
  echo "<BR>";
 }

[Verwijderd]

Druk de waarde van $sql2 eens af, daar gaat dus iets fout... Beetje debuggen kan geen kwaad

En een for-loop EN een while-loop Lijkt me niet dat dit goed werkt

En inderdaad, $pagina wordt altijd 1...

[ Voor 40% gewijzigd door Verwijderd op 12-10-2006 20:31 ]

[Janoz]

Ik zie zo snel de fout nog niet, misschien is het een idee om eens wat debug info toe te voegen. Druk bijvoorbeeld eens de waarde va pagina die je binnen krijgt af om te zien of deze klopt.

Die gewijzigde code is nog steeds onzin. Je hebt een lus in een lus. De while gaat alle resultaten bij langs (hij gaat immers door tot er geen resultaten zijn en fetch_array false opleverd) en vervolgens probeer je nog 9 keer die while lus te draaien (waarbij niks gebeurd omdat de recordset al aan het einde is). Verder doe je het tellen van het totaal aantal foto's onnodig zwaar. Je bent een complete resultset aan het op bouwen om vervolgens alleen te vragen hoeveel rows je terug krijgt. Dat kan onnoemelijk veel efficienter wanneer je gewoon een count in je query gebruikt. Je resultset bestaad dan immers maar uit 1 record met 1 kolom.


@hieronder: Dat zeg ik toch? Ik zie niet in wat het nut is van die 4 woorden met een smiley erachter.

[ Voor 6% gewijzigd door Janoz op 12-10-2006 20:39 ]

[Technicality]

Janoz schreef op donderdag 12 oktober 2006 @ 20:31:
Je bent een complete resultset aan het op bouwen om vervolgens alleen te vragen hoeveel rows je terug krijgt. Dat kan onnoemelijk veel efficienter wanneer je gewoon een count in je query gebruikt. Je resultset bestaad dan immers maar uit 1 record met 1 kolom.

oftewel SELECT COUNT etc

[ilovetechno]

Grote prutser schreef op donderdag 12 oktober 2006 @ 20:26:
Volgens mij stel je $pagina niet in? Moet zoiets zijn:

PHP:

$pagina = isset($_GET["pagina"]) ? $_GET['pagina'] : 1;

In deze code:

PHP:

$pagina =isset($_GET["pagina"]);
if(!$pagina){ 
    $pagina=1; 
}

Zal $pagina 1 of true zijn. Waarschijnlijk word die true naar een 1 omgezet, zodat je idd altijd op pagina 1 zit.

Helemaal top! Dom van mijzelf.

Nu werkt het zoals gewenst.

Technicality schreef op donderdag 12 oktober 2006 @ 20:33:
[...]

oftewel SELECT COUNT etc

Ik zal hier eens aandacht aa schenken.

Bedankt!