Насколько правильно такое написание кода?

Question

[Rufix]

Решил сделать задачку на JavaScript, которая будет заменять написанные в инпуте нецензурные слова на звездочки и выводить это предложение в параграф.
Задача вроде работает, но я не уверен, что она написана хорошо, т.к меня смущает написание "цикла внутри цикла". Хотелось бы узнать, насколько это нормальная практика, ну и услышать от вас какие у меня косяки в написании кода, чтобы сразу отучать себя от говнокода.

<input type="text" id="text">
	<p id="textOut"></p>

let abuse = ['блин', 'черт'];
let stars = '***'

let textInput = document.getElementById ('text');
let textOut = document.getElementById('textOut');
textInput.addEventListener ('blur', function censor () {
	let text = textInput.value;
	let words = text.split (' ');
	for (let i = 0; i < words.length; i++) {
		for (let k = 0; k < abuse.length; k++) {
			if (words [i] == abuse [k]) {
				words[i] = stars;
			}
		}
	}
	let newWords = words.join (' ');
	textOut.innerHTML = newWords;
	textInput.value = '';
})

Answer 1

[ShamanHead]

Слишком императивно.У js есть функции для работы с массивами, и для поиска элемента в массиве тоже.IndexOf

Answer 2

[Maxim Bredikhin]

Как вариант:

let abuse = ['блин', 'черт'];

let inp = document.querySelector('#text');
let out = document.querySelector('#textOut');

inp.addEventListener ('blur', () => {
  let words = inp.value.split(' ');
  
  words.forEach(element => {
  	if(!(abuse.indexOf(element) == -1)) {
  		element = '***';
  	}
  });
 
  out.innerHTML = words.join(' ');
  inp.value = '';
})

Answer 3

[Rsa97]

indexOf

Answer 4

[McBernar]

Точки, запятые?
Два цикла, конечно, не нужно. Есть поиск в массиве, есть has в мапах.

Answer 5

[Oleg]

Нуууу, я не мегапрофи, но я не читая описания, сразу понял что делает код, так что мне кажется код написан хорошо, понятно что его можно оптимизировать и тд и тп, но надо ли? Главное то читаемость)

Comments

[Developer]

Нет. Главное не читаемость, а быстрое выполнение кода с минимальным потреблением памяти.