как установить функцию onchange для каждого добавляемого элемента div - PullRequest
0 голосов
/ 02 февраля 2019

У меня есть кнопка, которая добавляет новый div с тегом select, каждая опция имеет свои атрибуты.моя проблема в том, что когда я создаю несколько div, функция ссылается только на последний div, который я создал, когда пытаюсь изменить первый тег выбора, функция не отвечает.

Я пытался присвоить всем div и входным данным разные идентификаторы, но это не сработало.

 <button type="button" class="cam_field_button btn waves-effect waves-light btn-outline-info" >הוסף צלם</button>


var max_fields = 25; //maximum input boxes allowed
var wrapper = $(".camera_men"); //Fields wrapper
var add_button = $(".cam_field_button"); //Add button ID

var x = 0; //initlal text box count
$(add_button).click(function (e) { //on add input button click
    e.preventDefault();
    if (x < max_fields) { //max input box allowed
        x++; //text box increment
        $(wrapper).append('<div>   <br> <select name="cam[]"  id="cam_select'+x+'"  class="form-control" >\n' +
            '                                            <option email="email" phone="phone"  value="צלם 1">צלם 1</option>\n' +
            '                                            <option email="email2" phone="phone2" value="צלם 2">צלם 2</option>\n' +
            '                                            <option email="email3" phone="phone3" value="צלם 3">צלם 3</option>\n' +
            '                                        </select><a href="#" class="remove_field">הסר צלם ' + x + '</a><div>  <input type="hidden" name="cam_email[]" id="cm_email' + x + '">\n' +
            '                                        <input type="hidden" name="cam_phone[]" id="cm_phone' + x + '">'); //add input box
        $('#cam_select'+x).on('change',function () {


            var email =  $('#cam_select'+x+' option:selected').attr("email");
            var phone =  $('#cam_select'+x+' option:selected').attr("phone");

            $('#cm_email'+x).val(email);
            $('#cm_phone'+x).val(phone);
        })
    }

Ответы [ 3 ]

0 голосов
/ 02 февраля 2019

@ Allesandro сделал правильный шаг.

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

Но вы должны сделать так называемый делегированный обработчик событий jQuery, который будет работать для вновь создаваемых элементов управления «на лету» (это ваш сценарий) и для элементов управления, соответствующих селектору, который был в HTMLуже при первоначальной загрузке документа: он выглядит следующим образом:

$(document).on('change', '.cam_select', function () {
  ...
})

Вы также можете быть более конкретным с курса и использовать wrapper для начального селектора вместо document.

Делегирование событий :

Делегирование событий позволяет нам подключить один родительский прослушиватель к родительскому элементу, который будет запускаться для всех потомков, соответствующих селектору, независимо от того, существуют ли эти потомки сейчасили добавляются в будущем.

0 голосов
/ 02 февраля 2019

Итак, я сделал некоторые изменения в вашем коде -

  1. Используйте классов вместо идентификаторов - потому что вы не можете иметь mutlipleID .
  2. Я использовал template literals здесь, потому что мне это нравится:)
  3. Посмотрите, как я использую динамический слушатель jquery, используя $(document).on('change', '.cam_select' + x, function(e) { -off('change'), чтобы гарантировать отсутствие дублирующих прослушивателей, хотя здесь их нет.
  4. Также мы можем получить элемент clicked , используя e.target, где e - событиеobject.
  5. Обратите внимание, что я добавил еще одну оболочку к добавленной разметке - cam и как я использую ее с помощью функции closest, а затемfind методы поиска братьев и сестер и двоюродных братьев в оболочке cam, которая была динамически вставлена.

Проверьтедемонстрация ниже иллюстрирует это:

$(document).ready(function() {
  var max_fields = 25;
  var wrapper = $(".camera_men"); 
  var add_button = $(".cam_field_button");
  var x = 0; 
  $(add_button).click(function(e) {
    e.preventDefault();
    if (x < max_fields) {
      x++;
      $(wrapper).append(`
        <div class="cam">
          <div>
            <select name="cam[]" class="form-control cam_select">
              <option email="email" phone="phone" value="צלם 1">צלם 1</option>
              <option email="email2" phone="phone2" value="צלם 2">צלם 2</option>
              <option email="email3" phone="phone3" value="צלם 3">צלם 3</option>
            </select>
	        <a href="#" class="remove_field">הסר צלם ${x}</a>
		  <div>
          <input type="input" name="cam_email[]" class="cm_email">
          <input type="input" name="cam_phone[]" class="cm_phone">
        </div>`);
      $(document).off('change').on('change', '.cam_select', function(e) {
        var email = $(e.target).find('option:selected').attr("email");
        var phone = $(e.target).find('option:selected').attr("phone");
        $(e.target).closest('.cam').find('.cm_email').val(email);
        $(e.target).closest('.cam').find('.cm_phone').val(phone);
      });
    }
  });
});
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
<button type="button" class="cam_field_button btn waves-effect waves-light btn-outline-info">הוסף צלם</button>

<div class="camera_men"></div>
0 голосов
/ 02 февраля 2019

Вы можете использовать селектор класса вместо идентификаторов

Пример:

...
$(wrapper).append(<select class="form-control cam_select" >...</select>...)
...
$('.cam_select').on('change',function () {

  var email =  $('option:selected',this).attr("email");
  var phone =  $('option:selected',this).attr("phone");

  $(this).parent().children('.cm_email').val(email);
  $(this).parent().children('.cm_email').val(phone);
})

Я не пробовал код, извините, если есть ошибки

...