PHP函数不返回值

Question

我有一个函数,给定一个文件名和目录路径,检查目录是否已经包含一个具有相同名称的文件,如果是这样,则返回一个修改过的文件名(通过在文件名的第一部分后附加一个数字).(get_filenames()函数是一个CodeIgniter帮助函数,它创建一个包含指定目录中所有文件名的数组.)

当我尝试打印出函数调用的返回结果时,我什么都没得到; 但是如果我在函数本身的else {}语句中打印$ new_filename,那么只需调用该函数(而不是打印它的值),它就可以了!

我需要返回函数中的值,而不是打印它,因为我实际上需要将结果分配给变量以进行进一步处理.(在下面的例子中,我刚刚打印了函数调用的结果来演示这一点.)

功能:

function avoid_conflicting_filenames($old_filename, $new_filename, $dir, $count)
{   
    $num = '';
    if ($count > 0):
        $num = $count;
    endif;

    $filename_arr = explode('.', $old_filename, -1);
    $new_filename = $filename_arr[0] . $num . '.' . $filename_arr[1];

    if (in_array($new_filename, get_filenames($dir))):      
        $count++;
        avoid_conflicting_filenames($old_filename, $new_filename, $dir, $count);
    else:
        return $new_filename;
    endif;
}

我在哪里调用函数:

print avoid_conflicting_filenames('file.jpg', '', 'path/to/file', 0);

这让我在过去的一天疯狂,所以任何帮助都将不胜感激!谢谢.

Answer 1

替换这个:

avoid_conflicting_filenames($old_filename, $new_filename, $dir, $count);

有了这个:

return avoid_conflicting_filenames($old_filename, $new_filename, $dir, $count);

你不是递归地思考它.您必须返回函数的返回值.

过去,if语法是什么？我在模板中容忍它,但对于代码？EW.

如果我正确理解代码,你也可以重写这个函数以避免递归,如下所示:

function avoid_conflicting_filenames($old_filename, $new_filename, $dir) {   
    $num = 0;
    $files = get_filenames($dir);
    $filename_arr = explode('.', $old_filename, -1);
    do {
        $new_filename = $filename_arr[0] . $num . '.' . $filename_arr[1];
        $num++;
    } while(in_array($new_filename, $files));
    return $new_filename;
}

我认为这更好,更容易获得,但这取决于你...