Você já criou as funções para filtrar homens e mulheres, outra para mapear cada funcionário com seu respectivo salário e outra para calcular a soma dos elementos de um array. Só faltou combiná-las na ordem certa:
const funcionarios = [
{ nome: "Joany", salario: 4500, sexo: "F"},
{ nome: 'Junior', salario: 4500, sexo: 'M'},
{ nome: 'Julio', salario: 3500, sexo: 'M'},
{ nome: 'Grazy', salario: 3000, sexo: 'F'},
{ nome: 'Rafael', salario: 6000, sexo: 'M'},
{ nome: 'Artur', salario: 7800, sexo: 'M'},
{ nome: 'Ana', salario: 3300, sexo: 'F'},
{ nome: 'Bianca', salario: 12500, sexo: 'F'},
{ nome: 'José', salario: 8400, sexo: 'M'},
{ nome: 'Maria', salario: 5500, sexo: 'F'}
];
const apenasMulher = a => a.sexo === 'F';
const apenasHomem = a => a.sexo === 'M';
// mudei o nome para "obtemSalario" pois "somaSalario" não é bem o que ela faz
const obtemSalario = sal => sal.salario;
const totalSalario = (sal, salAtual) => {
return sal + salAtual;
};
let totalMulheres = funcionarios.filter(apenasMulher).map(obtemSalario).reduce(totalSalario);
let totalHomens = funcionarios.filter(apenasHomem).map(obtemSalario).reduce(totalSalario);
console.log(totalMulheres);
console.log(totalHomens);
Ou seja, primeiro você filtra pelo critério desejado (filter
para ter só homens, ou só mulheres), depois mapeia-os para os seus salários (map
), e só depois você soma tudo (reduce
).
No seu código você fez funcionarios.map(somaSalario)
, ou seja, pegou os salários de todos os funcionários, e em seguida somou tudo (faltou o filter
para filtrar somente as mulheres). Na verdade você chamou o filter
depois, mas aí já não adianta porque o total já havia sido calculado.
Repare também que mudei o nome da função somaSalario
para obtemSalario
, pois ela não estava somando nada, só estava servindo para mapear cada funcionário para seu respectivo salário. Pode parecer um detalhe besta, mas dar nomes melhores ajuda na hora de programar.
Também coloquei ponto-e-vírgula no final das linhas. Pode parecer "frescura", e sei que o JavaScript "aceita" o código sem ponto e vírgula e "funciona", mas isso evita algumas situações bizarras que podem ocorrer se você não usá-los, como essa e essa (veja mais sobre isso aqui).
Alternativas
Só pra constar que há outras alternativas, algumas até eliminando a necessidade do filter
e/ou do map
:
function somar(dados, campo, sexo) {
// usando filter, map e reduce
//return dados.filter(e => e.sexo === sexo).map(e => e[campo]).reduce((a, b) => a + b);
// ou sem o map
//return dados.filter(e => e.sexo === sexo).reduce((a, b) => a + b[campo], 0);
// ou só com reduce
return dados.reduce((a, b) => a + (b.sexo === sexo ? b[campo] : 0), 0);
}
console.log(somar(funcionarios, 'salario', 'F')); // totalMulheres
console.log(somar(funcionarios, 'salario', 'M')); // totalHomens
Ou ainda, retornando um objeto com ambos os totais:
let { totalHomens, totalMulheres } = funcionarios.reduce(function (a, b) {
if (b.sexo === 'M') {
a.totalHomens += b.salario;
} else if (b.sexo === 'F') {
a.totalMulheres += b.salario;
}
return a;
}, { totalHomens: 0, totalMulheres: 0 });
console.log(totalMulheres);
console.log(totalHomens);
Esse último em particular eu já considero um exagero (e um certo abuso de reduce
), mas fica como curiosidade...
Sem filter
/map
/reduce
Mas será que precisava de tudo isso? Cada chamada de filter
e map
percorre os elementos do array e retorna outro array, e reduce
faz várias chamadas do callback totalSalario
para fazer uma simples soma. Ou seja, não só você está percorrendo várias vezes os mesmos elementos, como ainda está criando várias estruturas intermediárias à toa.
Eu acho muito mais simples fazer apenas um for
:
const funcionarios = [
{ nome: "Joany", salario: 4500, sexo: "F"},
{ nome: 'Junior', salario: 4500, sexo: 'M'},
{ nome: 'Julio', salario: 3500, sexo: 'M'},
{ nome: 'Grazy', salario: 3000, sexo: 'F'},
{ nome: 'Rafael', salario: 6000, sexo: 'M'},
{ nome: 'Artur', salario: 7800, sexo: 'M'},
{ nome: 'Ana', salario: 3300, sexo: 'F'},
{ nome: 'Bianca', salario: 12500, sexo: 'F'},
{ nome: 'José', salario: 8400, sexo: 'M'},
{ nome: 'Maria', salario: 5500, sexo: 'F'}
];
let totalHomens = 0;
let totalMulheres = 0;
for (const func of funcionarios) {
switch(func.sexo) {
case 'M':
totalHomens += func.salario;
break;
case 'F':
totalMulheres += func.salario;
break;
}
}
console.log(totalMulheres);
console.log(totalHomens);