WINN GUESTBOOK FR

Commentaire de aKheNathOn le 06/04/2010 10:07:41

c'est du beau boulot !

Juste une remarque :
<?php require('../data/dbcon.php'); ?>
<?php require('../data/functions.php'); ?>
<?php
session_start();

Quand tu ouvres et refermes les balises comme ça il y a le risque que PHP rajoute un line-break non ? Ta session risque d'en pâtir si les headers partent trop tôt ...

J'ai pas encore testé mais ça mérite un 10 facile vu la finition et le travail !

Bonne continuation,
akh

Commentaire de aKheNathOn le 06/04/2010 23:10:20

salut, dslé pour l'anglicisme, un line-break c'est un retour à la ligne tout simplement.

Autre truc pas très bon à ce sujet, les retours à la ligne sont composés de \r\n c'est un script fait très probablement avec un editeur sous windows, et dans chaque fichier tu as un ?>, avec en plus un saut de ligne, en require ou bien en include cela execute le buffer de sortie (exemple : admin/editpost.php)

Ce qui est à la "mode" en ce moment en PHP c'est "d'oublier" volontairement le ?> à la fin d'un fichier de librairie PHP pour éviter ce genre d'erreurs (j'ai vérifié pas de retour à la ligne pour functions et dbco mais le ?> reste inutile)

Du coup ma remarque est la suivante, lors d'une commande de type :

<?php ... ?> blabla

Le moteur PHP interpréte le <?php ... ?> puis passe à la sortie HTML blabla. En écrivant :

<?php require('../data/dbcon.php'); ?> [\r\n]
<?php require('../data/functions.php'); ?>

Il y a le risque à terme que le script ne fonctionne plus - actuellement php ne respecte pas vraiment la fin du ?> et fait un trim mais si un jour il devient plus stricte le session_open ne passera plus ...

:)) c'est un peu pointilleux comme remarque lol

Sinon ça me plait, pas assez orienté objet et classes mais on sent le gros effort de dév et j'aime bien la finition du projet :)