GOLDTEK BOOK V1.0 (LIVRE D'OR DYNAMIQUE)

Commentaire de kiki2sirom le 12/08/2006 23:47:10

j'ai regardé le code, j'ai pas pu tester encore...
plusieurs choses malgré tout, parfois importantes :
* le poster.php ne contient pas de php (?!?!?), appelle le poster.html !!! idem pour le index.php ->index.html !!
* il faut éviter de fermer puis rouvrir php systématiquement quand il n'y a pas de HTML !! (exemple aleatoire.php, ligne 35-36)
* là il faudrait que quelqu'un me confirme, mais moi j'ai appris à écrire une boucle comme ça "$i++" et non "++$i". Y a-t-il une règle ?
* mysql_connect('localhost', 'root', '');
mysql_select_db('ideetek');
à mettre dans un fichier de connection, et mettre un include après sur chaque php si nécessaire (imagine que tu veuilles par exemple sécuriser ton serveur MySQL et que tu mettes donc un mot de passe ou que tu changes le nom de ta BDD, il faudrait que tu le changes sur CHAQUE page, alors qu'avec un include, 1 SEULE fois)
* il est passé où le DOCTYPE ????
* <script language="Javascript"> à remplacer par <script type="text/javascript"> en fait l'attribut type est obligatoire pour les tags <script> et <style> (j'ai du déjà le dire au moins 20 fois depuis...!!!!)
* une fonction pour un addslashes et un htmlentities, je suis sceptique sur l'optimisation du code là-dessus (des spécialistes d'optimisation pourrait nous en dire plus ??)
* après une idée personnelle mais bon :
tu fais souvent ça j'ai remarqué, ça sert pas à grand chose puisque tu ne réutilises pas par exemple le '$totalDesMessages' autre part dans la page

$totalDesMessages = $donnees['nb_messages'];
$nombreDePages  = ceil($totalDesMessages / $nombreDeMessagesParPage);

pourquoi ne pas écrire plutôt (plus rapide) :
$nombreDePages  = ceil($donnees['nb_messages'] / $nombreDeMessagesParPage);

* il existe aussi les fonctions en javascript !! plutôt que de faire le test sur les champs non renseignés de lancer un "<meta http-equiv="refresh" content="0;url=poster.php?action=poster">"
je suis désolé mais moi pareil que pour la boucle, j'ai appris à faire des submit et à tester en javascript le remplissage des variables, et non de faire ça, m'enfin pfffffffffffffff...

De plus, une bonne partie du code s'est déjà vu sur une autre de tes sources ?? C'est ptet' pas trop la peine de reposter !!!

Je crois que c'est tout, si quelqu'un voit autre chose, qu'il n'hésite pas !!

Je testerai lundi la source, damn !! c'est le WE quand même ;-)

cdt

kiki

Commentaire de kiki2sirom le 13/08/2006 15:17:54

va voir là :

http://perso.orange.fr/coin.des.experts/reponses/faq9_64.html

cdt

kiki

Commentaire de samirpaul le 14/08/2006 10:35:31

Ok !
Mais moi je sais pas ce qui ne vas pas avec ce code ; mais en tout cas ça marche parfaitement !
alors pourquoi des tonnes de corrections ? est-ce pour me faire craindre la programmation ?

Commentaire de malalam le 14/08/2006 15:12:42 administrateur CS

Tu veux une note ? soit...je pensais attendre tes corrections avant de noter. mais si tu insistes...ma note pour ce code en l'état : 2/10.

Les modifications, on te les as expliquées. Le pourquoi aussi.

Commentaire de malalam le 15/08/2006 14:32:08 administrateur CS

On t'a clairement expliqué ce qui n'allait pas. Tu t'en fiches et tu veux être noté en l'état, c'est ton problème.

Commentaire de kankrelune le 18/08/2006 15:19:13

Normalement ++$i est légèrement plus rapide que $i++ après je sais pas si c'est valable pour une boucle... .. .

Pour samirpaul... mieux vaut des commentaires (constructifs) et pas de note que le contraire... faut perdre les réflexes d'écolier... .. . ;o)

@ tchaOo°