Во-первых ошибка здесь:
substr(str_shuffle($gener), 1,$length);
Тут должнен быть 0
, а не единица:
substr(str_shuffle($gener), 0,$length);
Иначе, запросив у ф-ции рандомную строку длиной 67 символов, вы с удивлением обнаружите, что вернется строка длиной 66 символов, хотя даже строка $gener
состоит из 67.
По-моему,
while (strlen($result) < $need_length)
$result .= $gener[rand(0, strlen($gener) - 1)];
выглядит убедительнее. Кроме того, этот вариант может выдать гораздо больше уникальных результатов. К примеру, ваш вариант не сможет выдать строку "99", а код приведенный мной — сможет.
Еще есть прямо-таки суровая ошибка: что если понадобится рандомная строка длиной 67+ символов?
Я бы, по крайней мере, изменил код как-то вот так:
function gener($length = 1) {
$gener = '123456789QqWwEeRrTtYyUuIiOoPp[[]]aAsSdDFfGgHhJjKkLl:;ZzXxCcVvBbNnMm';
$length_need = min($length, strlen($gener));
$result = '';
while (strlen($result) < $length)
$result .= substr(str_shuffle($gener), 0, $length_need); // *
return $result;
}
*Можно и более деликатно написать, чтобы избавиться от ситуации, когда перемешанную строку можно и не обрезать, а брать целиком.
PS А зачем в строке допустимых символов две пары скобок []
— для повышения вероятности?