Или лыжи не едут, или я тупой

[A'Kon]

Короче, решил в свободное время сделать простенькую авторизацию на c#, столкнулся с проблемой
Имеется поле с названием key и кнопка.

Код була

 public bool KeyActivate(string keyactiv)
        {
            string query = $"SELECT * FROM keys WHERE key='{keyactiv}';";
            try
            {
                if (OpenConnection())
                {
                    MySqlCommand cmd = new MySqlCommand(query, conn);
                    MySqlDataReader reader = cmd.ExecuteReader();

                    if (reader.Read())
                    {
                      
                        reader.Close();
                        conn.Close();
                        return true;
                    }
                    else
                    {
                        
                        reader.Close();
                        conn.Close();
                        return false;
                    }
                }
                else
                {
                    conn.Close();
                    return false;
                }
            }
            catch (Exception ex)
            {
                conn.Close();
                return false;
            }
        }

Код кнопки

private void metroButton5_Click(object sender, EventArgs e)
        {
            string keyactiv = key.Text;

            if (KeyActivate(keyactiv))
            {
                MessageBox.Show($"Ключ активирован!");
            }
            else
            {
                MessageBox.Show($"Ключа не существует!");
            }
        }

При нажатии кнопки ВСЕГДА выскакивает MessageBox.Show($"Ключа не существует!");
Таблица keys присутствует, столбец key также присутствует
key - VARCHAR 50 символов

 

[terry godmode_1]

как я понял на до мне кажется переписать key
 

как я понял на до мне кажется переписать key

либо я тупой

 

[A'Kon]

как я понял на до мне кажется переписать key

Шо ты блять несёшь, перестань набивать сообщения

 

[Wilde]

string query = $"SELECT * FROM keys WHERE key='{keyactiv}';";

Такое не прокатит, попробуй по-старому стандарту написать.

 

[saynz]

Спойлер

Function KeyExists(key) ‘функция проверки существования ключа реестра
Dim wshShell
Dim key2
On Error Resume Next'пропускаем ошибки
Set wshShell = CreateObject(«WScript.Shell»)
key2 = wshShell.RegRead(key) ‘читаем ключ
If Err.Number <> 0 Then ‘если ошибка — ключа нет
KeyExists = False
Else ‘значит есть
KeyExists = True
End If
On Error GoTo 0‘возвращаем ошибки в исходное положение
End Function

Попробуй это

 

[A'Kon]

string query = $"SELECT * FROM keys WHERE key='{keyactiv}';";

Такое не прокатит. Попробуй сразу вписать ключ, который есть в базе, чтобы проверить запрос

Интересно же почему такое не прокатит? Делал всё по аналогии с авторизацией

 

[saynz]

Интересно же почему такое не прокатит? Делал всё по аналогии с авторизацией

Может ключи не правильные?

 

[A'Kon]

Может ключи не правильные?

В каком смысле? В таблице столбец с типо VARCHAR, строка 123, ввожу в текстбокс 123, нажимаю кнопку и всегда не правильно пишет

 

[Wilde]

В каком смысле? В таблице столбец с типо VARCHAR, строка 123, ввожу в текстбокс 123, нажимаю кнопку и всегда не правильно пишет

Так выведи месагу, а то reader может быть пустой

 

[A'Kon]

Так выведи месагу, а то reader может быть пустой

Где? В буле?

 

[Wilde]

Где? В буле?

В буле

 

[A'Kon]

В буле

И как это должно выглядеть?

 

[Wilde]

И как это должно выглядеть?

Попробуй так проверить

   if (OpenConnection())
                {
                    MySqlCommand cmd = new MySqlCommand(query, conn);
                    MySqlDataReader reader = cmd.ExecuteReader();
                    reader.Read();
  
if(reader["key"].ToString() == keyactiv)
{ 
return true;
MessageBox.Show("Заебись");
reader.Close();
conn.Close();
}
else
{
MessageBox.Show("BB");
return false;
reader.Close();
conn.Close();
}
           
                }
                else
                {
                    conn.Close();
                    return false;
                }

 

[Крайслер]

return true; MessageBox.Show("Заебись"); reader.Close(); conn.Close();

у тебя не закроются дескрипторы

 

[Wilde]

у тебя не закроются дескрипторы

Потом он сам поправит, я для макета сделал. И да, они закроются

 

[A'Kon]

Попробуй так проверить

   if (OpenConnection())
                {
                    MySqlCommand cmd = new MySqlCommand(query, conn);
                    MySqlDataReader reader = cmd.ExecuteReader();
                    reader.Read();
 
if(reader.["key"].ToString() == keyactiv)
{
return true;
MessageBox.Show("Заебись");
reader.Close();
conn.Close();
}
else
{
MessageBox.Show("BB");
return false;
reader.Close();
conn.Close();
}
          
                }
                else
                {
                    conn.Close();
                    return false;
                }

В этой строчке ошибка ( жалуется на "[" )

 if (reader.["key"].ToString() == keyactiv)

Error    CS1001    Identifier expected

 

Потом он сам поправит, я для макета сделал. И да, они закроются

Там то я поправил, ибо возвращение тру в конце

 

[dogecore]

делать сразу подключение к дб пизда как небезопасно, но это так
попробуй убрать ; в конце

 

[Wilde]

В этой строчке ошибка ( жалуется на "[" )

 if (reader.["key"].ToString() == keyactiv)

Error    CS1001    Identifier expected

Убери точку reader.["key"]

 

[A'Kon]

делать сразу подключение к дб пизда как небезопасно, но это так
попробуй убрать ; в конце

О чем речь? Какая безопасность, я для себя делаю

 

[A'Kon]

Убери точку reader.["key"]

Энивей не работает