Неинициализированное смещение строки

Question

protected static function generateString ($length = 50, $Password){ 
     $characters = $Password; 
     $string = ''; 

     for ($p = 0; $p < $length; $p++) { 
      $string .= $characters[mt_rand(0, strlen($characters))]; // Error Line 
     } 
     echo $string; 
    } 

Hello. Я строй класса аутентификации, а функция выше дает мне некоторую печаль, я постоянно получаю представлен с сообщением об ошибке:

Примечания: Uninitialized строки смещении: 16

эха есть для в целях тестирования.

Я зову свою функцию с помощью:

public static function Compile($Password, $Length){ 
     self::generateString($Length,$Password); 
    } 

Auth::Compile('testi ngpassword', 10); 

Когда я бегу мой сценарий, иногда он генерирует без сообщений об ошибках; но при запуске несколько раз (3- 7) мне представляется сообщение, как указано выше? и иногда это уведомление показывает более двух строк?

Так почему же сообщение об ошибке генерируется случайным образом при повторном запуске? когда указанные аргументы не меняются?

Answer 1

рассмотрит следующую строку:

strlen($characters) 

, если длина строки 16 и вы делаете случайное число выбора между 0 и 16, вы даете ему возможность выбрать индекс после последнего, который ничего. Вам необходимо сделать это

strlen($characters) - 1 

Поскольку индекс фактически начинается с 0;