Pull to refresh

Comments 12

Очень похоже на эпизодические комментарии к какому-то коду. Если уж это статья, то основой должен быть текст, описывающий идею, подходы к реализации, выбранную архитектуру и прочее иллюстрированное кодом. А сейчас в основе код, к которому местами приложены комментарии, которые можно заменить правильными именами функций. Если называть функцию не hanadler(), а addToBlacklist (или BlockList, кому как), не hfandler(), а unBlockUserById() (или короче, смотря насколько из контекста понятно, что в параметрах — id того, кого разблокируем, а то я в питоне не шарю, меня message.text несколько путает), не proce(), а blockUserById(), то комментарий "теперь сделаем добавление в ЧС" теряет смысл. И выходит, что вся статья состоит просто из кода. Оптимизируем подход — статья состоит из ссылки на гитхаб.

За это и минусы, я полагаю.

// По коду тоже вопросы: почему флаг блокировки назван id, а не isBlocked? Почему бы не объединить функции блокировки и разблокировки в одну? Разница ведь тооько в значении, которое передаётся полю block в базе.

Ещё хочется добавить: считать количество пользователей не через count(*), а через select * и len(results) — это уже за гранью.

Я когда только с SQL познакомился, через 2-3 месяцев такое написал, а потом мне нужно было сходить в базу данных по ADSL. И тут я понял, зачем придуман count().

Полагаю, человек, который пишет обучающие статьи на хабре, не может пару месяцев назад познакомиться с языком. Или да?

В sqlite есть же rowid, зачем отдельно user_id заводить, который даже не auto_increment и не помечен уникальным? Или мне каких-то знаний не хватает?

Полагаю, user_id не auto_increment, потому что приходит от телеграмма. Но добавить pk к нему, конечно же, надо.

Статья имеет название и полностью ко по мне соответствует ему. Всё достаточно просто и легко написано и откомменчено автором. Помню, когда я искал гайд по инету для быстрого старта с телеграмным ботом, то 9 из 10 тупо убили мое время, коды отказывались работать, а здесь что не так с кодом? (Кроме моментов, где можно оптимизировать). И вообще мне непонятно, почему такие умники заходят читать эти статьи ? Вы не умеете создавать ботов на питоне?)) Или просто по жизни цель зайти и поминусовать чужих статей? ?

Лично я не умею писать ботов на питоне, потому что не знаю питон. Но даже не зная языка, я вижу, как код можно сделать лучше, о чём и пишу.

А по поводу того, что статья соответствует заголовку, дык, на хабре обычно публикуют статьи несколько другого рода, чем "скопируйте вот этот код и запустите".

// Я ничего не минусовал. Не уверен, что у меня даже есть такая возможность.

UFO just landed and posted this here

Не используйте форматные строки для динамического формирования запросов к БД! Это открытая дверь для SQL-инъекций!

После первой статьи уже стоило задуматься, почитать комментарии и на время затихнуть, изучая Python и SQL. Однако, увы, работы над ошибками нет.

Напомнило поговорку: "никогда не сдавайся - позорься до конца"...

Присоединяюсь к комментариям, что стоило прочитать комментарии к прошлому посту, сделать работу над ошибками... Но нет...

Подробно устанавливали питон целый урок, а потом сразу лихой код без объяснений ) Супер.

Sign up to leave a comment.

Articles