Créer un fichier contenant l'historique des connexion

Hichxm

Hichxm

Membre
Inscription
15 Octobre 2013
Messages
1 118
Réactions
227
Points
5 721
RGCoins
25
Hey,
aujourd'hui j'ai créer
un plugin pour vos site qui va tout simplement
générer un fichier "logs.txt"
​
Code: 
[26/10/15][13:33:53] 127.0.0.1 connect to 127.0.0.1/tuto/2/index.php
[26/10/15][13:34:17] 127.0.0.1 connect to 127.0.0.1/tuto/2/index.php
[26/10/15][13:34:18] 127.0.0.1 connect to 127.0.0.1/tuto/2/page1.php
[27/10/15][19:48:01] 127.0.0.1 connect to 127.0.0.1/tuto/2/index.php

Un tutoriel complet est disponible
sur github.

Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.
 
Disons que la qualite de ton code n'est pas un point fort :$
 
Volca780© a dit:
Hey,
aujourd'hui j'ai créer
un plugin pour vos site qui va tout simplement
générer un fichier "logs.txt"
​
Code: 
[26/10/15][13:33:53] 127.0.0.1 connect to 127.0.0.1/tuto/2/index.php
[26/10/15][13:34:17] 127.0.0.1 connect to 127.0.0.1/tuto/2/index.php
[26/10/15][13:34:18] 127.0.0.1 connect to 127.0.0.1/tuto/2/page1.php
[27/10/15][19:48:01] 127.0.0.1 connect to 127.0.0.1/tuto/2/index.php

Un tutoriel complet est disponible
sur github.

Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.
Cliquez pour agrandir...
Merci du partage :),
Mais ya plusieurs point et amélioration à faire peut-être ^^:
-Pourquoi créer une classe seulement pour une fonction ?
-Pourquoi passe tu par un array pour traité tes variable string ?
-Il faudrait peut être protégé protégé le fiché logs.txt pour plus de confidentialité ^^

Sinon c'est très bien comme sa hein :)
 
Mylifeispotatoes a dit:
Merci du partage :),
Mais ya plusieurs point et amélioration à faire peut-être ^^:
-Pourquoi créer une classe seulement pour une fonction ?
-Pourquoi passe tu par un array pour traité tes variable string ?
-Il faudrait peut être protégé protégé le fiché logs.txt pour plus de confidentialité ^^

Sinon c'est très bien comme sa hein :)
Cliquez pour agrandir...
AlbRk_ a dit:
Disons que la qualite de ton code n'est pas un point fort :$
Cliquez pour agrandir...
Je sais que le code que je créer n'est pas très complet et pas très sécurisé. Le traitement de fichier en PHP consomme énormément de CPU, je rentabilise sa en faisant un code moins complet :)
 
Mylifeispotatoes a dit:
Merci du partage :),
Mais ya plusieurs point et amélioration à faire peut-être ^^:
-Pourquoi créer une classe seulement pour une fonction ?
-Pourquoi passe tu par un array pour traité tes variable string ?
-Il faudrait peut être protégé protégé le fiché logs.txt pour plus de confidentialité ^^

Sinon c'est très bien comme sa hein :)
Cliquez pour agrandir...
Une classe ? C'est plus stylé qu'une fonction :troll:
Array ? C'est pour tout simplement baissé la consommation CPU ;)
 
Volca780© a dit:
Une classe ? C'est plus stylé qu'une fonction :troll:
Array ? C'est pour tout simplement baissé la consommation CPU ;)
Cliquez pour agrandir...
Le problème c'est que si tu veut vraiment optimisé tous sa, tu peut le faire en 3 ligne, sa mettra moin de temps à php d'allez cherché du texte en mémoire que de cherché dans le tableau ^^
 
Volca780© a dit:
Une classe ? C'est plus stylé qu'une fonction :troll:
Array ? C'est pour tout simplement baissé la consommation CPU ;)
Cliquez pour agrandir...
Sauf que tu demandes a ton serveur de faire plusieurs operation, alors que tu peux le faire simplement.

PHP: 
$data = date("d-m-Y H:i:S"). " : ".$_SERVER['REMOTE_ADDR']." connect to " .$_SERVER['SERVER_NAME'].$_SERVER['PHP_SELF'];
file_put_content($file, $data, FILE_APPEND);

De plus, apache gere deja les logs
 
Si je créer trois variable il y a auras plus de ram utilisé contrairement à si je créer un tableau
Mais la différence ne se voit pas, j'ai créer ceci de façons à ne pas faire buguer trop le script ;)
 
AlbRk_ a dit:
Sauf que tu demandes a ton serveur de faire plusieurs operation, alors que tu peux le faire simplement.

PHP: 
$data = date("d-m-Y H:i:S"). " : ".$_SERVER['REMOTE_ADDR']." connect to " .$_SERVER['SERVER_NAME'].$_SERVER['PHP_SELF'];
file_put_content($file, $data, FILE_APPEND);

De plus, apache gere deja les logs
Cliquez pour agrandir...
T'as pas tord mais les logs d'apache son assez bizarre et assez illisible
Le code que tu m'as envoyé sera beaucoup plus difficile à décrypter comparé au mien
 
Volca780© a dit:
T'as pas tord mais les logs d'apache son assez bizarre et assez illisible
Le code que tu m'as envoyé sera beaucoup plus difficile à décrypter comparé au mien
Cliquez pour agrandir...
Au contraire ;)

Un petit tour sur la doc pour comprendre que date("d-m-Y H:i:S") retourne une string qui contient la date et l'heure (
Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.
)
Et les logs d'apache sont assez clair, quand on prend le temps de les examiner
 
AlbRk_ a dit:
Au contraire ;)

Un petit tour sur la doc pour comprendre que date("d-m-Y H:i:S") retourne une string qui contient la date et l'heure (
Vous devez être inscrit pour voir les liens ! Inscrivez-vous ou connectez-vous ici.
)
Et les logs d'apache sont assez clair, quand on prend le temps de les examiner
Cliquez pour agrandir...
Ne t'inquiète pas je connais le doc php mais en faisant sa : date("H-Y-....") bas sa nous retourne H-Y et non H/Y enfin c'est n'est que un détail, et aussi ma class est facilement modulable comparé à la tienne ;)
 
Volca780© a dit:
Ne t'inquiète pas je connais le doc php mais en faisant sa : date("H-Y-....") bas sa nous retourne H-Y et non H/Y enfin c'est n'est que un détail
Cliquez pour agrandir...
Dans ces cas la, tu changes les symboles ;)

Volca780© a dit:
et aussi ma class est facilement modulable comparé à la tienne ;)
Cliquez pour agrandir...
Ca ne change pas grand chose au final, si tu regardes ;)

J'ecris les logs en 2 fonctions au lieu de 7-8 dans ton code ;)
Tu concatenes au total 6 fonctions alors que tu peux en ecrire qu'une seule.

Tu peux le message de log, le format de l'heure, tout comme toi ;)
 
Bas certes tu as raison mais franchement dit moi
entre ses deux codes lequel est le plus modulable ?
PHP: 
$data = date("d-m-Y H:i:S"). " : ".$_SERVER['REMOTE_ADDR']." connect to " .$_SERVER['SERVER_NAME'].$_SERVER['PHP_SELF'];
file_put_content($file, $data, FILE_APPEND);
PHP: 
$DATA = array();
$DATA['M'] = "[".date("d")."/".date("m")."/".date("y")."]";
$DATA['H'] = "[".date("H").":".date("i").":".date("s")."]";
$DATA['URL'] = $_SERVER['REMOTE_ADDR']." connect to " .$_SERVER['SERVER_NAME'].$_SERVER['PHP_SELF'];
$DATA['COMPLETE'] = $DATA['M'].$DATA['H']." ".$DATA['URL']."\n";
$files = fopen("logs.txt", "a+");
fputs($files, $DATA['COMPLETE']);
fclose($files);
 
Volca780© a dit:
Bas certes tu as raison mais franchement dit moi
entre ses deux codes lequel est le plus modulable ?
PHP: 
$data = date("d-m-Y H:i:S"). " : ".$_SERVER['REMOTE_ADDR']." connect to " .$_SERVER['SERVER_NAME'].$_SERVER['PHP_SELF'];
file_put_content($file, $data, FILE_APPEND);
PHP: 
$DATA = array();
$DATA['M'] = "[".date("d")."/".date("m")."/".date("y")."]";
$DATA['H'] = "[".date("H").":".date("i").":".date("s")."]";
$DATA['URL'] = $_SERVER['REMOTE_ADDR']." connect to " .$_SERVER['SERVER_NAME'].$_SERVER['PHP_SELF'];
$DATA['COMPLETE'] = $DATA['M'].$DATA['H']." ".$DATA['URL']."\n";
$files = fopen("logs.txt", "a+");
fputs($files, $DATA['COMPLETE']);
fclose($files);
Cliquez pour agrandir...
Je laisse les autres donner leurs avis ;)
 
Volca780© a dit:
Bas certes tu as raison mais franchement dit moi
entre ses deux codes lequel est le plus modulable ?
PHP: 
$data = date("d-m-Y H:i:S"). " : ".$_SERVER['REMOTE_ADDR']." connect to " .$_SERVER['SERVER_NAME'].$_SERVER['PHP_SELF'];
file_put_content($file, $data, FILE_APPEND);
PHP: 
$DATA = array();
$DATA['M'] = "[".date("d")."/".date("m")."/".date("y")."]";
$DATA['H'] = "[".date("H").":".date("i").":".date("s")."]";
$DATA['URL'] = $_SERVER['REMOTE_ADDR']." connect to " .$_SERVER['SERVER_NAME'].$_SERVER['PHP_SELF'];
$DATA['COMPLETE'] = $DATA['M'].$DATA['H']." ".$DATA['URL']."\n";
$files = fopen("logs.txt", "a+");
fputs($files, $DATA['COMPLETE']);
fclose($files);
Cliquez pour agrandir...
Celui de @AwH car il est plus claire et simple a comprendre, contrairement au tien
 
Retour
Haut