Algumas observações:
Se o campo senha está vazio, porque obrigar ao usuário a digitar o usuário novamente? basta definir o foco do cursor para o campo senha... txtPass.Focus();
Igualmente para a situação oposta.
Para verificar se uma string
é vazia, eu utilizo String.IsNullOrEmpty(txtPass.Text);
Logo abaixo de onde você abre a conexão, você declara duas variáveis:
string user = txtUser.Text;
string pass = txtPass.Text;
mas não as utiliza para nada.
E, na hora de executar o comando SQL, utilize parâmetros, e não concatene a string
dessa forma. Do jeito que está um SQL injection está muito fácil.
Fiz um código bem simples de como faria uma tela de login no seu caso:
O Form principal da aplicação (o que abre no método Main
com Application.Run
) deve ser seu form principal, e não o form de login. Por isso, coloco o dialogo de login e se caso o resultado for OK, continuo com a aplicação e abro o FormPrincipal
, no seu caso o Form1 ou FormGeral.
Program:
static class Program
{
/// <summary>
/// The main entry point for the application.
/// </summary>
[STAThread]
static void Main()
{
Application.EnableVisualStyles();
Application.SetCompatibleTextRenderingDefault(false);
FormLogin formLogin = new FormLogin();
if (formLogin.ShowDialog() == DialogResult.OK)
{
Application.Run(new FormPrincipal(formLogin.UsuarioLogado));
}
}
}
FormLogin:
Esse seria o código do FormLogin
:
public partial class FormLogin : Form
{
public Usuarios UsuarioLogado { get; set; }
public FormLogin()
{
InitializeComponent();
//O botão cancela, retorna 'Cancel'
buttonCancela.DialogResult = System.Windows.Forms.DialogResult.Cancel;
}
//Botão Login ou OK
private void buttonLogin_Click(object sender, EventArgs e)
{
try
{
if (!String.IsNullOrEmpty(txtUser.Text))
{
if (!String.IsNullOrEmpty(txtPass.Text))
{
//A rotina que valida o login do usuário, está dentro da
//classe Usuarios, e se for válido, retorna um objeto do
//tipo Usuarios, caso contrário, retorna null
UsuarioLogado = Usuarios.ValidarLogin(txtUser.Text, txtPass.Text);
if (UsuarioLogado != null)
{
//Se retornou o usuário, ou seja: é válido, retorna OK
this.DialogResult = System.Windows.Forms.DialogResult.OK;
}
else
{
txtUser.Text = txtPass.Text = "";
labelStatus.Text = "Usuario / Senha inválido";
}
}
else
{
labelStatus.Text = "Informe a senha do usuário";
txtPass.Focus();
}
}
else
{
labelStatus.Text = "Informe o nome de usuário";
txtUser.Focus();
}
}
catch (Exception ex)
{
labelStatus.Text = ex.Message;
}
}
}
Usuarios:
Essa seria uma classe de usuários, aqui ficam as propriedades dos usuários, e métodos de Insert / Update / Delete, além do método de validar login que é utilizado no FormLogin
public class Usuarios
{
public string Usuario { get; set; }
public string Senha { get; set; }
public string Nome { get; set; }
//Quaisquer outras propriedades
public static Usuarios ValidarLogin(string _user, string _senha)
{
SqlConnection con = new SqlConnection(@"Data Source=(LocalDB)\MSSQLLocalDB;AttachDbFilename='C:\Users\TutoDS\Desktop\Trabalho Programação - VideoClub\VideoClub\VideoClub\bdVideoClub.mdf';Integrated Security=True;Connect Timeout=30");
con.Open();
SqlCommand cmd = new SqlCommand("SELECT * FROM Login WHERE User= @usuario AND Pass = @senha;", con);
cmd.Parameters.Add(new SqlParameter("@usuario", _user));
cmd.Parameters.Add(new SqlParameter("@senha", _senha));
SqlDataReader reader = cmd.ExecuteReader();
if (reader.Read())
{
Usuarios obj = new Usuarios();
obj.Nome = reader["coluna_nome"].ToString();
obj.Usuario = reader["User"].ToString();
return obj;
}
else
return null;
}
}
FormPrincipal:
No FormPrincipal, no seu construtor, coloquei um parâmetro do tipo Usuarios que será o usuário logado na aplicação, a partir daí você pode ter qual usuário está utilizando o sistema.
public partial class FormPrincipal : Form
{
public FormPrincipal(Usuarios _usuarioLogado)
{
InitializeComponent();
labelUsuario.Text = _usuarioLogado.Nome;
}
}
Tentei não estender muito, se algum outro membro tiver sugestões fiquem à vontade. Qualquer dúvida, à disposição.
Fiz as alterações no seu código, sem levar em conta as observações que lhe passei, e conceitos de orientação a objeto:
SqlConnection sqlCon = null; //Conexão começa em Null
private string strCmd = @"Data Source=(LocalDB)\MSSQLLocalDB;AttachDbFilename='C:\Users\TutoDS\Desktop\VideoClub\VideoClub\VideoClub\bdVideoClub.mdf';Integrated Security=True"; //Conection String da BD
private string strSQL = string.Empty;
public bool logado = false;
public void Sign_in()
{
if (!String.IsNullOrEmpty(txtUser.Text))
{
if (!String.IsNullOrEmpty(txtPass.Text))
{
string usuarioLogado =null;
try
{
sqlCon = new SqlConnection(strCmd);
strSQL = "SELECT Nome FROM [Login] WHERE [User] = @utilizador AND [Pass] = @password";
sqlCon.Open();
SqlCommand cmd = new SqlCommand(strSQL, con);
cmd.Parameters.Add(new SqlParameter("@utilizador", txtUser.Text));
cmd.Parameters.Add(new SqlParameter("@password", txtPass.Text));
SqlDataReader reader = cmd.ExecuteReader();
if (reader.Read())
{
//usuário é válido e o nome está na variável usuarioLogado
logado = true;
usuarioLogado = reader["Nome"].ToString();
}
else
{
//usuário não é válido
txtUser.Text = txtPass.Text = "";
labelStatus.Text = "Usuario / Senha inválido";
}
}
catch (Exception ex)
{
MessageBox.Show(ex.Message);
}
}
else
{
labelStatus.Text = "Informe a senha do usuário";
txtPass.Focus();
}
}
else
{
labelStatus.Text = "Informe o nome de usuário";
txtUser.Focus();
}
}