0

Столкнулся с проблемой при написании сайта, а именно - не работает генерация пароля. Сама суть задачи в чем - есть поле для ввода пароля, а рядом "Сгенерировать", при нажатии на которое в поле ввода пароля вставляется сгенерированный пароль. Собственно, вот сам кусочек с полем для пароля:

<li>
   <label for="">Пароль</label>
   <span class="star">*</span>
   <input type="text" name="reg_pass" id="reg_pass">
   <span id="genpass">Сгенерировать</span>
</li>

Написал вот такой вот скрипт в отдельном файле на php для генерации пароля:

    <?php
    $number = 7;

    $arr = array('a','b','c','d','e','f',

                 'g','h','i','j','k','l',

                 'm','n','o','p','r','s',

                 't','u','v','x','y','z',

                 '1','2','3','4','5','6',

                 '7','8','9','0');

    // Генерируем пароль

    $pass = "";

    for($i = 0; $i < $number; $i++)

    {

      // Вычисляем случайный индекс массива

      $index = rand(0, count($arr) - 1);

      $pass .= $arr[$index];

    }


  echo $pass;
?>

А это уже ajax-функция, которая обрабатывает "Сгенерировать" и должна сразу выводить сгенерированный пароль в input, однако, увы, почему-то не работает... Располагается функция в отдельном файле, где есть еще и другие функции, которые успешно работают. Вот её код:

$('#genpass').click(function(){
 $.ajax({
  type: "POST",
  url: "/functions/genpass.php",
  dataType: "html",
  cache: false,
  success: function(data) {
  $('#reg_pass').val(data);
  }
});

}); 

Помогите, пожалуйста, разобраться, почему функция не работает...

3
  • Не по теме вопроса: генерирование паролей на сервере вызовет сомнения в безопасности у пользователей, а 7 символов для пароля слишком мало
    – andreymal
    22 мая 2017 в 13:57
  • По теме вопроса: ошибка-то какая-нибудь в консоли или где-то ещё печатается?
    – andreymal
    22 мая 2017 в 13:58
  • @andreymal нет, ошибку не наблюдаю
    – Dremjke
    22 мая 2017 в 14:35

2 ответа 2

1

Генерируйте пароль на клиентской строне, зачем зря сервер мучать?

function generatePassword(length = 8){
    let alphabet = 'qwertyuiopasdfghjklzxcvbnm';
    let pass = '';
    while(length--)
        pass += alphabet[~~(Math.random()*alphabet.length)];
    return pass;
}

UP:

Как это прикрутить:

$('#genpass').click(() => {
 $('#reg_pass').val(generatePassword());
}); 
7
  • изучил ваш скрипт) закинул его в директорию js а в своём ajax-скрипте вместо url: "/functions/genpass.php" указал url: "/js/genpassword.js", однако при нажатии "Сгенерировать" мне в поле ввода пароля выдаёт текст "function generatePassword(length = 7){ let alphabet = 'qwertyuiopasdfghjklzxcvbnm'; let pass = ''; while(length--) pass += alphabet[~~(Math.random()*alphabet.length)]; return pass;}" то есть - содержимое скрипта) где-то я туплю...
    – Dremjke
    22 мая 2017 в 14:44
  • @Dremjke обновил ответ
    – Darth
    22 мая 2017 в 14:50
  • не сочтите за тупость, но не совсем понял, как правильно "прикрутить"... вместо моей ajax-функции вставить?
    – Dremjke
    22 мая 2017 в 14:56
  • Ну да. Зачем вам отсылать лишний раз запрос на сервер, если всё то же самое можно сделать на клиенте
    – Darth
    22 мая 2017 в 15:00
  • всё, уже разобрался полностью с кодом, спасибо вам большое!)
    – Dremjke
    22 мая 2017 в 15:01
1

возвращай json из скрипта:

$ans= [
   'pass' => $pass,
];
return json_encode($ans)

и на клиенте получай json:

$('#genpass').click(function(){
   $.ajax({
       url: "/functions/genpass.php",
       cache: false,
       dataType: 'json',
       success: function(data.pass) {
           $('#reg_pass').val(data.pass);
       },
       error: function() {
           alert('Error'); 
       }
   });

 }); 
1
  • 1
    И вам спасибо, этот метод тоже пригодится!
    – Dremjke
    22 мая 2017 в 15:03

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.