Discussion :

[michel71]

Bonjour,

J'ai un problème avec le code ci-dessous. Je teste avec cette url

mozilla/5.0 (windows nt 6.1; wow64; rv:43.0) gecko/20100101 firefox/43.0

or le script me détecte "mozi" alors que je souhaite avoir une détection sur le mot complet (en clair je voudrais une détection si j'ai "mozilla" dans le tableau)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
$UA=$_SERVER["HTTP_USER_AGENT"];
$brow = strtolower($UA);
 
$bots = array (
"zeus.*webster",
"zeus",
"zyborg",
"mozi",
"voyager"
); 
 
$aspirateur="vide"
 
for($i=0;$i<count($bots);$i++)
{
if(preg_match("#$bots[$i]#",$brow)) $aspirateur="aspiOK";
}
echo $aspirateur;

[sabotage]

Je ne comprends pas bien : si tu ne veux pas obtenir "mozi" pourquoi tu mets "mozi" et pas "mozilla" ?

Au passage tu n'as pas besoin d'une expression regulière et utilise les boucle foreach :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
foreach ($bots as $bot) {
{
   if(strpos($brow, $bpt) !== FALSE) $aspirateur="aspiOK";
}

[michel71]

Bonjour,

Ok je vais utiliser "foreach ".

Je me suis surement mal expliqué, je veux virer du site les aspirateurs
par exemple dans cette chaine, que je passe en minuscule avant de la traiter, index.php HTTP/1.0" 200 7080 "-" "Mozilla/5.0 (compatible; MJ12bot/v1.4.5; http://www.majestic12.co.uk/bot.php?+)

je veux repérer la chaine mj12bot qui fait partie du tableau pour diriger le visiteur vers une page neutre.

J'ai testé la boucle que tu me propose mais ça ne marche pas

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
$bots = array (
"library",
"libwww-perl",
"linkchecker",
"lwp-trivial",
"microsoft control",
"minibot(naverrobot)",
"mj12bot",
"msiecrawler",
"msnbot-media",
"xenu link sleuth",
"yacy",
"yandex",
"zeus.*webster",
"zeus",
"zyborg",
"voyager"
); 
 
$brow=strtolower('index.php HTTP/1.0" 200 7080 "-" "Mozilla/5.0 (compatible; MJ12bot/v1.4.5; http://www.majestic12.co.uk/bot.php?+)"');
echo $brow; echo'<br>';
$aspirateur="vide";
 
foreach ($bots as $bot) 
{
   if(strpos($brow, $bot) !== FALSE) $aspirateur="aspiOK";
}
echo $aspirateur;

le problème c'est que si je met "mj12" dans mon tableau (à la place de mj12bot) le résultat est vrai, et j'ai peur de virer de braves gens qui ne sont pas des aspirateurs

Merci

[sabotage]

Encore une fois pourquoi voudrais-tu mettre "mj12" si c'est "mj12bot" que tu veux ?

Le code que tu nous montres détecte bien "mj12bot" et retourne "aspiok".

[michel71]

Simplement si un jour il y a une url avec "mj12" je ne veux pas la virer

[michel71]

Le code détecte aussi "mj12" !!

[sabotage]

Tu te trompes dans tes tests.
"mj12bot" renvoit "aspiok"
"mj12" renvoit "vide"

[michel71]

Désolé d'insister mais chez moi ça donne "aspiOK"
J'ai modifié le nom de la variable pour être sur que cette variable n'est pas utilisée ailleurs

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
$aspira="vide";
 
foreach ($bots as $bot) 
{
   if(strpos($brow, $bot) !== FALSE) $aspira="aspiOK";
}
echo $aspira;

[sabotage]

Tu devrais prendre le temps de relire tes messages, mes messages et ce que fais ton code car tu as le même résultat que moi et c'est bien ce qu'on veut :
"mj12bot" est bien intercepté et tu obtiens donc "aspiok".

[michel71]

Oui, tu as raison, en fait je pense que les résultats que j'avais étaient dû au cache de mon navigateur.
Je l'ai vidé et tout est bon.
Merci à toi et désolé d'avoir bêtement insisté