Skip to main content
se añadieron 274 caracteres en el cuerpo
Origen Enlace
Carlos Muñoz
  • 13.5k
  • 2
  • 47
  • 67

En el código hay los sgtessiguientes errores:

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"type="button"

En el código hay los sgtes errores:

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"

En el código hay los siguientes errores:

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"

se añadieron 274 caracteres en el cuerpo
Origen Enlace
Carlos Muñoz
  • 13.5k
  • 2
  • 47
  • 67

6. Validación de los input

El usuario podría ingresar valores que no son números en los inputs por lo que conviene validarlos con la función isNaN()

Por ejemplo si el valor ingresado no es un número se muestra una alert con el mensaje de error.

function depositar() {
    deposito = parseFloat(document.getElementsByName("deposito")[0].value);

    if (isNaN(deposito)) {
        alert("El valor ingresado no es número válido");
        return;
    }

    saldo = saldo + deposito;
}

function retirar() {
    retiro = parseFloat(document.getElementsByName("retiro")[0].value);

    if (isNaN(retiro)) {
        alert("El valor ingresado no es número válido");
        return;
    }

    if (retiro > saldo) {
        alert("Su fondo disposible no es suficiente");
        return;
    }

    saldo = saldo - retiro;    
}

7. Tag malformado

8. Errores de consistencia

El texto de los botones es inconsistente. "Depositar" comienza con mayúsculas pero "verficar" y "retirar" están en minúsculas

Correcciones:

<input type="button" value="Verificar" onclick="verificar()">

...

<input type="button" value="Retirar" onclick="retirar()">

El código completo te lo muestro a continuación:

    var saldo = 0,
    deposito, retiro;

    function depositar() {
        deposito = parseFloat(document.getElementsByName("deposito")[0].value); 

        if (isNaN(deposito)) {
            alert("El valor ingresado no es número válido");
            return;
        }

        saldo = saldo + deposito;
    }

    function retirar() {
        retiro = parseFloat(document.getElementsByName("retiro")[0].value);

        if (isNaN(retiro)) {
            alert("El valor ingresado no es número válido");
            return;
        }

        if (retiro > saldo) {
            alert("Su fondo disposible no es suficiente");
    } else {      return;
        }

        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 />
    su saldo actual es<br />
    <input type="text" name="verificacion">
    <input type="button" value="verificar"value="Verificar" onclick="verificar()">
    <br />
    <br />
    <input type="text" name="retiro" >
    <input type="button" value="retirar"value="Retirar" onclick="retirar()">
</form>

6. Tag malformado

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 />
    su saldo actual es<br />
    <input type="text" name="verificacion">
    <input type="button" value="verificar" onclick="verificar()">
    <br />
    <br />
    <input type="text" name="retiro" >
    <input type="button" value="retirar" onclick="retirar()">
</form>

6. Validación de los input

El usuario podría ingresar valores que no son números en los inputs por lo que conviene validarlos con la función isNaN()

Por ejemplo si el valor ingresado no es un número se muestra una alert con el mensaje de error.

function depositar() {
    deposito = parseFloat(document.getElementsByName("deposito")[0].value);

    if (isNaN(deposito)) {
        alert("El valor ingresado no es número válido");
        return;
    }

    saldo = saldo + deposito;
}

function retirar() {
    retiro = parseFloat(document.getElementsByName("retiro")[0].value);

    if (isNaN(retiro)) {
        alert("El valor ingresado no es número válido");
        return;
    }

    if (retiro > saldo) {
        alert("Su fondo disposible no es suficiente");
        return;
    }

    saldo = saldo - retiro;    
}

7. Tag malformado

8. Errores de consistencia

El texto de los botones es inconsistente. "Depositar" comienza con mayúsculas pero "verficar" y "retirar" están en minúsculas

Correcciones:

<input type="button" value="Verificar" onclick="verificar()">

...

<input type="button" value="Retirar" onclick="retirar()">

El código completo te lo muestro a continuación:

    var saldo = 0,
    deposito, retiro;

    function depositar() {
        deposito = parseFloat(document.getElementsByName("deposito")[0].value); 

        if (isNaN(deposito)) {
            alert("El valor ingresado no es número válido");
            return;
        }

        saldo = saldo + deposito;
    }

    function retirar() {
        retiro = parseFloat(document.getElementsByName("retiro")[0].value);

        if (isNaN(retiro)) {
            alert("El valor ingresado no es número válido");
            return;
        }

        if (retiro > saldo) {
            alert("Su fondo disposible no es suficiente");
            return;
        }

        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 />
    su saldo actual es<br />
    <input type="text" name="verificacion">
    <input type="button" value="Verificar" onclick="verificar()">
    <br />
    <br />
    <input type="text" name="retiro" >
    <input type="button" value="Retirar" onclick="retirar()">
</form>
se añadieron 274 caracteres en el cuerpo
Origen Enlace
Carlos Muñoz
  • 13.5k
  • 2
  • 47
  • 67
  1. 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:

  1. 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:

  1. 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:

  1. 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:

  1. 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>
  1. Dice tupe "text", debe decir type="text". También dice name "verificar", debe decir name="verificar"

Corrección:

  1. 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"

Corrección

  1. Los onclick de verficar y retirar deben ir el los botones y no en las cajas de texto

Correcciones

  1. 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:

  1. 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.

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>
    su saldo actual es</br>
    <input type="text" name="verificacion">
    <input type="button" value="verificar" onclick="verificar()">
    </br>
    </br>
    <input type="text" name="retiro" >
    <input type="button" value="retirar" onclick="retirar()">
</form>

1. Error de tipeo

Dice tupe "text", debe decir type="text". También dice name "verificar", debe decir name="verificar"

Corrección:

2. Usar type="button"

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"

Correcciones:

3. Ubicación de los onclick handlers

Los onclick de verficar y retirar deben ir el los botones y no en las cajas de texto

Correcciones:

4. Método inexistente

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:

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 />
    su saldo actual es<br />
    <input type="text" name="verificacion">
    <input type="button" value="verificar" onclick="verificar()">
    <br />
    <br />
    <input type="text" name="retiro" >
    <input type="button" value="retirar" onclick="retirar()">
</form>
se añadieron 589 caracteres en el cuerpo
Origen Enlace
Carlos Muñoz
  • 13.5k
  • 2
  • 47
  • 67
Loading
Origen Enlace
Carlos Muñoz
  • 13.5k
  • 2
  • 47
  • 67
Loading