Solução do desafio 11. #1008
Solução do desafio 11. #1008
Conversation
| }; | ||
|
|
||
| isPrimeNumber(minPrime); | ||
| for (var i = minPrime - 1; i <= maxPrime; i += 2) |
There was a problem hiding this comment.
O uso de var é menos recomendado em TypeScript/JavaScript moderno. Use let ou const.
There was a problem hiding this comment.
Aqui é por conta do uso recorrente de C#
| primeList.add(number); | ||
| }; | ||
|
|
||
| isPrimeNumber(minPrime); |
There was a problem hiding this comment.
Aqui não sei se era algum teste que ficou pra traz. Mas vc roda só para o minPrime a funçao.
isPrimeNumber(minPrime);
There was a problem hiding this comment.
Sim, porque como no for eu percorro apenas os ímpares maiores que 3, eu tenho que adicionar na lista o 2 primeiro.
There was a problem hiding this comment.
A abordagem atual pode ser aprimorada. Ao criar uma função, estamos encapsulando um trecho de código que pode ser reutilizado por você e por outras pessoas em diferentes contextos.
É importante que uma função faça apenas uma coisa. No caso atual, a função retorna falso quando o número não é primo e adiciona o número a um conjunto quando é primo.
Seria mais apropriado que a função se concentrasse apenas na verificação caso primo e retornasse true ou false com base nessa verificação.
Minha sugestão é que você modifique a função tratando o caso do número 2 internamente e realize nela apenas a verificação. Para qualquer primo true, não primo false.
Movendo a lógica de adição para fora da função. Por exemplo:
if (isPrimeNumber(i)) {
primeList.add(i);
}
|
Por conta que não foi especificado o tipo de importação e está como
commonjs invés de module
…On Thu, Sep 28, 2023, 09:53 Anthony Louis ***@***.***> wrote:
Fui testar tua solução e está aparecendo este erro
[image: image]
<https://user-images.githubusercontent.com/36685996/271289003-c7160e6c-a293-4167-86f3-e1512244aa79.png>
—
Reply to this email directly, view it on GitHub
<#1008 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIZPUA7ANMBV25NTCLIR4I3X4VXL3ANCNFSM6AAAAAA5FGQONE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
Tente executar pelo replit
https://replit.com/@WhoisBsa/PrimeInPi#index.ts
…On Thu, Sep 28, 2023, 10:02 Matheus B. Souza ***@***.***> wrote:
Poe conta que não foi especificado o tipo de importação e está como
commonjs invés de module
On Thu, Sep 28, 2023, 09:53 Anthony Louis ***@***.***>
wrote:
> Fui testar tua solução e está aparecendo este erro
> [image: image]
> <https://user-images.githubusercontent.com/36685996/271289003-c7160e6c-a293-4167-86f3-e1512244aa79.png>
>
> —
> Reply to this email directly, view it on GitHub
> <#1008 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AIZPUA7ANMBV25NTCLIR4I3X4VXL3ANCNFSM6AAAAAA5FGQONE>
> .
> You are receiving this because you authored the thread.Message ID:
> ***@***.***>
>
|
|
Eu rodei aqui usando o ts-node e funcionou |
|
Boa. É pq eu fiz em ts mesmo.
…On Thu, Sep 28, 2023, 10:03 Anthony Louis ***@***.***> wrote:
Eu rodei aqui usando o ts-node e funcionou
—
Reply to this email directly, view it on GitHub
<#1008 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIZPUA7XYFMYJCXUCES3FB3X4VYSNANCNFSM6AAAAAA5FGQONE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
No description provided.