около 20% времени неудачного сценария opendir. См. Пример

Question

Хит обновления несколько раз, и иногда я получаю «null».

Этот скрипт проходит через папку, чтобы получить все mp3-файлы и случайным образом выбирает их. Что я делаю неправильно? Благодаря

if ($handle = opendir('../../hope/upload/php/files/')) { 

    while (false !== ($entry = readdir($handle))) { 

     $entry = trim($entry); 
     if(preg_match('/.mp3/', $entry)) 
     { 
     $mp3[] = "$entry"; 
     } 
    } 


    closedir($handle); 
    $count = count($mp3); 

$rand = rand(0,$count -1); /// FIXED BY adding a -1 after count** 
$mp3 = $mp3[$rand]; 

if($mp3) 
    { 
    echo "http://MyWebsite.com/hope/upload/php/files/$mp3"; 
    } 
else 
    { 
    echo "null"; 
    } 
} 

Answer 1

Это происходит потому, что индексы массива идут от 0 к length - 1, но ваш скрипт генерирует случайный индекс от 0 к length. Предпочтительный способ исправить это было бы использовать array_rand():

$rand = array_rand($mp3); 
$mp3 = $mp3[$rand]; 

Answer 2

Вы случайным образом диапазон аут (максимальное целое является результатом count(), и помните, счетчик массива является один выше самого высокого показателя в порядковом массиве 0), и ваш код выглядит слишком многословным.

Try ...

$mp3s = glob('../../hope/upload/php/files/*.mp3'); 

$key = array_rand($mp3s); 

$randomMp3 = $mp3s[$key];