Как укоротить код?

Question

[Dmitrii]

Пробую новые функции,т.к. недавно только начал изучать python,сделал перезапуск программы через цикл и не понимаю как укоротить

#рандомайзер чисел
import random

def randomazer(random):
    print('Введите от какого и до какого числа рандом')
    print('Введите первое число')
    number = int(input())
    print('Введите второе число')
    number2 = int(input())
    random = random.randint(number,number2)
    print('Сгенерированное число - '+ str(random))

repeatWord = ('д','y')
    
def repeat():
    print('Сгенерировать новое число? (да или нет)')
    return input().lower().startswith(repeatWord)

while True:
    randomazer(random)
    if repeat():
        True  #перезапуск программы,просто чтобы внутри if что-то было,
              #я не знаю почему это работает
    else:
        break

с любым значением в if код работает правильно,но если цикл запускать просто с функцией,а не через if то выполняется неправильно.
И если в if работает любое значение,то ведь скорее всего он может быть и не нужен? :)

Answer 1

[dmshar]

Не знаю, что вы имеете ввиду, потому что что такое "если цикл запускать просто с функцией,а не через if" , что такое "выполняется неправильно" и что такое "в if работает(?) любое значение" - понять очень сложно, но вот так работает - по моему разумению - правильно.

repeatWord = ('д','y')
while True:
    print('Сгенерировать новое число? (да или нет)')
    if input().lower().startswith(repeatWord):
        pass
    else:
        break

Работает в цикле до тех пор, пока не наберете что-либо, отличное от 'д' или 'y'.

Comments

[Dmitrii]

Спасибо,думаю это то что нужно, просто "заглушка" без которой код работает по другому

Answer 2

[Nick]

Дело в том,что когда вы запускаете repeat она всегда возвращает значение,которое равно булевому True.Из-за этого смысла в if нет.И функцию randomazer тоже необязательно вызывать,т.к она не используется.
Вот укороченный код

repeatWord = ('д','y')
    
def repeat():
    print('Сгенерировать новое число? (да или нет)')
    return input().lower().startswith()

while True:
    repeat()

Comments

[dmshar]

Попытка запустить ваш код дает

return input().lower().startswith()
TypeError: startswith() takes at least 1 argument (0 given)

Не стоит постить сюда решения, не проверив его.

Но даже если его исправить на
return input().lower().startswith(repeatWord)

то вы получите бесконечный цикл. Как мне кажется ТС это вряд-ли устроит.

[Nick]

Если нужно,чтобы цикл останавливался когда пользователь вводит у,то :

repeatWord = ('д','y')
    
def repeat():
    print('Сгенерировать новое число? (да или нет)')
    return input().lower().startswith()

while True:
    if repeat() == 'y': break

И вы так и полностью не указали ТЗ.Не надо постить сюда вопросы без ТЗ

[dmshar]

Nick Korovkin, Какое ТЗ? Что я вам не указал?
А что до вашего "исправления". Ну запустите свой код, наконец. Перед тем, как его выкладывать. Он опять с ошибкой. Первая ошибка - случайность. А вторая?
Но еще больше меня умиляет то, что даже если ПОВТОРНО исправить вашу программу и вставить
return input().lower().startswith(repeatWord) то программа ТОЧНО не будет работать. Даже при введеном 'y'. Запустите, удостоверьтесь. И рекомендую вам изучить тему "функции в Python" и разобраться, что и как они возвращают.

[Nick]

Автор не указал в ТЗ когда должна заканчиваться программа.Исходя из кода автора получается,что программа заходит в бесконечный цикл.

Answer 3

[soremix]

Зачем укорачивать и так короткий код?
Что такое рандомайзер?

repeatWord = ('д','y')
    
def repeat():
    print('Сгенерировать новое число? (да или нет)')
    return input().lower().startswith(repeatWord)

while True:
    randomazer(random)
    repeat()

Comments

[dmshar]

Скажите, а при каком условии вам удастся закончить выполнение вашего варианта программы?

[soremix]

dmshar, это кусок кода без функционала, какая разница будет цикл завершаться или нет?

Answer 4

[Dmitrii]

Вставил весь код,дело в том что без if не включается :
return input().lower().startswith(repeatWord)

И при любом вводе он начинает заново,вместо того чтобы использовать только 'д' или 'y'