fichier de verrouillage en php

Question

J'ai eu un nouveau venu (l'adolescent d'à côté) écrire du code php pour suivre un peu d'utilisation sur mon site web. Je ne suis pas familier avec php, donc je demande un peu d'accès aux fichiers concurrents.

Mon application native (sous Windows) enregistre parfois des données sur mon site en tapant l'URL contenant mon script PHP. L'application native n'examine pas les données renvoyées.

 $fh = fopen($updateFile, 'a') or die("can't open file"); 
     fwrite($fh, $ip); 
     fwrite($fh, ', '); 
     fwrite($fh, $date); 
     fwrite($fh, ', '); 
     fwrite($fh, implode(', ', $_GET)); 
     fwrite($fh, "\r\n"); 
     fclose($fh); 

Ceci est un site à faible trafic et les données ne sont pas critiques. Mais que se passe-t-il si deux utilisateurs entrent en collision et que deux instances du script essaient chacune d'ajouter une ligne au fichier? Existe-t-il un verrouillage de fichier implicite en php?

Le code ci-dessus est-il au moins sûr de ne pas être verrouillé et de ne jamais retourner le contrôle à mon utilisateur? Le fichier peut-il être corrompu? Si j'ai le script ci-dessus supprimer le fichier tous les mois, que se passe-t-il si une autre instance du script est en cours d'écriture dans le fichier?

Answer 1

Vous devez mettre un verrou sur le fichier:

<?php 
$fp = fopen($updateFile, 'w+'); 
if(flock($fp, LOCK_EX)) { 
fwrite($fp, 'a'); 
flock($fp, LOCK_UN); 
} else { 
echo 'can\'t lock'; 
} 

fclose($fp); 

Answer 2

Depuis c'est un ajout au fichier, la meilleure façon serait de regrouper les données et l'écrire dans le fichier dans un fwrite(), fournir les données à écrire n'est pas plus grand que le tampon de fichier. Bien sûr, vous ne connaissez pas toujours la taille du tampon, donc flock(); est toujours une bonne option.