В чём может быть ошибка?

Question

[Basitkhan Mansurov]

есть такая функция, которое не принимает никаких значений, а просто генерирует случайный набор букв и цифр

function generate(){
		$keys = "qwertyuiopasdfghjklzxcvbnmQWERTYUIOPASDFGHJKLZXCVBNM123456789";
		for ($a=0; $a <= 12; $a++){
			$rand = rand(0,strlen($keys));
			$result .= $keys[$rand];
		}
		return $result;
	}

А в логах выдаёт ошибку
На эту строку $result .= $keys[$rand];

PHP Notice: Uninitialized string offset: 61 in /home/creativ5/public_html/admin_app/includes/project_create.php on line 126

Comments

[Basitkhan Mansurov]

Тестирую как отдельный модуль, --всё нормально. Да это не ошибка, это просто уведомление, но всё равно раздражает...

Answer 1

[Андрей]

$result = bin2hex(random_bytes(6));

Comments

[Basitkhan Mansurov]

А вот это тема =)

[Lander]

Basitkhan Mansurov, Ага. Только нифига не тот же результат. Набор символов другой :)

[Basitkhan Mansurov]

Lander, Ну мне главное, чтобы там был случайный набор символов, а тут как раз всё работает и ничего не ругается)

Answer 2

[SagePtr]

rand(0,strlen($keys)-1) надо

Comments

[Basitkhan Mansurov]

Разжуй пожалуйста

[SagePtr]

Basitkhan Mansurov, функция rand($a, $b) возвращает число от $a до $b включительно.
А в массивах и строках элементы имеют индексы от 0 до Размер-1, а не от 0 до Размер.
Потому вторым параметром функции rand нужно передавать не длину строки, а длину, уменьшенную на единицу (иначе в некоторых случаях будет возвращаться индекс, равный длине, выпадающий за пределы массива).
Причём эту ошибку вы не обязательно словите, а она будет с некоторой долей вероятности у вас выпадать, в следствие случайности.
Достаточно подробно разжевал, или остались непонятные моменты?

[Basitkhan Mansurov]

SagePtr, Вроде всё понятно, спасибо большое)