preparando para o review#1
Conversation
sidneyouteiro
left a comment
sidneyouteiro
left a comment
There was a problem hiding this comment.
Fala Jean! Trouxe um feedback de código do seu último projeto 🙂
Eu gostei bastante das soluções que você desenvolveu ao longo do projeto,
só tenho a acrescentar algumas coisas quanto a semântica e versionamento.
Para próximos projetos aconselho mensagens de commits mais descritivos e adotar convenções para nomear funções e variáveis.
Bom trabalho!
| cartas[0]= '<img src="images/bobrossparrot.gif">'; | ||
| cartas[2]= '<img src="images/explodyparrot.gif"> '; | ||
| cartas[4]= '<img src="images/fiestaparrot.gif"> '; | ||
| cartas[6]= '<img src="images/metalparrot.gif"> '; | ||
| cartas[8]= '<img src="images/revertitparrot.gif"> '; | ||
| cartas[10]= '<img src="images/tripletsparrot.gif"> '; | ||
| cartas[12]= '<img src="images/unicornparrot.gif">'; | ||
| cartas[1]= '<img src="images/bobrossparrot.gif">'; | ||
| cartas[3]= '<img src="images/explodyparrot.gif"> '; | ||
| cartas[5]= '<img src="images/fiestaparrot.gif"> '; | ||
| cartas[7]= '<img src="images/metalparrot.gif"> '; | ||
| cartas[9]= '<img src="images/revertitparrot.gif"> '; | ||
| cartas[11]= '<img src="images/tripletsparrot.gif"> '; | ||
| cartas[13]= '<img src="images/unicornparrot.gif">'; |
There was a problem hiding this comment.
Poderíamos armazenar apenas o nome do gif no array de cartas tendo em vista que é o único trecho que varia. Assim reduziria a quantidade de código duplicado.
|
|
||
|
|
||
|
|
||
| function sortearcartas(){ |
There was a problem hiding this comment.
É uma boa prática utilizarmos convenções de nomes de funções. Na Driven, utilizamos bastante o camelCase. Exemplo:
function sortearCartas(){ ... } ✅
| let qtdCartas = Number(prompt("Com quantas cartas deseja jogar?")); | ||
| let numerodejogadas=0; | ||
| let cartas = []; | ||
| let cartassorteadas = []; | ||
| let verificar = []; | ||
| let acertos = 0; |
There was a problem hiding this comment.
É uma boa prática utilizarmos convenções de nomes de variaveis. Na Driven, utilizamos bastante o camelCase. Exemplo:
let cartasSorteadas = []; ✅
1 participant
No description provided.