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

Primeiro envio do Desafio 03 por Talita Costa #1105

Merged
merged 6 commits into from
Oct 21, 2024

Conversation

talita048
Copy link
Contributor

Contatos:

Copy link
Member

@mpinheir mpinheir left a comment

Choose a reason for hiding this comment

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

@talita048 gostei.
Sugiro uma modificação:
Sua solução está permitindo que o usuário digitar um segundo número menor que o primeiro.
Implemente uma verificação para evitar isso e peça para o usuário digitar o número novamente caso isso acontece.

@mpinheir
Copy link
Member

@talita048 um outro problema da aplicação.
Eu digitei o primeiro número como um número negativo.
Note o que aconteceu.
Trate isso e evite este problema
Bem-vindo(a) ao detector de números palíndromos!
Um número palíndromo é aquele que permanece igual quando lido de trás para frente.
Insira dois números e vou te informar todos os palíndromos que existem entre eles :D
Digite o primeiro número: -3909123
Digite o segundo número: 10
Traceback (most recent call last):
File "/home/marcelo/Projects/OsProgramadores-Projetos/op-desafios/desafio-03/talita048/python/desafio03.py", line 11, in
reverso = int(str(limite_a)[::-1])
^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: invalid literal for int() with base 10: '3219093-'

@mpinheir
Copy link
Member

@talita048 mais um problema identificado.
A aplicação está permitindo que letras sejam digitadas.
Trate este problema
Bem-vindo(a) ao detector de números palíndromos!
Um número palíndromo é aquele que permanece igual quando lido de trás para frente.
Insira dois números e vou te informar todos os palíndromos que existem entre eles :D
Digite o primeiro número: a
Traceback (most recent call last):
File "/home/marcelo/Projects/OsProgramadores-Projetos/op-desafios/desafio-03/talita048/python/desafio03.py", line 5, in
limite_a = int(input("Digite o primeiro número: "))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: invalid literal for int() with base 10: 'a'

@magnussolidus magnussolidus added the python Pull requests that update Python code label Oct 19, 2024
Adicionadas funções que conferem se as entradas do usuário são números válidos e se o primeiro número é menor que o segundo.
@talita048
Copy link
Contributor Author

Código revisado!

Copy link
Member

@mpinheir mpinheir left a comment

Choose a reason for hiding this comment

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

@talita048 está quase lá mas precisa de um ajuste.
Note que eu digitei o segundo número menor que o primeiro e você mostrou corretamente o erro mas ao invés de me forçar a corrigir o segundo número você me levou de volta a ter que digitar o primeiro número.
Sugiro pedir para corrigir o segundo número apena
Vide mensagens abaixo e aguardo o Ajuste.
Bem-vindo(a) ao detector de números palíndromos!
Um número palíndromo é aquele que permanece igual quando lido de trás para frente.
Insira dois números e vou te informar todos os palíndromos que existem entre eles :D
Digite o primeiro número: 90909
Digite o segundo número: 3
O primeiro número deve ser menor que o segundo. Tente novamente.
Digite o primeiro número:

@mpinheir
Copy link
Member

@talita048 está quase lá mas precisa de um ajuste.
Note que eu digitei o segundo número menor que o primeiro e você mostrou corretamente o erro mas ao invés de me forçar a corrigir o segundo número você me levou de volta a ter que digitar o primeiro número.
Sugiro pedir para corrigir o segundo número apena
Vide mensagens abaixo e aguardo o Ajuste.
Bem-vindo(a) ao detector de números palíndromos!
Um número palíndromo é aquele que permanece igual quando lido de trás para frente.
Insira dois números e vou te informar todos os palíndromos que existem entre eles :D
Digite o primeiro número: 90909
Digite o segundo número: 3
O primeiro número deve ser menor que o segundo. Tente novamente.
Digite o primeiro número:

Caso o segundo número seja menor que o primeiro, o usuário só precisará corrigir o segundo número.
@talita048
Copy link
Contributor Author

Acredito ter resolvido (e com uma mudança muito simples!)
Gostei dessa modificação, aprendi muito com ela.

Copy link
Member

@mpinheir mpinheir left a comment

Choose a reason for hiding this comment

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

Parabéns!

@mpinheir mpinheir merged commit a8a0467 into OsProgramadores:master Oct 21, 2024
1 check passed
@talita048
Copy link
Contributor Author

Obrigada pela ajuda!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants