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

Finalizando o desafio #224

Merged
merged 2 commits into from
Oct 5, 2023
Merged

Finalizando o desafio #224

merged 2 commits into from
Oct 5, 2023

Conversation

guisaliba
Copy link
Contributor

(01) Cubo Simples

Explique um pouco como resolveu o desafio

Inicializei uma interface para aceitar inputs do usuário com require('readline') e então criei um método que itera sobre todos os inputs do usuário e retorna o valor de cada um deles ao cubo. Os valores são separados por um space com o método split(' ') do JavaScript.

Pontos interessantes

Não sabia como se fazia uma interface de input/output no Node, e foi legal aprender e implementá-la nesse desafio.

Observações

Não foi implementada uma validação sólida do tipo do input, que deveria ser sempre um número. Caso o usuário coloque qualquer entrada que não seja do tipo number, o método aceitará esse valor e irá logar apenas uma mensagem "The cube of [TIPO_DO_VALOR] is NaN."

Copy link
Collaborator

@cherryramatisdev cherryramatisdev left a comment

Choose a reason for hiding this comment

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

Você esta lendo corretamente do stdin, mas o output parece um pouco estranho com o que estamos esperando (que seria apenas o numero) !
image

@cherryramatisdev cherryramatisdev added the Necessita de correção PR que precisa ser corrigida label Oct 5, 2023
@guisaliba
Copy link
Contributor Author

Obrigado Cherry pelo review, tive que entender melhor como o readline do Node funciona. Acredito que agora esteja correto.

Copy link
Collaborator

@cherryramatisdev cherryramatisdev left a comment

Choose a reason for hiding this comment

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

Boooa! mto foda

image

@cherryramatisdev cherryramatisdev added hacktoberfest-accepted and removed Necessita de correção PR que precisa ser corrigida labels Oct 5, 2023
@cherryramatisdev cherryramatisdev merged commit fbd3016 into he4rt:main Oct 5, 2023
1 check passed
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.

3 participants