Creo que lo que tu interpretas como que se cierra es que en realidad no estás viendo ningún resultado del código que has escrito, esto es principalmente porque no has invocado a la función totalSalario()
, por lo tanto, su código nunca se ejecuta (y no produce ninguna salida).
Dicho esto, veo varios problemas:
- la función accede a variables globales, (externas a ella). Mientras esto es permitido, no siempre es una buena práctica. Sobre todo en tiempos modernos dónde hay promesas y los valores pueden cambiar mientras se está ejecutando la función. Además, como verás más adelante, también puede provocar confusión por el traslape de nombres de variables. En lo personal, prefiero generalmente pasar los valores como parámetros de la función, de esta manera queda claro con qué valores está operando (los que tenían las variables al momento de la llamada).
- Si pretendes calcular el salario, como más o menos indica el nombre de la función, entonces esta debiera retornar un valor y no lo hace.
- Quizás creas que has dejado el valor en la variable global
salario
, pero, de la forma que está escrita la función, lo que haces es declarar nuevas variables salario
, cuyo ámbito es solamente dentro del then
(la primera) y dentro del else
la segunda. El valor que asignas a esas variables no se verá reflejado en la variable global salario
, se pierde al salir del then
o del else
.
- La estructura del
if
deja por fuera el caso de que las horas trabajadas sea exactamente igual al máximo, no hace falta el segundo if
a menos que hubiese 3 formas distintas de cálculo.
Corrigiendo esos detalles, con esto verás el resultado del código (clic en el botón "Ejecutar" para verlo aquí mismo en el navegador):
var nombre = "Juan";
var apellido = "Lopez";
var valorHora = 3785;
var valorHoraExtra = 4731;
var salario = 0;
var maximohoras = 8;
var horasTrabajadas = prompt("ingres cantidad de horas: ");
salario = calcularSalario(nombre, apellido, valorHora, valorHoraExtra, maximohoras, horasTrabajadas);
console.log("fin.");
function calcularSalario(nombre, apellido, valorHora, valorHoraExtra, maximohoras, horasTrabajadas) {
var result = 0;
if (horasTrabajadas > maximohoras) {
result = valorHora * horasTrabajadas + valorHoraExtra;
} else {
result = valorHora * horasTrabajadas;
}
console.log(`El salario del usuario ${nombre} ${apellido} es: ${result}`);
return result;
}
El salario del usuario ${nombre} ${apellido} es: ${salario}
)