Как удалить элемент из массива?

Question

[Denioo]

Есть страничка с 3 кнопками, когда нажимаю на одну в функции js идет запись значения кнопки через push в массив, когда отжимаю идет удаление через pop. Суть в том, что если выбираю сначала 1 кнопку - идет запись в массив; затем 2 выбираю - так же идет запись в массив; но когда отжимаю первую кнопку - удаляются значения 2 кнопки. Как это исправить, чтоб все работало корректно в обе стороны?

Вот код, isSelected как раз та самая кнопка, если нажата = true, нет = false, а value значение присваиваемое в массив по нажатию:

function setPropertyShortName(propertySelection) {
			if (propertySelection.isSelected) {
				vm.selectedValue.push(propertySelection.value);				
			}
			if (!propertySelection.isSelected) {
				vm.selectedValue.pop();				
			}

//эта часть не важна, она работает корректно
			for (var i = 0; i < vm.properties.length; i++) {
				if (vm.properties[i].value === propertySelection.value) {
					vm.properties[i].isSelected = propertySelection.isSelected;
					//continue;
				}
			}
	        search(true);
        }

Answer 1

[Сергей delphinpro]

Метод .pop() вытаскивает последнее значение из массива.
Вам нужно вытаскивать произвольное.
Воспользуйтесь для этого методами slice / splice

Comments

[Denioo]

а не подскажите хотя бы примерно как написать запрос

[Сергей delphinpro]

Denioo, Документация по ссылкам на русском и с примерами. Даже не знаю что еще добавить..

[Denioo]

Да, спасибо, я смотрел вот только смотрите... У меня 3 кнопки, допустим я нажал все 3 такой последовательности 1, 3, 2. Когда я отжимаю допустим 3 кнопку должно удалиться значения 3 кнопки из массива, как написать подобный код немного не представляю пока что, вот и спросил у вас совета. Значения кнопок так же находятся в массиве возможно это вас спутало.

[Сергей delphinpro]

Denioo, в лоб примерно так:

Когда отжимаем кнопку, смотрим какое значение у этой кнопки.
Потом ищем это значение в массиве и получаем его индекс (indexOf)
Имея индекс удаляемого элемента воспользуемся методом splice

Примерный обработчик:

let arrValues = [ … ]; // массив с данными
function() {
  // пусть this — это кнопка

  let value = this.value;
  let index = arrValues.indexOf(value);
  if (index < 0) {
    // Значения не в массиве
    return;
  }

  arrValues.splice(index, 1); // удалить
}

[Denioo]

Сергей delphinpro, Спасибо!