关闭无法打开的文件有意义吗

Question

我正在审查另一位开发人员编写的代码。 该代码尝试打开文件，并根据成功/失败设置$canOpen 。

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

我发现奇怪的是，它也尝试关闭文件，但仅当if (!$fh)打开失败if (!$fh) 。 这有意义吗？ 成功打开文件后，关闭语句不应该位于else语句中吗？

Answer 1

不，没有道理。

如果满足!$fh作为条件，则意味着$fh包含布尔FALSE （或者，如果发生一些奇怪的内部PHP错误， $fh NULL ）。 因此，您有效执行的操作是：

fclose(FALSE);

...这毫无意义，将导致错误。

而且，如果您要做的只是填充$canOpen变量，并且对文件句柄不做任何有用的事情， is_file() ， is_readable()和is_writable()就足够了吗？

Answer 2

关闭未打开文件的唯一原因是出于恢复目的。 例如，始终在try...catch的finally块中关闭文件。

在您的情况下，它看起来像是编码错误。

Answer 3

如果将var_dump（$ fh）放在if的真实块中，则会发现它不是资源句柄。

php手册指出 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

不，这没有任何意义，如果您无法打开文件，则无需关闭文件。 此时，文件指针始终处于打开状态，因为如果文件不存在，它将创建一个文件。