не мной, но мне надо его ревьювить. Строгих гайдов по стилю кода (ещё) нет. У меня теперь такой вопрос.
В коде идёт работа с БД, есть такая функция:
async def select_all(table_name: str):
return await db.pool.fetch(f'SELECT * FROM {table_name}')
Она используется только со строковыми константами, никакого пользовательского ввода там нет. Стоит ли запрещать такой код или "и так сойдёт"? Мне он не нравится, однако если исправлять, то будет $table_count таких функций, где только имя таблицы отличается
Стоит
Селект * не надо юзать никогда. Нужно явно перечислять столбцы. Иначе возможны казусы с порядком/составом
Запросов будет больше чем количество таблиц. Потому что выборки нужны разные. Фильтрация по айди и не только, джойны, сортировка и т.п.
В коде нет ориентирования на порядок столбцов, ща заглянул, эта функция используется только для выборки всех name и id (я думаю, эти столбцы не исчезнут)
У тебя во всех таблицах есть name и id?
Так почему не указано, что она только их выгребает?
В 3/5 да В оставшихся двух используется только id из этой функции, и то для фильтрации (в ту часть кода я ещё не лез)
Вот у тебя уже два разных запроса
Обсуждают сегодня