Escribí este código que recorre todos los elementos con una ID y declara una variable con el mismo nombre que la ID que se refiere a ese elemento. El código funciona como se esperaba y al usarlo en la parte superior de mi programa ahorro mucho tiempo y líneas en mi programa. Sin embargo, casi parece bueno para ser verdad porque nunca antes lo había visto.

Sé que usar eval() es una mala práctica y me pregunto si usar window[] , como hago aquí, tiene el mismo efecto. Mi pregunta es si usar este código es una buena práctica o si debe evitarse.

 <div id="content1"></div> <div id="content2"></div> <div id="content3"></div> <div id="content4"></div> <script> var elements = document.querySelectorAll("[id]"); for (var i = 0; i < elements.length; i++){ window['elements[i].getAttribute("id")'] = document.getElementById("${elements[i].getAttribute('id')}"); }; content1.innerHTML = `<p>Hello World</p>`; </script>