Reacties op het script Een eenvoudig gastenboek
|
|
 Gepost op: 25 januari 2005 - 20:40 |
|
|
|
PHP expert
|
Goed gastenboek!
Maar wel wat simpel, mss wat meer functies? |
|
|
|
|
| Gepost op: 25 januari 2005 - 22:45 |
|
|
|
PHP gevorderde
|
mss cookie zetten tegen spam ?
en html uit zetten |
|
|
|
|
| Gepost op: 26 januari 2005 - 14:13 |
|
|
|
Lid
|
hoe bedoel je html uitzetten??
simpel was de bedoeling, het zal nog serieus worden uitgebreid. |
|
|
|
|
| Gepost op: 26 januari 2005 - 16:48 |
|
|
|
Crew algemeen
|
Het hoort simpel te zijn, zoals je zegt.
Maar het moet ook veilig zijn vind ik.
Ik zou er addslashes() + strip_tags() aan toevoegen.
Dan word het een stuk veiliger.
Ook bij het bericht, als je dan die text weghaalt die er staat, word er aangenomen dat het bericht juist is.
bericht, longtext, is dat niet beetje overdreven?
Ik heb ooit een text volgekregen (bericht, tpye = text) met een Lipsum dummy text.
Maar als iemand dat gaat lezen...
Bij datum ook zoiets, een texttype voor een datum.
Dan kun je beter een datetime meegeven, of eerder nog varchar ipv text...
Mzzls,
Rensjuh |
|
|
|
|
| Gepost op: 26 januari 2005 - 20:51 |
|
|
|
PHP expert
|
"index.php?link=gastenboek.php"
// dat had beter
$_SERVER['REQUEST_URI'];
// kunnen zijn
if(isset($_POST['door']) and isset($_POST['bericht']) and $_POST['door']!='Typ hier uw naam' and $_POST['bericht']!='Typ hier uw bericht!' ) {
// isset is niet nodig bij post fields, je kunt ook gewoon strlen() doen bijvoorbeeld, zodat je een minium aantal tekens in moet voeren
$bericht = $_POST['bericht'];
$door = $_POST['door'];
$datum = date("d-m-Y H:i:s", time());
$bericht = str_replace("\n", "<br>", $bericht);
// Het aanmaken van deze variabelen is onnodig, je kunt in je query gewoon de superglobals plaatsen.
// Wat ik ook niet echt slim vindt is dat je \n vervangt met <br>, daarvoor heb je nl2br() bij de output!
// En voor de datum heb je een veldtype genaamd DATETIME of DATE. Dan kun je de huidige datum gemakkelijk invoeren met NOW() in de query
$door = $obj->door;
$datum = $obj->datum;
$bericht = $obj->bericht;
// Ook weer niet nodig dat je deze variabelen omzet, kost in principe alleen maar generatie tijd.
"index.php?link=gastenboek.php" // dat had beter $_SERVER['REQUEST_URI']; // kunnen zijn if(isset($_POST['door']) and isset($_POST['bericht']) and $_POST['door']!='Typ hier uw naam' and $_POST['bericht']!='Typ hier uw bericht!' ) { // isset is niet nodig bij post fields, je kunt ook gewoon strlen() doen bijvoorbeeld, zodat je een minium aantal tekens in moet voeren $bericht = $_POST['bericht']; $door = $_POST['door']; // Het aanmaken van deze variabelen is onnodig, je kunt in je query gewoon de superglobals plaatsen. // Wat ik ook niet echt slim vindt is dat je \n vervangt met <br>, daarvoor heb je nl2br() bij de output! // En voor de datum heb je een veldtype genaamd DATETIME of DATE. Dan kun je de huidige datum gemakkelijk invoeren met NOW() in de query $door = $obj->door; $datum = $obj->datum; $bericht = $obj->bericht; // Ook weer niet nodig dat je deze variabelen omzet, kost in principe alleen maar generatie tijd.
|
|
|
|
|
| Gepost op: 27 januari 2005 - 17:46 |
|
|
|
Lid
|
Citaat: "index.php?link=gastenboek.php"
// dat had beter
$_SERVER['REQUEST_URI'];
// kunnen zijn
in de structuur van mijn site is het handiger het op te lossen op mijn manier
Citaat: if(isset($_POST['door']) and isset($_POST['bericht']) and $_POST['door']!='Typ hier uw naam' and $_POST['bericht']!='Typ hier uw bericht!' ) {
// isset is niet nodig bij post fields, je kunt ook gewoon strlen() doen bijvoorbeeld, zodat je een minium aantal tekens in moet voeren
wat zou dan de minimum lengte moeten zijn???
Citaat: $bericht = $_POST['bericht'];
$door = $_POST['door'];
$datum = date("d-m-Y H:i:s", time());
$bericht = str_replace("n", "<br>", $bericht);
// Het aanmaken van deze variabelen is onnodig, je kunt in je query gewoon de superglobals plaatsen.
// Wat ik ook niet echt slim vindt is dat je n vervangt met <br>, daarvoor heb je nl2br() bij de output!
// En voor de datum heb je een veldtype genaamd DATETIME of DATE. Dan kun je de huidige datum gemakkelijk invoeren met NOW() in de query
die variabelen zijn handiger om query's met te maken,
nl2br() kende ik nog niet
zoals ik al zei ben ik nog altijd een beginner en kan ik nog niet echt goed werken met die datetime en date(zoals ik al zei, er moet nog aan gewerkt worden)
|
|
|
|
|
| Gepost op: 28 januari 2005 - 14:50 |
|
|
|
PHP expert
|
1) Die structuur van jouw site interesseert ons weinig, het gaat erom dat jij hem post op sitemasters, en daar dien je jouw script dan op aan te passen.
2) Beetje logisch nadenken hè, een naam op z'n kleinst kan 2 tekens bevatten volgensmij (bijv. Ed), een bericht zou ik toch minimaal 4/5 tekens bevatten, anders komt er alleen maar gespam in je gastenboek.
3) Ja het kan handiger zijn, maar 't is maar dat je het weet, en IMO lijkt het niet handig, maar meningen verschillen.
4) nl2br() ken je dus vanaf nu wel  , maar dan had je alsnog beter de \n bij de output kunnen vervangen, want een <br> in een textfield (bewerken) staat niet mooi.
5) Date(time) moet je gewoon een keertje tutorial over zoeken, best interessant.
Veel succes verder |
|
|
|
|
| Gepost op: 29 januari 2005 - 13:33 |
|
|
|
Lid
|
ben aan het werken aan een uitbreiding, met emoticons, ik zal het dan aanpassen hoe jullie het graag zouden zien en opnieuw posten
greetings |
|
|
|
|
| Gepost op: 06 november 2005 - 16:28 |
|
|
|
Nieuw lid
|
hmm ik snap er echt niks van heeft iemand voor mij een frame waarin ik dit gb kan zetten dus met bellen enal ik heb geen verstand van php maar van css en html wel
alvast bedankt:!:  :!: |
|
|
|
|
| Gepost op: 28 juni 2006 - 21:03 |
|
|
|
Nieuw lid
|
Ik krijg een error als ik het gastenboek op mn site zet...
Parse Error, 500 error ?
zoiets...
:o |
|
|
|
|
| Gepost op: 29 juni 2006 - 18:13 |
|
|
|
Nieuw lid
|
Kan iemand mij helpen?
;-) |
|
|
|
|
| Gepost op: 01 juli 2006 - 12:52 |
|
|
|
Nieuw lid
|
| okee, Voor elkaar |
|
|
|
|
| Gepost op: 19 april 2007 - 18:52 |
|
|
|
Nieuw lid
|
Bij mij werkt het niet als ik een bericht wil posten dan zegt hij dat de pagina niet bestaat  |
|
|
|
|
| Gepost op: 05 februari 2010 - 21:06 |
|
|
|
Nieuw lid
|
JelmerJ schreef: Bij mij werkt het niet als ik een bericht wil posten dan zegt hij dat de pagina niet bestaatÂ
Verander GASTENBOEK.PHP in INDEX.PHP. Misschien helpt dat |
|
|
|
|
| Gepost op: 05 februari 2010 - 21:07 |
|
|
|
Nieuw lid
|
Leuk gemaakt, alleen nogal klein script
Misschien kan je er een admin script aan toevoegen? Ik wil hier wel aan meehelpen hoor :-)
Ik heb zelf ook een gastenboek gemaakt
Gvm |
|
|
|
|
| Gepost op: 26 mei 2010 - 09:41 |
|
|
|
Nieuw lid
|
Ik ben beginnend webdesigner en mijn ervaring met PHP is erg klein. Maar nu heb ik een opdracht waar ik wel PHP bij moet gebruiken. Ik moet een simpel gastenboek maken en na wat zoekwerk op internet kwam ik hier. Ik heb de stappen doorlopen en heb het voor een deel alles gedaan. Toch krijg ik 5 foutmelding en PHP kennis is zo beperkt dat ik niet verder.
Kan iemand mij misschien verder helpen?
MvG |
|
|
| Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom. |
|
|
|
Wat is jouw favoriete javascript framework? (S: 18, R: 2)
