login  Naam:   Wachtwoord: 
Registreer je!
Scripts > PHP > Gastenboeken > Gastenboek V1


Reacties op het script Gastenboek V1

Offline  Tuinstoel
Gepost op: 26 mei 2005 - 20:05
PHP expert



Het zou wel leuk zijn dat je mijn naam er bij vermeldt, bij de smilies functie;-)

Offline  nathanael
Gepost op: 26 mei 2005 - 20:27
Gouden medaille

HTML gevorderde



gedaan 

Offline  b4nkr0bz0r
Gepost op: 27 mei 2005 - 20:33
PHP gevorderde



wel lief dat je HTML enzo kan gebruiken

Offline  Legolas
Gepost op: 27 mei 2005 - 20:38
Onbekend



:(
geen controles op entities e.d.
verkeerd quotegebruik
fetchobject is sloom
paar verbeteringen

Offline  MechaVore
Gepost op: 27 mei 2005 - 22:09
Gouden medaille

PHP gevorderde



gewoon ff de reply's uit de database halen, waaronder mijne . en dan ff een controle erop maken.

op regel 59

vervang je $_POST['naam'] en al die andere dingen door:
htmlspecialchars(addslashes($_POST['naam']), ENT_QUOTES)

Offline  nathanael
Gepost op: 27 mei 2005 - 22:53
Gouden medaille

HTML gevorderde



Hey bedankt lightblade...

Ben nog een beginner PHP (eerste script) dus verder is alle verbeteringen welkom

@tuinstoel: Ik zeg toch dat ik beginner ben... ik snap al die dingen allemaal nog niet... 
Kun je dan minstens uitleggen wat error_reporting(E_ALL); betekend, of moet ik dan als altijd HIERO op google zoeken....

$kleur = ($i++ % 2) ? '#FFFFFF' : '#FAF9FF';
- Waar komt $i vandaan? Werken met error_reporting(E_ALL); dude

Heb ik hier van de site... zie snippets->Kleur van rij verwisselen

Offline  MechaVore
Gepost op: 27 mei 2005 - 22:57
Gouden medaille

PHP gevorderde



graag gedaan , ik was trouwens ook degene die je site liet doorverwijzen .. 

Offline  Legolas
Gepost op: 28 mei 2005 - 11:37
Onbekend



ok nog eentje:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php echo "".$datum.", ".$tijd.""; ?>

Kun je veel beter schrijven als:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?= $datum.", ".$tijd; ?>

Offline  Tuinstoel
Gepost op: 28 mei 2005 - 16:22
PHP expert



TYPE=MyISAM AUTO_INCREMENT=2 ;
- AUTO_INCREMENT=2 kan weg
----
mysql_connect("localhost", "user", "pass");
mysql_select_db("db_name");
- maak een goede foutafhandeling, anders krijg je lelijke errors
----
$select = mysql_query("SELECT * FROM gastenboek ORDER BY id DESC");
$datum = date("d-m-Y");
$tijd = date("H:m:s");
$qaantal = mysql_query("SELECT * FROM gastenboek");
$aantal = mysql_num_rows($qaantal);
- Je selecteert 2 x hetzelfde + je maakt in deze regels ook nog 2 onnodige (naar mijn mening) variabelen aan.
----
$kleur = ($i++ % 2) ? '#FFFFFF' : '#FAF9FF';
- Waar komt $i vandaan? Werken met error_reporting(E_ALL); dude
----
- Variabelen mogen nog steeds buiten quotes
----
if($obj->semail == "Ja")
- Booleans of 0/1 werkt toch wat makkelijker en beter
----
setcookie("gastenboek",1,Time()+3600*24); cookie geset voor 24uur om spam te verkomen
- // vergeten

Dus kortom:
- Werk met error_reporting(E_ALL);
- Leer wat netter scripten
- Gebruik goede variabele namen
- Vergeet geen tekens en woorden
- Variabelen buiten quotes
- gebruik maken van het DATETIME field in MySQL
- Geen onnodige variabelen aanmaken
- Geen dezelfde query's uitvoeren
- altijd met if en else werken als een functie true of false retourneerd

Offline  Euroinfo
Gepost op: 19 november 2005 - 15:27
Nieuw lid



Hoe kan ik de mailfunctie werkend maken?

Offline  nathanael
Gepost op: 19 november 2005 - 17:29
Gouden medaille

HTML gevorderde



Citaat:
[I][B]Euroinfo[/B] schreef op 19 November 2005 - 15:27[/I]

Hoe kan ik de mailfunctie werkend maken?


Deze is al werkend... Gebruik(te) heb zelf ook, met SUCCES 


Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom.
 
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.105s