Crea file in modo thread-safe

Question

Ho una matrice di nomi di file e ogni processo deve creare e scrivere solo in un singolo file.

Questo è quello che mi è venuto a:

foreach ($filenames as $VMidFile) { 
    if (file_exists($VMidFile)) { // A 
     continue; 
    } 

    $fp = fopen($VMidFile, 'c'); // B 

    if (!flock($fp, LOCK_EX | LOCK_NB)) { // C 
     continue; 
    } 

    if (!filesize($VMidFile)) { // D 
     // write to the file; 

     flock($fp, LOCK_UN); 
     fclose($fp); 
     break; 
    } 

    flock($fp, LOCK_UN); 
    fclose($fp); // E 
} 

Ma non mi piace che sto facendo leva sulla filesize.

Eventuali proposte per farlo in un altro (migliore) modo?

UPD: aggiunto le etichette per discutere facilmente

UPD 2: sto usando filesize perché non vedo altro modo affidabile per verificare se l'attuale filo creato il file (quindi è vuoto ancora)

UPD 3: la soluzione dovrebbe essere condizione corsa libera.

Answer 1

Una possibile, un po' brutta soluzione sarebbe quella di bloccare in un file di blocco e poi testare se il file esiste:

$lock = fopen("/tmp/".$filename."LOCK", "w"); // A 

if (!flock($lock, LOCK_EX)) { // B 
    continue; 
} 
if(!file_exists($filename)){ // C 
    //File doesn't exist so we know that this thread will create it 
    //Do stuff to $filename 
    flock($lock, LOCK_UN); // D 
    fclose($lock); 
}else{ 
    //File exists. This thread didn't create it (at least in this iteration). 
    flock($lock, LOCK_UN); 
    fclose($lock); 
} 

Ciò dovrebbe consentire l'accesso esclusivo al file e inoltre consente di decidere se la chiamata a fopen($VMidFile, 'c'); creerà il file.

Answer 2

Piuttosto che creare un file e sperando che non è interferito con:

1. creare un file temporanea
2. fare tutte le operazioni necessarie di file su di esso
3. rename alla nuova posizione se la posizione non esiste

Tecnicamente, poiché rename sovrascrive la destinazione, esiste la possibilità che i thread concorrenti continuino a scontrarsi. Questo è molto improbabile se si dispone di:

if(!file_exists($lcoation) { rename(... 

si potrebbe usare md5_file per verificare il contenuto del file è corretto dopo questo blocco.

Answer 3

si può garantire l'accesso esclusivo tramite semaphores (solo UNIX, e ha fornito l'estensione sysvsem è installato):

$s = sem_get(ftok($filename), 'foo'); 
sem_acquire($s); 

// Do some critical work... 

sem_release($s); 

In caso contrario è anche possibile utilizzare flock. Non richiede alcuna estensione speciali, ma secondo comments on PHP.net è un po 'più lento di quello che usano i semafori:

$a = fopen($file, 'w'); 
flock($a, LOCK_EX); 

// Critical stuff, again 

flock($a, LOCK_UN); 

Answer 4

Utilizzare la modalità 'x' anziché 'c' nella chiamata fopen. E controlla il $ fp risultante, se è falso, il file non è stato creato dal thread corrente, e dovresti continuare con il prossimo nome file.

Inoltre, a seconda delle impostazioni di installazione del PHP, è possibile mettere un @ davanti alla chiamata fopen per sopprimere eventuali avvisi se fopen ($ VMidFile, 'x') non è in grado di creare il file perché già esistente.

Questo dovrebbe funzionare anche senza gregge.