Skip to content

Conta bancaria - Letícia Amaral#17

Open
hinale wants to merge 4 commits into
reprograma:mainfrom
hinale:main
Open

Conta bancaria - Letícia Amaral#17
hinale wants to merge 4 commits into
reprograma:mainfrom
hinale:main

Conversation

@hinale

@hinale hinale commented Jun 28, 2023

Copy link
Copy Markdown

Exercício: considere um objeto que represente uma conta bancária, a conta possui o nome do titular, o saldo e o limite da conta. É possível fazer operações de consulta de saldo, saque e depósito. No caso de saque é necessário verificar se existe saldo suficiente para retirada, 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.

Resolução:

  • Foi usado uma classe para referenciar o mesmo objeto que continha nome, saldo e limite de uma conta bancária;
  • Criado funções para realizar as ações de sacar, depositar, checar saldo e alterar limite;
  • No commit anterior foi feito uma primeira versão com apenas funções que recebiam como argumento o objeto com os dados.

@Mariinoue

Copy link
Copy Markdown

Considerações

  • Código está bem organizado e legivel
  • Cada função tem uma responsabilidade
  • Os testes também seguem a ideia de tem uma responsabilidade
  • Interessante o uso de classes, pois eu tenho mais contato c/ prog. funcional

@Marianaliima Marianaliima left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

parabéns pela escrita do desafio, alguns pontos de melhoria seria utilizar variáveis no beforeEach para evitar repetição de código. Também algumas melhorias poderiam ser feitas na construção da lógica como por exemplo (!newlimit) não considera em caso que ele seja 0?

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.

3 participants