使用php显示目录中的随机图像

Question

我发现这个questions 对我正在做的事情有帮助，但我遇到的问题是 - 我不断收到这个$images = glob($imagesDir. '*.{jpg,jpeg,png,gif}', GLOB_BRACE); 的未定义索引错误

这是我的完整代码（它是 if/elseif 的一部分）：

elseif($result['avatar_type'] == 'Random'){ 
    $images = array(); //Initialize once at top of script
    $imagesDir = 'avatar/random/';
    if(count($images)==0){
      $images = glob($imagesDir. '*.{jpg,jpeg,png,gif}', GLOB_BRACE);
      shuffle($images);
      }
    $avatar = array_pop($images);
}

我想要做的是，如果数据库将 avatar_type 设置为 Random，然后显示 Random 目录中的随机图像，但就像我上面所说的，我不断收到未定义的索引错误。

有没有人发现我正在做的事情有什么问题？为什么我会收到这个错误？

Answer 1

一些建议：

这不是必需的，因为 glob 将返回一个数组：

$images = array(); //Initialize once at top of script

见http://php.net/manual/en/function.glob.php

如果 glob 之前返回 false，这将导致警告（但不是错误）：

$avatar = array_pop($images);

http://php.net/manual/en/function.array-pop.php

如果您确保检查手册中的返回类型，您就会知道要在代码中检查什么。

if (empty($var)) 很棒，因为它会检查 false、null 或 undefined 而不会引发错误。

此外，由于 array_pop 返回最后一个元素，而 glob 可能以相同的顺序返回元素，它不会像 array_rand 那样随机。

$avatarKey = array_rand($images, 1); //return 1 random result's key
$avatar = $images[$avatarKey]; //set the random image value (accessed w/ the rand key)

您的错误信息不应该是由 glob 行引起的，它实际上可能来自：

elseif($result['avatar_type'] == 'Random'){ 

如果 avatar_type 未在结果数组上设置或结果数组为空，您将获得未定义的索引。

为防止发生该错误，您应在尝试访问 avatar_type 键之前检查数组是否存在：

函数示例：

function getRandomAvatar($result)
{
    if (empty($result) || empty($result['avatar_type'])) {
        return;  //quit execution if the data is bad
    }
    //rest of code here -
}

内联代码示例：

if (empty($result) || empty($result['avatar_type'])) {
    //do nothing, render an error, whatever - stops execution of the next statement
} else {
    //this code will only run if $result and $result['avatar_type 
    //are set and wont cause errors
    if ('$result['avatar_type'] == 'Random') {
        //do code here

您的错误应该有一个行号。检查该行和它之前的行。