говнокод с asyncpg

0

4

async def notify_delete(channel, query, *, conn):
    status = await conn.execute(r'''DELETE FROM notify WHERE channel_id=$1 and query=$2''',
            snowflake2db(channel.id), query)
    return status != 'DELETE 0'

return status != 'DELETE 0'

насколько это плохо, и как надо правильно?

Ссылка

←	А посоветуйте фундаментальный труд по протоколам передачи данных

Вызов простых элементов GUI из скриптов

→

Оно строкой от постгреса приходит. Можешь парсер написать.

deadNightTiger ★★★★★
(04.11.18 20:05:28 MSK)

Ответ на: комментарий от deadNightTiger 04.11.18 20:05:28 MSK

ну, в данном случае ничего кроме 'DELETE 0' парсить не требуется. то есть так норм, постгрес ничего неожиданного выдать не может?

MyTrooName ★★★★★
(04.11.18 20:10:35 MSK) автор топика

Ответ на: комментарий от MyTrooName 04.11.18 20:10:35 MSK

Ну вообще может какой-нибудь error вернуть - может быть ошибка в параметрах, грантах, да «database read only» в конце концов. То есть если ты прям на 100% уверен в запросе и окружении то можно оставить, но по хорошему одной этой проверки не достаточно.

snowflake2db это что? А то первой ссылкой в гугле этот тред. Почему не использовать ORM?

micronekodesu ★★★
(04.11.18 21:32:55 MSK)

Ответ на: комментарий от micronekodesu 04.11.18 21:32:55 MSK

Ну вообще может какой-нибудь error вернуть - может быть ошибка в параметрах, грантах, да «database read only» в конце концов

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

Почему не использовать ORM?

к asyncio нормальной либы не нашел. решил, что лучше прямой контроль над запросами и индексами

snowflake2db это что?

нерелевантно. просто преобразование формата данных из строки c uint64 в int64

MyTrooName ★★★★★
(04.11.18 21:49:43 MSK) автор топика

Ответ на: комментарий от MyTrooName 04.11.18 21:49:43 MSK

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

Если ты используешь какой-нибудь там psycopg2 то да, только в таком случае тебе простого execute не хватит, надо потом прочитать результат через fetchall() например, да и вообще запрос надо выполнять не через подключение а через курсор. В любом случае, exception тоже же надо обработать.

micronekodesu ★★★
(04.11.18 21:55:00 MSK)