phpのロックファイルは多重起動を完璧に防止できるのか

Question

phpで例えば決済処理を重複して行わないために、

// 作成するファイル名の指定
$lock_file = __DIR__ . '/lock.txt';
 
// ファイルの存在確認
if( file_exists($lock_file) ){
    echo "多重起動はできません。\n";
    // スクリプトを終了する
    exit(0);
}
 
// 多重起動防止用ロックファイル作成
touch( $lock_file );
 
echo "スクリプト開始\n";
 
// 何か処理する
sleep(3);
 
echo "スクリプト終了\n";
 
// ロックファイルの削除
unlink( $lock_file );

参考ロックファイルコードサンプル
みたいな感じで、ロックファイルを用いることで多重起動を防止する例があると思うのですが、この方法だとファイルの存在確認とファイル作成の間に隙間があるので、バグが生じる可能性があるのかなと思います。（以下みたいに変更して、ターミナル2つ開いて同時に実行すると多重起動できてしまう）

// 作成するファイル名の指定
$lock_file = __DIR__ . '/lock.txt';

// ファイルの存在確認
if( file_exists($lock_file) ){
    echo "多重起動はできません。\n";
    // スクリプトを終了する
    exit(0);
}
sleep(10);

// 多重起動防止用ロックファイル作成
touch( $lock_file );

echo "スクリプト開始\n";

// 何か処理する


echo "スクリプト終了\n";

// ロックファイルの削除
unlink( $lock_file );

なのでより厳密には以下のようなコードの方が良いと思うのですが、僕の考えは合っていますでしょうか？（ロックファイルを使わずにセマフォを使うみたいな方法もありますが。）

<?php
// 作成するディレクトリ名の指定
$lock_file = __DIR__ . '/lock';

// ディレクトリの存在確認＆作成
if(! mkdir($lock_file, 0700) ){
    echo "多重起動はできません。\n";
    // スクリプトを終了する
    exit(0);
}

echo "スクリプト開始\n";

// 何か処理する
sleep(10);

echo "スクリプト終了\n";

// ディレクトリの削除
rmdir( $lock_file );

Comments

[yambejp]

DBは使わないのでしょうか？

[takuyaKK]

外部の決済処理サーバと通信することを想定しています。DBは特に使いません。

Answer 1

解決してますが、アドバイス。

ファイル作成で排他制御したつもりになっている参照サイトは論外として、
mkdirでの排他制御は、プログラムが異常終了した時にロックディレクトリが削除されず残ってしまう事の考慮が必要です。

OSごと落ちるようなケースは手動対応と割り切れば、例外処理で救うのでしょうか。
（あるいはタイムスタンプを見て古かったら無視するというのをやったことがあるが）

そういうことを気にしなくていいflock()が安心ではあります。

Comments

[takuyaKK]

確かにmkdirの場合は、そのへんの処理がめんどくさいですね。
そういうことを考えるとflockとかセマフォの方が優れているような気がしますね！
ありがとうございます????‍♂️

Answer 2

なのでより厳密には以下のようなコードの方が良いと思うのですが、僕の考えは合っていますでしょうか？

考えとしては合っていると思います。

ただ、PHPでファイルで排他制御をしようとするならfile_exists()ではなくfopen()やflock()を使う方が一般的かと思います。

参考）
https://www.softel.co.jp/blogs/tech/archives/5341

ただ、わりとややこしいところもあるので、それならmkdir()で実装したロックの方がシンプルでいいかもしれないですね。

Comments

[takuyaKK]

ありがとうございます！
fopenもアリですね！