Con la primera forma eliminé casi todo y dejé solo el body del HTML y luego definí variables, calculé promedio y verifiqué promedio para poder invocar las variables y que se muestre un alert según el caso.
Así dejé el HTML:
<body>
<script src="app.js" defer></script>
</body>
Así dejé el JS:
//variables
let nota1 = 7; let nota2 = 6; let nota3 = 3; let nota4 = 5;
//definir promedio
function calcularPromedio(nota1, nota2, nota3, nota4){
let promedio = (nota1 + nota2 + nota3 + nota4)/4
return promedio;
}
//verificar promedio
function verificarAprobacion(promedio){
return promedio >= 5 ? "Aprobado" : "Reprobado";
}
//invocar funciones y mostrar estado
let promedio = calcularPromedio(nota1, nota2, nota3, nota4);
let estado = verificarAprobacion(promedio);
alert(`"El estado es:" ${estado}`);
La segunda forma fue tomando algún elemento del HTML (yo solo dejé el h1 para que se vea texto), definir variables, calcular promedio y verificar promedio para poder invocar las variables y que se muestre un texto en el h1 pero con un template string para dejar todo en una línea y no tener que escribir un if-else entero.
Así dejé el HTML:
<body>
<div class="container">
<div class="container__contenido">
<div class="container__informaciones">
<div class="container__texto">
<h1></h1>
</div>
</div>
</div>
<script src="app.js" defer></script>
</body>
Así dejé el JS:
//variables
let nota1 = 7; let nota2 = 6; let nota3 = 3; let nota4 = 5;
//texto que se mostrará en pantalla
function textoHTML(elemento, texto){
let elementoHTML = document.querySelector(elemento);
elementoHTML.innerHTML = texto;
return
}
//definir promedio
function calcularPromedio(nota1, nota2, nota3, nota4){
let promedio = (nota1 + nota2 + nota3 + nota4)/4
return promedio;
}
//verificar promedio e invocar variables
function verificarAprobacion(promedio){
textoHTML('h1', `tu estado es ${promedio >= 5 ? "Aprobado" : "Reprobado"}`);
}
let promedio = calcularPromedio(nota1, nota2, nota3, nota4);
let estado = verificarAprobacion(promedio);