видели бы вы мой скрипт,когда я писал чтобы субтитры пофиксить))

1. после запятой пробел

Код с кучей ифов, который как говно. Можно это получше всё придумать Ну и close в конструкции with, но тебя уже поправили там

Мм, я если честно не совсем понимаю почему вы так не любите ифы. Только из за внешнего вида? Мм, хотяб дайте идейку пожалуйста как тут без них обойтись

PEP8,тебе про это говорят

Мм, я если честно не совсем понимаю почему вы так не любите ифы. Только из за внешнего вида? Мм, хотяб дайте идейку пожалуйста как тут без них обойтись

Можно все методы add_to ... перенести в класс типо Dispatcher Можно бы сделать чет типо items = {'Сухофрукты': 'dried_fruits', 'Овощи': 'vegetables', ...} PREFIX = 'add_to_' И дальше чет типо такого item = items[js[i]['klass']] handler_name = ''.join([PREFIX, item]) handler = getattr(Dispatcher, handler_name) handler( ... ) передаёшь аргументы

Мм, я если честно не совсем понимаю почему вы так не любите ифы. Только из за внешнего вида? Мм, хотяб дайте идейку пожалуйста как тут без них обойтись

Давай представим, что у тебя таких видов айтемов будет штук 50

PEP8,тебе про это говорят

Ааа, блин я подумал что по функциональности херня какая то или ошибка где то

Давай представим, что у тебя таких видов айтемов будет штук 50

Тут да, сделал бы по вашему совету

Тупо + 100 строк абсолютно шаблонного кода

ну и код офк говно

абсолютно не масштабируем

Ну тип да Но для novice простительно :)

???

Мой линтер кричит

И тут обычного warning становится недостаточно...

Мой линтер кричит

Годный линтер со смайлами

абсолютно не масштабируем

Мм, в каком смысде не масштабируем? если появится новая категория, достаточно добавить новую функцию и новое условие. Или вы про другое?

Мм, в каком смысде не масштабируем? если появится новая категория, достаточно добавить новую функцию и новое условие. Или вы про другое?

Не масштабируем это то, о чем говорил я

Представь, что видов айтемов будет 50

можно это говно отрефакторить? ?

Вот эта хуйня с 50 ифами никуда не будет годиться

Мой линтер кричит

А он не кричит что ему не нравится?

кричит

если навести на строку на которой кричит

А что за IDE? Pycharm же?

А что за IDE? Pycharm же?

nvim

Чорд

можно это говно отрефакторить? ?

В след проекте))))

там js[i] можно другой переменной присвоить

ибо очень часто юзается ?

после запятых пробел ставь

И вринципе не понятно, что такое js

и после операторов

Типо, переменная, которая ни о чем не говорит

И вринципе не понятно, что такое js

Это джсон файл полученный серваком

и после операторов

Ок

Это джсон файл полученный серваком

Ну вот вместо js лучше было б назвать переменную так, чтобы она описывала содержимое файла

Может loaded_json?

Это тоже нихера не говорит о том, что из себя представляют данные

Должно быть название как-то связанное с едой

Ну или че похожее, хз Но не js/ loaded_json уж точно

И как сказал один мужик в видео

Это PEP8, но это не питоничный код. (p.s. это пиздец)

Любому программисту, который будет читать код, и так будет понятно, что это json данные(по строке request.json) Но не понятно, что они из себя представляют

Любому программисту, который будет читать код, и так будет понятно, что это json данные(по строке request.json) Но не понятно, что они из себя представляют

судя по всему, оно возвращает это самое название фрукта, или не фрукта, в душе не ебу, что это

Нуу, можете меня бить, но мне больше понравилось как было до этого.))

судя по всему, оно возвращает это самое название фрукта, или не фрукта, в душе не ебу, что это

исходя из фрукта возвращают нее ебу что это))

Нуу, можете меня бить, но мне больше понравилось как было до этого.))

там были просто длинющие линии

надо урезать

но там было лучше :D

но это говно можно сделать ещё лучше

исходя из фрукта возвращают нее ебу что это))

в голосину

Нуу, можете меня бить, но мне больше понравилось как было до этого.))

Строка не должна быть длиннее 120 символов

Строка не должна быть длиннее 120 символов

79

ИТАК) всем спасибо за замечания и предложения. Я их все учту и постараюсь исправить. - пробелы - названия - поменьше ифов)

Ну тогда вроде окей, наверное )

И close в конструкции with

Зы. Кстати про ифы,Вы это еще не видели как я определяю название таблицы по активной радиокнопке))))

@Dakotaz ты пеп8 имеешь ввиду?

ERROR: S client not available

Про подучи теорию

Не только

Я бы посоветовал просто вот эту хуйню прочитать https://www.ozon.ru/context/detail/id/138437220/ Но не всем заходит, ибо когда опыта 0, то не всегда понятно, о чем идёт речь

Я бы посоветовал просто вот эту хуйню прочитать https://www.ozon.ru/context/detail/id/138437220/ Но не всем заходит, ибо когда опыта 0, то не всегда понятно, о чем идёт речь

Хорошая книжка

ой

ой

Чет говно

ой

syntax error

keys*

keys*

норма

keys* это распаковка листа в аргументы функции ?

норма

Нет, это syntax error.

Потому что распаковка это *keys

ой ?

?

Потому что распаковка это *keys

Не каждый день распаковываю ?

>id_zakaza УБИВАТ

>id_zakaza УБИВАТ

(не моё ?)

ещё и zak_ ну вообще, ну ппц

ещё и zak_ ну вообще, ну ппц

тут чувак код скинул, я рефакторить пытаюсь ?

Щас напишу свой говновариант

>id_zakaza УБИВАТ

Чеж триггерит то всех) за то всем понятно) мддда, над названиями надо поработать и этот проект на гитхаб я выкладывать не буду???

TypeError: 'HtmlElement' object is not callable

что это значит?

что ты попытался вызвать объект типа HtmlElement как функцию

Чеж триггерит то всех) за то всем понятно) мддда, над названиями надо поработать и этот проект на гитхаб я выкладывать не буду???

Вот например так https://gist.github.com/Asperas13/11038e456d5cdf37691e191d9392e4b8

что ты попытался вызвать объект типа HtmlElement как функцию

скину код сейчас

'''musics = pyquery_object('div.audio_row__inner') ready_songs = '' i = 0 for music in musics: i += 1 music_name_artist = music('div.audio_row__performers').find('a').text music_song = music('div.audio_row__title _audio_row__title') music_name_song = music_song.find('.//span[@class = "audio_row__title_inner _audio_row__title_inner"]').text ready_songs += '{0}. {1}- {2}/n'.format(i, music_name_artist, music_name_song) print(ready_songs) '''

пожалуйста, заключай код в ```