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

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

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

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

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

9 ответов

17 просмотров

Стоит

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

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

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

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

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

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

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

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

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

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

а через ESC-код ?
Alexey Kulakov
29
30500 за редактор? )
Владимир
47
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
13
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
program test; {$mode delphi} procedure proc(v: int32); overload; begin end; procedure proc(v: int64); overload; begin end; var x: uint64; begin proc(x); end. Уж не знаю...
notme
6
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
вы делали что-то подобное и как? может есть либы готовые? увидел картинку нокода, где всё линиями соединено и стало интересно попробовать то же в ddl на lua сделать. решил с ч...
Victor
8
Ребят в СИ можно реализовать ООП?
Николай
33
Подскажите пожалуйста, как в CustomDrawCell(Sender: TcxCustomGridTableView; ACanvas: TcxCanvas; AViewInfo: TcxGridTableDataCellViewInfo; var ADone: Boolean); получить наз...
A Z
7
Карта сайта