jamcastroo / challengeSwift

2 stars 0 forks source link

Melhoria na verificação de int #6

Open ciceronascimento opened 2 years ago

ciceronascimento commented 2 years ago

O app ficou muuuito bom! Bem intuitivo e muito util!!

Queria sugerir uma melhoria! Ao colocar o numero de repetições, coloquei por extenso e o app não respondeu.

Captura de Tela 2022-03-31 às 20 00 47

Vasculhando o código achei a verificação abaixo, vi que o app faz a verificação se é inteiro mas não retorna ao usuário, o que me fez pensar que o app havia congelado mas na vdd ele estava esperando eu digitar um inteiro

https://github.com/jamcastroo/challengeSwift/blob/7a8f031f45bf9950f3506b1988ca93bd840dab6e/GymVisualizer/Gym.swift#L46-L54

Acho que seria massa colocar um print para retorno do usuário como na sugestão abaixo:


        while answerInt == nil {
            //print sugerido
            print("Digite um numero inteiro")
            //cria uma constante para a resposta do usuario que é uma string
            let answer = readLine()! 
             //converte a resposta de string para inteiro, se a conversão der errado, recebe um nil e continua no loop 
             //se a conversão der certo, sai do loop e vai para a próxima instrução 
            answerInt = Int(answer) 
         } 
jamcastroo commented 2 years ago

O app ficou muuuito bom! Bem intuitivo e muito util!!

Queria sugerir uma melhoria! Ao colocar o numero de repetições, coloquei por extenso e o app não respondeu.

Captura de Tela 2022-03-31 às 20 00 47

Vasculhando o código achei a verificação abaixo, vi que o app faz a verificação se é inteiro mas não retorna ao usuário, o que me fez pensar que o app havia congelado mas na vdd ele estava esperando eu digitar um inteiro

https://github.com/jamcastroo/challengeSwift/blob/7a8f031f45bf9950f3506b1988ca93bd840dab6e/GymVisualizer/Gym.swift#L46-L54

Acho que seria massa colocar um print para retorno do usuário como na sugestão abaixo:

        while answerInt == nil {
            //print sugerido
            print("Digite um numero inteiro")
            //cria uma constante para a resposta do usuario que é uma string
            let answer = readLine()! 
             //converte a resposta de string para inteiro, se a conversão der errado, recebe um nil e continua no loop 
             //se a conversão der certo, sai do loop e vai para a próxima instrução 
            answerInt = Int(answer) 
         } 

Show, muito legal as sugestões ! com certeza são melhorias possíveis mesmo no futuro :))