@ru_python

Страница 3590 из 9768
CthUlhUzzz
23.08.2017
21:46:20
В остальных местах логгирование попросту не нужно

b0g3r
23.08.2017
21:46:53
string.ascii_lowercase + string.ascii_uppercase

string.ascii_letters есть

CthUlhUzzz
23.08.2017
21:47:06
Тут-то что не так?

Google
CthUlhUzzz
23.08.2017
21:47:09
Лол

b0g3r
23.08.2017
21:47:10
Или чет такое

CthUlhUzzz
23.08.2017
21:47:31
Да, действительно

b0g3r
23.08.2017
21:47:53
assert не в тестах

assert можно отключить при запуске питона, raise - нет

CthUlhUzzz
23.08.2017
21:49:02
Ассерты там тоже чисто для себя натыкал

Может на твой код посмотрим?)

А то чет разошелся

b0g3r
23.08.2017
21:50:18
Можно

CthUlhUzzz
23.08.2017
21:50:33
Ну го

b0g3r
23.08.2017
21:51:16
https://github.com/b0g3r/nosleepbot?files=1 вот тут код годичной давности Ставлю ему 8/10 по шкале херовости

state.states.State

Ахуенно

Google
CthUlhUzzz
23.08.2017
21:54:08
import os if 'DEBUG' in os.environ: class Delay: check = 10 # 1200 alarm = 10 # 60 wake_up = 10 # 60 else: class Delay: check = 1200 alarm = 60 wake_up = 60

b0g3r
23.08.2017
21:54:08
Лучше включайтесь сюда: https://github.com/spbgti

Отличный дебаг)

CthUlhUzzz
23.08.2017
21:54:45
Можно просто объявить класс а потом атрибуты втыкать

b0g3r
23.08.2017
21:54:51
Главное непонятные цифры в комментах ставить

CthUlhUzzz
23.08.2017
21:54:52
Извне

b0g3r
23.08.2017
21:54:57
Себе на будущее

Можно просто объявить класс а потом атрибуты втыкать
Да мне в этот код самому смотреть страшно, хоть переписывай

Зачем там класс вообще, если и словаря бы хватило? Нипанятна

CthUlhUzzz
23.08.2017
21:56:41
.idea в гитигнор можно было кинуть

Простым обывателям он никчему ХД

b0g3r
23.08.2017
21:57:50
А он там есть!

CthUlhUzzz
23.08.2017
21:58:09
Но он там не нужен

b0g3r
23.08.2017
21:58:52
В гитигноре

CthUlhUzzz
23.08.2017
21:58:57
А, лол

b0g3r
23.08.2017
21:59:01
Ага))

Nikolay
23.08.2017
22:35:33
От object'а наследовать?
ну как бы всегда полезно, почему нет

CthUlhUzzz
23.08.2017
22:36:32
Потому, что в 3-м питоне уже не object

type

Да и зачем лишние телодвижения эти?

Google
CthUlhUzzz
23.08.2017
22:37:06
Если все и так прекрасно работает

От тайпа обычно наследуют метаклассы

Остальное - в пекло

Nikolay
23.08.2017
22:37:42
никуда в третьем питоне object не делся

просто наследование стало неявным по дефолту

CthUlhUzzz
23.08.2017
22:38:42
чо
New style objects

Или как их там обозвали

Все объекты

Nikolay
23.08.2017
22:39:15
New style objects
ты, видимо, путаешь с new style classes

CthUlhUzzz
23.08.2017
22:39:21
Ну да

Nikolay
23.08.2017
22:39:24
только это не имеет отношения к третьему питону

CthUlhUzzz
23.08.2017
22:39:37
Могу ошибаться

Nikolay
23.08.2017
22:39:41
new style classes существуют с питона 2.5, кажется

CthUlhUzzz
23.08.2017
22:39:43
Но вопрос был не в этом

Nikolay
23.08.2017
22:39:49
если не 2.4

ну это я так

ворчу

CthUlhUzzz
23.08.2017
22:40:55
Смотрю я на эту всю простыню из сеттеров и геттеров в коде

Google
CthUlhUzzz
23.08.2017
22:41:02
Пора писать орм для редиса

Nikolay
23.08.2017
22:41:19
уже ж есть

CthUlhUzzz
23.08.2017
22:41:39
Я посмотрел на них

Чет не то

Кинь линк

Nikolay
23.08.2017
22:42:12
https://github.com/b0g3r/nosleepbot?files=1 вот тут код годичной давности Ставлю ему 8/10 по шкале херовости
а чо класс _State не абстрактный? разве ты его напрямую где-то инстанциируешь? и зачем подчеркивание, если есть all?

https://github.com/b0g3r/nosleepbot/blob/master/model.py фуфуфу, импорты со звездочкой

да еще и в неправильном порядке

https://github.com/b0g3r/nosleepbot/blob/master/scheduler.py#L15-L17 и нафига такая магия, если достаточно переменной одной глобальной?

Admin
ERROR: S client not available

Nikolay
23.08.2017
22:47:20
https://github.com/CthUlhUzzz/telegram-bot-constructor
https://github.com/CthUlhUzzz/telegram-bot-constructor/blob/master/telegram_bot_constructor/helpers.py#L15 во-первых, не надо называть параметр type. Во-вторых, у тебя два ифа проверяют одно и то же условие.

CthUlhUzzz
23.08.2017
22:49:05
Так лучше?

Я тот код к херам выпилил

Nikolay
23.08.2017
22:49:37
https://github.com/CthUlhUzzz/telegram-bot-constructor
https://github.com/CthUlhUzzz/telegram-bot-constructor/blob/master/telegram_bot_constructor/operator_client.py чо декораторы без functools.wraps? некошерно. А еще на строчке 104 магическая константа какая-то

CthUlhUzzz
23.08.2017
22:50:16
for _ in range(25):

Так принято

Когда не используешь значение

Nikolay
23.08.2017
22:50:30
for _ in range(25):
что такое 25?

CthUlhUzzz
23.08.2017
22:50:33
А

Ты об этом

Google
CthUlhUzzz
23.08.2017
22:50:41
Хардкодед щит

Nikolay
23.08.2017
22:50:41
я не про _, разумеется

вот именно

CthUlhUzzz
23.08.2017
22:50:46
Без него не работает

Это проблема скорее библиотеки для редиса, чем моя

Nikolay
23.08.2017
22:51:30
причем тут редис, в твоем же коде константа

ты 25 раз пытаешься получить сообщение по пабсабу

почему 25?

CthUlhUzzz
23.08.2017
22:52:25
Потому, что я так хочу)

Nikolay
23.08.2017
22:52:40
ну вынеси в конфиг хотя бы

zero-one-infinity rule же никто не отменял

CthUlhUzzz
23.08.2017
22:53:29
Ты прав, но мне лень

ХД

Хочешь кости помыть, лучше выбери что-нибудь позаконченнее)

Вот например

https://github.com/CthUlhUzzz/tiny-eda

Асинхронщина

Как ты любишь

Nikolay
23.08.2017
22:59:12
https://github.com/CthUlhUzzz/tiny-eda
логгер неправильно используешь

в нем не надо явно процентом форматировать, надо просто через запятую аргументы передавать, он по дефолту умеет в printf-нотацию

CthUlhUzzz
23.08.2017
23:00:33
Крутяк, не знал

Aleksey
23.08.2017
23:14:15
Друзья перебираю циклом for , в значениях присутствуют как числа так и списки, как сделать проверку , что сейчас список а не число? Что-то наподобие как есть строковый метод isupper()?

Страница 3590 из 9768