Вопрос Хороший ли код получился?

[pidjan]

Короче, ранее я изучал c# и мы с учителем по программированию делал программу "ближайшее число от 0 до 100", так вот, я решил это сделать для пайтона, хороший ли код получился и чистый ли он?

from random import randint as rand
print('Введите ближайшее число от 0 до 100')
i = int(input())
number = rand(0, 100)
s = 10
while i != number and s > 1:
    if not 0 < i and i < 100:
        print('Вы ввели некорректное число! Число должно быть в диапазоне от 0 до 100!')
    if i - 1 == number or i + 1 == number or i == number:
        print('Вы выиграли! Загаданное число:', number)
        break
    elif i < number and s >= 1:
        s-=1
        print('У вас осталось:', s, ' попыток! Загаданное число больше вашего числа!')
    elif i > number and s >= 1:
        s-=1
        print('У вас осталось:', s, ' попыток! Загаданное число меньше вашего числа!')
    print('Введите ближайшее число от 0 до 100')
    i = int(input())
    if i != number and s < 1:
        print('Вы не угадали( Загаданное число:', number)

 

[DTRR]

Лютого говнокода не видно, назвать тебя бездарностью - язык не повернётся :)

 

[vasya2137]

мне нрав)

 

[Pasterbyking]

мне нрав)

мне тоже

 

[SQL]

Использовал бы словари (АНАЛог свичей)

 

[ded78667]

# -*- coding: utf-8 -*-

from random import randint as rand


i = int(input('Введите ближайшее число от 0 до 100: '))
number = rand(0, 100)
s = 10

if __name__ == "__main__":
    while i != number and s > 1:
        if not 0 < i and i < 100:
            print('Вы ввели некорректное число! Число должно быть в диапазоне от 0 до 100!')
        if i - 1 == number or i + 1 == number or i == number:
            print(f'Вы выиграли! Загаданное число:{number}')
            break
        if i < number and s >= 1:
            s-=1
            print(f'У вас осталось {s} попыток! Загаданное число больше вашего числа!')
        if i > number and s >= 1:
            s-=1
            print(f'У вас осталось {s} попыток! Загаданное число меньше вашего числа!')
        i = int(input('Введите ближайшее число от 0 до 100: '))
        if i != number and s < 1:
            print(f'Вы не угадали :( Загаданное число: {number}')

1)делай отступы от импортов
2)не.юзай.елиф.



ps: от питона 3.6 можно юзать f-стринги

 

[pidjan]

# -*- coding: utf-8 -*-

from random import randint as rand


i = int(input('Введите ближайшее число от 0 до 100: '))
number = rand(0, 100)
s = 10

if __name__ == "__main__":
    while i != number and s > 1:
        if not 0 < i and i < 100:
            print('Вы ввели некорректное число! Число должно быть в диапазоне от 0 до 100!')
        if i - 1 == number or i + 1 == number or i == number:
            print(f'Вы выиграли! Загаданное число:{number}')
            break
        if i < number and s >= 1:
            s-=1
            print(f'У вас осталось {s} попыток! Загаданное число больше вашего числа!')
        if i > number and s >= 1:
            s-=1
            print(f'У вас осталось {s} попыток! Загаданное число меньше вашего числа!')
        i = int(input('Введите ближайшее число от 0 до 100: '))
        if i != number and s < 1:
            print(f'Вы не угадали :( Загаданное число: {number}')

1)делай отступы от импортов
2)не.юзай.елиф.



ps: от питона 3.6 можно юзать f-стринги

И зачем ты сделал if __name__ == "__main__", если это логически не нужно?
И так-же, отступы делают все по своему

 

[DTRR]

if __name__ == "__main__",

Нужен для написания модуля скорее всего :)

 

[SQL]

Нужен для написания модуля скорее всего :)

Блять, это просто мультипоток

 

[WANDERS]

Нормально, но можно было и лучше .