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;
Вполне допустимая команда "удалить таблицу"
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.
Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?
Когда вы исправите это в своем приложении, начните смотреть на это - но внимательно посмотрите на то, что вы делаете. Откуда вы получаете данные изображения?
tbl
это новый DataTable - он не содержит строк, поэтому счетчик всегда будет равен нулю.
Вы также не делаете ничего полезного с вашим Датаредером!
И, пожалуйста, сделайте себе одолжение: прекратите использовать имена Visual Studio по умолчанию для всего - вы можете помнить, что "TextBox8" - это номер мобильного телефона сегодня, но когда вам придется изменить его через три недели, вы это сделаете? Используйте описательные имена - например, "tbMobileNo", - и ваш код станет легче читать, более самодокументируемым, легче поддерживать - и на удивление быстрее кодировать, потому что Intellisense может добраться до "tbMobile" за три нажатия клавиш, где "TextBox8" занимает размышление и 8 нажатий клавиш...
Member 14630006
спасибо за помощь! , ИК, что ты имеешь в виду, им тестирование проекта теперь это не основной код, так что я использовать, чтобы добраться до изображения, то я организовать мой код, и я использую тестовую базу данных прямо сейчас, чтобы увидеть, какие проблемы я буду иметь
OriginalGriff
Не делайте этого: всегда предполагайте, что то, что вы пишете, является "реальным кодом" - потому что это будет так, вы не всегда сможете вернуться, чтобы пересмотреть его, как только он заработает. И тогда вам придется иметь дело с дерьмовым шоу, которое происходит из кода, который вы знали, как сделать правильно, но не сделали, и это может занять дни или даже недели в реальном мире.