SAVOIR SI LE VISITEUR EST UN ROBOT D'UN MOTEUR DE RECHERCHE

Commentaire de MadM@tt le 29/04/2007 13:35:49

Merci pour le commentaire, par contre j'aurais besoin d'un peu d'explication :
pourquoi privilegier foreach plutot que for ? c'est plus rapide ?

Et de ton code j'en déduis que l'instruction "return" fait terminer la fonction ? (ce qui effectivement est bien mieux que ce que j'ai fait)

Pour le fichier, bonne idée, mais si ce code est appelé à chaque page vue ça risque de faire perdre beaucoup de temps s'il faut que j'ouvre le fichier à chaque fois, et que je parse le contenu.

Merci

Commentaire de MadM@tt le 29/04/2007 18:18:43

Ok merci beaucoup pour les infos, php a tellement de fonctions pour faire la meme chose qu'il est difficile de savoir laquelle utiliser.

Je vais m'empresser de modifier ça, surtout que j'utilise cette méthode dans toutes mes boucles for de mon site ^^

Commentaire de neigedhiver le 29/04/2007 20:44:42

Re,

Non non, strpos() est une très bonne solution, si on ne se contente pas de vérifier que la chaine a été trouvée, mais si on vérifie aussi la position (valeur retournée par strpos() justement !). En plus, c'est plus court :

if (strpos($IPtab[$t], $IP) === 0)
{
    return true;
}

Return va automatiquement sortir de la fonction et retourner true quand la chaine est trouvée.

Commentaire de Astalavista le 30/04/2007 01:37:57

Je voit que j'ai aidé :), je te remercie  neigedhiver d'avoir affirmer ce que j'ai marqué ...

Ha oui, il y a encore une petite erreur :
for($t = 1; $t <= $max; $t = $t++)
S'utilise plutôt comme ca :
for($t = 1; $t <= $max; $t++)

ton count doit s'écrire en minuscule

à mon avis, le foreach est un peut plus adapté.
Si un jour tu supprime la clef 19 (par exemple), il faudra changer toutes tes autres clef.
Mais bon, si tu insiste pour utiliser ton for, ajoute tes IP de cette façon :
$IPtab[] = '66.249.';
$IPtab[] = '207.68.146.';
L'ajout se fera à la suite, et 0 par défaut.

Commentaire de kankrelune le 30/04/2007 13:57:31

for($t=1,$nb=count($IPtab);$t<=$nb;++$t)

Histoire d'optimiser encore un peu... perso je ne suis pas fan de cette méthode car si le bot à changé d'adresse ip ou s'il n'est pas répertorié ton script ne sert à rien... le mieux serait de faire un lien caché vers une page ou de compter le temps entre chaque ouverture de page et de stocker l'ip s'il sagit d'un bot... .. .

@ tchaOo°

Commentaire de MadM@tt le 30/04/2007 14:24:59

Voilà mis à jour ^^

C'est sympa d'avoir du monde pour repasser sur son code ;) merci beaucoup

Commentaire de MadM@tt le 30/04/2007 14:58:42

oui je te comprend, je suis assez actif sur vbfrance je vois de quoi tu parle ^^

Commentaire de kankrelune le 30/04/2007 15:13:35

Pareil que neigedhiver... .. . ;o)

Pour finir plutot que de faire...

$IP = $_SERVER['REMOTE_ADDR']
$max = count($IPtab) - 1;
for($t = 0; $t <= $max; $t++) {
    if (strpos($IPtab[$t], $IP) === 0)
    {
        return true;
    }
}
return false;

autant faire

if(isset($_SERVER['REMOTE_ADDR']) && !empty($_SERVER['REMOTE_ADDR']))
{
    for($t=0,$max=count($IPtab);$t<$max;++$t)
    {
        if (strpos($IPtab[$t], $_SERVER['REMOTE_ADDR']) === 0)
           return true;
    }
}
return false;

;o)

@ tchaOo°

Commentaire de kankrelune le 30/04/2007 15:34:38

pas exactement... en fait $i++ c'est de la post incrémentassion et ++$i de la pré incrementassion... .. .

Dans le premier cas l'instruction, si instruction il y a, est exécutée puis la variable est incrémentée de 1... dans le second cas la variable est incrémentée puis l'instruction est exécutée... exemple...

$a = 1;
$b = $a++;

echo '$a = '.$a.',$b = '.$b; // affichera  $a = 2,$b = 1

---------------

$a = 1;
$b = ++$a;

echo '$a = '.$a.',$b = '.$b; // affichera  $a = 2,$b = 2

Hormis cette petite différence la pré incrémentassion est légèrement plus rapide que la post incrémentassion car en pré inc php ne se pose pas de question et incrémente directement la variable alors qu'en post inc php garde en mémoire qu'il doit incrémenter la variable après avoir exécuté l'instruction en cours... .. . ;o)

@ tchaOo°

Commentaire de MadM@tt le 30/04/2007 16:44:15

Ok mais alors si mon tableau va de 0 à count() - 1, il faut donc une boucle :
for($t = 0, $max = count($IPtab) - 1; $t < $max; ++$t)  ??

Commentaire de neigedhiver le 30/04/2007 17:10:16

Alors je vais encore chipoter (j'adooooore ça).

if(isset($_SERVER['REMOTE_ADDR']) && !empty($_SERVER['REMOTE_ADDR']))

Là aussi, syntaxiquement, tu vérifies si une variable potentiellement non instanciée est vide... Le fait de vérifier qu'elle existe ne la fait pas exister si tel n'est pas le cas...
Puor bien faire, il faudrait alors faire un autre test avec empty() dans le bloc if.
Non ?

Commentaire de MadM@tt le 30/04/2007 20:08:54

@Kankrelune > Ok j'avais oublié la nuance < et <= merci

Commentaire de kankrelune le 01/05/2007 12:08:07

@ neigedhiver... non car la première instruction de la condition étant fausse la seconde n'est pas vérifiée... .. .

@  Astalavista... pas forcement... foreach travail sur une copie du tableau, for non... après c'est une question de perf et de façon de faire... pour gagner en rapidité j'aurais tendance à faire...

$i = -1;
while(isset($IPtab[++$i]))
{
   ...
}

@ tchaOo°

Commentaire de neigedhiver le 01/05/2007 19:48:12

@kankrelune : Bon sang de bon soir... Merci... Je ne savais pas exactement étaient évaluées les conditions multiples par if... J'ai fait un test pour vérifier tes dires (c'est pas que j'ai pas confiance, mais là, au moins, je peux me permettre de l'affirmer à mon tour si besoin, puisque j'ai expérimenté).
Donc merci pour ces petits détails qui enrichissent tous les jours ma culture de geek...

Commentaire de cpp4ever le 07/05/2007 14:05:43

vous etes au courant que le module php a des centaines de fonctions prédéfinie.
par exemple in_array :

if (in_array ($_SERVER['REMOTE_ADDR'], $IPtab)) {
return TRUE;
}
else {
return FALSE;
}

quant aux ip des bots je crois qu'il faut faire le tour des moteurs de recherche.

Commentaire de cpp4ever le 07/05/2007 15:13:25

oups, effectivement
faut m'excuser, je croise tous le temps des coders
qui ne se donne jamais la peine d'aller voir le manuel de php,
que j'ai pris de mauvaises habitudes.

Commentaire de MadM@tt le 29/06/2007 15:12:39

Merci pour ta précision, c'est mis à jour.

Et merci pour ta liste d'IP

Commentaire de sy125gi le 10/07/2007 19:20:59

Et en fait, il faut metre ce fichier à quel endroit du site?