Refatorar função eguamat.media()

[lucaspompeun]

egua/src/lib/eguamat.js

Lines 343 to 351 in 5671a03

	module.exports.media = function(a) {
	if (isNaN(num) || num === null)
	throw new RuntimeError(
	this.token,
	"Você deve prover valores para media(a)."
	);
	return smtr(a)/a.length;
	};

[barbozafernando]

@lucaspompeun, essa média seria o cálculo de média normal mesmo, né? Tipo, se eu passo 5 valores, somo todos e divido por 5. Seria isso?

[lucaspompeun]

@barbozafernando positivo, é isso mesmo. Nesses caso o argumento precisa ser uma lista.

[barbozafernando]

Pompeu, pensei em duas formas de implementar essa função.

• A primeira, é o cara chamando media() e passando quantos parâmetros quiser pra fazer a média.
  Seria algo como:

var mat = importar('eguamat');
escreva(mat.media(2, 5, 10, 20));
// 9.25 

• A segunda, seria como vc falou. O cara já passar uma lista pra fazer o cálculo.

Daria pra implementar os dois modos. O que acha, @lucaspompeun ?

[lucaspompeun]

Acho perfeito fazer os dois.

[barbozafernando]

Mas pensando aqui, Pompeu. Cara, mas será que se fizer dos dois jeitos, não vai dar na mesma? Vou te mostrar o porquê.

module.exports.media = function () {
  // Pega cada item que veio por parâmetro e coloca em uma posição diferente em um array.
  const args = [...arguments];

  // Soma todos os itens.
  const valoresSomados = args.reduce((acumulador, itemAtual) => acumulador + itemAtual, 0);

  // Faz o cáculo da média em si e retorna.
  return (valoresSomados / args.length);
};

Então desse modo, na implementação já colocaria tudo em um array, então não faria muito sentido permitir o usuário mandar um array de valores ou apenas os valores separados por vírgula, porque no final das contas, vai dar no mesmo.
Pensando assim, o método então receberia apenas os valores que o usuário gostaria de cácular a média e obviamente, todos tem de ser do tipo número.

O que acham, @lucaspompeun e @leonelsanchesdasilva ?

[lucaspompeun]

Mas pensando aqui, Pompeu. Cara, mas será que se fizer dos dois jeitos, não vai dar na mesma? Vou te mostrar o porquê.

module.exports.media = function () {
  // Pega cada item que veio por parâmetro e coloca em uma posição diferente em um array.
  const args = [...arguments];

  // Soma todos os itens.
  const valoresSomados = args.reduce((acumulador, itemAtual) => acumulador + itemAtual, 0);

  // Faz o cáculo da média em si e retorna.
  return (valoresSomados / args.length);
};

Então desse modo, na implementação já colocaria tudo em um array, então não faria muito sentido permitir o usuário mandar um array de valores ou apenas os valores separados por vírgula, porque no final das contas, vai dar no mesmo.
Pensando assim, o método então receberia apenas os valores que o usuário gostaria de cácular a média e obviamente, todos tem de ser do tipo número.

O que acham, @lucaspompeun e @leonelsanchesdasilva ?

Eu particularmente gosto que tenha como entrada um vetor pois ajuda o usuário a entender uma das aplicações de vetores. Esse é o motivo pelo qual gosto de ter o vetor como entrada. O que vocês acham?

[leonelsanchesdasilva]

Mas pensando aqui, Pompeu. Cara, mas será que se fizer dos dois jeitos, não vai dar na mesma? Vou te mostrar o porquê.

module.exports.media = function () {
  // Pega cada item que veio por parâmetro e coloca em uma posição diferente em um array.
  const args = [...arguments];

  // Soma todos os itens.
  const valoresSomados = args.reduce((acumulador, itemAtual) => acumulador + itemAtual, 0);

  // Faz o cáculo da média em si e retorna.
  return (valoresSomados / args.length);
};

Então desse modo, na implementação já colocaria tudo em um array, então não faria muito sentido permitir o usuário mandar um array de valores ou apenas os valores separados por vírgula, porque no final das contas, vai dar no mesmo.
Pensando assim, o método então receberia apenas os valores que o usuário gostaria de cácular a média e obviamente, todos tem de ser do tipo número.

O que acham, @lucaspompeun e @leonelsanchesdasilva ?

A implementação considerando número variável de argumentos é possível, mas vai aumentar o tratamento dos argumentos.

Nada me impediria, por exemplo, de passar uma série indefinida de arrays numéricos, o que já faria o método falhar na hora de fazer a redução. Um .flat() de todos os arrays antes teria que ser feito.

Eu faria media() apenas aceitando um vetor único de números. Se houver necessidade de implementar uma versão mais robusta, isso pode ser feito depois.

[barbozafernando]

Mas pensando aqui, Pompeu. Cara, mas será que se fizer dos dois jeitos, não vai dar na mesma? Vou te mostrar o porquê.

module.exports.media = function () {
  // Pega cada item que veio por parâmetro e coloca em uma posição diferente em um array.
  const args = [...arguments];

  // Soma todos os itens.
  const valoresSomados = args.reduce((acumulador, itemAtual) => acumulador + itemAtual, 0);

  // Faz o cáculo da média em si e retorna.
  return (valoresSomados / args.length);
};

Então desse modo, na implementação já colocaria tudo em um array, então não faria muito sentido permitir o usuário mandar um array de valores ou apenas os valores separados por vírgula, porque no final das contas, vai dar no mesmo.
Pensando assim, o método então receberia apenas os valores que o usuário gostaria de cácular a média e obviamente, todos tem de ser do tipo número.
O que acham, @lucaspompeun e @leonelsanchesdasilva ?

A implementação considerando número variável de argumentos é possível, mas vai aumentar o tratamento dos argumentos.

Nada me impediria, por exemplo, de passar uma série indefinida de arrays numéricos, o que já faria o método falhar na hora de fazer a redução. Um .flat() de todos os arrays antes teria que ser feito.

Eu faria media() apenas aceitando um vetor único de números. Se houver necessidade de implementar uma versão mais robusta, isso pode ser feito depois.

Eu estava fazendo exatamente isso. Estava tentando implementar desse jeito aí, e cara, terei que fazer vários tratamentos nos argumentos. No momento, talvez não seja necessário mesmo. Concordo em aceitar apenas o array. O que acha, Pompeu?

[lucaspompeun]

Podemos ficar com o argumento único de array.