如果“file_exists”是错误的被设计为返回路径而不是 TRUE？

Question

file_exists 函数成功时返回 TRUE，但我认为如果它返回值而不只是 TRUE，会更有用传递的$filename。我这样想有错吗？

如果函数是这样设计的，我们就可以使用：

$file = file_exists("dir/file.ext");

if($file)
{
    // do something
}

... 而不是更复杂的：

$file = "dir/file.ext";

$success = file_exists("dir/file.ext");

if($success)
{
    // do something
}

Answer 1

我不明白为什么这会是一个改进。请考虑：

1. 实际上，您会将 file_exists 调用放在条件内，因此不会有额外的 $success 变量
2. 名称 file_exists 预先决定了代码的读者认为该函数将返回是/否答案（布尔值）

所以总之，我认为除了返回值的类型之外什么都不改变是一个坏主意。

Answer 2

文件存在吗？基本上是一个是或否问题，因此它给出是或否（布尔）答案。

该文件存在吗？ “是”，它存在。

所以它的名字恰如其分

Answer 3

那毫无意义。它还可以返回文件权限、文件大小、最后修改日期，而且它仍然与文件名一样有意义。它只不过是一个给出布尔值的检查，你可以根据它做出决定。

顺便说一句，我没有得到你的例子，有什么比以下更容易的：

$file = "dir/file.ext";
if(file_exists($file))
{
    // do something with $file
}

或者如果你以后确实需要返回值，你可以这样做：

$file = "dir/file.ext";
if($success=file_exists($file))
{
    // do something with $file
}

Answer 4

好吧，没有什么能真正阻止您编写这个函数：

function get_filename_if_exists($fname) {
    return (file_exists($fname) ? $fname : FALSE );
}

恕我直言，它是有问题的，因为它有多种返回类型，并且不会为您提供比从 file_exists() 获得的更多信息 - 因此比毫无意义更糟糕；但如果你真的愿意的话，也可以这样搬起石头砸自己的脚。

（由于此处和其他答案中所述的原因，这种行为不太可能被改进到 file_exists() 中）

Answer 5

那么fopen有什么用呢？

$res = fopen("my/path.txt");
if ($res===FALSE) {
  // File does not exists or an error while opening
}

如果您特别想知道文件是否存在，那么 file_exists 函数会执行它应该执行的操作。

if (FALSE===file_exists("my/path.txt")) {
  // Stop here, 
}