random.randint
вычисляет новое случайное число каждый раз, когда оно вызывается.Кроме того, вам не нужен цикл for
для печати переменной.
Вместо этого вызовите random.randint
один раз, сохраните число в переменной, а затем повторно используйте в последующей логике.Вы также можете использовать range
, например, if x in range(1, 4)
представляет более ясную логику, чем if x in {1, 2, 3}
.
x = random.randint(1, 8)
if x in range(1, 4):
print("https://media.giphy.com/media/vgPDmpQsQsjug/giphy.gif")
elif x in range(4, 7):
print("http://i0.kym-cdn.com/photos/images/original/000/676/205/57b.gif")
else:
print("Bye")
Стилистически вы должны придерживаться правила "отступа из 4 символов".Хотя ваш код неверен, он не соответствует общепринятым соглашениям.