- Dice
tupe "text"
, debe decir type="text"
. También dice name "verificar"
, debe decir name="verificar"
1. Error de tipeo
Corrección:Dice tupe "text"
, debe decir type="text"
. También dice name "verificar"
, debe decir name="verificar"
Corrección:
- Cambia el
type
de los botones a button
. Ya que todo el código es por lado cliente, no necesitas que el botón haga un POST HTTP así que conviene más usar type="button"
2. Usar type="button"
CorrecciónCambia el type
de los botones a button
. Ya que todo el código es por lado cliente, no necesitas que el botón haga un POST HTTP así que conviene más usar type="button"
Correcciones:
- Los
onclick
de verficar
y retirar
deben ir el los botones y no en las cajas de texto
3. Ubicación de los onclick
handlers
CorreccionesLos onclick
de verficar
y retirar
deben ir el los botones y no en las cajas de texto
Correcciones:
- No existe
document.getElementByName
. El método correcto es getElementsByName
con Elements en plural. Debido a que devuelve un array con varios resultados debes escoger el primer elemento que es el que tiene el input
que estás buscando
4. Método inexistente
Correcciones:No existe document.getElementByName
. El método correcto es getElementsByName
con Elements en plural. Debido a que devuelve un array con varios resultados debes escoger el primer elemento que es el que tiene el input
que estás buscando
Correcciones:
- Los inputs
verificar
y retirar
se llaman igual que las funciones. Tienes que ponerles nombres diferentes ya están haciendo conflicto a la hora de ejecutar el onclick()
5. Colisión de nombres
Los inputs verificar
y retirar
se llaman igual que las funciones. Tienes que ponerles nombres diferentes ya están haciendo conflicto a la hora de ejecutar el onclick()
Por ejemplo yo les he cambiado el nombre a los inputs en su declaración y en el código javascript.
Correcciones:
6. Tag malformado
El tag </br>
está malformado. Lo correcto es <br />
El código completo te lo muestro a continuación:
var saldo = 0,
deposito, retiro;
function depositar() {
deposito = parseFloat(document.getElementsByName("deposito")[0].value);
saldo = saldo + deposito;
}
function retirar() {
retiro = parseFloat(document.getElementsByName("retiro")[0].value);
if (retiro > saldo) {
alert("Su fondo disposible no es suficiente");
} else {
saldo = saldo - retiro;
}
}
function verificar() {
document.getElementsByName("verificacion")[0].value = saldo;
}
<form action="" method="post">
<input type="text" name="deposito" placeholder="Ingrese un deposito" />
<input type="button" value="Depositar" onclick="depositar()">
<<br /br>>
<<br /br>>
su saldo actual es<es<br /br>>
<input type="text" name="verificacion">
<input type="button" value="verificar" onclick="verificar()">
<<br /br>>
<<br /br>>
<input type="text" name="retiro" >
<input type="button" value="retirar" onclick="retirar()">
</form>