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

Challenges/4 emprestimo #201

Closed
wants to merge 2 commits into from
Closed

Challenges/4 emprestimo #201

wants to merge 2 commits into from

Conversation

JoaoPaulo-creator
Copy link

4 - Emprestimo

Explique um pouco como resolveu o desafio

Inicialmente li as regras de negócio propostas para o exercício, e então a parte mais difícil foi apenas implementar os cálculos utilizando das libs/pacotes nativos de golang, visto que não tenho muita familiaridade com a linguagem.

Pontos interessantes

Mesmo tendo em mente o que deveria ser feito, quando estava próximo de concluir o desafio, percebi que precisava cobrir alguns casos de uso, como por exemplo, validar não ser possível a criação de um empréstimo quando o valor for zerado.

Observações

O código está um pouco macarrônico, principalmente nas validações dos argumentos inputados, mas é possível melhorar, apenas preciso praticar Go um pouco mais.

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Salve @JoaoPaulo-creator, você esqueceu de mandar o desafio 0!

@JoaoPaulo-creator
Copy link
Author

O desavio 0 já foi realizado, inclusive já foi mergeado na main :c

Copy link
Contributor

@fernanduandrade fernanduandrade left a comment

Choose a reason for hiding this comment

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

primo a pr está mandando 2 desafios ao mesmo tempo e está com a branch errada
pode refazer por favor?

o intuito é um desafio por pr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants