Refatorar verificação de parâmetros e remover string do retorno

eguatech / egua

Linguagem de programação em português, simples e moderna

https://egua.dev/idegua/

MIT License

822 stars 39 forks source link

Refatorar verificação de parâmetros e remover string do retorno #85

Closed lucaspompeun closed 3 years ago

lucaspompeun commented 3 years ago

https://github.com/eguatech/egua/blob/5671a030a7fd3fcef57dbde0846cf5613b25f566/src/lib/eguamat.js#L62-L71

leonelsanchesdasilva commented 3 years ago

@lucaspompeun O que eu entendi dessa função é o seguinte:

A função recebe dois números, a e b,
a é a distância entre um ponto da abscissa e outro;
b é o valor central no eixo x;
O resultado será a geração de 7 pontos no eixo x.

Perguntas:

Qual a utilidade dessa função? Em que tipo de exercício é usada?
Por que apenas a é verificada?
Por que o número de pontos gerados não é variável?

Tendo essas respostas, eu posso refatorar essa função também (escolhendo um nome melhor, aliás).

lucaspompeun commented 3 years ago

@leonelsanchesdasilva essa função foi incluída por outro desenvolvedor e ele acabou por sair do desenvolvimento da linguagem antes mesmo de documentá-la. Você acha que deveríamos removê-la ou tentaremos encontrar de fato uma aplicação à mesma?

leonelsanchesdasilva commented 3 years ago

@lucaspompeun Eu faria um trabalho melhor de parâmetros, renomearia para gerarPontosAbscissa e faria algo mais dinâmico, com mais ou menos de 7 pontos, mantendo o padrão de 7 pontos.

lucaspompeun commented 3 years ago

@leonelsanchesdasilva vou deixar essa decisão com você haha Você decide se manteremos ou removeremos.