Reacties op het script Gastenboek class V1.0
|
|
 Gepost op: 10 december 2007 - 14:59 |
|
|
|
PHP expert
|
Paar kleine opmerkingen die ik direct opmerk.
-Je kijkt of $setting gelijk is aan 'all'. Dit zou ik niet doen maar ik zou gebruik maken van een extra functie.
- Hetzelfde bij setSetting(). Maak een functie setSettings waarbij men meerdere settings kan doorgeven via een array.
- Dat van die regulaire expressie heb ik al verteld  .
- Daarnaast moet je ook wat rekening houden met de types. Zo is aantal_messages een integer en geen string. Ik weet dat php daar niets van aantrekt maar dit is wel handig voor de code gebruiker. Dit werkt wel maar toch is het belangrijk.
En zet de SQL code er ook bij |
|
|
|
|
| Gepost op: 10 december 2007 - 15:15 |
|
|
|
Crew algemeen
|
getMessage, getEmail; waarom dan? Als je het zo zou doen, zou ik je variabele $message en $email private maken; zoals het nu is heeft het namelijk geen nut...
Ik kan toch net zo goed dit doen:
echo $gastenboek->email; ipv echo $gastenboek->getEmail();?
voor de rest iig wel een mooi script met een deftige logica achter |
|
|
|
|
| Gepost op: 12 december 2007 - 14:32 |
|
|
|
PHP expert
|
nog over addMessage().
Waarom voer je die checks niet uit op het moment dat je set*() uitvoert?
Dat is het hele idee achter die functies.
Ook maakt dat addMessage een stuk duidelijker.
Verder zou ik dan een array bij houden met welke onderdelen goed zijn.
Iets als volgt:
<?php
$aGecheckt = array('email'=>true,'bericht'=>true); // naam is dus nog niet goed.
$aVerplicht = array('email'=>true,'bericht'=>true,'naam'=>true);
$aFout = array_diff_key($aVerplicht,$aGecheckt);?>
<?php $aGecheckt = array('email'=>true ,'bericht'=>true ); // naam is dus nog niet goed. $aVerplicht = array('email'=>true ,'bericht'=>true ,'naam'=>true ); $aFout = array_diff_key($aVerplicht,$aGecheckt);?>
|
|
|
|
|
| Gepost op: 12 december 2007 - 15:20 |
|
|
|
Beheerder
|
| Slim ikkedikke. Ik ga er even naar kijken. |
|
|
|
|
| Gepost op: 19 december 2007 - 18:43 |
|
|
|
HTML beginner
|
| het zou eigenlijk in 2 klassen moeten opgedeeld zijn. |
|
|
|
|
| Gepost op: 19 december 2007 - 19:56 |
|
|
|
Beheerder
|
|
|
|
|
| Gepost op: 30 december 2007 - 10:59 |
|
|
|
HTML beginner
|
| Het is natuurlijk subjectief en relatief aan de performantie, maar ik zou het netter vinden als je een gastenboekcontroller hebt met een collection van gastenboekberichten. Op die manier kan je je gastenboekberichten eventueel nog op een andere manier benaderen. Het zorgt voor een betere modulariteit, alhoewel het in dit voorbeeld misschien wat overdreven is. |
|
|
| Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom. |
|
|
|
Wat is jouw favoriete javascript framework? (S: 18, R: 2)
