Estoy desarrollando mi primera aplicacion como proyecto de univ. usando MySQL con JAVA, me estoy dando cuenta que hay metodos y rutinas que no cierran las conexiones y los statements a pesar que le doy sus respectivas ordenes para que lo haga, algo debo hacer de manera correcta para que eso no suceda, que me sugieren?
ADVERTENCIA: ESTE CODIGO QUE AQUI SE MUESTRA CONTIENE ERRORES DE INSTRUCCIONES, PUEDES TOMARLO COMO EJEMPLO PARA SABER COMO CREAR UN LOGIN Y COMO REALIZAR DISTINTAS OPERACIONES CON UNA TABLA DE USUARIOS PERO LA CLASE DE CONEXION GENERA CONEXIONES DE MAS Y NO LAS CIERRA LEE BIEN EL POST COMPLETO PARA NO CAER EN EL MISMO ERROR QUE YO.
Esta es la clase que utilizo para conectarme.
package metodos;
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.SQLException;
import java.util.logging.Level;
import java.util.logging.Logger;
import javax.swing.JOptionPane;
public class Conexion {
private static final String DB = "sil";
private static final String DBuser = "root";
private static final String DBpwd = "****";
private static final String DBurl = "jdbc:mysql://localhost:3306/" + DB;
Connection Conector;
public Conexion() {
try {
Class.forName("com.mysql.jdbc.Driver");
Conector = DriverManager.getConnection(DBurl, DBuser, DBpwd);
} catch (ClassNotFoundException classNotFoundException) {
JOptionPane.showMessageDialog(null, "Error con el driver JDBC");
} catch (SQLException exception) {
try {
JOptionPane.showMessageDialog(null, "Error en la conexion");
Conector.close();
} catch (SQLException ex) {
Logger.getLogger(Conexion.class.getName()).log(Level.SEVERE, null, ex);
}
}
}
public Connection getConexion() {
return Conector;
}
/**
* ¿ES NECESARIO ESTE METODO?
* Este metodo cierra la conexion, para su uso
* el manejo de recursos del sistema.
*/
/*public void cierraConexion() {
try {
Conector.close();
} catch (Exception exception) {
// TODO: handle exception
System.out.println("Error cerrando la conexion");
}
}//Cierre del constructor*/
}//Fin de la clase Conexion
Y esta clase la que utiliza la conexion y genera conexiones en exceso.
package metodos;
import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.util.logging.Level;
import java.util.logging.Logger;
import javax.swing.JOptionPane;
public class OperacionesUsuarios {
Conexion Conecta = new Conexion();
Connection Conector;
public Object[][] AccedeUsuario(String login, String clave) {
Conector = Conecta.getConexion();
int registros = 0;
int usu_id;
try {
String Query = "SELECT count(1) as cont" + " FROM usuarios";
Statement St = Conector.createStatement();
/*
PreparedStatement pstm = Conn.prepareStatement(Query);
ResultSet Rs = pstm.executeQuery();
*/
try (ResultSet Rs = St.executeQuery(Query)) {
/*
PreparedStatement pstm = Conn.prepareStatement(Query);
ResultSet Rs = pstm.executeQuery();
*/
Rs.next();
registros = Rs.getInt("cont");
Rs.close();
}
} catch (SQLException sqle) {
JOptionPane.showMessageDialog(null, "Error en la consulta MySQL");
Logger.getLogger(OperacionesLicorerias.class.getName()).log(Level.SEVERE, null, sqle);
}
Object[][] data = new Object[registros][3];
if (login.length() != 0 && clave.length() != 0) {
String usu_login;
String usu_password;
try {
String Query = "SELECT * FROM sil.usuarios WHERE login = '"
+ login + "' AND clave = '" + clave + "'";
Statement St = Conector.createStatement();
/* PreparedStatement pstm = Conn.prepareStatement(Query);
try (ResultSet Rs = pstm.executeQuery()) {
*/
try (ResultSet Rs = St.executeQuery(Query)) {
int i = 0;
while (Rs.next()) {
usu_id = Rs.getInt("idUsuarios");
usu_login = Rs.getString("login");
usu_password = Rs.getString("clave");
data[i][0] = usu_id;
data[i][1] = usu_login;
data[i][2] = usu_password;
i++;
}
}
} catch (SQLException sqle) {
JOptionPane.showMessageDialog(null, "Error en la consulta MySQL");
Logger.getLogger(OperacionesLicorerias.class.getName()).log(Level.SEVERE, null, sqle);
} finally {
try {
if (Conector != null) {
Conector.close();
}
} catch (SQLException sqle) {
JOptionPane.showMessageDialog(null, "Error al cerrar la conexion");
Logger.getLogger(OperacionesLicorerias.class.getName()).log(Level.SEVERE, null, sqle);
}
}
}
return data;
}
public void RegistrandoUsuario(String nombreUsuarios, String login, String clave) {
Conector = Conecta.getConexion();
try {
String Query = "INSERT INTO `sil`.`usuarios` (`nombreUsuarios`,"
+ " `login`, `clave`) VALUES ('" + nombreUsuarios + "', '"
+ login + "', '" + clave + "');";
try (Statement St = Conector.createStatement()) {
St.executeUpdate(Query);
JOptionPane.showMessageDialog(null, "Datos ingresados correctamente!");
St.close();
Conector.close();
}
} catch (SQLException SQLe) {
JOptionPane.showMessageDialog(null, "Error en la consulta");
}
/*finally {
if (Conector != null) {
try {
Conector.close();
} catch (SQLException sqle) {
JOptionPane.showMessageDialog(null, "Error al cerrar la conexion");
Logger.getLogger(OperacionesLicorerias.class.getName()).log(Level.SEVERE, null, sqle);
}
}
}*/
}
public ResultSet BuscaUsuario(String login) {
Conector = Conecta.getConexion();
String Query = "SELECT * FROM sil.usuarios where login = ?";
Statement St = null;
ResultSet Rs = null;
PreparedStatement Pst = null;
try {
Pst = Conector.prepareStatement(Query);
Pst.setString(1, login);
Rs = Pst.executeQuery();
while (Rs.next()) {
JOptionPane.showMessageDialog(null, " | ID: " + Rs.getString("idUsuarios")
+ " | Login: " + Rs.getString("login")
+ " | Nombre: " + Rs.getString("nombreUsuarios"));
St.close();
Rs.close();
}*/
} catch (SQLException SQLe) {
JOptionPane.showMessageDialog(null, "Error en la consulta");
Logger.getLogger(OperacionesLicorerias.class.getName()).log(Level.SEVERE, null, SQLe);
}
return Rs;
}
public void BorraUsuario(String login) {
Conector = Conecta.getConexion();
try {
String Query = "DELETE FROM usuarios WHERE `login`='" + login + "';";
Statement St = Conector.createStatement();
St.executeUpdate(Query);
JOptionPane.showMessageDialog(null, "Registro borrado con exito!");
St.close();
} catch (SQLException ex) {
JOptionPane.showMessageDialog(null, "Error en la actualizacion de datos");
} finally {
if (Conector != null) {
try {
Conector.close();
} catch (SQLException sqle) {
JOptionPane.showMessageDialog(null, "Error al cerrar la conexion");
Logger.getLogger(OperacionesLicorerias.class.getName()).log(Level.SEVERE, null, sqle);
}
}
}
}
}
Ya anoche pude resolver el problema, segun podran observar en mi codigo, aparentemente estoy construyendo un objeto o instanciando una clase, que ya ejecuta la conexion, y ademas que creo un objeto de tipo Connection al cual que le digo que ejecute el metodo .getConnection, modifique el codigo y elimine el metodo getConnection() de mi clase Conexion, añadi a mi metodo Conexion el tipo de Connection y por ahi fueron los tiros.
Para darme cuenta en donde estaba mi error, tuve que ir comentando las instrucciones donde habia metodos de conexion, paso por paso probando el proyecto e ir descartando que era lo que me generaba conexiones de mas que no se cerraban, y pude determinar que el problema estaba en mi clase Conexion.
Si hay alguna informacion que alguien quiera preguntarme seria perfecto, como ven estoy aprendiendo por mi cuenta y aun me falta manejar muchos temas.