Utiliza std::count_if
:
using std::literals::string_literals;
auto mujeres = std::count_if(std::begin(employee), std::end(employee),
[](const person &p) {
return p.sex == "femenino"s;
});
Cuentas las mujeres, los hombres resultarán de restar la cantidad de mujeres a max
, esto te permite evitar contar por separado cada sexo, evitando la necesidad de un if
, que afectaría muy negativamente al rendimiento.
La función std::count_if
está disponible al incluir la cabecera <algorithm>
, te contará todos los elementos entre el iterador de inicio (primer parámetro) y el iterador de final (segundo parámetro) que cumplan con una condición (el tercer parámetro).
Las funciones std::begin
y std::end
están disponibles en la cabecera <iterator>
y son las que calculan el inicio y final de tu formación employee
.
La instrucción using std::literals::string_literals;
es la que te permite usar los literales definidos por el usuario de cadenas de caracteres, por eso en la lambda pasada a std::count_if
podemos escribir
return p.sex == "femenino"s;
en lugar de
return p.sex == std::string{"femenino"};
Puedes no usar los iteradores y aprovechar que employee
es una formación y aplicar aritmética de punteros:
auto mujeres = std::count_if(employee, employee + n,
[](const person &p) {
return p.sex == "femenino"s;
});
Así también esquivas el posible problema de contar mal cuando n
y max
no contienen el mismo valor.
###Otras cosas a tener en cuenta.
En
Otras cosas a tener en cuenta.
En C++ struct
no forma parte del tipo, así que no necesitas anteponerlo para declarar estructuras.
En C++ las funciones que no reciben parámetros basta con escribirlas con un paréntesis vacío.
La función main
debe devolver un valor.
No se qué tipo tiene max
, pero si no es un tipo conocido en tiempo de compilación, tendrás problemas de compatibilidad con tu código, lee este hilo para saber más al respecto.
Las variables de los for
se pueden declarar en los propios bucles, no es necesario hacerlo fuera.