Почему SQL запрос ничего не возвращает?

Question

[newPsevdonim]

Есть бд в которую постоянно стекаются данные, я с интервалом в пять секунд извлекаю их из нее. Сначала смотрю последнее время и потом от него отнимаю 5 секунд и уже с ним делаю запрос. Но если в бд натекла только одна запись то запрос ничего не возвращает почему так может происходить?

last_time = self.client.execute('SELECT time_local FROM {} ORDER'
                                            ' BY time_local DESC LIMIT 1'.format(table_name))
self.last_time[tag] = last_time[0][0]
print(self.last_time[tag])
table = self.client.execute('SELECT {} FROM {} WHERE client_id = \'{}\''
                                        'AND time_local >= \'{}\' ORDER BY'
                                        ' time_local DESC'.format(table_fields,
                                                                  table_name,
                                                                  tag,
                                                                  self.last_time[tag] - timedelta(seconds=5)))

Comments

[Akina]

А зачем два запроса-то?

почему так может происходить?

Потому что второй запрос содержит дополнительное условие по client_id.

[alexalexes]

Но если в бд натекла только одна запись то запрос ничего не возвращает почему так может происходить?

Все верно. Результат выборки - ноль строк.

self.last_time[tag] = last_time[0][0]
...
self.last_time[tag] - timedelta(seconds=5)

Эти телодвижения вне sql опишите в sql, тогда первый запрос будет не нужен.

Answer 1

[Сергей П]

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

но говнокод тот еще, конечно. Не удивительно, что для вас не прозрачна причина такого поведения кода, когда он так плохо написан.

Короче, в первом зпросе забыли добавить "WHERE client_id =..."

Comments

[newPsevdonim]

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

[Сергей П]

newPsevdonim, я вижу кусочек кода и сужу именно по нему.
В более качественно написанном коде это бросалось бы в глаза сразу и вы не потеряли бы из виду этот нюанс.

Давайте попробую докопаться до кода детально.

# Тут не буду писать про форматиование, ниже покажу как лучше 
# и прозрачнее форматировать такие участки кода
# Но здесь отмечу, что в питоне есть замечательные f-строки
# для более читабельной подстановки параметров.
# Вообще любые позиционные параметры, в отличии от именованных,
# сегда сильно снижают читабельность, размазывая контекст необходимого внимания
# по коду.
last_time = self.client.execute('SELECT time_local FROM {} ORDER'
                                            ' BY time_local DESC LIMIT 1'.format(table_name))
# Здесь появляется непонятная переменная tag, которая никак не пояснена в вопросе:
self.last_time[tag] = last_time[0][0]
print(self.last_time[tag])
# А вот здесь снова нечитабельные и зачастую опасные позиционные подстановки
# среди которых ОЧЕНЬ опасное и мусорное экранирование, которое грозит нам 
# потенциальными SQL-инъекциями.
# А ещё в контексте вашего кода одно и то же у вас называется 
# двумя непохожими именами: tag и client_id. Для того, кто будет итать ваш код
# одинаковые вещи должны называться одинаково. 
# Это, я считаю, наравне с форматированием, основная причина вашей ошибки тут.
table = self.client.execute('SELECT {} FROM {} WHERE client_id = \'{}\''
                                        'AND time_local >= \'{}\' ORDER BY'
                                        ' time_local DESC'.format(table_fields,
                                                                  table_name,
                                                                  tag,
                                                                  self.last_time[tag] - timedelta(seconds=5)))

Давайте отформатируем код и он сразу станет гораздо читабельнее:

st_time = self.client.execute(f'''
    SELECT time_local 
    FROM {table_name} 
    ORDER BY time_local DESC 
    LIMIT 1
''')
self.last_time[tag] = last_time[0][0]
print(self.last_time[tag])
time_5_seconds_ago = self.last_time[tag] - timedelta(seconds=5)
table = self.client.execute(f'''
    SELECT {table_fields} 
    FROM {table_name} 
    WHERE 
        client_id = {str(tag)!r}
        AND time_local >= {str(time_5_seconds_ago)!r}
    ORDER BY time_local DESC
''')

Строчек получилось чуть больше, но читабельность и прозрачность кода повысилась, хотя код по-прежнему далеко не идеален.
Не знаю что там за клиентскую библиотеку для работы с БД вы используете, но у всех библиотек есть встроенный механизм передачи и экранирования параметров. Его нужно использовать и это вопрос не только хорошего стиля, но и вопрос безопасности от SQL-инъекций.
Даже если конкретно здесь инъекция пока невозможна, сам факт того, что у вас уже сформировалась привычка такой передачи параметров в запросы уже делает вас, как программиста, не безопасным.
А этот код, возможно, будут править другие люди, и они, возможно, захотят добавить ещё какие-нибудь параметры и пробросят их так же, как тут уже сделано, и вот там может случиться что угодно.
Ваш код, даже если в данном случае конкретно здесь он безопасен, своим форматированием и способом передачи параметров в запрос маскирует потенциально опасные ситуации.

Да, в SQL зачастую нельзя штатным параметром передать, например, перечень столбцов или имя таблицы, но эти вещи вы скорее всего не получаете через какие-то интерфейсы от ользователя. Наверно список столбцов вы джойните через запятую в коде и могли бы приложить какие-то усилия для валидации имен столбцов. Надёжно провалидировать имя таблицы тоже можно, или взять его из какого-то внутреннего безопасного словаря, заданного константой в коде.
Эти вещи можно подставить и строковыми параметрами, это часть запроса.
А вот параметры SQL-запроса не должны подставляться в строку. Они должны передаваться в execute позиционно или по имени отдельно, чтобы клиентская библиотека надёжно позаботилась об их экранировании.
То, как экранировали строковые параметры вы - плохая опасная привычка.

Не принимайте на свой счет мои слова о говнокоде. Все иногда его пишут. Главное воспринимать с пользой конструктивную критику и учиться на ошибках. Иногда наговнокодить - это не стыдно. Стдно делать это изо дня в день и не работать над собой для улучшения навыка.

Обосновал?