我有一个名为ConnectionDerby的类,在该类中我设置了连接到嵌入式数据库(Derby)的参数,我想减少连接的冗余,实际方法和连接工作,但我想增加性能,所以类的代码是这样的:
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import javax.swing.JOptionPane;
public class ConnectionDerby {
private Connection conn = null;
private Statement sttm = null;
public Connection CrearBD(String query) {
try {
//Obtenemos el Driver de Derby
Class.forName("org.apache.derby.jdbc.EmbeddedDriver");
conn = DriverManager.getConnection("jdbc:derby:.\\BD\\inventario.db;create=true");
if (conn != null) {
//JOptionPane.showMessageDialog(null, "Base de Datos Lista");
try {
PreparedStatement pstm = conn.prepareStatement(query);
pstm.execute();
pstm.close();
//JOptionPane.showMessageDialog(null, "Base de Datos Creada Correctamente");
System.out.println("SENTENCIA SQL EFECTUADA CORRECTAMENTE");
} catch (SQLException ex) {
//JOptionPane.showMessageDialog(null, ex.getLocalizedMessage());
System.out.println(ex.getMessage());
JOptionPane.showMessageDialog(null, "NO SE PUDO EFECTUAR LA SENTENCIA SQL", "Error", JOptionPane.ERROR_MESSAGE);
//JOptionPane.showMessageDialog(null, "NO SE PUDO EFECTUAR LA SENTENCIA SQL");
}
}
} catch (SQLException e) {
System.out.println(e.getMessage());
JOptionPane.showMessageDialog(null, "NO SE PUDO EFECTUAR LA SENTENCIA SQL", "Error", JOptionPane.ERROR_MESSAGE);
//JOptionPane.showMessageDialog(null, "TRONO LA APLICACION EN EJECUTAR LAS SENTENCIAS SQL parte 2");
} catch (ClassNotFoundException e) {
System.out.println(e.getMessage());
JOptionPane.showMessageDialog(null, "NO SE PUDO EFECTUAR LA SENTENCIA SQL", "Error", JOptionPane.ERROR_MESSAGE);
//JOptionPane.showMessageDialog(null, "TRONO LA APLICACION EN EJECUTAR LAS SENTENCIAS SQL parte 3");
}
return conn;
}
public Connection AccederBD() {
try {
//Obtenemos el Driver de Derby
Class.forName("org.apache.derby.jdbc.EmbeddedDriver");
//Obtenemos la Conexión
conn = DriverManager.getConnection("jdbc:derby:.\\BD\\inventario.db");
if (conn != null) {
System.out.println("Base de Datos Ya Leida Correctamente");
//JOptionPane.showMessageDialog(null, "Base de Datos Ya Leida Correctamente");
}
} catch (SQLException e) {
System.out.println(e.getMessage());
System.out.println("NO SE ENCONTRO LA BASE DE DATOS");
System.out.println("CREANDO BASE DE DATOS EN DERBY DATABASE");
String createTableProyecto = "CREATE TABLE proyecto\n"
+ "(\n"
+ "idproyecto INTEGER NOT NULL GENERATED ALWAYS AS IDENTITY (START WITH 1, INCREMENT BY 1),\n"
+ "nombre VARCHAR(50) NOT NULL,\n"
+ "descripcion VARCHAR(1000),\n"
+ "CONSTRAINT proyecto_pk PRIMARY KEY (idproyecto)\n"
+ ")";
String createTablePrimera = "CREATE TABLE primera\n"
+ "(\n"
+ "idprimera INTEGER NOT NULL GENERATED ALWAYS AS IDENTITY (START WITH 1, INCREMENT BY 1),\n"
+ "nombre VARCHAR(75) NOT NULL,\n"
+ "descripcion VARCHAR(1000),\n"
+ "CONSTRAINT primera_pk PRIMARY KEY (idprimera)\n"
+ ") ";
String createTableSegunda = "CREATE TABLE segunda\n"
+ "(\n"
+ "idsegunda INTEGER NOT NULL GENERATED ALWAYS AS IDENTITY (START WITH 1, INCREMENT BY 1),\n"
+ "nombre VARCHAR(75) NOT NULL,\n"
+ "descripcion VARCHAR(1000),\n"
+ "CONSTRAINT segunda_pk PRIMARY KEY (idsegunda)\n"
+ ")";
String createTableEncabezado = "CREATE TABLE encabezado\n"
+ "(\n"
+ "idencabezado INTEGER NOT NULL GENERATED ALWAYS AS IDENTITY (START WITH 1, INCREMENT BY 1),\n"
+ "idproyecto INTEGER NOT NULL,\n"
+ "idprimera INTEGER NOT NULL,\n"
+ "idsegunda INTEGER NOT NULL,\n"
+ "fecha DATE NOT NULL,\n"
+ "CONSTRAINT encabezado_pk PRIMARY KEY (idencabezado)\n"
+ ")";
String createRelationCotizacionIdTopCoat = "ALTER TABLE ENCABEZADO ADD FOREIGN KEY (IDSEGUNDA) REFERENCES SEGUNDA(IDSEGUNDA)";
String createRelationCotizacionIdPrimer = "ALTER TABLE ENCABEZADO ADD FOREIGN KEY (IDPRIMERA) REFERENCES PRIMERA(IDPRIMERA)";
String createRelationCotizacionIdProyecto = "ALTER TABLE ENCABEZADO ADD FOREIGN KEY (IDPROYECTO) REFERENCES PROYECTO(IDPROYECTO)";
CrearBD(createTableProyecto);
CrearBD(createTablePrimera);
CrearBD(createTableSegunda);
CrearBD(createTableEncabezado);
CrearBD(createRelationCotizacionIdTopCoat);
CrearBD(createRelationCotizacionIdPrimer);
CrearBD(createRelationCotizacionIdProyecto);
//*************PRUEBAS*****************
} catch (ClassNotFoundException e) {
System.out.println(e.getMessage());
System.out.println("ERROR DE TIPO ClassNotFoundException");
//JOptionPane.showMessageDialog(null, "TRONO LA APLICACION EN ACCEDER A LA BASE DE DATOS parte 2");
}
return conn;
}
public void UID(String sqlcad) {
try {
//Obtenemos el Driver de Derby
Class.forName("org.apache.derby.jdbc.EmbeddedDriver");
conn = DriverManager.getConnection("jdbc:derby:.\\BD\\inventario.db");
sttm = conn.createStatement(ResultSet.TYPE_SCROLL_SENSITIVE, ResultSet.CONCUR_READ_ONLY);
sttm.executeUpdate(sqlcad);
System.out.println("Conexión Exitosa a la Base de Datos");
//JOptionPane.showMessageDialog(null, "Conexión exitosa");
sttm.close();
conn.close();
if (conn != null) {
System.out.println("Consulta Realizada Correctamente");
//JOptionPane.showMessageDialog(null, "Base de Datos Ya Leida Correctamente");
}
} catch (SQLException e) {
System.out.println("Error= " + e.getMessage());
} catch (ClassNotFoundException e) {
System.out.println("Error= " + e.getMessage());
}
}
public ResultSet getvalores(String sqlcad) {
ResultSet rs = null;
try {
Class.forName("org.apache.derby.jdbc.EmbeddedDriver");
conn = DriverManager.getConnection("jdbc:derby:.\\BD\\inventario.db");
sttm = conn.createStatement(ResultSet.TYPE_SCROLL_SENSITIVE, ResultSet.CONCUR_READ_ONLY);
//String sqlcad = "Select nombre, descripcion from primera";
rs = sttm.executeQuery(sqlcad);
return rs;
} catch (Exception e) {
System.out.println("Error= " + e.getMessage());
return rs;
}
}
}
我使用另一个类来设置查询查询的字符串,如下所示:
import java.sql.ResultSet;
public class Primer {
ConnectionDerby c = new ConnectionDerby();
public void insertMaestroPrimera(String nombre, String descripcion) {
c.UID("INSERT INTO primera(nombre, descripcion) values('" + nombre + "','" + descripcion + "')");
}
public void updateMaestroPrimera(int id, String nombre, String descripcion) {
c.UID("UPDATE primera set nombre='" + nombre + "',descripcion='" + descripcion + "' where idprimera=" + id);
}
public void deleteMaestroPrimer(int id) {
c.UID("DELETE FROM primera where idprimera=" + id);
}
public ResultSet llenarTable() {
ResultSet rs = null;
String sql = "SELECT nombre, descripcion from primera ORDER BY nombre";
rs = c.getvalores(sql);
return rs;
}
public ResultSet obtenerDatos(String value) {
ResultSet rs = null;
String sql = "SELECT idprimera, nombre, descripcion from primera WHERE nombre = '" + value + "'";
rs = c.getvalores(sql);
return rs;
}
}
所以我想要一些建议,因为当我在表encabezado中进行更新时,我得到了以下错误:“Error = Statement.executeUpdate()无法使用返回ResultSet的语句调用。”和“错误=在请求的时间内无法获得锁定”。我认为这是因为太多的池连接,所以请任何人可以帮助我提高性能或者sql是错的我不知道,但其他表(primera,segunda,proyecto)中的CRUD工作得很好!谢谢你顺便的时间!
答案 0 :(得分:2)
您当前的设计存在两个大问题:
Connection
变量必须在本地声明每种方法。您将Connection
声明为字段,因此如果访问同一时间执行不同操作的同一ConnectionDerby
对象实例的两个线程将陷入竞争状态,则会出现奇怪的问题,例如在此处发布错误消息:错误= Statement.executeUpdate()无法使用返回ResultSet的语句调用,这意味着线程A尝试执行更新,而线程B尝试执行读取操作,两者都在同时,在同一个实例上,竞争条件混合了Connection
和Statement
个实例。
您正在手动打开连接。这是一个坏主意,因为打开连接在性能方面成本很高。您应该使用数据库连接池。
说完这些之后,您可以通过以下方式提高性能并解决项目中的问题:
Connection
课程中删除Statement
和ConnectionDerby
字段,而在每个方法中添加新的Connection
和Statement
个变量。使用它们后,请记得始终关闭资源。更多信息:
可能你没有使用JBoss,但答案中的概念也适用于这种情况。