Skip to content

revisão#15

Open
Piedrovi wants to merge 1 commit intomasterfrom
revisao-js
Open

revisão#15
Piedrovi wants to merge 1 commit intomasterfrom
revisao-js

Conversation

@Piedrovi
Copy link
Collaborator

###revisao

pouca coisa

  • 7 quests

quase nada

-problemas v=com a internet hoje

@Piedrovi Piedrovi requested a review from alinelabenu May 24, 2022 02:04
Copy link

@labenu-bot labenu-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Parabéns pela entrega!

Requisitos do projeto ✅

Implementações Feito
Exercício 1 -
Exercício 2 -
Exercício 2 Desafio -
Exercício 3 -
Exercício 3 Desafio -
Exercício 4 -
Exercício 4 Desafio -
Exercício 5 -
Exercício 6 -
Exercício 7 -
Exercício 8 -
Exercício 9 -
Exercício 10 -
Exercício 11 -
Exercício 12 -
Exercício 13A -
Exercício 13B -
Exercício 14 -
Exercício 15A -
Exercício 15B Desafio -
Feedback do código Feito
Indentação correta -
Boa nomeação de variáveis -
Uso correto do camelCase -

Comentários da pessoa avaliadora

Olá Pedro!

Primeiramente, parabéns pela entrega!

Porém, preciso deixar um aviso: como foi alertado no início do arquivo dos exercícios, as funções não deveriam ter sido alteradas ou comentadas ou alteradas, isso envolve tanto a declaração delas, quanto as chaves de fechamento, pois isso quebra todo o seu código, tanto pelo JS não entender onde termina a função, quanto pelo arquivo de testes que roda junto com o arquivo de exercícios.

Exercício 1 e 2:
A lógica está correta, mas você deveria ter usado o array presente no parâmetro da função.

Exercício 3:
Incompleto, faltou retornar o array ordenado.

Exercício 4 e 5:
Precisarias ter salvo os números pares em um array e retornado ele.
A lógica de verificar os nº pares está correta.
No caso do 5, precisarias adicionar ao novo array o número multiplicado por ele mesmo.

Exercício 6:
Na linha 52, você deveria ter atribuído o valor de array[i] a numbigger

Exercício 8:
A verificação seria similar aos exercícios 4 e 5, mas ao invés de um array, seria um tipo de contador que iria de 0 ao "n" nº par.

Pontos de atenção, para a leitura de código:
Evite deixar todo o código comentado, isso deixa bastante confuso o que era exercício e o que era código que você "descartou".
Tenha mais atenção com a indentação, camelCase e nomenclatura de variáveis. Sobre a indentação, você pode usar o atalho do vscode que arruma automaticamente (alt + shift + F no windows ou Shift + Option + F no mac).

No mais, parabéns por mais uma semana de curso!

Tenha uma ótima semana!!!

Análise realizada por: Clara

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants