El codigo me trollea

[legomolina]

Hola, tengo el siguiente codigo:

Código Javascript:

function vacios() {
            pass=document.getElementbyId('pass');
            user=document.getElementsById('user');
            alert('Hola');
        }
 

y luego esto

Código HTML:

<a href="javascript:vacios();">Alert</a> 

Pues si quito antes del alert las variables funciona pero asi ya no.
Pero no se porque.
A ver si alguien sabe porque

[ClubIce]

Código Javascript:

function vacios() {
pass=document.getElementById('pass');
user=document.getElementById('user');
alert('Hola');
}
 

por el codigo que pasaste, el error esta en el nombre de la funcion, es document.getElementById no document.getElementbyId ni document.getElementsById

utiliza la consola del navegador para identificar estos pequeños errores que son muy comunes.

[Artificium]

Debe suceder que no existe ningún elemento con id pass o user.

[legomolina]

Si que existe:
Es un formulario de log-in y lo que quiero es que si los campos 'user' o 'pass', usuario y contraseña, estan vacios, que salte un alert diciendo que estan vacios y si no que se envie el form. Eso lo se hacer pero es raro que no se muestre el alert como digo arriba.

[caricatos]

Hola:

Lo correcto es hacer la validación desde el evento de envío submit del tag form, devolviendo false si es incorrecto, así se cancela el envío...

Aparte del error de getElementbyId (la "b" debe ser mayúscula), no sería necesario controlar el atributo id...

<form onsubmit="return vacios()"...

Luego tampoco se ve la condición...

function vacíos() {
if (pass.value == "" || usser.value == "") {
//aquí la alerta;
return false;
}
}

Saludos

[legomolina]

Entonces, return false; deberia estar dentro de la condicion del if, no?
Porque si no no se enviaria

[caricatos]

¡Correcto!...

Fíjate como están las llaves.

Saludos

[legomolina]

No hace falta poner un return true?
en un else?

[caricatos]

Iniciado por legomolina

No hace falta poner un return true?
en un else?

Solo se cancela con return + algo que equivalga a false... (1==2), (2*2 ==5)... cualquier otra cosa no sirve para la cancelación (incluyendo 0 ó -1.

Saludos

[legomolina]

Gracias. Al final funciono con tu codigo-