[WDPT SAO 05-2021] Renan Oliveira

[RenanOliveira20]

No description provided.

[d-kunrath]

Ótimo código Renan!
Abaixo algumas pequenas considerações para arrumar, assim o seu Jasmine volta a passar os testes.

E uma dica para completar os 2 pontos que faltam passar no Jasmine também (tirando o Bônus 8):
O jasmine aplica as próprias condições de teste, ou seja, não usa as variáveis e nem se importa com os console.logs que vc usou nesse arquivo, ele se importa apenas com as funções que criou.
Dito isso, no avg(), ele espera um resultado e já dou uma dica aqui: que tenha exatamente 2 casas decimais.
Já no sum() ele espera que vc throw Error quando há uma condição específica.

Espero que vc fique curioso como usar isso. haha
De resto, fez tudo! parabéns!

[d-kunrath]

Cuidado com os tipos de dados.
vc está comparando uma string "" com um número. Aqui naõ deu problema algum, mas cuidado futuramente.
Sugiro iniciar a variável como número mesmo.
let maxNumber = 0

[d-kunrath]

Atenção aqui:
Vc está declarando duas variáveis com o mesmo nome.
Na linha 53 já existe const mixedArr = (...)!
Isso fez o seu jasmine quebrar pois ele não consegue letr o JS todo.

[RenanOliveira20]

Valeu pelos feedbacks vou aderir ao meu código! Em ter., 25 de mai. de 2021 23:21, DK ***@***.***> escreveu:

…

***@***.**** commented on this pull request. Ótimo código Renan! Abaixo algumas pequenas considerações para arrumar, assim o seu Jasmine volta a passar os testes. E uma dica para completar os 2 pontos que faltam passar no Jasmine também (tirando o Bônus 8): O jasmine aplica as próprias condições de teste, ou seja, não usa as variáveis e nem se importa com os console.logs que vc usou nesse arquivo, ele se importa *apenas* com as funções que criou. Dito isso, no avg(), ele espera um resultado e já dou uma dica aqui: que tenha *exatamente* 2 casas decimais. Já no sum() ele espera que vc throw Error quando há uma condição específica. Espero que vc fique curioso como usar isso. haha De resto, fez tudo! parabéns! ------------------------------ In src/functions-and-arrays.js <#2366 (comment)> : > @@ -1,20 +1,129 @@ // Iteration #1: Find the maximum - - +function maxOfTwoNumbers(number1, number2){ + let maxNumber = ''; + if(number1 > number2){ + maxNumber = number1; + }else if(number1 < number2){ + maxNumber = number2; + }else if(number1 === number2){ + maxNumber = number1; + }else{ + maxNumber = 'mesmos valores'; + } + return maxNumber; Cuidado com os tipos de dados. vc está comparando uma string "" com um número. Aqui naõ deu problema algum, mas cuidado futuramente. Sugiro iniciar a variável como número mesmo. let maxNumber = 0 ------------------------------ In src/functions-and-arrays.js <#2366 (comment)> : > // Level 2: Array of strings const wordsArr = ['seat', 'correspond', 'linen', 'motif', 'hole', 'smell', 'smart', 'chaos', 'fuel', 'palace']; + function averageWordLength(array){ + let valorTotal = 0 + let totalDeItens = 0 + if(array.length === 0){ + return null + }else if(array.length !== 0){ + array.forEach(posicaoAtual => { + valorTotal += posicaoAtual.length; + totalDeItens ++; + }); + } + return valorTotal / totalDeItens + } + averageNumbers(wordsArr) + const mixedArr = [ 6 , 12 , 'miami' , 1 , true , 'barca' , '200' , 'lisboa' , 8 , 10 ] ; Atenção aqui: Vc está declarando duas variáveis com o mesmo nome. Na linha 53 já existe const mixedArr = (...)! Isso fez o seu jasmine quebrar pois ele não consegue letr o JS todo. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2366 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AT5XSPPOIXI52NKKR5Q5YG3TPRLKZANCNFSM45PIAEUA> .

[stale]

This pull request has been automatically marked as stale because it didn't have any recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[stale]

This pull request is closed. Thank you.