1
private void lodButton_Click(object sender, EventArgs e)
        {
            String loginUser = login.Text;
            String passUser = password.Text;

            DB db = new DB();

            DataTable table = new DataTable();

            MySqlDataAdapter adapter = new MySqlDataAdapter();

            MySqlCommand command = new MySqlCommand("SELECT * FROM 'users' WHERE 'login' = '@uL' AND 'password' = '@uP'", db.getConnection());

            command.Parameters.Add("@uL", MySqlDbType.VarChar).Value = loginUser;
            command.Parameters.Add("@uP", MySqlDbType.VarChar).Value = passUser;

            adapter.SelectCommand = command;
            adapter.Fill(table);

            if (table.Rows.Count > 0)
                MessageBox.Show("Yes");
            else
                MessageBox.Show("no");
        }
18
  • 1
    вы везде ставите кавычки для названий полей неправильные. Либо используйте правильные (бэк тики) либо уберите совсем. `SELECT * FROM users WHERE login = '@uL' AND password = '@uP'
    – A K
    14 мая 2021 в 11:06
  • 1
    Не устану повторять: хранить в базе пароли в открытом виде - низкий, аморальный поступок.
    – aepot
    14 мая 2021 в 11:14
  • 1
    "SELECT * FROM users WHERE login = @uL AND password = @uP"
    – aepot
    14 мая 2021 в 11:20
  • 1
    спасибо,все заработало
    – crow
    14 мая 2021 в 11:26
  • 1

1 ответ 1

1
private void lodButton_Click(object sender, EventArgs e)
        {
            String loginUser = login.Text;
            String passUser = password.Text;

            DB db = new DB();

            DataTable table = new DataTable();

            MySqlDataAdapter adapter = new MySqlDataAdapter();

            MySqlCommand command = new MySqlCommand("SELECT * FROM `users` WHERE `login` = @uL AND `password` = @uP", db.getConnection());

            command.Parameters.Add("@uL", MySqlDbType.VarChar).Value = loginUser;
            command.Parameters.Add("@uP", MySqlDbType.VarChar).Value = passUser;

            adapter.SelectCommand = command;
            adapter.Fill(table);

            if (table.Rows.Count > 0)
                MessageBox.Show("Yes");
            else
                MessageBox.Show("no");
        }
1
  • 1
    Меня всегда удивляло, зачем заполняют DataTable, когда нужно проверить, существует ли запись (в данном случае user) в таблице. Используйте один из запросов отсюда. Выкиньте DataTable и MySqlDataAdapter. Используйте метод ExecuteScalar(). 14 мая 2021 в 12:13

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.