Bueno, vamos a ir viendo qué errores o mejoras hay en tu código...
- No utilizar variables globales. Las variables globales, en contraposición con las variables locales, son las que se declaran fuera de cualquier función. Esto está desaconsejado. En todo caso se pueden utilizar para declarar constantes.
- En la línea 11 llamas a la función bucle() pasando <a> (con valor 0) como argumento. Pasas un parámetro para ignorar su valor lo cual no debe hacerse y segundo que ese bucle no se va a ejecutar porque <estu> en ese punto vale 0 y <a> vale 1.
- Después todo funciona normal hasta la línea 15. Aquí se llama de nuevo a bucle() y dentro de bucle() se llama a PromEdadIng() que tiene el error. Has definido que esa función devuelve un valor entero (int) pero en el caso de que se cumpla el if() no se devuelve nada.
Si no se crea un bloque con llaves después de una estructura de control se entiende que únicamente la siguiente instrucción pertenece al bloque (no importa la indentación en C/C++).
// Esto:
if(condicion) cout << "Hola" << endl; cout << "Adios" << endl;
// es equivalente a esto:
if(condicion) {
cout << "Hola" << endl;
}
cout << "Adios" << endl;
En tu caso el <return edad> pertenece al bloque del else por lo que si se cumple la condición del if() no hay ningún return. Además la condición del if() va a ser siempre cierta. En C/C++ el operador '=' es el operador de asignación (asigna un valor a una variable) y si asignas a una variable cualquier valor distinto de 0, el resultado es true. El operador de comparación de igualdad es '=='.
Encontrado el error, a mejorar el código...
Primero, como ya he dicho, hay que evitar el uso de variables globales. Vamos a utilizar una constante para ver mejor su uso adecuado.
#include <iostream>
using namespace std;
const int MAX_ESTUDIANTES = 10; // Como maximo el programa admitira 10 estudiantes
int main() {
int estudiantes; // variable local para el numero de estudiantes a introducir
// Ahora pedimos el numero de estudiantes y debera estar entre 1 y el maximo que hayamos definido antes
// Para ello se utiliza un filtro con un bucle do-while() que pedira un valor mientras el que se haya introducido no sea valido
do {
cout << "Introduce el numero de estudiantes [1 - " << MAX_ESTUDIANTES << "]: ";
cin >> estudiantes;
} while(estudiantes < 1 || estudiantes > MAX_ESTUDIANTES);
}
Ahora toca ver el resto de la funcionalidad. Primero voy a hacerlo todo en el main() sin utilizar más funciones y después utilizando funciones para ver el proceso. Según veo en el código, el propósito es mostrar la edad promedio de los estudiantes de ingeniería.
#include <iostream>
using namespace std;
const int MAX_ESTUDIANTES = 10; // Como maximo el programa admitira 10 estudiantes
int main() {
int estudiantes; // variable local para el numero de estudiantes a introducir
// Ahora pedimos el numero de estudiantes y debera estar entre 1 y el maximo que hayamos definido antes
// Para ello se utiliza un filtro con un bucle do-while() que pedira un valor mientras el que se haya introducido no sea valido
do {
cout << "Introduce el numero de estudiantes [1 - " << MAX_ESTUDIANTES << "]: ";
cin >> estudiantes;
} while(estudiantes < 1 || estudiantes > MAX_ESTUDIANTES);
int estudios;
int edad, edadTotalIngenieria = 0; // Importante inicializar a 0 el acumulador
for(int i = 0; i < estudiantes; ++i) { // Es mejor contar desde 0 para cuando avances mas en el lenguaje
cout << "Introduce la edad del estudiante " << (i+1) << ": ";
cin >> edad; // Guardamos la edad introducida en <edad>
cout << "Introduce los estudios del estudiante " << (i+1) << "(1 - Ingenieria | 2 - Otro): ";
cin >> estudios;
if(estudios == 1) edadTotalIngenieria += edad; // Si es estudiante de ingenieria, sumamos su edad a la del resto de ingenieria
}
// Hay que convertir al menos una variable en double para que el resultado sea double y no int
double edadPromedioIngenieria = (double)edadTotalIngenieria / estudiantes;
cout << "La edad promedio de los estudiantes de ingenieria es: " << edadPromedioIngenieria << endl;
return 0; // En la funcion main() no es necesario. En el resto de funciones que devuelven algo, el return es obligatorio
}
Ahora para crear una función hay que buscar un fragmento de código que forme como un bloque y que pueda ser utilizado varias veces preferiblemente. En este caso el mejor ejemplo que veo es el principio del programa. Hay un fragmento de código para pedir un valor entre dos límites y repetirse hasta que se introduce un valor válido.
Es una buena práctica que las funciones tengan verbos en su nombre para indicar lo que hacen. Por ejemplo se puede llamar pedirEstudiantesEnRango()
int pedirEstudiantesEnRango() {
int estudiantes;
do {
cout << "Introduce el numero de estudiantes [1 - " << MAX_ESTUDIANTES << "]: ";
cin >> estudiantes;
} while(estudiantes < 1 || estudiantes > MAX_ESTUDIANTES);
return estudiantes; // Importante devolver el valor
}
int main() {
int estudiantes = pedirEstudiantesEnRango();
// el resto del programa
}
En la función anterior podemos usar MAX_ESTUDIANTES sin pasarlo como argumento porque es una constante global. Sin embargo, ¿qué pasaría si en algún momento del programa queremos pedir un número de estudiantes en otro rango? Habría que cambiar todo el programa. Entonces podemos agregar parámetros para indicar el mínimo y el máximo.
int pedirEstudiantesEnRango(int minimo, int maximo) {
int estudiantes;
do {
cout << "Introduce el numero de estudiantes [" << minimo " - " << maximo << "]: ";
cin >> estudiantes;
} while(estudiantes < minimo || estudiantes > maximo);
return estudiantes; // Importante devolver el valor
}
int main() {
int estudiantes = pedirEstudiantesEnRango(1, MAX_ESTUDIANTES);
// el resto del programa
}
Como se ve en el código anterior, si se utilizan parámetros es para pasar valores necesarios de una función a otra.
A partir de todo esto ya te toca seguir a ti y siempre puedes preguntar de nuevo si tienes algún problema o alguna duda.
Suerte.
PD: Las dos siguientes líneas son iguales con la diferencia de que la primera se entiende mejor que la segunda...
Además, si cuentas el bucle for() desde 0, no hace falta ni restar 1.
int retorno = f() - 1;
int retorno = (1 - f()) * -1;