Skip to content

Projeto blackjack#14

Open
Piedrovi wants to merge 3 commits intomasterfrom
PROJETO-BLACKJACK
Open

Projeto blackjack#14
Piedrovi wants to merge 3 commits intomasterfrom
PROJETO-BLACKJACK

Conversation

@Piedrovi
Copy link
Collaborator

projeto do black jack vulgo 21

@Piedrovi Piedrovi requested a review from alinelabenu May 21, 2022 01:49
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
O usuário deve encontrar uma mensagem de boas vindas no console ao abrir o programa -
O usuário deve ser questionado se deseja iniciar uma nova rodada
Se o usuário recusar iniciar uma nova rodada, o programa deve imprimir uma mensagem informando o fim do jogo -
Se o usuário aceitar iniciar uma nova rodada, o programa deve exibir as carta do jogador e do computador
Ao final do jogo, o programa deve imprimir o resultado informando quem ganhou
Quão fácil foi achar as informações no projeto através de alerts e logs? Difícil
[Desafio] O jogo esconde a uma das cartas do computador ao início da rodada -
[Desafio] O usuário é questionado se deseja comprar mais cartas -
[Desafio] Se o usuário somar mais de 21, o mesmo perde automaticamente -
[Desafio] Se o computador somar mais de 21, o mesmo perde automaticamente -
[Desafio] Implementou a verificação para impedir um par de A na primeira mão -
Funcionalidade extra -
Feedback do código Feito
Utilizou corretamente a função de sortear cartas
Nomenclatura de variáveis e funções fazem sentido -
Boa indentação
Lógica separada em funções -
Não deixou variáveis inutilizadas no código
Não deixou console.log desnecessários no código
Criou um bom readme com informações relevantes para o projeto -

Comentários da pessoa avaliadora

Olá Pedro!

Parabéns pela entrega deste projeto!

Está funcionando bem, mas alguns detalhes no console acabaram deixando um pouco confuso.
Por exemplo, você coloca que "sua carta inicial/secundária é..." e "seus pontos..." tanto para as cartas do jogador quanto do computador. Apesar de estar correta a lógica, isso confunde o usuário, afinal quem é "you" que "win"?

No seu código falta atenção ao uso do camelCase e, apesar de você ter dado nomes distintos às variáveis usar "carta", "cartas", "cartacpu" e "cartaspc" dá a entender que são cartas de jogadores distintos e até arrays de cartas, por ter o plural em algumas. O ideal é procurar dar nomes que realmente signifiquem aquilo que a variável traz como valor. Algo como 'cartaUsuario1' e 'cartaUsuario2' ou 'cartaPc1' e 'cartaPc2' são bons exemplos.

No mais, parabéns novamente pelo projeto!

Tenha uma excelente 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