login  Naam:   Wachtwoord: 
Registreer je!
 Forum

lus maken lukt me niet

Offline banana - 29/01/2005 12:44 (laatste wijziging 29/01/2005 12:46)
Avatar van bananaOnbekend hey hallo,
ik ben thomas ben nieuw hier op sitemasters.
ik zat eerst altijd op webmastercity maar deze
site vind ik beter 
nu heb ik al meteen een probleem hier met een lus
dit hieronder is de code waar het om draait.
er is uiteraard verbinding gemaakt met db
wie kan helpen?
php code - Bekijk de code zonder highlighting - Klap code in 
  1. $sqlre = "SELECT * FROM reacties WHERE re=".$_GET['id'];
  2. $resultre = mysql_query($sqlre) or die(mysql_error());
  3. $countre = mysql_num_rows($resultre);
  4. while($object=mysql_fetch_object($resultre))
  5.  
  6. $datumre= $objectre->datum{0}.$objectre->datum{1}.'-'.$objectre->datum{2}.$objectre->datum{3}.'-'.$objectre->datum{4}.$objectre->datum{5}.$objectre->datum{6}.$objectre->datum{7};
  7.  
  8. if($object->rang==1){ $rang = 'Gebruiker'; }
  9. if($object->rang==2){ $rang = 'VIP gebruiker'; }
  10. if($object->rang==3){ $rang = 'Moderator'; }
  11. if($object->rang==4){ $rang = 'Admin'; }
  12. else { $rangre = 'Gebruiker'; }

9 antwoorden

Gesponsorde links
Offline nemesiskoen - 29/01/2005 14:02 (laatste wijziging 29/01/2005 14:02)
Avatar van nemesiskoen Gouden medaille

PHP expert		 zou je niet beter elseif kunnen gebrukiken, en je while openen met een {

kan je ook zeggen WAT het niet doet, en wat het WEL doet

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  2. $sqlre = "SELECT * FROM reacties WHERE re='". $_GET['id'] ."'"; 
  3. $resultre = mysql_query($sqlre) or die(mysql_error()); 
  4. $countre = mysql_num_rows($resultre); 
  5. while($object=mysql_fetch_object($resultre)){
  6.  
  7.       $datumre= $objectre->datum{0}.$objectre->datum{1}.'-'.$objectre->datum{2}.$objectre->datum{3}.'-'.$objectre->datum{4}.$objectre->datum{5}.$objectre->datum{6}.$objectre->datum{7}; 
  8.  
  9.       if($object->rang==1){ $rang = 'Gebruiker'; } 
  10.       elseif($object->rang==2){ $rang = 'VIP gebruiker'; } 
  11.       elseif($object->rang==3){ $rang = 'Moderator'; } 
  12.       elseif($object->rang==4){ $rang = 'Admin'; } 
  13.       else { $rangre = 'Gebruiker'; } 
  14. }
  15. ?>
Offline banana - 29/01/2005 22:18
Avatar van banana Onbekend dit is mijn code nu, ik heb de lus er uit gelaten
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  2. include('menu.php');
  3. include('connect_1.php');
  4.  
  5. $sql = "SELECT * FROM onderwerpen WHERE id=".$_GET['id'];
  6. $result = mysql_query($sql) or die(mysql_error());
  7. $object = mysql_fetch_object($result);
  8.  
  9. $datum = $object->datum{0}.$object->datum{1}.'-'.$object->datum{2}.$object->datum{3}.'-'.$object->datum{4}.$object->datum{5}.$object->datum{6}.$object->datum{7};
  10.  
  11. if($object->rang==1){ $rang = 'Gebruiker'; }
  12. if($object->rang==2){ $rang = 'VIP gebruiker'; }
  13. if($object->rang==3){ $rang = 'Moderator'; }
  14. if($object->rang==4){ $rang = 'Admin'; }
  15. else { $rang = 'Gebruiker'; }
  16.  
  17. print('<table style="border: 1px solid #CCCCCC;" width="100%" cellspacing="0" cellpadding="3px">');
  18. print('<tr height="26px"><td background="layout/berichtkop.jpg"><table border="0" cellpadding="0" cellspacing="0" width="100%"><tr><td align="left" width="50%">'.$object->titel.'</td><td align="right" width="50%">[ '.$datum.'&nbsp;&nbsp;'.$object->tijd.' ]</td></tr></table></td></tr></table>');
  19. print('<table style="border: 1px solid #CCCCCC; border-top:0px;" width="100%" cellspacing="0" cellpadding="3px" bgcolor="#FAFFFF">');
  20. print('<td width="106px" align="middle">');
  21.  
  22. print($object->naam);
  23. print('<img src="'.$object->avater.'" width="70px" height="70px"><br>');
  24. print('Rang: '.$rang.'<br>');
  25. print('Posts: '.$object->posts);
  26. print('</td><td bgcolor="#CCCCCC" width="1px" style="padding:0px;"></td>');
  27. print('<td align="left" valign="top">');
  28. print($object->inhoud);
  29. print('</td></tr></table>');
  30.  
  31. print('<br>');
  32.  
  33. $sqlre = "SELECT * FROM reacties WHERE re=".$_GET['id'];
  34. $resultre = mysql_query($sqlre) or die(mysql_error());
  35. $countre = mysql_num_rows($resultre);
  36. $objectre=mysql_fetch_object($sqlre);
  37.  
  38. $datumre= $objectre->datum{0}.$objectre->datum{1}.'-'.$objectre->datum{2}.$objectre->datum{3}.'-'.$objectre->datum{4}.$objectre->datum{5}.$objectre->datum{6}.$objectre->datum{7};
  39.  
  40. if($objectre->rang==1){ $rangre = 'Gebruiker'; }
  41. if($objectre->rang==2){ $rangre = 'VIP gebruiker'; }
  42. if($objectre->rang==3){ $rangre = 'Moderator'; }
  43. if($objectre->rang==4){ $rangre = 'Admin'; }
  44.  
  45.  
  46. print('<table style="border: 1px solid #CCCCCC;" width="100%" cellspacing="0" cellpadding="3px">');
  47. print('<tr height="26px"><td background="layout/berichtkop.jpg"><table border="0" cellpadding="0" cellspacing="0" width="100%"><tr><td align="left" width="50%">Re: '.$object->titel.'</td><td align="right" width="50%">[ '.$datumre.'&nbsp;&nbsp;'.$objectre->tijd.' ]</td></tr></table></td></tr></table>');
  48. print('<table style="border: 1px solid #CCCCCC; border-top:0px;" width="100%" cellspacing="0" cellpadding="3px" bgcolor="#FAFFFF">');
  49. print('<td width="106px" align="middle">');
  50.  
  51. print($objectre->naam);
  52. print('<img src="'.$objectre->avater.'" width="70px" height="70px"><br>');
  53. print('Rang: '.$rangre.'<br>');
  54. print('Posts: '.$objectre->posts);
  55. print('</td><td bgcolor="#CCCCCC" width="1px" style="padding:0px;"></td>');
  56. print('<td align="left" valign="top">');
  57. print($objectre->inhoud);
  58. print('</td></tr></table>');
  59. ?>
Offline Addow - 29/01/2005 22:31 (laatste wijziging 29/01/2005 22:32)
Avatar van Addow Gouden medaille

PHP beginner		 Brrrr...dit steekt vol fouten hoor, en alleszins niet al te netjes gescript. Heb nu iets te weinig tijd om 't allemaal te gaan overtypen, maar een paar tips:

1] je selectiequery's : "SELECT * FROM reacties WHERE re='".$_GET['id']."'" -> Properder
2] 4 iffen & 1 else, die else wordt enkel uitgevoerd wanneer niet aan de laatste if wordt voldaan. (Dus je zal nooit als VIP-gebruiker weergegeven kunnen worden)
3] Liefst "avatar" ipv "avater" eh 
4] Waar wil je je lus?
Offline nemesiskoen - 30/01/2005 01:03 (laatste wijziging 30/01/2005 01:07)
Avatar van nemesiskoen Gouden medaille

PHP expert		 maar WAT doet het verkeerd banana, daar heb je nogaltij niet op geantwoord
Offline Thomas - 30/01/2005 01:10
Avatar van Thomas Moderator Euh... Als je de wijsneus uit wilt hangen moet je natuurlijk wel weten waar je het over hebt - het is volgens mij toch echt avatar, ik heb nog nooit van een avater gehoord...

Voor if-statements die telkens hetzelfde controleren, maar telkens een andere waarde is het switch-statement uitgevonden...

En het zou idd handig zijn om:
- HTML niet te echo'en
- fatsoenlijk in te springen
- te vertellen wat nou het probleem is idd 
Offline nemesiskoen - 30/01/2005 09:11
Avatar van nemesiskoen Gouden medaille

PHP expert		 na even googelen

avatar = engels
avater = nederlands

het is een nederlandstalige site (want er komen woorden als gebruiker in voor), daarom denk ik dat het aangewezen is het nederlandstalige woord te gebruiken
Offline banana - 30/01/2005 10:44 (laatste wijziging 30/01/2005 10:46)
Avatar van banana Onbekend t was inderdaad avater - avatar is een schrijffoutje, op jullie vraag
de variabelen waar re achter staat worden niet weergeven, ik weet zeker dat ze er wel zijn.
en de lus moet om het stukje van de reactie komen, dit dus:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. //speciale variablen
  2. $datumre= $objectre->datum{0}.$objectre->datum{1}.'-'.$objectre->datum{2}.$objectre->datum{3}.'-'.$objectre->datum{4}.$objectre->datum{5}.$objectre->datum{6}.$objectre->datum{7};
  3.  
  4. if($objectre->rang==1){ $rangre = 'Gebruiker'; }
  5. if($objectre->rang==2){ $rangre = 'VIP gebruiker'; }
  6. if($objectre->rang==3){ $rangre = 'Moderator'; }
  7. if($objectre->rang==4){ $rangre = 'Admin'; }
  8.  
  9. //weergeef reacties
  10. print('<table style="border: 1px solid #CCCCCC;" width="100%" cellspacing="0" cellpadding="3px">');
  11. print('<tr height="26px"><td background="layout/berichtkop.jpg"><table border="0" cellpadding="0" cellspacing="0" width="100%"><tr><td align="left" width="50%">Re: '.$object->titel.'</td><td align="right" width="50%">[ '.$datumre.'&nbsp;&nbsp;'.$objectre->tijd.' ]</td></tr></table></td></tr></table>');
  12. print('<table style="border: 1px solid #CCCCCC; border-top:0px;" width="100%" cellspacing="0" cellpadding="3px" bgcolor="#FAFFFF">');
  13. print('<td width="106px" align="middle">');
  14. print($objectre->naam);
  15. print('<img src="'.$objectre->avater.'" width="70px" height="70px"><br>');
  16. print('Rang: '.$rangre.'<br>');
  17. print('Posts: '.$objectre->posts);
  18. print('</td><td bgcolor="#CCCCCC" width="1px" style="padding:0px;"></td>');
  19. print('<td align="left" valign="top">');
  20. print($objectre->inhoud);
  21. print('</td></tr></table>');

[edit]
en op de tip fatsoenlijk in te springen: ik hoef nergens in te springen..
Offline Gerard - 30/01/2005 14:09 (laatste wijziging 30/01/2005 14:09)
Avatar van Gerard Ouwe rakker
Citaat:
en op de tip fatsoenlijk in te springen: ik hoef nergens in te springen..


Je hoeft niks, tis alleen maar dat je later zelf nog weet waar je in godsnaam mee bezig was :]

Overzichtelijk scripten is dat
Offline banana - 30/01/2005 19:15
Avatar van banana Onbekend jah dat snap ik ook en ik doe het ook maar in dit stukje script zie ik nergens waar het zou moeten
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2025 Sitemasters.be - Regels - Laadtijd: 0.207s