Cambiar clase de un objecto clickando en otro objeto

[geclos]

Hola soy bastante nuevo en esto de javascript.

El tema es el siguiente.

Tengo una lista de links de class "menulink":

Código:

<div class="linkactivo">
                	<a href="quees14cg.html" class="menulink">Qué es 14CG?</a>
                	</div>
                	<div class="linkinactivo">
                    <a href="vision.html" class="menulink">Visión, Misión y Valores</a>
                    </div>
                    <div class="linkinactivo">
                    <a href="equipo.html" class="menulink">Equipo</a>
                    </div>
                    <div class="linkinactivo">
                    <a href="comotrabajamos.html" class="menulink">Como trabajamos</a>
                    </div>

La idea es que al clickar en alguno de estos links, pueda cambiar la class de otros objetos situados mas abajo y de class "contentvisible" y "contentnovisible" (dependiendo de si quiero que sean visibles o no).

Este es el javascript que tengo de momento:

Código:

window.onload = initAll();

function initAll() {
	var allLinks = document.getElementsByTagName("a");
	for ( var i = 0; i<allLinks.lenght; i++) {
		if (allLinks[i].className == "menulink") {
			allLinks[i].onClick = toogleMenu();
		}
	}
}

function toggleMenu() {
				var startId = this.href.lastIndexOf("/")+1;
				var stopId = this.href.lastIndexOf(".");
				
				var myId = this.href.substring(startId,stopId);
				var myIdClass = document.getElementById(myId).className;
				
				if (myIdClass == "contentvisible") {
        	myIdClass = "contentnovisible";
			} else {
				myIdClass = "contentvisible";
				}
				return false;
			}

Como vereis por el codigo, para obtener el Id del objeto que quiero cambiar lo que hago es sacarlo del href del link que he apretado con la funcion substring.

El problema es que al cargar la pagina nada de esto ocurre y si apreto en alguno de los links me redirige al href correspondiente, que no existe.

Gracias de antemano por la ayuda.

Un saludo.

[alexg88]

Tienes algunos fallos que he solucionado y ya se ejecuta, en una de las sentencias:

Código Javascript:

var myIdClass = document.getElementById(myId).className;
 

document.getElementById(myId) da null (no hay ningún elemento con ese id) y, por tanto, no hay propiedad className. Ya verás como lo arreglas.

Código Javascript:

window.onload = initAll;
 
function initAll() {
    var allLinks = document.getElementsByTagName("a");
    for ( var i = 0; i<allLinks.length; i++) {
        if (allLinks[i].className == "menulink") {
            allLinks[i].onclick = toggleMenu;
        }
    }
}
 
function toggleMenu() {
                var startId = this.href.lastIndexOf("/")+1;
                var stopId = this.href.lastIndexOf(".");
                
                var myId = this.href.substring(startId,stopId);
                var myIdClass = document.getElementById(myId).className;
                
                if (myIdClass == "contentvisible") {
            myIdClass = "contentnovisible";
            } else {
                myIdClass = "contentvisible";
                }
                return false;
            }
 

Fíjate en que no hay que poner los paréntesis cuando asignas una función a un evento (por que sino le estás asignando el resultado de llamarla) y el evento es onclick en minúsculas

[geclos]

Gracias por las correcciones.

Aunque el debugger del Google Chrome ya no me da ningun error sigue sin funcionar el script.

Cuando clickeo en los links sencillamente no ocurre nada, no me redirige a ningun href (lo cual ya es un avance) pero tampoco hace el cambio de class que en teoria tendria que hacer.

Alguna idea de cual puede ser el problema ?

Gracias!

[geclos]

Estoy pensando que tal vez estoy haciendo algo mal en el codigo del script para coger la id del objeto cuya class quiero cambiar.

Para aclarar cosas pego los div cuya class quiero cambiar:

Código:

<div id="quees14cg" class="contentvisible">               
       		</div>
            
            <div id="vision" class="contentnovisible">
            </div>
            
            <div id="equipo" class="contentnovisible">
            </div>
            
            <div id="comotrabajamos" class="contentnovisible">
            </div>

Como puede ver la id de los objetos cuya class quiero cambiar corresponde al href del link correspondiente. En teoria la id se coge a partir del href que se clickea con esta parte del codigo script:

Código:

var startId = this.href.lastIndexOf("/")+1;
                var stopId = this.href.lastIndexOf(".");
                
                var myId = this.href.substring(startId,stopId);
                var myIdClass = document.getElementById(myId).className;

Alguien ve algun problema en todo esto ?

Gracias y un saludo!