[OP]

Python. Оценка качества кода. Просьба указать, что можно улучшить. Использовал линтер flake8, так что код соответствует pep8.
cclass text_image():
def treagle_up(self):
i = 0
while i <= 12:
print(""i)
i += 1

def treagle_down(self):
i = 11
while i >= 0:
print(""i)
i -= 1


print('Input what image you wont get: \n\
1. triangle in up. \n\
2. triangle in down. \n\
3. triangle mix')
choice = input(': ')
output_image = text_image()
if choice == 1:
output_image.treagle_up()
elif choice == 2:
output_image.treagle_down()
if choice == 3:
output_image.treagle_up()
output_image.treagle_down()

Макака убила отступы.
Код: https://dropmefiles.com/LQQGE

Бамп.

Оно же не запустится даже, какой толстый троленг

>>237634799
>>237634765

[email: sage]

Какой пеп8 когда ты копируешь в виде обычного текста без сохранения форматирования?

Что ты хочешь тут узнать от нас? Какие ты задачи решаешь, чтобы понять что улучшать? вдруг задача была именно в написании этого куска говна и тут и улучшать-то нечего.

input возвращает строку, а ты её сравниваешь с числом. choice == 1 никогда не будет true потому что "1" != 1. А что будет если ввести не 1,2 или 3? а что будет если ввести строку? иди нахуй

>>237634828
Похуй на отступы. У тебя в синтаксисе беды уровня cclass и "";i

>>237634598 (OP)
НЕ ВКЛЮЧАЙТЕ ЭТОТ КОД ОН СКАЧАТ ВИРУС НА ВАШ КОМПБЮТЕР!!!

[email: sage]

>>237634765

https://pastebin.com/tools

Сюда закидывай а не файлы кидай

>>237634598 (OP)
>self
Блджад, объясните по человечески нахуй это надо писать?!!
Пожалуйста.

[email: sage]

>>237635106
Методы классов в питуне первым аргументом принимают инстанс этого класса чтобы иметь доступ к другим методам и аттрибутам этого класса через self. Статикметоды принимают первым аргументом сам класс а не его экземпляр

>>237635108 →
>>237635106

__init__ или любая функция некого класса Object
self.huerga = 5
zalupa = 10

classobject = Object()
classobject.huerga # 5
classobject.zalupa # error var даз нот ексист

>>237635236
> инстанс этого класса
Еще проще можно, для тех кого няня головой об край кровати била?
>>237635682
Не понял, пардон.

>>237635236
Ебать ты идиот по жизни

>>237634922
https://pastebin.com/qZuYCMZP
Это из-за imageboard.
>>237634866
Спасибо. Пока просто тренируюсь в программировании. Подскажи, пожалуйста, как писать что-то более серьёзное? Это задание из чужой работы, которую увидел, что просил решить другой анон в треде. Было просто интересно реализовать и проверить свои навыки.
Как исправить? Я правильно понимаю, что необходимо проверять является ли строка числом через string.isdigit()?

>>237636115
> строка
значение

>>237635806
>Еще проще можно
self первым аргументом это ООП. Объектно Ориентированное Программирование позволяет эффективно расходовать оперативную память, кэш процессора, и даже энергомощность блока питания за счёт того, что позволяет манипулировать остатками ячеек памяти уже пройденных компилятором, путём представления ячеек (или их совокупностей) через т. н. объекты. Классы описывают эти самые объекты. Всё очень просто.

>>237634598 (OP)
Зачем тебе класс и объекты, когда ты нигде не используешь их состояние? Тут класс явно лишний.
Ты используешь волшебные числа (12 и 11), взятые абсолютно с потолка. Я понимаю что конкретно этот код не очень сложный, но ты бы их объявил как-нибудь.
Твои циклы получились на три строчки каждый вместо for i in range(n), это выглядит немного нелепо.
> код соответствует pep8
Это конечно хорошо, но ты играешь в разработку, вместо того, чтобы научиться программировать.

>>237638547
Спасибо. Но разве соблюдение pep8 не важно?

>>237639582
>pep8
>линтер
>нет пробелов вокруг умножения

[email: sage]

>>237643384
flake8 ошибку не выдаёт, не?

>>237643512
sage...

>>237643512
https://www.python.org/dev/peps/pep-0008/#other-recommendations
Еще чего-нибудь найти в доках за тебя?
Линтеры настраивать можно, если что.