Seu código tem alguns problemas. Primeiro, não misture lógica de visualização JOptionPane
com lógica de banco de dados.
Considerando que a sua classe ConnectionFactory
é a desta questão, ela têm sérios problemas no gerenciamento da conexão. Vou consertá-la com base no que fiz nessa outra resposta aqui e também nessa:
package dao;
import java.sql.DriverManager;
import java.sql.SQLException;
public final class ParametrosDeConexao {
private final String url;
private final String usuario;
private final String senha;
public ParametrosDeConexao(String url, String usuario, String senha) {
this.url = url;
this.usuario = usuario;
this.senha = senha;
}
public Connection conectar() throws SQLException {
return DriverManager.getConnection(url, usuario, senha);
}
}
Com isso, seu código fica que nem esse abaixo. Observe que o campo CONECTAR
representa o que era sua classe ConnectionFactory
:
private static final ParametrosDeConexao CONECTAR =
new ParametrosDeConexao(
"jdbc:postgresql://localhost:5432/topografiaJava",
"postgres",
"1");
private static final String LOGIN_SQL =
"SELECT login FROM pessoa WHERE login = ?";
public boolean validarLogin(String login) throws SQLException {
try (
Connection c = CONECTAR.conectar();
PreparedStatement ps = c.prepareStatement(LOGIN_SQL)
) {
ps.setString(1, login);
try (ResultSet rs = st.executeQuery()) {
return rs.next();
}
}
}
Observe como o método validarLogin(String)
ficou bastante enxuto. Para isso:
Lembre-se de usar o try-with-resources. A menos que você esteja trabalhando com o Java 6 ou anterior, NUNCA deixe de utilizá-lo com Connection
, PreparedStatement
ou ResultSet
.
Lembre-se de preencher os parâmetros do PreparedStatement
adequadamente (com o setString(1, login)
neste caso.
O return rs.next();
vai dar o true
ou false
que você quer dependendo se há ou não resultado no ResultSet
. No seu caso, é desnecessário ler esse resultado.
O Class.forName
não é mais necessário. Veja mais sobre isso aqui.