MoneyLover
то что ты из ру - я уже понял
MoneyLover
для него русские номера отображаются ( на его же скрине )
MoneyLover
сервис мультиязычный просто)
MoneyLover
по айпи детектит
🍁 Jaŭhien
Коля, беги!
MoneyLover
В общем, если есть кто из Киева кто осилит данный стэк, прошу в личку
MoneyLover
старое описание продукта
17
Sasha
Сегодня четверг вообще то
еще среда. Думал, что в адмету попал, а это питон... странная тут революция происходит.
🍁 Jaŭhien
А вчера вообще вторник был
Anonymous
Коля, ты за Путина голосовал наверное? судя по твоей проницательности
Не актуально ужо. Тут либо за Зе, либо за По.
Сорян, не удержался. Без оффтопа
MoneyLover
извиняюсь, у меня несколько вариантов сотрудничества. Есть торговая марка и .ua и ООО
можно работать как на партнерских так и на зп, но все нужно обсуждать
🍁 Jaŭhien
Не актуально ужо. Тут либо за Зе, либо за По.
Сорян, не удержался. Без оффтопа
Уже ж разобрались. Комика в президенты, шоколадного короля на свалку
Tigran
Tishka17
А в чем с ними проблема?
Я пока не улавливаю суть
Ладно, я не знаю что такое save в pewee. Но если есть отдельный вызов, вероятно он нужен не всегда
Tishka17
Вообще, save больше похож что должен быть в репозитории или слое бизнес логики, но точно не в самой сущности
Anonymous
Ладно, я не знаю что такое save в pewee. Но если есть отдельный вызов, вероятно он нужен не всегда
Ташка, херовая же тема - категорично советовать там, где не шаришь или шаришь неоч.
Aragaer
я могу ошибаться и с чем-то путать, но по-моему создание экземпляра сущности не заносит его автоматически в базу данных. save() нужен именно для этого. В частности он присваивает id
Roman
Привет. Как работает import одной и той же библиотеки многократно использующийся в разных частях сткрипта (например import os в основном теле скрипта, в модулях и в классах), импортируется библиотека 1 раз или каждый раз при встрече этой инструкции?
Aragaer
не стоит
Aragaer
вообще лучше не использовать локальных импортов
Tishka17
Roman
вообще лучше не использовать локальных импортов
описывать импорты в глобальной области видимости?
Aragaer
да
Anonymous
вообще лучше не использовать локальных импортов
Кстати, мне енто всегда казалось таким, дискуссионным вопросом. Если что-то редкое, то зачем егоипрортить на глобальном уровне?
Tishka17
Кстати, мне енто всегда казалось таким, дискуссионным вопросом. Если что-то редкое, то зачем егоипрортить на глобальном уровне?
Это глобальная переменная что так что так
🍁 Jaŭhien
Aragaer
если что-то редкое, разовое и локальное, то я и сам делаю локальные импорты
Tishka17
Я делаю локальные только если они от конфига зависят или других подобных факторов
pishite
Подскажите не могу вникнуть в функции, а имеено в зарезервированные слова: global, local, nonlocal
Tishka17
Например, тип коннектора к базе
pishite
Кто-то может простым языком объяснить
Tishka17
Или реализация интерфейса другого
pishite
Слишком заумно в книге
Anonymous
Что можете о коде в этом боте? https://github.com/MaxiFilippov/news_bot
Anonymous
Так удобней работать с кодом
Не улучшает, кстати. У тебя в хедере ебашится импорт какой-то лузлы. Сам файл на несколько тысяч строк. Создается ложное впечатление, что енто что-то распространенное и часто юзабельное. А по факту там один вызов где-то.
Так быть не должно
Tigran
Что можете о коде в этом боте? https://github.com/MaxiFilippov/news_bot
Что в нём фантастически много кода и непонятно, с чего даже начинать читать
Anonymous
Что в нём фантастически много кода и непонятно, с чего даже начинать читать
Извиняюсь, забыл залить с комментариями)
Anonymous
Что в нём фантастически много кода и непонятно, с чего даже начинать читать
index.py - главный файл
Tigran
ну я догадался, да
Roman
Не улучшает, кстати. У тебя в хедере ебашится импорт какой-то лузлы. Сам файл на несколько тысяч строк. Создается ложное впечатление, что енто что-то распространенное и часто юзабельное. А по факту там один вызов где-то.
Так быть не должно
Вот и вопрос с этим возник - использую два класса-обертки над threading - практически эта библиотека используется только в 2-х методах класса (создание и остановка). Описываю импорт глобально, но чую, что нужно в описание классов перенести
Tigran
Что можете о коде в этом боте? https://github.com/MaxiFilippov/news_bot
А ещё повсеместные import * очень затрудняют чтение
Anonymous
Вот и вопрос с этим возник - использую два класса-обертки над threading - практически эта библиотека используется только в 2-х методах класса (создание и остановка). Описываю импорт глобально, но чую, что нужно в описание классов перенести
Имхо, "чует" тут не надо. Если импорт в хедере добавляет вопросов, то локально, если все и без ентого понятно, тоне парься.
Anonymous
Имхо, лучше подозвать коллегу, который не знаком с кодом и спросить, что легче читается
Roman
еще вопрос, как отлавливаете выход из скрипта, вызванный извне? (особенно Ctrl + Z интересует) (мне нужно грамотно треды останавливать, чтобы потом не висели в системе)
Roman
Имхо, лучше подозвать коллегу, который не знаком с кодом и спросить, что легче читается
Увы, работа с этим не связана
Tishka17
Не улучшает, кстати. У тебя в хедере ебашится импорт какой-то лузлы. Сам файл на несколько тысяч строк. Создается ложное впечатление, что енто что-то распространенное и часто юзабельное. А по факту там один вызов где-то.
Так быть не должно
Если файл на несколько тысяч строк его ничего не улучшит кроме деления на части
Anonymous
Если файл на несколько тысяч строк его ничего не улучшит кроме деления на части
Енто с какого перепугу?
Иван
index.py - главный файл
1. Entrypoint обычно лучше именовать main или как-то так
2. global не оч хорошо в коде юзать
3. Много лишних комментариев там, где они не нужны. Не нужно писать над функцией, что она делает. В идеале, по названию и коду должно быть понятно
âDêšwu öBipci
Кто в курсе, в селери id задачи это строка?
âDêšwu öBipci
Написано uuid, но я чёт не уверен какой тип именно это означает
Roman
Anonymous
С файлом такого размера сложно уже работать
Аргументы будуть? А то пока неоч. Начать можно с простого - пайчарм облегчает работу.
А закончить фразой - сишники поржут.
Иван
index.py - главный файл
Вот эта вся функция выглядит очень не очень
https://github.com/MaxiFilippov/news_bot/blob/master/change_all_settings.py#L25
🍁 Jaŭhien
Аргументы будуть? А то пока неоч. Начать можно с простого - пайчарм облегчает работу.
А закончить фразой - сишники поржут.
Нервно посмеются. А после пустят скупую слезу
Иван
Лучше избегать больших функций, которые мало делают
Aragaer
ориентир, который я где-то уцепил и мне он очень понравился
Aragaer
если файл больше 200 строк, то в нем наверно овердохрена всего и надо бы начать делить
Nikolay
Брехня
Aragaer
200. Двести
Tigran
ну это не про го, конечно)
âDêšwu öBipci
Я и в проектах на 1к+ строк нормально ориентируюсь. Даже спустя годы.
Aragaer
пожалуй да, не про го
âDêšwu öBipci
На один файл, да.
Aragaer
я плохо ориентируюсь, но у меня вообще с памятью и внимательностью плохо
Aragaer
200 строк ок
Aragaer
кстати, я на жаве такого ограничения старался придерживаться
Richard
200 строк файл?
âDêšwu öBipci
Если нормально и логично разбивать на блоки кода, то все читаемо.
Anonymous
Ташке ща контрольный в голову засажу: uvloop в основном файле, loop.c, содержит 163 000 строк.