Discussion :

[kris92130]

Bonjour à tous,

Je suis débutant en PHP, j'ai fait un script qui récupère les adresses IP des visiteurs de mon site dans un fichier (TrackFile.txt) SAUF si cettte adresse IP se trouve dans un tableau ($ipInterdites).
Problème : mon script fonctionne très bien si j'ai une seule adresse IP rentrée dans $ipInterdites (ex. 'aa.bbb.cc.ddd') => cette adresse IP n'est pas récupérée dans le fichier... mais il ne fonctionne plus dès que je rajoute une 2eme IP (ex. 'e.ff.gg.hhh'), => la 1ere IP est insérée dans le fichier alors qu'elle devrait pas l'être !

Qqun peut-il m'expliquer pourquoi SVP ?
Merci infiniment.

Code :

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

1
2
3
4
5
6
7
8
9
$ip = $_SERVER['REMOTE_ADDR'];
$ipInterdites = array('aa.bbb.cc.ddd','e.ff.gg.hhh');
foreach($ipInterdites as $v){
    if($ip!= $v){
        $trackingFile = fopen("TrackFile.txt", "a+");
        fputs($trackingFile, $ip."\t".$date."\t".$heure."\n");
        fclose($trackingFile);
    }
}

[Bovino]

Ben ça semble pourtant assez logique quand même...
Si tu as deux valeurs dans ton tableau d'IP interdites, ça signifie qu'au moins une des deux valeurs ne correspond pas à l'IP testée et donc que tu rentreras au moins une fois dans le if...

[Celira]

+1 sur Bovino.
Plutôt qu'un foreach et un if, utilise plutot in_​array :

Code :

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

1
2
3
4
5
6
7
8
$ip = $_SERVER['REMOTE_ADDR'];
$ipInterdites = array('aa.bbb.cc.ddd','e.ff.gg.hhh');
 
if(!in_array($ip, $ipInterdites)){
    $trackingFile = fopen("TrackFile.txt", "a+");
    fputs($trackingFile, $ip."\t".$date."\t".$heure."\n");
    fclose($trackingFile);
}

[kris92130]

En effet ... merci beaucoup

[Bovino]

Code :

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

$ipInterdites = array('aa.bbb.cc.ddd','e.ff.gg.hhh');

Tu as donc 2 IP interdites. Imaginons que je viens sur ton site avec l'IP aa.bbb.cc.ddd.

Code :

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

foreach($ipInterdites as $v){

Premier tour de boucle, $v vaut 'aa.bbb.cc.ddd'.

Code :

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

if($ip!= $v){

la condition n'est pas vérifiée puisque les deux IP sont identiques, on n'entre pas dans le if et on sort de cette itération.
Seconde itération, IP vaut 'e.ff.gg.hhh'.

Code :

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

if($ip!= $v){

La condition est vraie dans ce cas, on entre dans le if et l'IP est inscrite dans le fichier.

Conclusion : tu manques un peu de logique !

[Deuzz]

je ne comprends pas pourquoi l'IP interdite se retrouve insérée dans le fichier !

Parce que l'IPinterdite2 est différente de l'IPinterdite1.

Ta comparaison revient à dire : Pour chaque ipinterdite : la comparer avec l'ipvisiteur.
or ce que tu souhaites est : Pour chaque ipvisiteur : la comparer avec toutes les ipinterdites.

[kris92130]

Oui j'ai compris... dur dur de se remettre à la programmation après plusieurs années d'interruption lol !

Encore merci.