Reforço-Js-Lista1#14
Open
rokorde wants to merge 2 commits into
Open
Conversation
labenu-bot
previously approved these changes
Jan 4, 2022
labenu-bot
left a comment
There was a problem hiding this comment.
Parabéns pela entrega!
Requisitos do projeto ✅
| Implementações | Feito |
|---|---|
| Exercício 1 | - |
| Exercício 2 | ✅ |
| Exercício 3 | - |
| Exercício 4 | ✅ |
| 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 13 | - |
| Exercício 14 | - |
| Exercício 15 | - |
| Feedback do código | Feito |
|---|---|
| Indentação correta | - |
| Boa nomeação de variáveis | ✅ |
| Uso correto do camelCase | ✅ |
Comentários da pessoa avaliadora
Olá Rô, Parabéns pela entrega e pelo esforço
Segue meu feedBack:
- Questão 1: A função não precisa de um return pois a intenção é que ela apenas imprima a área
- Questão 5: Criou um vetor com os valores “fixos” ao invés de usar valores lidos do prompt
- Questão 6: Apesar de receber a string por parâmetro nunca a utiliza. Cria uma nova variável para armazenar a frase “oi” e jogar ela pra maiúscula
Não deixe de enviar as dúvidas no canal ou no plantão
Análise realizada por: Izabella Silva
labenu-bot
approved these changes
Jan 7, 2022
labenu-bot
left a comment
There was a problem hiding this comment.
Parabéns pela entrega!
Requisitos do projeto ✅
| Implementações | Feito |
|---|---|
| Exercício 1 | - |
| Exercício 2 | ✅ |
| Exercício 3 | - |
| Exercício 4 | ✅ |
| 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 13 | - |
| Exercício 14 | - |
| Exercício 15 | - |
| Feedback do código | Feito |
|---|---|
| Indentação correta | - |
| Boa nomeação de variáveis | - |
| Uso correto do camelCase | - |
Comentários da pessoa avaliadora
Olá Ro, parabéns pelo esforço, continue se esforçando. Sempre que receber um feedback, tente localizar o erro e corrigi-lo, será melhor para seu aprendizado
Segue meu feedback:
- questão 1: A função não precisa de um return pois a intenção é que ela apenas imprima a área
- questão 5: Criou um vetor com os valores “fixos” ao invés de usar valores lidos do prompt
- questão 6:
- Apesar de receber a string por parâmetro nunca a utiliza, lembre-se que quando receb parâmetro, não precisa criar uma nova variável, a não ser que você precise para incluir na sua lógica
- Cria uma nova variável para armazenar a frase “oi” e jogar ela pra maiúscula
- questão 9: você já recebe o array como parâmetro, logo, não precisa criar novamente, atenção também no que pede o exercicio, pediu para retornar e você imprimiu
- Sugiro revisar funções
- Uma pena não ter continuado, pois poderia dar mais sugestões e correções, sugiro que mesmo que faça errado, enviei, pois assim poderemos dar feedback para você ver seu erro e corrigir
Análise realizada por: Izabella Silva
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
POR FAVOR, EDITE ESSA MENSAGEM INSERINDO AS INFORMAÇÕES DO SEU PROJETO. COMECE APAGANDO ESSA LINHA.
INSIRA O NOME NOME DO PROJETO AQUI
O que funciona
O que não funciona
Link Surge
A PARTIR DA SEMANA 5, INSIRA AQUI O LINK DO SURGE. ANTES DISSO, APAGUE ESSAS DUAS LINHAS.
Imagens
TIRE PRINTS DAS TELAS DO SEU SITE E COLE AQUI