Tarjeta de Crédito de Melyna Bisley Pernia Ostos#42
Tarjeta de Crédito de Melyna Bisley Pernia Ostos#42MelynaPernia wants to merge 1 commit intoLaboratoria-learning:masterfrom
Conversation
| @@ -0,0 +1,56 @@ | |||
|
|
|||
| // creando una funcion llamada isValidCard con un parámetro numberCard | |||
There was a problem hiding this comment.
Es bueno agregar comentarios, pero hay que encontrar un balance entre no tener comentarios y agregar demasiados.
Recuerda que los comentarios solo deben ser agregados cuando explican algo que no es posible entender con solo leer el código.
Por ejemplo, el comentario "creando una funcion llamada isValidCard con un parámetro numberCard" es innecesario, porque var isValidCard = function(numberCard) { me dice exactamente lo mismo que el comentario.
Revisa los comentarios que tienes en tu código y solo deja los que agregan información que no es clara al leer el código.
| // creando una funcion llamada isValidCard con un parámetro numberCard | ||
| var isValidCard = function(numberCard) { | ||
| // Declarando la variable arrNumberReverse y se asigna los valores ingresados al revés | ||
| var arrNumberReverse = ((numberCard.toString()).split('')).reverse(); |
There was a problem hiding this comment.
Solo agrega paréntesis cuando es necesario o ayuda a entender mejor el código. En este caso, ((numbrCard.toString()).spilt('')).reverse() puede ser numberCard.toString().split('').reverse();, y queda más fácil de entender.
| var sumDigit = 0; | ||
|
|
||
| // Creando un bucle For que recorre el arreglo arrNumberReverse | ||
| for (var i = 0;i < arrNumberReverse.length;i++) { |
There was a problem hiding this comment.
Agrega espacios despues de ;. En este caso: for (var i = 0; i < arrNumberReverse.length; i++) {
|
|
||
| // Creando un bucle para recorrer el nuevo arreglo llamado newArr | ||
| for (var j = 0;j < newArr.length;j++) { | ||
| // Si el residuo de la posición mas uno es cero |
There was a problem hiding this comment.
Este es un buen caso para ver cuando un comentario es necesario. El comentario actual dice en palabras exactamente lo que el codigo hace, por lo que no agrega informacion.
Pero lo que realmente estas haciendo en este checkeo es ver si el numero es par, y eso no es claro al leer el codigo.
Un comentario bueno en este caso seria // Revisa si el digito es par. De esa forma al leer el codigo queda claro la razon por la que se hace el checkeo.
| // Creando una variable sum inicializado en 0 | ||
| var sum = 0; | ||
| // Creando una variable sumDigit inicializado en 0 | ||
| var sumDigit = 0; |
There was a problem hiding this comment.
Puedes declarar esta variable donde la usas mas abajo. En general es bueno declarar las variables cerca del lugar donde son usadas.
| // Declarando la variable arrNumberReverse y se asigna los valores ingresados al revés | ||
| var arrNumberReverse = ((numberCard.toString()).split('')).reverse(); | ||
| // Creando un arreglo vacío | ||
| var newArr = []; |
There was a problem hiding this comment.
Es bueno usar nombres para variables que sean representativos de la informacion que contienen. En este caso newArr podria ser cualquier arreglo. En este caso, algo como cardNumberAsIntArr o algo similar serviria para hacer mas claro para que se usara la variable.
| // creando la variable numberCard para ingresar el número de la tarjeta de crédito | ||
| var numberCard = parseInt(prompt('Ingrese su tarjeta de credito')); | ||
| // Mientras el número de la tarjeta ingresada sea undefined seguir con el bucle | ||
| } while (numberCard === undefined); |
There was a problem hiding this comment.
Recuerda que debes validar tambien que la tarjeta tiene 16 digitos.
3 participants
No description provided.