Discussion :

[Lysgard]

Bonjour,
Léger problème dans un de mes scripts PHP.
Je suis en train de développer une fonction qui enregistre dans un fichier txt les erreurs récupérées dans un try... catch, en les classant de la date la plus récente à la plus ancienne.
Mais je ne parviens pas à les classer dans cet ordre, toutes les nouvelles données sont inscrites à la suite de mon fichier, et non pas au début, comme je le voudrais.
Exemple de mon script :

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
 
try
{
    $connexion = new PDO('mysql:host='.$PARAM_hote.';port='.$PARAM_port.';dbname='.$PARAM_db, $PARAM_user, $PARAM_pwd);
}
catch(Exception $e)
{
    $erreur = $e->getMessage();
    $numero = $e->getCode();
    $dateErreur = date("Y-m-d H:i:s"); //AAAA-MM-JJ HH:MM:SS
    $f = "erreurs/rapport.txt";
 
    // Ouverture du fichier.
    $rapport = fopen($f, "r+"); // r+ : Ouvre en lecture et écriture, et place le pointeur de fichier au début du fichier. 
 
    // Récupération du contenu.
    $message = file_get_contents($f);
 
    // Place le pointeur au début du fichier.
    rewind($rapport);
 
    // Écriture dans le fichier.
    $message .= "[".$dateErreur."] - Erreur ".$numero." : ".$erreur;
    fputs($rapport, $message);
 
    // Fermeture du fichier.
    fclose($rapport);
}

Et voici ce que je récupère dans mon fichier :
[2011-06-24 22:22:27] - Erreur 0 : SQLSTATE[28000] [1045] Access denied for user 'root'@'localhost' (using password: YES)
[2011-06-24 22:22:30] - Erreur 0 : SQLSTATE[28000] [1045] Access denied for user 'root'@'localhost' (using password: YES)
[2011-06-24 22:22:33] - Erreur 0 : SQLSTATE[28000] [1045] Access denied for user 'root'@'localhost' (using password: YES)

Or donc, ce que je veux, c'est avoir :
[2011-06-24 22:22:33] - Erreur 0 : SQLSTATE[28000] [1045] Access denied for user 'root'@'localhost' (using password: YES)
[2011-06-24 22:22:30] - Erreur 0 : SQLSTATE[28000] [1045] Access denied for user 'root'@'localhost' (using password: YES)
[2011-06-24 22:22:27] - Erreur 0 : SQLSTATE[28000] [1045] Access denied for user 'root'@'localhost' (using password: YES)

Si vous pouviez m'éclairer.
Merci d'avance.

[provirus]

file_get_contents() lit le fichier et n'as pas besoin qu'il soit déjà ouvert puisqu'il prend le nom du fichier et non un file_descriptor. Tu peux donc l'appeler en premier et ensuite ouvrir le fichier avec fopen().

Puisque file_get_contents() ne touche pas au file_descriptor, tu n'as pas besoin de faire un rewind().

Puisque tu vas grossir le fichier en remettant tout le texte, autant mettre comme mode "w".

Dernier truc pour l'ordre, tu fais "$message .= " ce qui veut dire que tu ajoutes à la fin de cette variable.

Le code final serait donc

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
// Récupération du contenu.
$message = file_get_contents($f);
 
// Ouverture du fichier.
$rapport = fopen($f, "w");
 
// Écriture dans le fichier.
$nouveaumessage = "[".$dateErreur."] - Erreur ".$numero." : ".$erreur;
$nouveaumessage .= $message;
fwrite($rapport, $nouveaumessage);
 
// Fermeture du fichier.
fclose($rapport);

Ensuite, il y a une considération techniques à voir: plusieurs scripts appelés en même temps par plusieurs utilisateurs vont-ils s'écraser mutuellement (peut-être mettre un lock)

[Lysgard]

Merci de ton aide, je viens juste de voir mon erreur.

C'est bien évidemment

Code :

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

$message = "[".$dateErreur."] - Erreur ".$numero." : ".$messageErreur.$message;

Et non pas -_-

Code :

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

$message .= "[".$dateErreur."] - Erreur ".$numero." : ".$erreur;

Néanmoins, j'ai bien pris note de tes diverses remarques, et il est vrai que tes propositions me permettraient d'optimiser mon code.

Quant au lock, c'est en effet une piste à suivre, même si ici, cela ne sera pas nécessairement utile, il n'y aura qu'une personne qui utilisera ce script.

Merci.