Skip to main content
conteúdo editado
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
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.

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 exemplo.

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 exemplo.

adicionou 143 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
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 exemplo.

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();
    }
}

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 exemplo.

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 exemplo.

Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

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();
    }
}

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 exemplo.