Sistema bancario#26
Open
lumarin2802 wants to merge 2 commits into
Open
Conversation
labenu-bot
approved these changes
Jan 31, 2022
labenu-bot
left a comment
There was a problem hiding this comment.
Parabéns pela entrega!
Requisitos do projeto ✅
| Implementações | Feito |
|---|---|
| É possível criar uma conta no banco se tiver idade igual ou maior do que 18 anos | - |
| Um usuário deve conseguir verificar o saldo da sua conta, passando o seu nome e o seu CPF | - |
| É possível pagar uma conta passando: um valor, uma descrição e uma data de pagamento | - |
| Se a data do pagamento da conta não for informada, deve-se considerar que o pagamento é para ser feito no mesmo dia | - |
| Não é possível agendar um pagamento para um dia anterior | - |
| Não é possível pagar uma conta com o valor maior do que saldo. | - |
| É possível fazer uma transferência bancária somente se for informado o nome, CPF, nome do destinatário, o CPF do destinatário e o valor | - |
| Se a data da transferência bancária não for informada, deve-se considerar que o pagamento é para ser feito no mesmo dia | - |
| Um usuário consegue adicionar saldo à sua conta, passando nome, CPF e o valor que desejar | - |
| Se a data de um pagamento não for informada, a data do pagamento é direcionada para o mesmo dia` | - |
| Não é possível criar usuários diferentes com o mesmo CPF [Desafio] | - |
| Criou um endpoint get que recebe um CPF como parâmetro e retorna o saldo da conta do usuário. Emitiu um erro se o CPF informado não for do usuário [Desafio] | - |
| Criou um endpoint put que recebe um nome, um CPF e um valor para adicionar um valor no saldo do usuário. Emitiu um erro se o CPF informado não for do usuário [Desafio] | - |
| Criou um método GET na entidade users para pegar todos os usuários existentes no array de usuários. | ✅ |
| Funcionalidades Extra | - |
| Feedback do código | Feito |
|---|---|
| todos os gastos do usuário é guardado em um extrado, que no código é um array de objetos com as propriedades: valor, data e descrição | - |
| Criou um tipo para representar uma conta para o usuário | ✅ |
| Criou um array global que armazene usuários na aplicação | ✅ |
| Criou um tipo para representar as transações que serão salvas no extrato | ✅ |
| Criou um endpoint que utiliza o método POST da entidade users para cadastrar um usuário em um array de usuários. | ✅ |
| Exibiu uma mensagem de erro se o usuário for menor de 18 anos. | - |
| Foi criado um array para armazenar as transações de um cliente | ✅ |
| Boa indentação | ✅ |
| Qualidade de código | Médio |
Comentários da pessoa avaliadora
Olá Luciana!
Primeiramente, parabéns pela sua entrega!
Alguns comentários sobre seu código:
Inicialmente o código estava quebrando pois faltou tipar o erro (linhas 33 e 45). Ele não aceita não ser tipado, mas como não temos um type específico pra ele, pode usar "error: any".
No endpoint de criar novo usuário faltou só conferir se o CPF já existia e se o usuário tem mais de 18 anos, no mais, está correta sua lógica.
Por boa prática, colocamos os endpoints organizados por entidade (user, por exemplo) e então métodos, começando pelo GET, POST, PUT e DEL, nesta ordem.
No mais, parabéns pela sua dedicação! Continue assim!!!
Tenha uma excelente semana!
Análise realizada por: Clara
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.
SISTEMA BANCARIO
O que funciona
O que não funciona
Link Surge