Member 12604208 Ответов: 1

Я хочу узнать ежемесячную продажу в промежутке между заданными временами из поля datetime


Я использую vs2008 и backend ms-access
В таблице есть два поля bdate (только дата хранения), endtime (дата и время хранения)

Я хочу отображать записи из таблицы между двумя датами только для данного времени.
Я использую datetimepicker для condition. Никаких записей не найдено...

Pl дайте решение

Что я уже пробовал:

SELECT * FROM sale where bdate between #" & Format(fromtime.Value, "dd/MMM/yyyy") & "# and #" & Format(totime.Value, "dd/MMM/yyyy") & "# and (endtime between #" & fromtime.Value.ToLongTimeString & "# and " & totime.Value.ToShortTimeString & "#)"

1 Ответов

Рейтинг:
1

OriginalGriff

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

Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:

SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу"
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?

В этом случае вам это сойдет с рук, потому что DateTimePicker всегда возвращает только действительные даты, но даже тогда это плохая идея, потому что если вы позволите расширить это приложение, чтобы использовать серверную базу данных, то фактическая дата, которую вы передадите, может быть очень легко интерпретирована неправильно. заведите привычку использовать параметры для каждого доступа к БД, и многие будущие проблемы исчезнут.

Если также означает, что вы не получаете нужное количество символов "#" неправильно ... это должно быть четное число! С параметрами они вам не нужны, и все это гораздо проще читать.