Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Entrega do exercicio para casa - semana01 #36

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

TiaErikaDev
Copy link

Commit 01 - Erika Mayara Pimentel

Copy link

@bialocatelli bialocatelli left a comment

Choose a reason for hiding this comment

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

Oi, Erika! Deixo aqui meu code review do exercício da primeira semana.


consultarSaldo(){
return this.saldo;
}

Choose a reason for hiding this comment

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

Notei que está certinho sua consulta de saldo. Se caso vc quiser, também é possível fazer a consulta do limite seguindo esse mesma estrutura de código.

//caso o cliente não tenha saldo suficiente para a operação, mas possua limite disponível (e suficiente), o saque poderá ocorrer,
//nesses casos o saldo do cliente ficará negativo após o saque. Além disso, o limite de uma conta pode ser reajustado (para mais e para menos) ou desativado.
//Use a abordagem Red - Green - Refactor para desenvolver essa aplicação.

Choose a reason for hiding this comment

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

Deixar a questão do exercício na mesma página da resolução é ótimo! Pra quem tá revisando ou lendo o código pela primeira vez fica muito mais claro.

}
}
return this.consultarSaldo()
}

Choose a reason for hiding this comment

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

achei bem válida essa estrutura de if. Ainda não tinha feito assim.


console.log(Erika.reajustarLimiteMenos(1000))
console.log(Erika)

Choose a reason for hiding this comment

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

Vou te deixar um comentário que também fizeram pra mim. Me parece que os console log não é necessário subir. Seria só mais para nos orientar enquanto codamos. No mais, seu código está limpo, muito bem feito mesmo.

test("Validar a função reajustarLimiteMenos: se o novo limite for maior que o limite atual", () => {
expect(() => Erika.reajustarLimiteMenos(3500)).toThrow("O novo limite deve ser menor que o limite atual")
})
})

Choose a reason for hiding this comment

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

Seus testes me pareceram bem completos. Achei todos bem organizados e fáceis de ler. Deixo uma sugestão, se caso vc quiser, poderia testar se caso o usuário colocasse zero ou um valor negativo no saque e depósito se retornaria uma mensagem de erro.
Achei que vc colocou tudo em prática do que foi visto em aula. A estrutura do seu código tá ótima!

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.

None yet

2 participants