Remove event listeners when no longer needed to prevent memory leaks
search.addEventListener('keyup', function(){
1'use strict'23var search = document.querySelector('[type=search]');4var code = document.querySelector('pre');56search.addEventListener('keyup', function(){7 var xhr = new XMLHttpRequest;8 xhr.open('GET', '/search/' + search.value, true);9 xhr.onreadystatechange = function(){10 if (xhr.readyState === 4) {11 code.textContent = xhr.responseText;12 }13 };14 xhr.send();15}, false);
Code quality findings 5
Use let or const to avoid scope issues and hoisting
var search = document.querySelector('[type=search]');
Use let or const to avoid scope issues and hoisting
var code = document.querySelector('pre');
Use let or const to avoid scope issues and hoisting
var xhr = new XMLHttpRequest;
Use strict equality (===) to prevent type coercion bugs
if (xhr.readyState === 4) {