WEB TOP LISTE

Commentaire de coucou747 le 03/11/2005 22:28:10

salut, j'ai testé ton code sans installer la base, c'est très joli, mais voila : rien ne fonctionne et on n'a que des warnings : explications : je n'ai pas installé la base (quand j'ai vu qu'on avait pas de messages error fatale, mais seulement que des warnings, j'ai pas installé la base, mais j'ai lu le code...)

Voila ce qu'il y avait de mauvais dans le code :

Fichier admin\ajouter_sites.php
   $action = isset($_POST['action']) ? $_POST['action'] : 0;
   $mode = isset($_POST['mode']) ? $_POST['mode'] : 0;
   if (!empty($action))
tu vérifie par deux fois ne fait que le champ action soit renseigné... ça fait une optimisation possible...
De plus, la variable $action peut être soit une chaine de caractère, soit un nombre, c'est pas super joli, on sait que PHP est moyennement typé, mais c'est pas une raison !
Pour des questions de compatibilitée, on ne met pas <? mais <?php
Si tu veux mettre des doubles quotes, c'est ton choix (c'est plus lent à l'éxécution), mais sache que mélanger doubles et simple, c'est immonde : on choisit ses méthodes de programmations, mais on y reste fidèle...
dans : admin\mise_a_jour.php
   $j = 0;
   while ($j < $n)
   {
   mysql_query("UPDATE topliste_sites_partenaire SET out=0,cliks=0,moyenne=0");
   $j++;
   }
tu connais for ? for est plus rapide et plus claire que while...
dans : liens.php
<?php echo ''.$fond_page.'' ?>
c'est mignon, mais t'as pas pensé à faire :
<?php echo $fond_page; ?> (sans les guillemets mais avec un ;)
Sinon, dans admin\index.php, t'as de multiples echo à suivres, on se demande pourquoi t'en as pas fait qu'un...
En php, utiliser les @ n'est pas conseillé car ça peut donner un comportement totalement étrange à ton programme sans même que tu ne le sache...

dans ton fichier javascript :
<!-- Begin
ceci n'est pas un commentaire en javascript !
document.form.speudo.value
cette méthode marche, mais est dépressiée : on accède à un élément par son id.
if (!(/^\w+([\.-]?\w+)*@\w+([\.-]?\w+)*(\.\w{2,3})+$/.test(form.email.value))){
normalement, l'objet n'est pas l'expression régulière, mais la chaine de caractère (enfin, si, mais c'est compliqué : l'objet auquel on s'interesse... l'expreg n'est ici qu'un argument de méthode normalement...)

en php comme en javascript, t'as des accolades en trop...

Pour le code html, tu devrais choisir entre minuscules et majuscules, mais ne pas mélanger. Un peu de CSS ne ferait pas de mal... (pour enlever les atributs qui peuvent être remplis en CSS)
height="30" => pas d'unitées ?

Tes expressions régulières de mails sont fausses : on a le droit aux caractères : + { et } dans les mails (j'ai vraiment l'impression de me répéter...)

Commentaire de Antidote le 07/11/2005 21:13:31

$action = isset($_POST['action']) ? $_POST['action'] : 0;
$mode = isset($_POST['mode']) ? $_POST['mode'] : 0;

if (!empty($action))

Même vérif ? pas d'accord. La première vérifie l'existance et met une valeur par défaut si n'existe pas la seconde test si elle contient quelques chose ?

si j'ai une url ma_page.php?action=&mode=

tu veux vérifier comment autrement que par ce moyen ?

Commentaire de Antidote le 07/11/2005 23:48:42

Bien je poursuis ce que tu dis, moi j'ai souvent à faire a ce genre de cas.

ma_page.php?action=

le isset($_GET['action']) renvoie true
le empty($_GET['action']) renvoie true

La variable existe et est vide.

Comme pour la méthode POST, c'est pour ça que selon ce qu'il veut obtenir je pense que ce double emploie est en fait bien nécessaire dans certain cas ou l'on veut une variable qui contienne forcément quelques choses.

Ceci dit, la fonction empty apparement ne provoquerai pas d'erreur et renverrai false si la variable n'existe pas. Ce qui voudrais dire qu'on pourrai se passez du isset()... J'ai vu ceci utiliser plus d'une fois de cette manière. Côté fiabilité j'ai pas testé.

Commentaire de coucou747 le 08/11/2005 17:54:26

bah écoute, je suis en term S, et j'aimes bien la rigeur... Tu sais,  j'ai fait bien plus de 5000 lignes dans mes projets, mais j'ai codé proprement pour que tout le monde puisse en bénéficier sans forcément être dégouté par la vitesse du code, ou par les problèmes rencontrés lors du portage (sur mon serveur, j'avais autorisé les <?, je l'ai éteind pour cause de lancement d'apache-perl, mais quand je le ralumerais, il se peut que <? soit interdit...)

Commentaire de coucou747 le 08/11/2005 18:27:42

détails de codings... chacun sa façon de voir les choses, je dirais plutot : façon d'améliorer ta source...

Commentaire de Antidote le 14/11/2005 21:50:24

Je ne fais jamais ça parce que je ne sais pas quel condition ba testé d'abord php avec un AND

si la variable n'existe pas qui di que empty ne va pas renvoyer une erreur ?

Commentaire de Antidote le 15/11/2005 19:53:42

et dans ce sens ?

if ( !empty(trim($_POST['action'])) && isset($_POST['action']) )

D'accord ça ne fait pas d'erreur je le sais j'ai testé depuis x année

mais pourquoi ? théoriquement ça devrait en faire une, alors pourquoi ça n'en fait pas ? c'est voulu ? parce que php à une compilation qui fait que et qui me dit que ça durera ainsi ?

ba rien.

Commentaire de Antidote le 16/11/2005 18:46:56

j'aime bien ta vision de la bouteille ^_^

Je sais ce que ça fait réellement mais je voulais porter le point sur la réflexion que ce cas peut apporter.

Je pense que tout les programmeurs devrait raisonner ainsi avant d'utiliser une fonction qui marche.

Commentaire de FhX le 17/11/2005 11:55:29

"if ( !empty(trim($_POST['action'])) && isset($_POST['action']) )

D'accord ça ne fait pas d'erreur je le sais j'ai testé depuis x année
"
Encore heureux, le && est la pour tester les 2 cotés de la condition. Si l'une est fausse, alors la condition est fausse.

"mais pourquoi ? théoriquement ça devrait en faire une, alors pourquoi ça n'en fait pas ? c'est voulu ? parce que php à une compilation qui fait que et qui me dit que ça durera ainsi ?"

La différence est notable quand tu utilises un "ou logique". En effet, en premier, tu va lire la première condition. Si la premiere condition est vrai, alors la condition entière est vrai (on ne vérifie plus le reste, ou alors PHP le fait mais ne provoque pas d'erreurs).

L'ordre n'a d'importance que si tu utilises des conditions imbriquées ou alors un "ou logique", à savoir "or" ou "||". C'est tout :)