file_exists() 在 while 循环内的 php5 中不起作用

Question

file_exists 不工作。我看了几个例子，还是不行。程序未检测到该文件。我的文件路径是/var/www/osbs/PHPAPI/recording.mp3，网站根目录在osbs里面。该文件的位置在 PHPAPI 中，这就是为什么我没有将完整路径放在 file_put_contents 中的原因。该程序能够制作原始的recording.mp3，但不能制作它的任何附加版本。

<?php
$actual_name = pathinfo("PHPAPI/recording.mp3",PATHINFO_FILENAME);
$original_name = $actual_name;
$extension = pathinfo("PHPAPI/recording.mp3",PATHINFO_EXTENSION);

if ($_GET["RecordingUrl"]) {
     if (file_exists("/var/www/osbs/PHPAPI/".$actual_name.".".$extension)) {
        $actual_name = find_new_name($original_name, $extension);
     }
     else {
        $actual_name = $original_name;
     }
     $name = $actual_name.".".$extension;
     file_put_contents($name, file_get_contents($_GET["RecordingUrl"]));
}

function find_new_name ( $file, $extension ) {
    $name = $file.".".$extension;
    $i = 0;
    while(file_exists("/var/www/osbs/PHPAPI/".$name)){
        $new_name = $file.$i;
        $name = $new_name.".".$extension;
        $i++;
    }
    return $new_name;
}
 ?>

Answer 1

您的问题与file_put_contents 有关。您需要指定一个完整路径，并且您只指定一个文件名。在使用之前尝试echoing $name，你会发现它不是路径，只是文件名。

我建议您在文件开头设置一个带有路径的常量，而不是有时依赖相对路径，有时依赖绝对路径。

<?php
const SAVE_PATH = "/var/www/osbs/";

$actual_name = pathinfo(SAVE_PATH."PHPAPI/recording.mp3",PATHINFO_FILENAME);
$original_name = $actual_name;
$extension = pathinfo(SAVE_PATH."PHPAPI/recording.mp3",PATHINFO_EXTENSION);

if (isset($_GET["RecordingUrl"]) && $_GET["RecordingUrl"]) {
     if (file_exists(SAVE_PATH."PHPAPI/".$actual_name.".".$extension)) {
        $actual_name = find_new_name($original_name, $extension);
     }
     else {
        $actual_name = $original_name;
     }
     $name = $actual_name.".".$extension;

     file_put_contents(SAVE_PATH.'PHPAPI/'.$name, file_get_contents($_GET["RecordingUrl"]));
}

function find_new_name ( $file, $extension ) {
    $name = $file.".".$extension;
    $i = 0;
    while(file_exists(SAVE_PATH."PHPAPI/".$name)){
        $new_name = $file.$i;
        $name = $new_name.".".$extension;
        $i++;
    }
    return $new_name;
}
 ?>

我改变了什么：

1. 定义了const SAVE_PATH = "/var/www/osbs/";
2. 在任何地方都使用新常量。不再有时而相对，时而绝对，一切都是绝对的。
3. 在file_put_contents 中使用了常量（这是实际修复，您需要一个完整的路径）
4. 添加了一个额外的检查来确保RecordingUrl isset，否则当它没有设置时你会得到一个PHP警告。

Answer 2

问题似乎出在脚本的第一行：

$actual_name = pathinfo("PHPAPI/recording.mp3", PATHINFO_FILENAME);

这会将recording.mp3 分配给$actual_filename。然后，您通过将扩展名连接到文件名来检查recording.mp3.mp3。我想你想使用PATHINFO_BASENAME 它将返回文件名无扩展名。

Answer 3

您确定路径吗？ /PHPAPI 在 /PHPAPI 中查找文件，而不是预期的 /var/www/osbs/PHPAPI/。你应该检查PHPAPI/$filename。

Answer 4

你必须把你所有的逻辑都放在find_new_name() 函数中。这将使您的代码更清晰

if ($_GET["RecordingUrl"]) {
  $name = find_new_name("PHPAPI/recording.mp3");
  file_put_contents($name, file_get_contents($_GET["RecordingUrl"]));
}

function find_new_name($name) {
  $info = pathinfo($name);
  $name = $info['basename'];
  $i = 0;
  while (file_exists("$info[dirname]/$name")) {
    $name = sprintf('%s%d.%s', $info['filename'], ++$i, $info['extension']);
  }
  return "$info[dirname]/$name";
}

Answer 5

您忘记了 file_put_contents() 的路径。 应该是：

file_put_contents("PHPAPI/".$name, file_get_contents($_GET["RecordingUrl"]));

或者：

file_put_contents("/var/www/osbs/PHPAPI/".$name, file_get_contents($_GET["RecordingUrl"]));

Answer 6

您对文件的 URL 和它的 PATH

感到困惑

您的 httdoc（或 public_html ）根目录是 /var/www/osbs/PHPAPI

但您的文件系统根目录是 '/'

试试

file_put_contents( __DIR__.'/'.$name, file_get_contents($_GET["RecordingUrl"]));

你的代码中有很多不好的做法

Answer 7

'file_exists' 和其他一些文件调用（如 fstat）由 php 缓存。这记录在 file_exists 的手册中。当文件不存在时您的第一次调用将被保存并在后续调用中返回。在调用之间使用“clearstatcache()”来清除缓存。

Answer 8

一个“小”重构：

1. 无处不在的绝对路径
2. 函数透明，名称更一目了然，参数使用更简单
3. 防止恶意输入（$_POST 确实不适合）

4. 为什么要实际复制（）时使用file_put_contents（）

   <?php
   
   define("SRC_PATH", "/var/www/osbs/whereverYourSrcIs/");
   define("SAVE_PATH", "/var/www/osbs/PHPAPI/");
   
   function findAvailableName($name) {
       $i = 1;
       $pathinfo = pathinfo($name);
       while(file_exists($name)) {
           $name = $pathinfo['dirname'] . '/' . $pathinfo['filename'] . "." . $i++ . "." . $pathinfo['extension'];
       }
       return $name;
   }
   
   if (isset($_GET["RecordingUrl"]) && $_GET["RecordingUrl"]) {
   
       if (strpos('/' . $_GET['RecordingUrl'] . '/', '/../') !== false) {
           die("invalid input, don't be evil");
       }
   
       copy(SRC_PATH . $_GET["RecordingUrl"], findAvailableName(SAVE_PATH . "recording.mp3"));
   }