PHP beginner |
|
Heb wel een aantal leuke vraagjes wegens je script techniek/logica
Waarom dit (regel 49):
Vertaling van code:
Als $count minder dan 10 is volgende code uitvoeren.
Wat als er nou meer dan 10 berichten zijn?
Dan laat hij opeens niks meer zien....
Want hij die if sluit af op regel 137. en er staat geen else.
[begin_edit]
Ik zie nu dat je LIMIT 10 hebt staan in je query
Ik trek deze mening dus terug
Maar als er limit 10 staat. dan krijg je echt nooit meer dan 10 resultaten terug
Dus die regel is heel erg overbodig..
[end_edit]
Waarom roep je bij elk schoutbox bericht een nieuwe tabel op? (regel 51 tot en met 55)
Je kunt veel beter elke keer een nieuwe rij invoegen in de tabel met <tr> enzo.
Waarom deze code op regel 29:
if(!(file_exists('connect.inc.php')))
Daar kan je tog het volgende van maken:
if(!file_exists('connect.inc.php'))
Doet precies hetzelfde
Waarom dit op regel 57:
if(!(isset($_POST['post'])))
if(!(isset($_POST['post'])))
Kan je net zo goed de volgende doen:
if(!isset($_POST['post']))
// Of zelfs beter
if (empty($_POST['post']))
if(!isset($_POST['post'])) // Of zelfs beter if (empty($_POST['post']))
En de volgende code (regel 84,100,116):
Hier kan je weer die empty() gebruiken
[back_on_topic]
Je antwoord op je vraag staat hierboven beschreven.
Nu is het aan jouw de taak om het toe te passen
Dus nu probeer ik het met wat uitleg.
Voor de while loop moet je een tellertje bijhouden
Dus dan doen we dit als voorbeeld
In je while loop moet je zorgen dat het tellertje met 1 optelt
Dit kan je doen door het volgende te doen:
Nu ga je zorgen dat je een kleur genereert.
Dus daar pakken we de volgende code voor:
$kleur = ($tellertje %2 == 0) ? 'black' : 'red';
$kleur = ($tellertje %2 == 0) ? 'black' : 'red';
en nu pas gaan we beginnen met wat html die je in je while loopje wilt
Ik neem als voorbeeld jouw structuur aan.
echo "<table style='background:".$kleur."'>";
echo "<table style='background:".$kleur."'>";
Zo kan het gebruikt worden.
Je kan in plaats van <table> het ook in <tr> verwerken of zelfs <td>
In jouw geval hoe jouw code nu is moet je dat niet doen
Nu is het aan jouw de taak om het simpel stapje voor stapje te verwerken in jouw script.
[off_topic]
Wil je ons wel uitleg geven over jouw script structuur/logica?
Zie het begin van dit bericht over welke punten enzo.
Als je namelijk later hier je werk van gaat maken en je doet dit op een ict school, dan word je heel raar aangekeken.
Somige Sima's vinden het denk ik ook een klein beetje aan de raare kant.
Als je later werk hebt en je doet dit dan heb je niet heel lang dat werk meer
Dus je moet het leren om het meteen op een bepaalde structuur te scripten.
Als je dan een fout erin hebt, en je geeft het script aan iemand anders kan die gemakkelijker de fout eruit halen
Ik hoop dat je nog even wat uitleg geeft over je logica en structuur
Somige sima's zijn er erg benieuwd naar denk ik
Groetjes jeroen. |