OriginalGriff
Не используйте метод 2. Никогда не используйте метод 2. Никогда не объединяйте строки для построения 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;
Вполне допустимая команда "удалить таблицу"
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.
Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?
Если ваш код не получает ошибку, но база данных не обновляется, то это означает, что предложение UPDATE WHERE не соответствует ни одной строке.
А почему бы и нет? Ну, это большой вопрос! Возможно, значение идентификатора, которое вы передаете, не то, что вы думали, или вы можете получить доступ к неправильной базе данных, или новые значения, которые вы устанавливаете для них, совпадают со старыми. Мы не можем сказать наверняка.
Так что все будет зависеть от тебя.
К счастью, у вас есть инструмент, который поможет вам выяснить, что происходит: отладчик. Как вы его используете, зависит от вашей системы компилятора, но быстрый поиск в Google имени вашей IDE и "отладчика" должен дать вам необходимую информацию.
Поместите точку останова в первую строку функции и запустите код через отладчик. Затем посмотрите на свой код и на свои данные и определите, что должно произойти вручную. Посмотри на свой
id
переменная и проверьте точно, что она содержит - это тот идентификатор, который вы ожидаете? Если это не так, то почему бы и нет?
Если это так, используйте SSMS для просмотра содержимого БД и изучения этой строки. Существует ли она вообще? Каковы его текущие значения? Отличаются ли они от ценностей, которые вы передаете?
Затем по одному шагу в каждой строке проверяйте, что то, что вы ожидали, произойдет именно так, как и произошло. Когда это не так, тогда у вас есть проблема, и вы можете вернуться назад (или запустить ее снова и посмотреть более внимательно), чтобы выяснить, почему.
Я также хотел бы посмотреть на возвращаемое значение из операции SQL:
int rowsAffected = sql.ExecuteNonQuery();
Извините, но мы не можем сделать это за вас - пришло время вам освоить новый (и очень, очень полезный) навык: отладку!