Не срабатывает if в цикле, что делать?

Question

[Роман Смирнов]

Почему-то не срабатывает if в цикле, не могу понять почему

<input type="text" id="key">    	
<input type="text" id="input"> 					 							 								 								
<p id="parag"></p>
 <input type="submit" id="sub">
			
			var input = document.getElementById('input');
			var keyInput = document.getElementById('key');
			var parag = document.getElementById('parag');
			var sub = document.getElementById('sub');
			sub.addEventListener('click', function(valArr, strKey) {
				
			var valArr = input.value.split(' ');
			var strKey = keyInput.value;
				console.log(valArr);
				console.log(typeof(strKey));
			for (var i = valArr.length - 1; i >= 0; i--) {<ol>

				console.log(strKey);
				console.log(valArr);
			if(valArr[i].includes(strKey)) { 
			delete valArr[i] 
			} 
			} 
			return valArr;
				console.log(valArr);
			});

Comments

[Павел Корнилов]

Это что за чудеса программирования?

function(valArr, strKey) {
      var valArr = input.value.split(' ');
      var strKey = keyInput.value;
// ...
}

И это все еще с учетом того, что обработчик события получает первым аргументом сам объект события!

[Роман Смирнов]

KorniloFF,
Пытался сделать так, чтобы за первый и второй параметр принимались значения из инпутов

function deleteElemsWith (arr, str) { 
for (let i = arr.length - 1; i >= 0; i--) { 
if(arr[i].includes(str)) { 
delete arr[i] 
} 
} 
return arr; 
} 

let newArr = deleteElemsWith(['Привет', 'как', 'Дела'], 'ак'); 
console.log(newArr);

[Павел Корнилов]

Роман Смирнов, ну это совсем не правильно. К тому же первым параметром будет все равно объект. Почитайте про обработчики событий.

[Роман Смирнов]

KorniloFF, спасибо, только я всё равно не понял, как мне сделать, чтобы параметры функции можно было записать через инпуты

[Павел Корнилов]

Ну не умею я ногой ухо чесать. Зачем нужно задавать именно параметры, если из функции можно получить значения инпутов в локальные переменные и оперировать ими?

[Роман Смирнов]

KorniloFF,
Видимо опять всё не так делаю, не работает

var input = document.getElementById('input');
			var keyInput = document.getElementById('key');
			var parag = document.getElementById('parag');
			var sub = document.getElementById('sub');
			sub.addEventListener('click', grebl);
								 
			function grebl(){
				var value = input.value.split(' ');
				var key = keyInput.value;
				
			for (var i = value.length - 1; i >= 0; i--) {
				
			if(value[i].includes(key)) { 
			delete value[i] 
			} 
			} 
			return value;
				console.log(value);
			}

[Павел Корнилов]

Роман Смирнов, да, не так.
А вот так все работает - https://codepen.io/anon/pen/mqjoJR?editors=1111

Answer 1

[ThunderCat]

вы явно что-то стандартное делаете явно геморроидальным способом. Какова цель сего изврата? И там точно нет возможности все это нормально сделать?

PS: слегка напрягает отсутствие тегов script

Comments

[Роман Смирнов]

Вы явно что-то стандартное делаете явно геморроидальным способом.

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

function deleteElemsWith (arr, str) { 
for (let i = arr.length - 1; i >= 0; i--) { 
if(arr[i].includes(str)) { 
delete arr[i] 
} 
} 
return arr; 
} 

let newArr = deleteElemsWith(['Привет', 'как', 'Дела'], 'ак'); 
console.log(newArr);

Скрипт в другом файле находится, поэтому и не стал вставлять тег

[ThunderCat]

Роман Смирнов,
не проще что-то типа такого?

var arr = $('.input').val().split(' ');
arr.each(function(i,v){
  if(v.indexOf("ak") != -1){
    delete arr[i];
  }
})

писано вживую, могу накосячить с нюансами.

[Роман Смирнов]

ThunderCat, это jquery? Если да, то нужен обычный js