Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tarea 1 de la clase 6 #38

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

Tarea 1 de la clase 6 #38

wants to merge 19 commits into from

Conversation

Marcos8i22o
Copy link

A revisar

Marcos8i22o and others added 19 commits February 1, 2020 19:37
Estoy subiendo el ejercicio 1 de la Clase 1 a Github
se sube el ejercicio para corregir
Ejercicio terminado. Falta corregir si está bien.
Se sube el ejercicio 1 incompleto para revisar por qué no funciona la función "resetearResultados" que tendría que restablecer los valores de los inputs a su estado inicial.
A revisar la función resetearResultados cuyo fin es restablecer los inputs a su estado original.
* Se agregó el índice i en la función resetearResultados en el método .remove.
* Se colocó la función resetearResultados dentro de la función resetear habiendola probado antes al hacer click en el botón Siguiente sin el resultado esperado.
Se corrigieron las funciones mostrarMayor, mostrarMenor y mostrarPromedio para que al clickear sobre el botón "Limpiar", borre los valores ingresados  antes
Creados los botones para:

-Agregar y quitar miembros
-Calcular
-Limpiar formulario
Se agregaron funciones para calcular:

* mayor y menor salario anual,
* salario anual y mensual promedio

Se cambió el elemento $nuevoMiembro por el elemento $contenedorMiembros.

Se agregaron las funciones para mostrar los salarios.
- Se optimizó el código para que muestre el botón Calcular sólo después de haber ingresado un miembro.

- Se muestran los resultados del análisis solamente con una sola función pasandole los parámetros que sean necesarios según el caso.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant