Увеличивающийся счетчик не меняет значение другой переменной

Question

Увеличивающийся счетчик не меняет значение другой переменной

1

почему значение переменной [counter], используемое в другой переменной, не изменяется при изменении значения счетчика? Это как-то связано со ссылкой?

var counter = 0;

var allFields = document.getElementsByClassName('test')[counter].getElementsByClassName("pizza");

console.log(allFields);

counter++;

console.log(allFields); // expected result is to have 2 pizza elements.

<div class="test">
	<div class="pizza"></div>
</div>
<div class="test">
	<div class="pizza"></div>
	<div class="pizza"></div>
</div>
<div class="test">
	<div class="pizza"></div>
	<div class="pizza"></div>
  <div class="pizza"></div>
</div>
<div class="test">
	<div class="pizza"></div>
	<div class="pizza"></div>
  <div class="pizza"></div>
  <div class="pizza"></div>
</div>

J. Doe 18 июнь 2018, в 23:30

Источник

0

Когда вы запускаете этот код, counter равен 0. Он не увеличивается до следующего.
nurdyguy 18 июнь 2018, в 21:07
0

Да, вы должны пройтись по длине выбранных элементов
Derek Pollard 18 июнь 2018, в 21:08
0

О, на самом деле, "Это что-то делать со ссылкой?" да. Это целое число, поэтому оно передается по значению, и объект сохраняется в этой точке. При дальнейшем увеличении исходное значение остается неизменным.
nurdyguy 18 июнь 2018, в 21:09
0

Как эффективно увеличить счетчик, чтобы изменить все allFields соответственно?
J. Doe 18 июнь 2018, в 21:11
0

Оставьте document.getElementsByClassName('test') в виде массива и затем обращайтесь к нему, когда вам это нужно, через индекс counter . Как только вы сохраняете один элемент этого массива в переменную, он сохраняется и не будет обновляться так, как вы хотите.
nurdyguy 18 июнь 2018, в 21:17

Показать ещё 3 комментария

Теги:

javascript

2 ответа

1

Как только вы установите

var allFields = document.getElementsByClassName('test')[counter]

объект создается. Вы не можете динамически изменять объект, просто изменяя counter. Если вы хотите это сделать, просто сохраните

var allFields = document.getElementsByClassName('test')

и используйте [counter] когда вам нужно.

nurdyguy 18 июнь 2018, в 20:09

Ещё вопросы

Когда вы запускаете этот код, counter равен 0. Он не увеличивается до следующего.
Да, вы должны пройтись по длине выбранных элементов
О, на самом деле, "Это что-то делать со ссылкой?" да. Это целое число, поэтому оно передается по значению, и объект сохраняется в этой точке. При дальнейшем увеличении исходное значение остается неизменным.
Как эффективно увеличить счетчик, чтобы изменить все allFields соответственно?
Оставьте document.getElementsByClassName('test') в виде массива и затем обращайтесь к нему, когда вам это нужно, через индекс counter . Как только вы сохраняете один элемент этого массива в переменную, он сохраняется и не будет обновляться так, как вы хотите.

hsz · Accepted Answer · 2018-06-18T19-56-00.000Z

Когда вы назначили поля для allFields с помощью counter = 0 - они больше не будут меняться - вам нужно снова назначить эту переменную.

Я бы предложил вам создать его как функцию:

var allFields = function(i) {
  return document.getElementsByClassName('test')[counter].getElementsByClassName("pizza");
}

var counter = 0;
console.log(allFields(counter));

counter++;
console.log(allFields(counter));

Другой, лучший способ - создать цикл, например:

for (var i = 0, i < 2; i++) {
  var allFields = document.getElementsByClassName('test')[i].getElementsByClassName("pizza");

  console.log(allFields);
}

Не говоря уже о том, что петли нет
Его недоразумение состоит в том, что он думал, что counter был по ссылке, поэтому, когда он увеличивает его, затем делает другой console.log он ожидал, что оригинальный объект изменится.
«Другой, лучший способ» - это жесткое кодирование?