Question

estoy intentando hacer una prueba pero tengo un problema al tratar de modificar el color de fondo del elemento de mi menu, tengo un menu de navegacion horizontal con tres elementos todos los elementos llevan un color de fondo azul, quiero que cuando se haga click en uno de estos elementos se cambia de color de fondo a verde, indiferentemente de cual se haya presionado.

<ul id="menu">
  <li>item1 </li>
  <li>item2</li>
  <li>item3</li>
</ul>

var items = document.getElementsBytagName('li')
items.addEventListener(onclick, function(){

                     style.backgroundColor="green"
})

cuando se hace un click sobre un elemento li este deberia de cambiar de color de fondo
esto debe ocurrir con todos los elementos, cada vez que se haga click sobre cualquier li debe cambiar de fondo solo el li que fue clickeado.
esto efecto es comun en todas las paginas, por ejemplo en la pagina de la w3c cuando se esta es una seccion del menu este se pone verde, cuando se cambia a otra seccion esto se pone en verde, espero haber sido suficientemnte expresivo

Answer 1

Todo tu código es erróneo, además de ineficiente. Lo de la eficiencia lo dejaremos para otro día, porque introduce conceptos avanzados que te puede liar, así que haré las correcciones sobre tu código:

1. La función se llama getElementsByTagName, con la T mayúscula, y no es la primera vez que te corrijo esto
2. Efectivamente tienes que usar un for, porque el método anterior te da un array de items
3. onclick tiene que ser click, e ir entre comillas, porque si no lo interpreta como una variable
4. style es una variable no definida.

Absolutamente TODOS los errores te los dice o bien un IDE o bien la consola de desarrollador que viene incorporada con cualquier navegador, por lo que me hace sospechar que no utilizas ni lo uno ni lo otro.

El código que funciona, partiendo del tuyo, es este:

    var items = document.getElementsByTagName('li');
    var previous = null;
    for(var i=0; i<items.length; i++) {
    items[i].addEventListener('click', function () {
        if (previous) {
            previous.style.backgroundColor = "blue";
        }
        this.style.backgroundColor = "green";
        previous = this;
    })
    }

Por supuesto, esto se haría mejor con hojas de estilo y selectores, "apagando" aquel elemento que tenga la clase "selected" o similar, para no tener que guardar el previous.