Почему не работает код C#?

Question

[Dommager]

Делаю авторизацию, логины и пароли беру из БД. Все записи берутся корректно (выводил в MessageBox значения из тест бокса, одновременно со значениями из базы данных - идентичны). Когда начинаю сравнивать в коде, все отрицательно...
Не понимаю почему так происходит.

SqlCommand command = new SqlCommand("SELECT * FROM [Admin]", sqlConnection);

sqlReader = await command.ExecuteReaderAsync();

await sqlReader.ReadAsync();

if (textBox1.Text ==  Convert.ToString(sqlReader["login"]) )
{
   if (textBox2.Text == Convert.ToString(sqlReader["pass"]) )
   {
      Form1 f1 = new Form1();
      f1.ShowDialog();
   }
}
else
   MessageBox.Show("Неправильный логин или пароль");

Comments

[Дмитрий Ковальский]

Вы ведь понимаете что этот код плох примерно по 5-10 причинам? Из-за чего его трудно отлаживать и приходится задавать подобные вопросы.

Answer 1

[Ivan Arxont]

Если в messagebox у вас всё верно, а при сравнение в коде нет, то сравните посимвольно и узнаете где ошибка, может в бд у вас добавлен какой-нибудь пробел, или перевод строки, или смесь русских-латинских символов. Сейчас попробовал ваш код - всё работает.

var sqlConnection = new SqlConnection("Data Source=localhost;" +
                                                  "Initial Catalog=tosterTest;" +
                                                  "Integrated Security=SSPI;");

            sqlConnection.Open();

            var command = new SqlCommand("SELECT * FROM [Admin]", sqlConnection);

            var sqlReader = await command.ExecuteReaderAsync();

            await sqlReader.ReadAsync();

            if (   textBox1.Text == sqlReader["login"].ToString()
                && textBox2.Text == sqlReader["pass"].ToString())
            {
                MessageBox.Show("Всё верно!");
            }
            else
                MessageBox.Show("Неправильный логин или пароль");

Comments

[Ivan Arxont]

Ну и хешируйте пароль и не храните его простым текстом.

[Роман]

Да и тут еще один момент ТСу, булевы операции хорошая штука.

Answer 2

[Pavel Shvedov]

Попробуй сравнивать строки с помощью метода Equals(), сам нагуглишь, надеюсь, как его использовать

Comments

[eRKa]

Pavel Shvedov, Оператор сравнения для строк переопределен, он сначала сравнивает ссылки и если они не равны, тогда сравнивает побайтово, т.е. внутри этот самый equals и применяется.