開くことができなかったファイルを閉じることは意味がありますか

Question

私は別の開発者によって書かれたコードを検討しています。コードはファイルを開こうとし、成功/失敗に基づいて$canOpenを設定します。私は奇妙見つける何

$fh = @fopen('files.php', 'a+');   
if (!$fh){ 
    fclose($fh); 
    $canOpen = false; 
} else { 
    $canOpen = true; 
} 

は、それはまた、ファイルを閉じようとしていることが、オープンはif (!$fh)を失敗したときのみです。これは理にかなっていますか？ファイルが正常に開かれたときに、close文をelse文に入れるべきではありませんか？

Answer 1

いいえ、意味がありません。

!$fhを条件として満たされた場合、それは$fhは（いくつかの奇妙な内部PHPエラー、NULLの場合に、又はおそらく）ブールFALSEを含むことを意味します。あなたが効果的にやっていることは次のとおりです。

fclose(FALSE); 

...無意味でエラーになります。

そして、あなたは $canOpen変数を移入し、ファイルハンドルに有用な何もしていないやろうとしているすべては、 is_file()、 is_readable()と is_writable()の組み合わせは十分ではないでしょうか？

Answer 2

未開封のファイルを復元する唯一の理由は、回復力を目的としている場合のみです。たとえば、finallyブロックのファイルを常に閉じる（try...catch）。

あなたの場合は、コーディングエラーのようです。

Answer 3

ifの真のブロックにvar_dump（$ fh）を置くと、それはリソースハンドルではないことがわかります。

php manual states fcloseがリソースを取ること。したがって、fcloseは、開くことができないファイルに対して呼び出すことはできません。

<?php 
$fh = @fopen('files.php', 'a+'); 

if (!$fh){ 
    var_dump($fh); 
    fclose($fh); // this will cause an error 
    $canOpen = false; 
} else { 
    $canOpen = true; 
} 

Answer 4

いいえ、意味がありません。ファイルを開くことができない場合は、それを閉じる必要はありません。ファイルが存在しない場合は、ファイルを作成するため、この時点でファイルポインタは常に開いています。