Что в нём фантастически много кода и непонятно, с чего даже начинать читать

index.py - главный файл

ну я догадался, да

Не улучшает, кстати. У тебя в хедере ебашится импорт какой-то лузлы. Сам файл на несколько тысяч строк. Создается ложное впечатление, что енто что-то распространенное и часто юзабельное. А по факту там один вызов где-то. Так быть не должно

Вот и вопрос с этим возник - использую два класса-обертки над threading - практически эта библиотека используется только в 2-х методах класса (создание и остановка). Описываю импорт глобально, но чую, что нужно в описание классов перенести

Что можете о коде в этом боте? https://github.com/MaxiFilippov/news_bot

А ещё повсеместные import * очень затрудняют чтение

Вот и вопрос с этим возник - использую два класса-обертки над threading - практически эта библиотека используется только в 2-х методах класса (создание и остановка). Описываю импорт глобально, но чую, что нужно в описание классов перенести

Имхо, "чует" тут не надо. Если импорт в хедере добавляет вопросов, то локально, если все и без ентого понятно, тоне парься.

Имхо, лучше подозвать коллегу, который не знаком с кодом и спросить, что легче читается

еще вопрос, как отлавливаете выход из скрипта, вызванный извне? (особенно Ctrl + Z интересует) (мне нужно грамотно треды останавливать, чтобы потом не висели в системе)

Имхо, лучше подозвать коллегу, который не знаком с кодом и спросить, что легче читается

Увы, работа с этим не связана

Не улучшает, кстати. У тебя в хедере ебашится импорт какой-то лузлы. Сам файл на несколько тысяч строк. Создается ложное впечатление, что енто что-то распространенное и часто юзабельное. А по факту там один вызов где-то. Так быть не должно

Если файл на несколько тысяч строк его ничего не улучшит кроме деления на части

Если файл на несколько тысяч строк его ничего не улучшит кроме деления на части

Енто с какого перепугу?

index.py - главный файл

1. Entrypoint обычно лучше именовать main или как-то так 2. global не оч хорошо в коде юзать 3. Много лишних комментариев там, где они не нужны. Не нужно писать над функцией, что она делает. В идеале, по названию и коду должно быть понятно

Кто в курсе, в селери id задачи это строка?

Написано uuid, но я чёт не уверен какой тип именно это означает

Енто с какого перепугу?

С файлом такого размера сложно уже работать

С файлом такого размера сложно уже работать

Аргументы будуть? А то пока неоч. Начать можно с простого - пайчарм облегчает работу. А закончить фразой - сишники поржут.

index.py - главный файл

Вот эта вся функция выглядит очень не очень https://github.com/MaxiFilippov/news_bot/blob/master/change_all_settings.py#L25

Аргументы будуть? А то пока неоч. Начать можно с простого - пайчарм облегчает работу. А закончить фразой - сишники поржут.

Нервно посмеются. А после пустят скупую слезу

Написано uuid, но я чёт не уверен какой тип именно это означает

Бамп

Лучше избегать больших функций, которые мало делают

ориентир, который я где-то уцепил и мне он очень понравился

если файл больше 200 строк, то в нем наверно овердохрена всего и надо бы начать делить

Брехня

200. Двести

ну это не про го, конечно)

Я и в проектах на 1к+ строк нормально ориентируюсь. Даже спустя годы.

пожалуй да, не про го

На один файл, да.

я плохо ориентируюсь, но у меня вообще с памятью и внимательностью плохо

200 строк ок

кстати, я на жаве такого ограничения старался придерживаться

200 строк файл?

Если нормально и логично разбивать на блоки кода, то все читаемо.

Ташке ща контрольный в голову засажу: uvloop в основном файле, loop.c, содержит 163 000 строк.

да. В каждом файле проекта стараться иметь не более 200 строк

Аргументы будуть? А то пока неоч. Начать можно с простого - пайчарм облегчает работу. А закончить фразой - сишники поржут.

1. Больше проблем при слиянии 2. Больше риск наплодить циклических связей и вообще сильно связать код 3. Сложнее навигация и поиск нужных вещей 4. Чем более кода, тем больше вероятность нарушения single responsibility

Кстати, а зачем тебе ctrl + Z ловить?

Ташке ща контрольный в голову засажу: uvloop в основном файле, loop.c, содержит 163 000 строк.

Ник мой хотяб пиши нормально

Написано uuid, но я чёт не уверен какой тип именно это означает

Ну бамп :с

Ну бамп :с

А гуглить пробовал?

Ташке ща контрольный в голову засажу: uvloop в основном файле, loop.c, содержит 163 000 строк.

Он автосгенерированный ведь.

https://ru.m.wikipedia.org/wiki/UUID

А гуглить пробовал?

Пробовал, ничего не нашел...

Меня интересует именно в селери который дают

Ташке ща контрольный в голову засажу: uvloop в основном файле, loop.c, содержит 163 000 строк.

Ссылку плиз

Пробовал, ничего не нашел...

https://ru.wikipedia.org/wiki/UUID

Ссылку плиз

Это файл, генерённый из .pyx, в котором три тысячи строк.

Ссылку плиз

Ну сам открой. У тебя же должен он в репах стоять

https://github.com/MagicStack/uvloop/blob/master/uvloop/loop.pyx

Окей, Пикс вижу

https://ru.wikipedia.org/wiki/UUID

То есть в любом случае байты?

То есть в любом случае байты?

Ставлю на шестнадцатеричную строку

Он автосгенерированный ведь.

Ну енто не аргумент в рамках дискуссии про большие файлы

Кстати, а зачем тебе ctrl + Z ловить?

треды висят в линуксе тогда

Ставлю на шестнадцатеричную строку

Это что ещё за хтонь

треды висят в линуксе тогда

Ну да, ты же суспендишь тред

Это что ещё за хтонь

Ты блять вики читал, которую я тебе скинул?

Да...

Я туплю уже к вечеру слегка

Мне его крч надо в базу засунуть И я не могу понять какой тип юзать для поля

Ну да, ты же суспендишь тред

тогда вопрос по другому: как правильнее прирывать скрипт с консоли, чтобы отловить эксепш и грамотно завершить работу скрипта (в тредах могут остаться зависшие подключения к бд)

Мне его крч надо в базу засунуть И я не могу понять какой тип юзать для поля

хочешь — суй в строку, хочешь — суй в байты

Хочешь, отдельный тип есть в некоторых бд

хочешь — суй в строку, хочешь — суй в байты

Ок, спасибо

хочешь, я убью коллизии, что мешают совать

(извините)

тогда вопрос по другому: как правильнее прирывать скрипт с консоли, чтобы отловить эксепш и грамотно завершить работу скрипта (в тредах могут остаться зависшие подключения к бд)

https://docs.python.org/3/library/signal.html

C-z, кстати, невьебенно полезная команда, о которой часто забывают, привыкнув к С-с. Когда многопоточный демонг начинает жрать сервер, то С-с нихуя не поможет. А вот С-z - да.

А я тут недавно узнал о волшебной комбинации ~., которая убивает зависшее ssh-соединение

потрясающе

А я тут недавно узнал о волшебной комбинации ~., которая убивает зависшее ssh-соединение

Хм, интересно

А я тут недавно узнал о волшебной комбинации ~., которая убивает зависшее ssh-соединение

при пустой строке ввода

надо сначала энтер нажать, а потом ~.

ну да, наверное

https://docs.python.org/3/library/signal.html

спасибо, не работал с сигнал)

А я недавно телегу вшатал)) Она не смогла прожевать после перезагрузки зависшие "соединения" с файлами - и падало на одном логе с json. Это было мило.

Ну сам открой. У тебя же должен он в репах стоять

Ну в итоге там 3к строк и я не уверен, почему так. Как и не вижу причин считать что так надо всегда делать

Всем привет. Ребят, помогите пожалуйста. Скачал pytube, сделал по инструкции, но ошибка Traceback (most recent call last): File "/mnt/0a5cb80d-673d-4f97-a33b-188bfdef8a77/videos/video.py", line 6, in <module> yt = YouTube('https://www.youtube.com/watch?v=8Egjs4bW4UA') File "/mnt/0a5cb80d-673d-4f97-a33b-188bfdef8a77/videos/env/lib/python3.7/site-packages/pytube/__main__.py", line 88, in __init__ self.prefetch_init() File "/mnt/0a5cb80d-673d-4f97-a33b-188bfdef8a77/videos/env/lib/python3.7/site-packages/pytube/__main__.py", line 96, in prefetch_init self.prefetch() File "/mnt/0a5cb80d-673d-4f97-a33b-188bfdef8a77/videos/env/lib/python3.7/site-packages/pytube/__main__.py", line 170, in prefetch age_restricted=self.age_restricted, File "/mnt/0a5cb80d-673d-4f97-a33b-188bfdef8a77/videos/env/lib/python3.7/site-packages/pytube/extract.py", line 121, in video_info_url group=0, File "/mnt/0a5cb80d-673d-4f97-a33b-188bfdef8a77/videos/env/lib/python3.7/site-packages/pytube/helpers.py", line 66, in regex_search .format(pattern=pattern), pytube.exceptions.RegexMatchError: regex pattern (\W[\'"]?t[\'"]?: ?[\'"](.+?)[\'"]) had zero matches (

Ну в итоге там 3к строк и я не уверен, почему так. Как и не вижу причин считать что так надо всегда делать

Енто просто первый попавшийся пример. Ну открой другие приложухи крупные - там будет тоже самое. И на 20к строк без генерации.

Есть спецы по селениуму ?

На самом деле нет никаких объективных причин винить размер файла. Ну, если мы там не рассматриваем какие-то гигабайты. Нормально структурируемый код на то и структурируемый, чтобы не париться

Всем привет. Ребят, помогите пожалуйста. Скачал pytube, сделал по инструкции, но ошибка Traceback (most recent call last): File "/mnt/0a5cb80d-673d-4f97-a33b-188bfdef8a77/videos/video.py", line 6, in <module> yt = YouTube('https://www.youtube.com/watch?v=8Egjs4bW4UA') File "/mnt/0a5cb80d-673d-4f97-a33b-188bfdef8a77/videos/env/lib/python3.7/site-packages/pytube/__main__.py", line 88, in __init__ self.prefetch_init() File "/mnt/0a5cb80d-673d-4f97-a33b-188bfdef8a77/videos/env/lib/python3.7/site-packages/pytube/__main__.py", line 96, in prefetch_init self.prefetch() File "/mnt/0a5cb80d-673d-4f97-a33b-188bfdef8a77/videos/env/lib/python3.7/site-packages/pytube/__main__.py", line 170, in prefetch age_restricted=self.age_restricted, File "/mnt/0a5cb80d-673d-4f97-a33b-188bfdef8a77/videos/env/lib/python3.7/site-packages/pytube/extract.py", line 121, in video_info_url group=0, File "/mnt/0a5cb80d-673d-4f97-a33b-188bfdef8a77/videos/env/lib/python3.7/site-packages/pytube/helpers.py", line 66, in regex_search .format(pattern=pattern), pytube.exceptions.RegexMatchError: regex pattern (\W[\'"]?t[\'"]?: ?[\'"](.+?)[\'"]) had zero matches (

Хз, похоже на баг в либе

Как супом спарсить HTML, после нажатия кнопки через селениум ?

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

Как супом спарсить HTML, после нажатия кнопки через селениум ?

soup = BeautifulSoup(driver.page_source)

Но селениум тоже прекрасно умеет парсить

soup = BeautifulSoup(driver.page_source)

Точно, забыл про драйвер, спасибо

Но селениум тоже прекрасно умеет парсить

Супом быстрее же, разве нет ?

существует ли структурное программирование в питоне?

Супом быстрее же, разве нет ?

Черт его знает. Если использовать lxml как бэкенд - мб и быстрей, а мб и нет

Браузеры тоже не дураки пишут

1. Entrypoint обычно лучше именовать main или как-то так 2. global не оч хорошо в коде юзать 3. Много лишних комментариев там, где они не нужны. Не нужно писать над функцией, что она делает. В идеале, по названию и коду должно быть понятно

Хорошо, спасибо за ответы!

существует ли структурное программирование в питоне?

Структурное - это ифы, форы. Почему нет?