public String IdentificarAcessoDAL(FuncionarioDTO acesso) {
try {
conexao.cmd.Connection = conexao.conexao;
conexao.cmd.CommandText = "SELECT CARGO.NOME_CARGO FROM LOGIN INNER JOIN FUNCIONARIO ON FUNCIONARIO.CPF_FUNCIONARIO = LOGIN.CPF_FUNCIONARIO INNER JOIN CARGO ON CARGO.ID_CARGO = FUNCIONARIO.ID_CARGO WHERE LOGIN.USUARIO_LOGIN = '" + acesso.User + "' AND LOGIN.SENHA_LOGIN = '" + acesso.Password + "'";
conexao.conexao.Open();
return funcionarioDTO.Funcao = conexao.cmd.ExecuteScalar().ToString();
} finally {
conexao.conexao.Close();
}
}
Pronto, simples.
O código ainda não está bom. Certamente esse controle de conexão é algo mal feito e isto precisaria ser arrumado, mas não é o foco da pergunta. A arquitetura desta aplicação certamente é fonte de problemas que está tendo e terá no futuro, eu simplificaria isto.
A solução foi simples porque estava fazendo algo desnecessário aí. Pra que capturar uma exceção para fazer nada? Exceções existem para propagar o erro, só capture quando sabe o que fazer com ele e resolvê-lo adequadamente, não faz sentido capturar para fazer nada.
Eu preferia fazer assim que é mais simples de gerenciar:
public String IdentificarAcessoDAL(FuncionarioDTO acesso) {
using (var connection = new SqlConnection(connectionString)) {
var command = new SqlCommand("SELECT CARGO.NOME_CARGO FROM LOGIN INNER JOIN FUNCIONARIO ON FUNCIONARIO.CPF_FUNCIONARIO = LOGIN.CPF_FUNCIONARIO INNER JOIN CARGO ON CARGO.ID_CARGO = FUNCIONARIO.ID_CARGO WHERE LOGIN.USUARIO_LOGIN = '" + acesso.User + "' AND LOGIN.SENHA_LOGIN = '" + acesso.Password + "'", connection);
command.Connection.Open();
command.ExecuteNonQuery();
}
}
Coloquei no GitHub para referência futura.
Mentira, eu faria mais simples ainda, mas depende de uma arquitetura específica, não é o foco da pergunta e é algo amplo demais para por em uma resposta.
ATENÇÂO! Eu não consertei o grave problema de segurança que tem na aplicação, foquei só no problema específico. Veja o correto. Outro exemplexemploo.