par MacIntoc » 29 Sep 2007, 20:04
frank9321 a écrit:Toujours dans index.php, ajoute un filtrage des fichiers appelés en paramètre pour éviter un index.php?p=blabla qui renvoie une erreur PHP.
Pourquois changerais-tu index.php?p=accueil par autre chose!?
Pasque rien n'empêche l'utilisateur de le faire
C'est juste une question de sécurité.
Autre chose, tu devrais séparer complètement le PHP du HTML. Dans un fichier, tu génèrerais toutes les données PHP que tu veux afficher et dans un deuxième, tu structurerais ton affichage avec HTML.
Pour cela il faudrait un systeme de templates. Et, a vrai dire, la, j'ai un peu la flemme. En plus, le site n'a pas beaucoup de visiteurs par jour, donc je ne vois pas a quoi cela pourait servir..
Pas besoin d'un système de template ??
C'est juste pour améliorer la clarté des sources.
Par exemple :
page.php:
- Code:
<?php
$sql='SELECT data
FROM table';
$query=mysql_query($sql);
while($result=mysql_fetch_array($query)) $datas[]=$result;
include('page.html.php');?>
page.html.php:
- Code:
<ul><?php
foreach($datas as $data){?>
<li><?php echo $data[0]?></li><?php
}?>
</ul>
Si vous avez d'autres critiques, dites-le moi.
Géré les fichiers et les répertoires sous forme d'arborescence plutôt que de liste barbare ?
[edit]A propos, pour ton idée de splitter automatiquement un gros ficher en plusieurs petits de 2Mo, garde en tête que Free limite le temps d'exécution des scripts.
Par la suite, pour facilité la navigation, tu devrais gérer l'arborescence vue par l'utilisateur à travers une BDD. Ca permettrait à l'utilisateur de ne voir qu'un fichier de 20Mo là où il y en aurait 10 de 2Mo sur le serveur (et de transférer n'importe qu'elle type de fichier, y compris des fichiers php, sans risque pour la sécurité).[/edit]
[quote="frank9321"]
[quote]Toujours dans index.php, ajoute un filtrage des fichiers appelés en paramètre pour éviter un index.php?p=blabla qui renvoie une erreur PHP. [/quote]
Pourquois changerais-tu index.php?p=accueil par autre chose!?[/quote]
Pasque rien n'empêche l'utilisateur de le faire :)
C'est juste une question de sécurité.
[quote][quote]Autre chose, tu devrais séparer complètement le PHP du HTML. Dans un fichier, tu génèrerais toutes les données PHP que tu veux afficher et dans un deuxième, tu structurerais ton affichage avec HTML. [/quote]
Pour cela il faudrait un systeme de templates. Et, a vrai dire, la, j'ai un peu la flemme. En plus, le site n'a pas beaucoup de visiteurs par jour, donc je ne vois pas a quoi cela pourait servir..[/quote]
Pas besoin d'un système de template ??
C'est juste pour améliorer la clarté des sources.
Par exemple :
page.php:
[code]
<?php
$sql='SELECT data
FROM table';
$query=mysql_query($sql);
while($result=mysql_fetch_array($query)) $datas[]=$result;
include('page.html.php');?>
[/code]
page.html.php:
[code]
<ul><?php
foreach($datas as $data){?>
<li><?php echo $data[0]?></li><?php
}?>
</ul>
[/code]
[quote]Si vous avez d'autres critiques, dites-le moi.[/quote]
Géré les fichiers et les répertoires sous forme d'arborescence plutôt que de liste barbare ? :mrgreen:
[edit]A propos, pour ton idée de splitter automatiquement un gros ficher en plusieurs petits de 2Mo, garde en tête que Free limite le temps d'exécution des scripts.
Par la suite, pour facilité la navigation, tu devrais gérer l'arborescence vue par l'utilisateur à travers une BDD. Ca permettrait à l'utilisateur de ne voir qu'un fichier de 20Mo là où il y en aurait 10 de 2Mo sur le serveur (et de transférer n'importe qu'elle type de fichier, y compris des fichiers php, sans risque pour la sécurité).[/edit]
