login  Naam:   Wachtwoord: 
Registreer je!
 Forum

foutje in script (denk ik)

Offline jeffreydv - 01/10/2005 21:06
Avatar van jeffreydvHTML beginner ik begrijp het niet meer...
hieronder staat het script van wat ik in mijn nieuwsbrief heb staan.
maar het probleem is, dat er soms opeens iets staat met een andere kleur, dus zomaar 1 regel met een andere lettertype kleur.
of er staat zomaar opeens <br> tussen.
kan iemand zien wat ik fout doe?

  1. <?php
  2. $sql = "SELECT * FROM dvd WHERE datum ORDER BY id DESC LIMIT 20";
  3. $query = mysql_query($sql) or die("<b>Foutje:</b><br>".mysql_error());
  4. $count = mysql_num_rows($query);
  5. if(empty($count)){
  6. //niks
  7. }
  8. else{
  9.  
  10.  
  11. while( $show = mysql_fetch_object( $query ) ) {
  12.  
  13. echo "<a OnMouseOut=\"window.status=''; return true\" OnMouseOver=\"window.status='$show->artiest - $show->titel'; return true\" onFocus=\"this.blur()\" href=javascript:popUp('$LedensysteemLocatie"."show.php?id=$show->id')><font color=black face=veranda style='font-size: 9px; text-decoration:none'>$show->artiest - $show->titel</font></a><Br>";
  14.  
  15.  
  16.  
  17. }
  18.  
  19. }
  20. ?>

3 antwoorden

Gesponsorde links
Offline Gerard - 02/10/2005 00:34 (laatste wijziging 02/10/2005 00:36)
Avatar van Gerard Ouwe rakker Hoewel ik niet kan verklaren waarom dat gebeurt heb ik toch even je scriptje herschreven. Ik wist niet goed wat je met de where deed in je query dus daarom heb ik maar gewoon dat stukje eruit gelaten.

  1. <?php
  2. $qExtractItems = mysql_query("SELECT * FROM dvd ORDER BY id DESC LIMIT 20");
  3.  
  4. if (mysql_num_rows($qExtractItems) > 0) {
  5.  
  6. while ($aItemInfo = mysql_fetch_assoc($qExtractItems)) {
  7.  
  8. echo "<a OnMouseOut=\"window.status=''; return true\" OnMouseOver=\"window.status='".$aItemInfo['artiest']." - ".$aItemInfo['titel']."'; return true\" onFocus=\"this.blur()\" href=javascript:popUp('".$LedensysteemLocatie."show.php?id=".$aItemInfo['id']."')><font style='font-family: Verdana; color: #000000; font-size: 9px; text-decoration:none'>".$aItemInfo['artiest']." - ".$aItemInfo['titel']."</font></a><br />";
  9.  
  10. }
  11.  
  12. }
  13. ?>


De reden waarom ik dit heb gedaan is omdat er wat flaws in jouw versie zitten. Vooral het controleren van de $count slaat nergens op. Wanneer je met mysql_num_rows() het aantal rijen op gaat halen wat MySQL heeft returned komt er dus 0 in de variabele te staan mocht er niks zijn. Die variabele is dus NOOIT empty.

Daarnaast moet je je variabelen buiten de quotes gaan houden.
Offline Duikboot - 02/10/2005 01:12
Avatar van Duikboot Grafische gevorderde Zoals Proximus al zegt klopt die where niet. Je doet'

WHERE datum

Alleen datum heeft geen waarde. Je zou het dan zo moeten doen:

WHERE datum = '$datum' (of WHERE datum = $datum als de datum getallen zijn)

De waarde van $datum geef je mee op de volgende manier:

pagina.php?datum=12062005

Maar dat wist je natuurlijk al 
Offline jeffreydv - 02/10/2005 10:02
Avatar van jeffreydv HTML beginner bedankt.. kzal et ff probere..
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.195s