OriginalGriff
Для начала, не делай этого так! Здесь столько всего не так...
1) Никогда не объединяйте строки для построения 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;
Вполне допустимая команда "удалить таблицу"
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.
Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?
И делать это на странице входа на сайт? Где любой, где угодно может делать то, что хочет? Это почти преступная глупость.
2) Никогда не храните пароли в открытом тексте - это серьезная угроза безопасности. Здесь есть некоторая информация о том, как это сделать:
Хранение паролей: как это сделать.[
^] Объедините это с SQL-инъекцией, и я смогу войти в систему как кто я хочу, когда я хочу, даже не зная ни одного пароля...
3) получите правильный SQL.
new SqlCommand("SELECT count([Role]) Users WHERE ([Role] = 'Admin' ");
Без 'FROM' перед 'Users' это вызовет исключение SQL.
Исправьте их в таком порядке в приоритетном порядке на протяжении всего вашего приложения: или вы потеряете свою базу данных - черт возьми, вы лучший помощник попробуете это сделать, чтобы увидеть, что происходит, и просто увидеть выражение вашего лица...
Gatsby29
Спасибо тебе, приятель, за заботу и длинный ответ. Я просто новичок, и я пытаюсь сделать это приложение для практики, позже я буду реализовывать хеширование и другие материалы. Просто пытаюсь заставить этот базовый логин работать правильно, и у меня есть проблема в этом операторе if или заполнении таблицы данных, я не уверен, что даже с хорошей командой sql не работает.
OriginalGriff
Никогда, никогда не делай этого!
Если вам говорят: "это и это опасно", не думайте, что "сейчас все будет хорошо, я вернусь позже и все исправлю". Вы этого не сделаете, это скучно, и у вас есть более интересные / неотложные проекты на вашей тарелке.
Вместо этого возьмите в привычку делать это правильно с самого начала. Это намного, намного быстрее, чем пытаться исправить ситуацию после серьезной проблемы, вызванной вашими ярлыками. И гораздо легче рано приобрести "правильные привычки", чем пытаться переключиться на них позже!