Avatar
Adur S

@adurs

Buenas, esta es la primera vez que intento aprender a programar y me gustaria si alguien tiene tiempo me diera su opinion respecto al codigo que he creado con la idea del cajero automatico, le he añadido alguna cosilla, pero no se si existen mejores formas de implementar esos pequeños añadidos. Alguna idea para mejorar el codigo o simplificarlo(limpiarlo, reducirlo) un poco? Algun error que pueda ser mejorado? En VSC la linea 12 me da error(u) la 13 seguramente tambien y no me queda muy claro el porqué, ya que en rept.it el codigo funciona perfectamente, alguna idea? Me gustaria seguir aprendiendo y me interesan opiniones Muchisimas gracias

https://repl.it/@AdurS/SuperMiniCertifications#index.js

JW
Jesús W Sanchez

@jesuswsanchez

Bueno podrías iniciar por validar el dinero que se deposita así mismo el que se retira por ejemplo haciendo una prueba super rápida se puede dar cuenta que al ingresar un monto negativo lo deja ingresar y esto debería ser un error. Recuerda primero resuelves el problema, luego te centras en mejorar la lógica del programa y depuras.

RP
Ronny Pumaricra Meneses

@ronnypumaricrameneses

Vi tu código y algo que creo se puede mejorar es declarar variables locales con let en las funciones, en lugar de modificar los parámetros.

1function login () {
2 let u = prompt("Username???")
3 let p = prompt("Password???")
4 // Más código
5}
RP
Ronny Pumaricra Meneses

@ronnypumaricrameneses

Un error en el programa es que al ingresar un dato no numérico lo acepta y el monto guardado se convierte en NaN.

Solución

Valida el dato de entrada con isNaN()

1if (!isNaN(value) && value >= 0) {
2 // Más código
3}
Avatar

Escribe una respuesta