preparando review#1
Conversation
thiagoribb
left a comment
thiagoribb
left a comment
There was a problem hiding this comment.
Oii, Juliana!
De maneira geral, mostrou domínio dos conceitos apresentados em aula e apresentou um código muito bem organizado e conciso. Comentei alguns pontos pelo Pull Request.
| align-items: center; | ||
| } | ||
|
|
||
| .boxcards { |
There was a problem hiding this comment.
É interessante colocar hífen nas classes com nomes compostos. É uma convenção que facilita a leitura do código. Chamamos esse padrão de kebab-case, é o padrão mais usado no mercado para CSS.
| @@ -1,97 +0,0 @@ | |||
| let cardsnumber = parseInt(prompt("Qual o número de cartas? Número par de 4 a 14")); | |||
There was a problem hiding this comment.
Em javascript, usamos o padrão de camelCase pra nomes compostos de variáveis e funções. É o padrão mais utilizado no mercado.
| "<img class ='gif disappears' src='imgs/unicornparrot.gif' alt='papagaio unicórnio'>" | ||
| ] | ||
|
|
||
| //embaralha array das imagens |
There was a problem hiding this comment.
Devemos evitar esses comentários pois isso pode acabar poluindo o código a longo prazo.
| let cardsnumber = parseInt(prompt("Qual o número de cartas? Número par de 4 a 14")); | ||
| let win = 0; | ||
| let total = 0; | ||
| let front; |
There was a problem hiding this comment.
Essa variável foi declarada globalmente mas acabou não utilizando. Devemos ter cuidado pra não deixar código desnecessário no projeto.
| console.log(cards); | ||
| console.log(arraycards[i]); |
There was a problem hiding this comment.
Não é recomendado deixar comandos como esse console.log no nosso código. Isso vale para as outras vezes em que isso aparece no código.
| } | ||
| } | ||
|
|
||
| function game() { |
There was a problem hiding this comment.
É interessante definir nomes mais semânticos pras funções, de maneira que ela represente de forma sucinta a intenção daquela função, assim como o nome da sua função test
1 participant
No description provided.