phpである処理をすると数を１足して、ある処理をするとその数字から１引

質問

phpである処理をすると数を１足して、ある処理をするとその数字から１引くような事をしたいのですが、上手くいきません。（まれに数字にずれが生じてしまいます。）

１足していくもの
$fp = fopen("./data/registry/buy.dat","r+");
$registry_count = fgets($fp,1024);
if($registry_count == ""){
$registry_count = 0;
}
$registry_count++;
flock($fp,LOCK_EX);
fseek($fp,0);
fputs($fp,$registry_count);
flock($fp,LOCK_UN);
fclose($fp);

１引いていくもの
$fp = fopen("./data/registry/buy.dat","r+");
$registry_count = fgets($fp,1024);
if($registry_count != "0"){
$registry_count--;
}
flock($fp,LOCK_EX);
fseek($fp,0);
ftruncate($fp,0);
fputs($fp,$registry_count);
flock($fp,LOCK_UN);
fclose($fp);

どなたかご教授ください。
よろしくお願いいたします。

回答No.1

Hohenheim · ベストアンサー

flockの位置が間違っています。
fopen直後にロックをかけるべきです。

-------------------------------------
$fp = fopen("./data/registry/buy.dat","r+");
flock($fp,LOCK_EX);
$registry_count = fgets($fp,1024);
以下略

お礼

ご指摘の点を修正して多分、問題なく動作いたしました。
ありがとうございます。

補足

早速ご回答ありがとうございます。
ご指摘の通り、flock関数の使い方が間違っていました。
原因として考えられるのはflock以外に
何かあればご指導のほどよろしくお願いいたします。

回答No.3

qaqa246

>やはり「===」でしょうか？
個人的にはstrcmp()関数を利用するようにしている。

お礼

ご指摘の点を修正して多分、問題なく動作いたしました。
ありがとうございます。

回答No.2

qaqa246

>if($registry_count == ""){
文字列比較に「==」を使うのはやめましょう。

補足

ご回答ありあがとうございます。
やはり「===」でしょうか？