hola
Para autenticar deberias usar algo como esto
public static bool Autentificar(String pUsuarios, String pContrasena)
{
using(SqlConnection cn = Conexion())
{
cn.Open();
string query = "Select * From Usuarios Where Usuario = @usuario and PwdCompare(@password,Contrasena) = 1 ";
SqlCommand comando = new SqlCommand(, cn);
comando.Parameters.AddWithValue("@usuario", pUsuarios);
comando.Parameters.AddWithValue("@password", pContrasena);
SqlDataReader reader = comando.ExecuteReader();
if(reader.Read())
{
return true;
}
}
return false;
}
No se de dodne salio eso del -1 o del 50 que retornas, pero en una autenticacion eso no aplica
Despues de autenticar obtienes al usuario
public class Usuario
{
public int Id {get;set;}
public int Nombre {get;set;}
}
public static Usuario ObtenerUsuario(String pUsuarios)
{
using(SqlConnection cn = Conexion())
{
cn.Open();
string query = "Select Id, Nombre From Usuarios Where Usuario = @usuario";
SqlCommand comando = new SqlCommand(, cn);
comando.Parameters.AddWithValue("@usuario", pUsuarios);
SqlDataReader reader = comando.ExecuteReader();
if(reader.Read())
{
return new Usuario()
{
Id = Convert.ToInt32(reader["Id"]),
Nombre = Convert.ToInt32(reader["Nombre"])
};
}
}
return null;
}
defines una entidad que lo represente y asignas las propiedades
Ademas usa siempre parametros, eso de concatenar los valores en un string es una pesima practica
>>he estado pensando en mostraro en un textbox, bueno que este oculto para de ahi guardarlo
porque harias eso, sino vas a mostrando ponlo en una variablea nivel del form o si quieres que sea mas global la defines como static
Un control oculto, de verdad?, que ocurrencias
saludos
Leandro Tuttini
Blog
MVP Profile
Buenos Aires
Argentina
