La idea es que event.preventDefault() se pone al inicio de la función para evitar la acción por defecto que haría el evento submit. Luego si los inputs son correctos haces el submit mediante this.submit();
Pero ya te he dicho que sin el código HTML no puedo asegurar que funcione. Pero tiene toda la pinta de que estés mezclando código HTML con javascript. Aquí tienes un tutorial de iniciación con eventos:
http://librosweb.es/javascript/capitulo_6.html que te explica como separar el contenido( código HTML) de la parte interactiva (código Javascript ).
Un saludo!