¿que falla en mi codigo?

[angel1993]

Código Javascript:

<script language="javascript"> 
function cambiar(pipam){
if(document.getElementbyid(pipam).src=="imagenes/privado.png"){
document.getElementbyid(pipam).src="imagenes/publico.png";
}else{
document.getElementbyid(pipam).src="imagenes/privado.png";
}}
</script>
 

Código HTML:

<span onclick="cambiar('privado')"><img   src="imagenes/publico.png" width="25" height="25" alt="Privado/Publico" id="privado" /></span>
 

Cuando clikeo en la imagen no pasa nada

[zerokilled]

no estas llamando a la funcion correcta. javascript es sensible a las mayuscula y minusculas, fijate que getElementById no lo tienes correcto.

[angel1993]

:), tengo esto

Código Javascript:

<script language="javascript"> 
function cambiar(pipam){
if (document.getElementById(pipam).src == "imagenes/privado.png"){
document.getElementById(pipam).src="imagenes/publico.png";
}else{
document.getElementById(pipam).src="imagenes/privado.png";
}}
 
</script>
 

Ahora si funciona a medias, osea si el src esta en "imagenes/publico.png" me la cambia a "imagenes/privado.png", pero si esta en "imagenes/privado.png" no la cambia a "imagenes/publico.png", no tengo ni idea por que sera.. ¿e if esta bien escrito no?
lo he comparado con otros que si me funcionan y esta bien :S ni idea por que puede ser..

[caricatos]

Hola:

Aunque el atributo src tenga una ruta relativa, javascript te va a procesar la ruta absoluta, así que la pregunta deberías hacerla contemplándo eso... por ejemplo:

ruta = document.getElementById(pipam).src.split("/"); if (ruta[ruta.length - 1] == "privado.png")...

Saludos

[zerokilled]

en el sentido sintáctico el codigo esta bien escrito. el problema es que la propiedad src lee la ruta completa. es decir, la ruta aparece con todo su protocolo, domionio, directorio, y nombre de archivo. hay varias formas obtener el valor que necesitas. en tu caso, uno bien facil seria que en lugar de leer la propiedad src, utiliza el metodo getAttribute para obtener el valor tal como aparece en el codigo fuente.

@edit,
hola caricatos, otra vez nos tropezamos

[angel1993]

Iniciado por caricatos

Hola:

Aunque el atributo src tenga una ruta relativa, javascript te va a procesar la ruta absoluta, así que la pregunta deberías hacerla contemplándo eso... por ejemplo:

ruta = document.getElementById(pipam).src.split("/"); if (ruta[ruta.length - 1] == "privado.png")...

Saludos

¿por que el - 1? me he estuado el codigo, pero me falta saber eso..

[caricatos]

Iniciado por zerokilled

@edit,
hola caricatos, otra vez nos tropezamos

¡Je, je! ... y lo que nos queda... ,

Iniciado por angel1993

¿por que el - 1? me he estuado el codigo, pero me falta saber eso..

Los arrays se indexan desde el cero, así que si el array obtenido es de n elementos, el último (o sea el que nos interesa) será el n - 1...

Saludos

[angel1993]

ahh, gracias tio :)