Hola, bienvenido a los foros...
En principio no veo en esos campos los id's... si usas getelementbyId, ese id debe existir (no debes confundirte con el name)...
Luego también tienes otras cosas para mejorar, como usar el evento submit con botones de ese tipo... pero de eso ya habrá ocasiones de hablar...
Saludos
