170 похожих чатов

Всем привет. Нужно ваше мнение, у меня дилемма. Есть код, написан

не мной, но мне надо его ревьювить. Строгих гайдов по стилю кода (ещё) нет. У меня теперь такой вопрос.

В коде идёт работа с БД, есть такая функция:
async def select_all(table_name: str):
return await db.pool.fetch(f'SELECT * FROM {table_name}')

Она используется только со строковыми константами, никакого пользовательского ввода там нет. Стоит ли запрещать такой код или "и так сойдёт"? Мне он не нравится, однако если исправлять, то будет $table_count таких функций, где только имя таблицы отличается

9 ответов

21 просмотр

Стоит

Селект * не надо юзать никогда. Нужно явно перечислять столбцы. Иначе возможны казусы с порядком/составом

Запросов будет больше чем количество таблиц. Потому что выборки нужны разные. Фильтрация по айди и не только, джойны, сортировка и т.п.

Evgen-Fil // hehe Автор вопроса
Tishka17
Селект * не надо юзать никогда. Нужно явно перечис...

В коде нет ориентирования на порядок столбцов, ща заглянул, эта функция используется только для выборки всех name и id (я думаю, эти столбцы не исчезнут)

Evgen Fil // hehe
В коде нет ориентирования на порядок столбцов, ща ...

Так почему не указано, что она только их выгребает?

Evgen-Fil // hehe Автор вопроса
Tishka17
У тебя во всех таблицах есть name и id?

В 3/5 да В оставшихся двух используется только id из этой функции, и то для фильтрации (в ту часть кода я ещё не лез)

Похожие вопросы

Обсуждают сегодня

Господа, а что сейчас вообще с рынком труда на делфи происходит? Какова ситуация?
Rꙮman Yankꙮvsky
29
А вообще, что может смущать в самой Julia - бы сказал, что нет единого стандартного подхода по многим моментам, поэтому многое выглядит как "хаки" и произвол. Короче говоря, с...
Viktor G.
2
30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Карта сайта