Pude notar que no estás asignando la conexión a MySqlCommand
.
using (MySqlConnection conexion = new MySqlConnection("server=127.0.0.1;database=estacionamiento2;Uid=root;pwd=;"))
{
conexion.Open();
MySqlCommand comando = new MySqlCommand(String.Format("select * from usuarios where usuario = '" + ustxt.Text + "' and password = PASSWORD('" + textBox1.Text + "')"));
comando.Connection = conexion; // <----- Agrega esta línea
MySqlDataReader reader = ...
//Código omitido para mantener el foco en el área de interés
}
También puedes utilizar el constructor de MySqlCommand
que recibe como parámetros la instrucción sql y la conexión.
Recomendaciones:
- En relación a los objetos
MySqlCommand
y MySqlDataReader
deben ser descartados (Dispose) bien sea de forma implicita (dentro de un bloque using
tal como tienes la conexión) o de manera explicita invocando su método Dispose()
en el bloque finally
de un try-catch-finally
o try-finally
.
Esta recomendación aplica para cualquier clase que implementa la interface IDisposable
.
- Para saber si un
MySqlDataReader
contiene registros, es mejor utilizar la propiedad reader.HasRows
.
Siguiendo estas recomendaciones tu código puede escribirse de la siguiente manera:
A partir de C# 8 puedes utilizar el patrón de declaración using
using MySqlConnection conexion = new MySqlConnection("server=127.0.0.1;database=estacionamiento2;Uid=root;pwd=;");
using MySqlCommand comando = new MySqlCommand("instruccion sql", conexion);
try
{
conexion.Open();
using MySqlDataReader reader = comando.ExecuteReader();
if (reader.HasRows)
{
Form1 frm = new Form1();
frm.Visible = true;
}
else
{
MessageBox.Show("Datos erroneos");
}
}
catch(Exception ex)
{
//instrucciones cuando ocurre un error
}
En cualquier versión de C#
using (MySqlConnection conexion = new MySqlConnection("server=127.0.0.1;database=estacionamiento2;Uid=root;pwd=;"))
using (MySqlCommand comando = new MySqlCommand("instruccion sql", conexion))
{
try
{
conexion.Open();
using( MySqlDataReader reader = comando.ExecuteReader() )
{
if (reader.HasRows)
{
Form1 frm = new Form1();
frm.Visible = true;
}
else
{
MessageBox.Show("Datos erroneos");
}
}
}
catch(Exception ex)
{
//instrucciones cuando ocurre un error
}
}
- Por favor ten en cuenta el comentario de @Arriel relacionado con la Inyección SQL