se ejecuto limpieza del codigo, reformateo del codigo y optimizacion de los imports por parte del IDE

src/danielcortes/xyz/controllers/ArqueoController.java

@@ -37,10 +37,8 @@ import danielcortes.xyz.views.ArqueoView;
 import danielcortes.xyz.views.CalcularFondoView;
 import danielcortes.xyz.views.components.NumberFormatedTextField;
 
-import javax.swing.AbstractAction;
+import javax.swing.*;
-import javax.swing.JComponent;
+import java.awt.*;
-import javax.swing.KeyStroke;
-import java.awt.Color;
 import java.awt.event.ActionEvent;
 
 /**

src/danielcortes/xyz/controllers/EgresosController.java

@@ -34,7 +34,9 @@ import danielcortes.xyz.views.EgresosView;
 import danielcortes.xyz.views.components.EgresosTableModel;
 
 import javax.swing.*;
-import java.awt.event.*;
+import java.awt.event.ActionEvent;
+import java.awt.event.MouseAdapter;
+import java.awt.event.MouseEvent;
 
 /**
  * Controlador el cual esta orientado a manejar la vista de EgresosView
@@ -68,6 +70,7 @@ public class EgresosController {
 
     /**
      * Getter!
+     *
      * @return
      */
     public EgresoDAO getEgresoDAO() {
@@ -76,6 +79,7 @@ public class EgresosController {
 
     /**
      * Getter
+     *
      * @return
      */
     public TipoEgresoDAO getTipoEgresoDAO() {
@@ -151,7 +155,6 @@ public class EgresosController {
      * Primero llama a normalizar los inputs y a ocultar los mensajes de error
      * Luego si es que esta colocada la flag de editing se llama al metodo editarEgreso y si no, se llama a guardarEgreso
      * Al terminar esto, se llama a resetear el focus en los inputs y a actualizar el total de egresos
-     *
      */
     private void guardarActionListener() {
         this.normalizeInputs();
@@ -274,6 +277,7 @@ public class EgresosController {
 
     /**
      * llama a los metodos necesarios para validar los inputs entregados
+     *
      * @return true cuando todas las validaciones retoran true, si no, false
      */
     private boolean validateInput(String nro, String descripcion, TipoEgreso tipoEgreso, Caja caja) {
@@ -291,6 +295,7 @@ public class EgresosController {
      * - Es null
      * - Esta vacio
      * Cuando el primer caso sea true, colocara un mensaje de error correspondiente en el jlabel correspondiente
+     *
      * @return Si cualquiera de estos casos son true se retornara false, si no, se retorna true
      */
     private boolean validateNro(String nro) {
@@ -315,6 +320,7 @@ public class EgresosController {
      * - Es null
      * - Esta vacio
      * Cuando el primer caso sea true, colocara un mensaje de error correspondiente en el jlabel correspondiente
+     *
      * @return Si cualquiera de estos casos son true se retornara false, si no, se retorna true
      */
     private boolean validateDescripcion(String descripcion) {
@@ -336,6 +342,7 @@ public class EgresosController {
      * Valida la variable tipoEgreso contra los casos
      * - Es null
      * Cuando este caso sea true, colocara un mensaje de error correspondiente en el jlabel correspondiente
+     *
      * @return Si este caso es  true se retornara false, si no, se retorna true
      */
     private boolean validateTipoEgreso(TipoEgreso tipoEgreso) {
@@ -350,6 +357,7 @@ public class EgresosController {
     /**
      * Valida la variable caja contra los casos
      * - Es null
+     *
      * @return Si este caso es  true se retornara false, si no, se retorna true
      */
     private boolean validateCaja(Caja caja) {
@@ -394,6 +402,7 @@ public class EgresosController {
 
     private class GuardarAction extends AbstractAction {
         EgresosController controller;
+
         GuardarAction(EgresosController controller) {
             this.controller = controller;
         }

src/danielcortes/xyz/controllers/InformesController.java

@@ -28,8 +28,8 @@ import danielcortes.xyz.informes.InformeEgresos;
 import danielcortes.xyz.informes.InformeLibroDeVentas;
 import danielcortes.xyz.models.tipo_egreso.TipoEgreso;
 import danielcortes.xyz.utils.StringUtils;
-import danielcortes.xyz.views.MonthSelectDialog;
 import danielcortes.xyz.views.InformesView;
+import danielcortes.xyz.views.MonthSelectDialog;
 import danielcortes.xyz.views.TipoEgresoSelectDialog;
 
 import javax.swing.*;

src/danielcortes/xyz/controllers/IngresosController.java

@@ -33,7 +33,9 @@ import danielcortes.xyz.views.IngresosView;
 import danielcortes.xyz.views.components.IngresosTableModel;
 
 import javax.swing.*;
-import java.awt.event.*;
+import java.awt.event.ActionEvent;
+import java.awt.event.MouseAdapter;
+import java.awt.event.MouseEvent;
 
 /**
  * Controlador el cual esta orientado a manejar la vista de IngresosView
@@ -74,6 +76,7 @@ public class IngresosController {
 
     /**
      * Getter!!!
+     *
      * @return
      */
     public TipoIngresoDAO getTipoIngresoDAO() {
@@ -265,7 +268,6 @@ public class IngresosController {
     /**
      * Edita el ingreso tras llamar a validar el input
      * Tras esto actualiza el ingreso en la tabla, llama a actualizar el total de ingresos, a limpiar los campos de input y a desactivar la flag de editing.
-     *
      */
     private void editarIngreso(int valor, String nroZInicial, String nroZFinal, String nroInicial, String nroFinal, TipoIngreso tipoIngreso, Caja caja) {
         if (this.validateInput(nroZInicial, nroZFinal, nroInicial, nroFinal, tipoIngreso, caja)) {
@@ -285,6 +287,7 @@ public class IngresosController {
 
     /**
      * Llama a los metodos necesarios para validar el input
+     *
      * @return true cuando todas las validaciones retoran true, si no, false
      */
     private boolean validateInput(String nroZInicial, String nroZFinal, String nroInicial, String nroFinal, TipoIngreso tipoIngreso, Caja caja) {
@@ -301,6 +304,7 @@ public class IngresosController {
     /**
      * Valida la variable caja este caso
      * - Es null
+     *
      * @return Si este caso es true se retornara false, si no, se retorna true
      */
     private boolean validateCaja(Caja caja) {
@@ -312,6 +316,7 @@ public class IngresosController {
      * - Es null
      * - Esta vacio
      * Cuando el primer caso sea true, colocara un mensaje de error correspondiente en el jlabel correspondiente
+     *
      * @return Si cualquiera de estos casos son true se retornara false, si no, se retorna true
      */
     private boolean validateNroInicial(String nroInicial) {
@@ -334,6 +339,7 @@ public class IngresosController {
      * - Es null
      * - Esta vacio
      * Cuando el primer caso sea true, colocara un mensaje de error correspondiente en el jlabel correspondiente
+     *
      * @return Si cualquiera de estos casos son true se retornara false, si no, se retorna true
      */
     private boolean validateNroFinal(String nroFinal) {
@@ -355,6 +361,7 @@ public class IngresosController {
      * Valida la variable caja este caso
      * - Es null
      * Cuando sea true, colocara un mensaje de error en el jlabel correspondiente
+     *
      * @return Si este caso es true se retornara false, si no, se retorna true
      */
     private boolean validateTipoIngreso(TipoIngreso tipoIngreso) {
@@ -420,6 +427,7 @@ public class IngresosController {
 
     private class GuardarAction extends AbstractAction {
         IngresosController controller;
+
         GuardarAction(IngresosController controller) {
             this.controller = controller;
         }

src/danielcortes/xyz/controllers/ManagerController.java

@@ -63,7 +63,7 @@ public class ManagerController {
      * Crea el controlador
      * Necesita todos las interfaces DAO para poder asignarselos a sus vistas,
      * esto con el objetivo que sean facilmente intercambiables.
-     *
+     * <p>
      * Llama a los metodos que:
      * - Cargan el contenido del CardLayout
      * - Selecciona una fecha inicial

src/danielcortes/xyz/data/ConnectionHolder.java

@@ -27,5 +27,5 @@ package danielcortes.xyz.data;
 import java.sql.SQLException;
 
 public interface ConnectionHolder {
-    public java.sql.Connection getConnection() throws SQLException;
+    java.sql.Connection getConnection() throws SQLException;
 }

src/danielcortes/xyz/informes/InformeEgresos.java

@@ -27,22 +27,22 @@ package danielcortes.xyz.informes;
 import danielcortes.xyz.models.caja.SQLiteCajaDAO;
 import danielcortes.xyz.models.informes.egresos.InformeEgresosContent;
 import danielcortes.xyz.models.informes.egresos.SQLiteInformeEgresosContentDAO;
-import danielcortes.xyz.models.tipo_egreso.TipoEgreso;
 import danielcortes.xyz.utils.Pair;
 import org.apache.poi.hssf.usermodel.HSSFWorkbook;
 import org.apache.poi.ss.usermodel.*;
 import org.apache.poi.ss.util.CellRangeAddress;
 import org.apache.poi.ss.util.PropertyTemplate;
 
-import java.io.File;
-import java.io.FileOutputStream;
 import java.io.IOException;
 import java.io.OutputStream;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.time.LocalDate;
 import java.time.ZoneId;
-import java.util.*;
+import java.util.ArrayList;
+import java.util.Date;
+import java.util.HashMap;
+import java.util.List;
 
 public class InformeEgresos {
     private final String[] titles = {

src/danielcortes/xyz/informes/InformeLibroDeVentas.java

@@ -32,8 +32,6 @@ import org.apache.poi.ss.usermodel.*;
 import org.apache.poi.ss.util.CellRangeAddress;
 import org.apache.poi.ss.util.PropertyTemplate;
 
-import java.io.File;
-import java.io.FileOutputStream;
 import java.io.IOException;
 import java.io.OutputStream;
 import java.nio.file.Files;

src/danielcortes/xyz/models/caja/CajaDAO.java

@@ -25,7 +25,6 @@
 package danielcortes.xyz.models.caja;
 
 import danielcortes.xyz.data.ConnectionHolder;
-import danielcortes.xyz.data.SQLiteConnectionHolder;
 
 import java.sql.ResultSet;
 import java.sql.SQLException;
@@ -37,9 +36,13 @@ public abstract class CajaDAO {
     protected ConnectionHolder connectionHolder;
 
     public abstract List<Caja> findAll();
+
     public abstract Caja findById(int id);
+
     public abstract Caja findByFecha(LocalDate fecha);
+
     public abstract boolean insertCaja(Caja caja);
+
     public abstract boolean updateCaja(Caja caja);
 
     public abstract void createCajasForMonth(LocalDate month);

src/danielcortes/xyz/models/caja/SQLiteCajaDAO.java

@@ -48,8 +48,7 @@ public class SQLiteCajaDAO extends CajaDAO {
     @Override
     public List<Caja> findAll() {
         List<Caja> cajaList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from caja");
             ResultSet rs = ps.executeQuery();
 
@@ -57,18 +56,17 @@ public class SQLiteCajaDAO extends CajaDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }
+
         return cajaList;
     }
 
     @Override
     public Caja findById(int id) {
         Caja caja = null;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from caja where id = ?");
 
             ps.setInt(1, id);
@@ -79,7 +77,6 @@ public class SQLiteCajaDAO extends CajaDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }
@@ -89,8 +86,7 @@ public class SQLiteCajaDAO extends CajaDAO {
     @Override
     public Caja findByFecha(LocalDate fecha) {
         Caja caja = null;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from caja where fecha = ?");
 
             ps.setString(1, fecha.toString());
@@ -105,7 +101,6 @@ public class SQLiteCajaDAO extends CajaDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }
@@ -115,8 +110,7 @@ public class SQLiteCajaDAO extends CajaDAO {
     @Override
     public boolean insertCaja(Caja caja) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("insert into caja (fecha) values (?)");
 
             ps.setString(1, caja.getFecha().toString());
@@ -131,7 +125,6 @@ public class SQLiteCajaDAO extends CajaDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;
@@ -142,8 +135,7 @@ public class SQLiteCajaDAO extends CajaDAO {
     @Override
     public boolean updateCaja(Caja caja) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("update caja set fecha = ? where id = ?");
             ps.setString(1, caja.getFecha().toString());
             ps.setInt(2, caja.getId());
@@ -151,7 +143,6 @@ public class SQLiteCajaDAO extends CajaDAO {
             updates = ps.executeUpdate();
 
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;

src/danielcortes/xyz/models/calculo_fondo/CalculoFondoDAO.java

@@ -37,11 +37,15 @@ public abstract class CalculoFondoDAO {
     protected ConnectionHolder connectionHolder;
 
     public abstract List<CalculoFondo> findAll();
+
     public abstract List<CalculoFondo> findByCaja(Caja caja);
+
     public abstract CalculoFondo findById(int id);
 
     public abstract boolean insertCalculoFondo(CalculoFondo calculoFondo);
+
     public abstract boolean updateCalculoFondo(CalculoFondo calculoFondo);
+
     public abstract boolean deleteCalculoFondo(CalculoFondo calculoFondo);
 
     public abstract int getTotalCalculoFondo(Caja caja);

src/danielcortes/xyz/models/calculo_fondo/SQLiteCalculoFondoDAO.java

@@ -42,8 +42,7 @@ public class SQLiteCalculoFondoDAO extends CalculoFondoDAO {
     @Override
     public List<CalculoFondo> findAll() {
         List<CalculoFondo> calculoFondoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from calculo_fondo");
             ResultSet rs = ps.executeQuery();
 
@@ -51,17 +50,16 @@ public class SQLiteCalculoFondoDAO extends CalculoFondoDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }
         return calculoFondoList;
     }
+
     @Override
     public List<CalculoFondo> findByCaja(Caja caja) {
         List<CalculoFondo> calculoFondoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from calculo_fondo where caja_id = ?");
             ps.setInt(1, caja.getId());
 
@@ -71,7 +69,6 @@ public class SQLiteCalculoFondoDAO extends CalculoFondoDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }
@@ -81,8 +78,7 @@ public class SQLiteCalculoFondoDAO extends CalculoFondoDAO {
     @Override
     public CalculoFondo findById(int id) {
         CalculoFondo calculoFondo = null;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from calculo_fondo where id = ?");
             ps.setInt(1, id);
             ResultSet rs = ps.executeQuery();
@@ -91,7 +87,6 @@ public class SQLiteCalculoFondoDAO extends CalculoFondoDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }
@@ -101,8 +96,7 @@ public class SQLiteCalculoFondoDAO extends CalculoFondoDAO {
     @Override
     public boolean insertCalculoFondo(CalculoFondo calculoFondo) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("insert into calculo_fondo (valor, descripcion, caja_id) values (?, ?, ?)");
             ps.setInt(1, calculoFondo.getValor());
             ps.setString(2, calculoFondo.getDescripcion());
@@ -118,7 +112,6 @@ public class SQLiteCalculoFondoDAO extends CalculoFondoDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;
@@ -129,8 +122,7 @@ public class SQLiteCalculoFondoDAO extends CalculoFondoDAO {
     @Override
     public boolean updateCalculoFondo(CalculoFondo calculoFondo) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("update calculo_fondo set valor = ?, descripcion = ?, caja_id = ?  where id = ?");
             ps.setInt(1, calculoFondo.getValor());
             ps.setString(2, calculoFondo.getDescripcion());
@@ -140,7 +132,6 @@ public class SQLiteCalculoFondoDAO extends CalculoFondoDAO {
             updates = ps.executeUpdate();
 
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;
@@ -151,15 +142,13 @@ public class SQLiteCalculoFondoDAO extends CalculoFondoDAO {
     @Override
     public boolean deleteCalculoFondo(CalculoFondo calculoFondo) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("delete from calculo_fondo where id = ?");
             ps.setInt(1, calculoFondo.getId());
 
             updates = ps.executeUpdate();
 
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;
@@ -170,8 +159,7 @@ public class SQLiteCalculoFondoDAO extends CalculoFondoDAO {
     @Override
     public int getTotalCalculoFondo(Caja caja) {
         int sum = 0;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select sum(valor) from calculo_fondo where caja_id = ?");
             ps.setInt(1, caja.getId());
 
@@ -180,7 +168,6 @@ public class SQLiteCalculoFondoDAO extends CalculoFondoDAO {
             sum = rs.getInt(1);
 
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }

src/danielcortes/xyz/models/documentos/DocumentosDAO.java

@@ -38,12 +38,17 @@ public abstract class DocumentosDAO {
     protected ConnectionHolder connectionHolder;
 
     public abstract List<Documentos> findAll();
+
     public abstract Documentos findById(int id);
+
     public abstract Documentos findByCaja(Caja caja);
 
     public abstract boolean insertDocumentos(Documentos documentos);
+
     public abstract boolean insertDefaultDocumentos(Documentos documentos);
+
     public abstract boolean updateDocumentos(Documentos documentos);
+
     public abstract boolean deleteDocumentos(Documentos documentos);
 
     public abstract int getTotalDocumentos(Caja caja);

src/danielcortes/xyz/models/documentos/SQLiteDocumentosDAO.java

@@ -24,11 +24,8 @@
 
 package danielcortes.xyz.models.documentos;
 
-import danielcortes.xyz.data.ConnectionHolder;
 import danielcortes.xyz.data.SQLiteConnectionHolder;
 import danielcortes.xyz.models.caja.Caja;
-import danielcortes.xyz.models.caja.CajaDAO;
-import danielcortes.xyz.models.caja.SQLiteCajaDAO;
 
 import java.sql.Connection;
 import java.sql.PreparedStatement;
@@ -45,8 +42,7 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
     @Override
     public List<Documentos> findAll() {
         List<Documentos> documentosList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from documentos");
             ResultSet rs = ps.executeQuery();
 
@@ -54,7 +50,6 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }
@@ -64,8 +59,7 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
     @Override
     public Documentos findById(int id) {
         Documentos documentos = null;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from documentos where id = ?");
             ps.setInt(1, id);
 
@@ -78,7 +72,6 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }
@@ -88,8 +81,7 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
     @Override
     public Documentos findByCaja(Caja caja) {
         Documentos documentos = null;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from documentos where caja_id = ?");
             ps.setInt(1, caja.getId());
 
@@ -102,7 +94,6 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }
@@ -112,8 +103,7 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
     @Override
     public boolean insertDocumentos(Documentos documentos) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("insert into documentos (cheques, tarjetas, retiros, caja_id) values (?,?,?,?)");
             ps.setInt(1, documentos.getCheques());
             ps.setInt(2, documentos.getTarjetas());
@@ -130,7 +120,6 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;
@@ -141,8 +130,7 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
     @Override
     public boolean insertDefaultDocumentos(Documentos documentos) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("insert into documentos (cheques, tarjetas, retiros, caja_id) values (0,0,0,?)");
             ps.setInt(1, documentos.getCaja().getId());
 
@@ -156,7 +144,6 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;
@@ -167,8 +154,7 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
     @Override
     public boolean updateDocumentos(Documentos documentos) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("update documentos set tarjetas = ?, cheques = ?, retiros = ?, caja_id = ? where id = ?");
             ps.setInt(1, documentos.getTarjetas());
             ps.setInt(2, documentos.getCheques());
@@ -178,7 +164,6 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
             updates = ps.executeUpdate();
 
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;
@@ -189,15 +174,13 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
     @Override
     public boolean deleteDocumentos(Documentos documentos) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("delete from documentos where id = ?");
             ps.setInt(1, documentos.getId());
 
             updates = ps.executeUpdate();
 
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;
@@ -208,8 +191,7 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
     @Override
     public int getTotalDocumentos(Caja caja) {
         int total = 0;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select cheques + tarjetas + retiros from documentos where caja_id = ?");
             ps.setInt(1, caja.getId());
 
@@ -219,7 +201,6 @@ public class SQLiteDocumentosDAO extends DocumentosDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }

src/danielcortes/xyz/models/efectivo/EfectivoDAO.java

@@ -38,12 +38,17 @@ public abstract class EfectivoDAO {
     protected ConnectionHolder connectionHolder;
 
     public abstract List<Efectivo> findAll();
+
     public abstract Efectivo findById(int id);
+
     public abstract Efectivo findByCaja(Caja caja);
 
     public abstract boolean insertEfectivo(Efectivo efectivo);
+
     public abstract boolean insertDefaultEfectivo(Efectivo efectivo);
+
     public abstract boolean updateEfectivo(Efectivo efectivo);
+
     public abstract boolean deleteEfectivo(Efectivo efectivo);
 
     public abstract int getTotalEfectivo(Caja caja);

src/danielcortes/xyz/models/efectivo/SQLiteEfectivoDAO.java

@@ -24,11 +24,8 @@
 
 package danielcortes.xyz.models.efectivo;
 
-import danielcortes.xyz.data.ConnectionHolder;
 import danielcortes.xyz.data.SQLiteConnectionHolder;
 import danielcortes.xyz.models.caja.Caja;
-import danielcortes.xyz.models.caja.CajaDAO;
-import danielcortes.xyz.models.caja.SQLiteCajaDAO;
 
 import java.sql.Connection;
 import java.sql.PreparedStatement;
@@ -45,8 +42,7 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
     @Override
     public List<Efectivo> findAll() {
         List<Efectivo> efectivoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from efectivos");
             ResultSet rs = ps.executeQuery();
 
@@ -54,7 +50,6 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }
@@ -65,8 +60,7 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
     @Override
     public Efectivo findById(int id) {
         Efectivo efectivo = null;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from efectivos where id = ?");
             ps.setInt(1, id);
             ResultSet rs = ps.executeQuery();
@@ -75,7 +69,6 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }
@@ -86,8 +79,7 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
     @Override
     public Efectivo findByCaja(Caja caja) {
         Efectivo efectivo = null;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from efectivos where caja_id = ?");
             ps.setInt(1, caja.getId());
             ResultSet rs = ps.executeQuery();
@@ -99,7 +91,6 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }
@@ -110,8 +101,7 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
     @Override
     public boolean insertEfectivo(Efectivo efectivo) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("insert into efectivos (veinte_mil, diez_mil, cinco_mil, dos_mil, mil, quinientos, cien, cincuenta, diez, caja_id) values (?,?,?,?,?,?,?,?,?,?)");
             ps.setInt(1, efectivo.getVeinteMil());
             ps.setInt(2, efectivo.getDiezMil());
@@ -134,7 +124,6 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;
@@ -145,8 +134,7 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
     @Override
     public boolean insertDefaultEfectivo(Efectivo efectivo) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("insert into efectivos (veinte_mil, diez_mil, cinco_mil, dos_mil, mil, quinientos, cien, cincuenta, diez, caja_id) values (0,0,0,0,0,0,0,0,0,?)");
             ps.setInt(1, efectivo.getCaja().getId());
 
@@ -160,7 +148,6 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;
@@ -171,8 +158,7 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
     @Override
     public boolean updateEfectivo(Efectivo efectivo) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("update efectivos set veinte_mil = ?, diez_mil = ?, cinco_mil = ?, dos_mil = ?, mil = ?, quinientos = ?, cien = ?, cincuenta = ?, diez = ?, caja_id = ? where id = ?");
             ps.setInt(1, efectivo.getVeinteMil());
             ps.setInt(2, efectivo.getDiezMil());
@@ -189,7 +175,6 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
             updates = ps.executeUpdate();
 
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;
@@ -200,15 +185,13 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
     @Override
     public boolean deleteEfectivo(Efectivo efectivo) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("delete from efectivos where id = ?");
             ps.setInt(1, efectivo.getId());
 
             updates = ps.executeUpdate();
 
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
             return false;
@@ -219,8 +202,7 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
     @Override
     public int getTotalEfectivo(Caja caja) {
         int total = 0;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select veinte_mil + diez_mil + cinco_mil + dos_mil + mil + quinientos + cien + cincuenta + diez from efectivos where caja_id = ?");
             ps.setInt(1, caja.getId());
 
@@ -230,7 +212,6 @@ public class SQLiteEfectivoDAO extends EfectivoDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }

src/danielcortes/xyz/models/egreso/EgresoDAO.java

@@ -41,13 +41,19 @@ public abstract class EgresoDAO {
     protected ConnectionHolder connectionHolder;
 
     public abstract List<Egreso> findAll();
+
     public abstract List<Egreso> findById(int id);
+
     public abstract List<Egreso> findByCaja(Caja caja);
+
     public abstract List<Egreso> findByNro(String nro);
+
     public abstract List<Egreso> findByTipoEgreso(TipoEgreso tipoEgreso);
 
     public abstract boolean insertEgreso(Egreso egreso);
+
     public abstract boolean updateEgreso(Egreso egreso);
+
     public abstract boolean deleteEgreso(Egreso egreso);
 
     public abstract int getTotalEgreso(Caja caja);

src/danielcortes/xyz/models/egreso/SQLiteEgresoDAO.java

@@ -24,14 +24,9 @@
 
 package danielcortes.xyz.models.egreso;
 
-import danielcortes.xyz.data.ConnectionHolder;
 import danielcortes.xyz.data.SQLiteConnectionHolder;
 import danielcortes.xyz.models.caja.Caja;
-import danielcortes.xyz.models.caja.CajaDAO;
-import danielcortes.xyz.models.caja.SQLiteCajaDAO;
-import danielcortes.xyz.models.tipo_egreso.SQLiteTipoEgresoDAO;
 import danielcortes.xyz.models.tipo_egreso.TipoEgreso;
-import danielcortes.xyz.models.tipo_egreso.TipoEgresoDAO;
 
 import java.sql.Connection;
 import java.sql.PreparedStatement;
@@ -48,8 +43,7 @@ public class SQLiteEgresoDAO extends EgresoDAO {
     @Override
     public List<Egreso> findAll() {
         List<Egreso> egresoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from egresos");
             ResultSet rs = ps.executeQuery();
 
@@ -67,8 +61,7 @@ public class SQLiteEgresoDAO extends EgresoDAO {
     @Override
     public List<Egreso> findById(int id) {
         List<Egreso> egresoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from egresos where id = ?");
             ps.setInt(1, id);
             ResultSet rs = ps.executeQuery();
@@ -87,8 +80,7 @@ public class SQLiteEgresoDAO extends EgresoDAO {
     @Override
     public List<Egreso> findByCaja(Caja caja) {
         List<Egreso> egresoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from egresos where caja_id = ?");
             ps.setInt(1, caja.getId());
             ResultSet rs = ps.executeQuery();
@@ -107,8 +99,7 @@ public class SQLiteEgresoDAO extends EgresoDAO {
     @Override
     public List<Egreso> findByNro(String nro) {
         List<Egreso> egresoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from egresos where nro = ?");
             ps.setString(1, nro);
             ResultSet rs = ps.executeQuery();
@@ -127,8 +118,7 @@ public class SQLiteEgresoDAO extends EgresoDAO {
     @Override
     public List<Egreso> findByTipoEgreso(TipoEgreso tipoEgreso) {
         List<Egreso> egresoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from egresos where tipo_egreso_id = ?");
             ps.setInt(1, tipoEgreso.getId());
             ResultSet rs = ps.executeQuery();
@@ -147,8 +137,7 @@ public class SQLiteEgresoDAO extends EgresoDAO {
     @Override
     public boolean insertEgreso(Egreso egreso) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("insert into egresos (nro, descripcion, valor, tipo_egreso_id, caja_id) values (?,?,?,?,?)");
             ps.setString(1, egreso.getNro());
             ps.setString(2, egreso.getDescripcion());
@@ -176,8 +165,7 @@ public class SQLiteEgresoDAO extends EgresoDAO {
     @Override
     public boolean updateEgreso(Egreso egreso) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("update egresos set nro = ?, descripcion = ?, valor = ?, tipo_egreso_id = ?, caja_id = ? where id = ? ");
             ps.setString(1, egreso.getNro());
             ps.setString(2, egreso.getDescripcion());
@@ -200,8 +188,7 @@ public class SQLiteEgresoDAO extends EgresoDAO {
     @Override
     public boolean deleteEgreso(Egreso egreso) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("delete from egresos where id = ? ");
             ps.setInt(1, egreso.getId());
 
@@ -219,8 +206,7 @@ public class SQLiteEgresoDAO extends EgresoDAO {
     @Override
     public int getTotalEgreso(Caja caja) {
         int total = 0;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select sum(valor) from egresos where caja_id = ?");
             ps.setInt(1, caja.getId());
             ResultSet rs = ps.executeQuery();

src/danielcortes/xyz/models/informes/egresos/InformeEgresosContentDAO.java

@@ -24,12 +24,17 @@
 
 package danielcortes.xyz.models.informes.egresos;
 
+import danielcortes.xyz.data.ConnectionHolder;
+
 import java.time.LocalDate;
 import java.util.List;
 
 public abstract class InformeEgresosContentDAO {
+    protected ConnectionHolder connectionHolder;
+
     /**
      * Genera el informe con nombre muy largo
+     *
      * @param month mes sobre el cual se quiere le informe
      * @return lista del objeto que contiene los datos necesarios para el informe
      */

src/danielcortes/xyz/models/informes/egresos/SQLiteInformeEgresosContentDAO.java

@@ -37,11 +37,14 @@ import java.util.List;
 public class SQLiteInformeEgresosContentDAO extends InformeEgresosContentDAO {
     private List<InformeEgresosContent> list;
 
+    public SQLiteInformeEgresosContentDAO() {
+        this.connectionHolder = new SQLiteConnectionHolder();
+    }
+
     @Override
     public List<InformeEgresosContent> getInformeEgresosFactuasMateriaPrima(LocalDate date, int tipoEgresoId) {
         list = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = new SQLiteConnectionHolder().getConnection();
 
             String query = "select caja.fecha as \"fecha\",\n" +
                     "egresos.nro as \"nro\",\n" +

src/danielcortes/xyz/models/informes/libro_de_ventas/InformeLibroDeVentasContentDAO.java

@@ -48,12 +48,17 @@
 
 package danielcortes.xyz.models.informes.libro_de_ventas;
 
+import danielcortes.xyz.data.ConnectionHolder;
+
 import java.time.LocalDate;
 import java.util.Collection;
 
 public abstract class InformeLibroDeVentasContentDAO {
+    protected ConnectionHolder connectionHolder;
+
     /**
      * Genera el contenido del informes mensual
+     *
      * @param date fecha que esta dentro del mes en el que se necesita el informes
      * @return Lista con las columnas principales necesarias para el informes
      */

src/danielcortes/xyz/models/informes/libro_de_ventas/SQLiteInformeLibroDeVentasContentDAO.java

@@ -48,7 +48,6 @@
 
 package danielcortes.xyz.models.informes.libro_de_ventas;
 
-import danielcortes.xyz.data.SQLiteConnectionHolder;
 import danielcortes.xyz.utils.NaturalOrderComparator;
 
 import java.sql.Connection;
@@ -56,7 +55,9 @@ import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.time.LocalDate;
-import java.util.*;
+import java.util.Collection;
+import java.util.Comparator;
+import java.util.HashMap;
 
 public class SQLiteInformeLibroDeVentasContentDAO extends InformeLibroDeVentasContentDAO {
     private HashMap<Integer, InformeLibroDeVentasContent> map;
@@ -67,8 +68,7 @@ public class SQLiteInformeLibroDeVentasContentDAO extends InformeLibroDeVentasCo
 
     @Override
     public Collection<InformeLibroDeVentasContent> getInformeMensual(LocalDate date) {
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = new SQLiteConnectionHolder().getConnection();
             String queryTotales =
                     "select " +
                             "caja.id as \"caja\",\n" +
@@ -110,7 +110,6 @@ public class SQLiteInformeLibroDeVentasContentDAO extends InformeLibroDeVentasCo
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }

src/danielcortes/xyz/models/ingreso/IngresoDAO.java

@@ -42,12 +42,17 @@ public abstract class IngresoDAO {
     protected ConnectionHolder connectionHolder;
 
     public abstract List<Ingreso> findAll();
+
     public abstract List<Ingreso> findByCaja(Caja caja);
+
     public abstract List<Ingreso> findById(int id);
+
     public abstract List<Ingreso> findByTipoIngreso(TipoIngreso tipoIngreso);
 
     public abstract boolean insertIngreso(Ingreso ingreso);
+
     public abstract boolean updateIngreso(Ingreso ingreso);
+
     public abstract boolean deleteIngreso(Ingreso ingreso);
 
     public abstract int getTotalIngreso(Caja caja);

src/danielcortes/xyz/models/ingreso/SQLiteIngresoDAO.java

@@ -24,14 +24,9 @@
 
 package danielcortes.xyz.models.ingreso;
 
-import danielcortes.xyz.data.ConnectionHolder;
 import danielcortes.xyz.data.SQLiteConnectionHolder;
 import danielcortes.xyz.models.caja.Caja;
-import danielcortes.xyz.models.caja.CajaDAO;
-import danielcortes.xyz.models.caja.SQLiteCajaDAO;
-import danielcortes.xyz.models.tipo_ingreso.SQLiteTipoIngresoDAO;
 import danielcortes.xyz.models.tipo_ingreso.TipoIngreso;
-import danielcortes.xyz.models.tipo_ingreso.TipoIngresoDAO;
 
 import java.sql.Connection;
 import java.sql.PreparedStatement;
@@ -48,8 +43,7 @@ public class SQLiteIngresoDAO extends IngresoDAO {
     @Override
     public List<Ingreso> findAll() {
         List<Ingreso> ingresosList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from ingresos");
             ResultSet rs = ps.executeQuery();
 
@@ -67,8 +61,7 @@ public class SQLiteIngresoDAO extends IngresoDAO {
     @Override
     public List<Ingreso> findByCaja(Caja caja) {
         List<Ingreso> ingresosList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from ingresos where caja_id = ?");
             ps.setInt(1, caja.getId());
             ResultSet rs = ps.executeQuery();
@@ -87,8 +80,7 @@ public class SQLiteIngresoDAO extends IngresoDAO {
     @Override
     public List<Ingreso> findById(int id) {
         List<Ingreso> ingresosList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from ingresos where id = ?");
             ps.setInt(1, id);
             ResultSet rs = ps.executeQuery();
@@ -107,8 +99,7 @@ public class SQLiteIngresoDAO extends IngresoDAO {
     @Override
     public List<Ingreso> findByTipoIngreso(TipoIngreso tipoIngreso) {
         List<Ingreso> ingresosList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select ingresos.* from ingresos inner join tipos_ingreso on (ingresos.tipo_ingreso_id = tipos_ingreso.id) where ingresos.tipo_ingreso_id = ?");
             ps.setInt(1, tipoIngreso.getId());
             ResultSet rs = ps.executeQuery();
@@ -127,8 +118,7 @@ public class SQLiteIngresoDAO extends IngresoDAO {
     @Override
     public boolean insertIngreso(Ingreso ingreso) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("insert into ingresos (valor, nro_z_inicial, nro_z_final, nro_inicial, nro_final, tipo_ingreso_id, caja_id) values (?,?,?,?,?,?,?)");
             ps.setInt(1, ingreso.getValor());
             ps.setString(2, ingreso.getNroZInicial());
@@ -158,8 +148,7 @@ public class SQLiteIngresoDAO extends IngresoDAO {
     @Override
     public boolean updateIngreso(Ingreso ingreso) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("update ingresos set valor = ?, nro_z_inicial = ?, nro_z_final = ?, nro_inicial = ?, nro_final = ?, tipo_ingreso_id = ?, caja_id = ? where id = ?");
             ps.setInt(1, ingreso.getValor());
             ps.setString(2, ingreso.getNroZInicial());
@@ -184,8 +173,7 @@ public class SQLiteIngresoDAO extends IngresoDAO {
     @Override
     public boolean deleteIngreso(Ingreso ingreso) {
         int updates;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("delete from ingresos where id = ?");
             ps.setInt(1, ingreso.getId());
             updates = ps.executeUpdate();
@@ -201,8 +189,7 @@ public class SQLiteIngresoDAO extends IngresoDAO {
     @Override
     public int getTotalIngreso(Caja caja) {
         int total = 0;
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select sum(valor) from ingresos where caja_id = ?");
             ps.setInt(1, caja.getId());
             ResultSet rs = ps.executeQuery();

src/danielcortes/xyz/models/tipo_egreso/SQLiteTipoEgresoDAO.java

@@ -24,7 +24,6 @@
 
 package danielcortes.xyz.models.tipo_egreso;
 
-import danielcortes.xyz.data.ConnectionHolder;
 import danielcortes.xyz.data.SQLiteConnectionHolder;
 
 import java.sql.Connection;
@@ -42,8 +41,7 @@ public class SQLiteTipoEgresoDAO extends TipoEgresoDAO {
     @Override
     public List<TipoEgreso> findAll() {
         List<TipoEgreso> tipoEgresoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from tipos_egreso");
             ResultSet rs = ps.executeQuery();
 
@@ -61,8 +59,7 @@ public class SQLiteTipoEgresoDAO extends TipoEgresoDAO {
     @Override
     public List<TipoEgreso> findById(int id) {
         List<TipoEgreso> tipoEgresoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from tipos_egreso where id = ?");
             ps.setInt(1, id);
             ResultSet rs = ps.executeQuery();
@@ -81,8 +78,7 @@ public class SQLiteTipoEgresoDAO extends TipoEgresoDAO {
     @Override
     public List<TipoEgreso> findByNombre(String nombre) {
         List<TipoEgreso> tipoEgresoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from tipos_egreso where nombre = ?");
             ps.setString(1, nombre);
             ResultSet rs = ps.executeQuery();

src/danielcortes/xyz/models/tipo_egreso/TipoEgreso.java

@@ -61,7 +61,8 @@ public class TipoEgreso {
         this.nombre = nombre;
     }
 
-    public TipoEgreso(){}
+    public TipoEgreso() {
+    }
 
     public String getNombre() {
         return nombre;

src/danielcortes/xyz/models/tipo_egreso/TipoEgresoDAO.java

@@ -59,11 +59,15 @@ public abstract class TipoEgresoDAO {
     protected ConnectionHolder connectionHolder;
 
     public abstract List<TipoEgreso> findAll();
+
     public abstract List<TipoEgreso> findById(int id);
+
     public abstract List<TipoEgreso> findByNombre(String nombre);
 
     public abstract boolean insertTipoEgreso(TipoEgreso tipoEgreso);
+
     public abstract boolean updateTipoEgreso(TipoEgreso tipoEgreso);
+
     public abstract boolean deleteTipoEgreso(TipoEgreso tipoEgreso);
 
     List<TipoEgreso> tipoEgresoFromResultSet(ResultSet rs) throws SQLException {

src/danielcortes/xyz/models/tipo_ingreso/SQLiteTipoIngresoDAO.java

@@ -24,7 +24,6 @@
 
 package danielcortes.xyz.models.tipo_ingreso;
 
-import danielcortes.xyz.data.ConnectionHolder;
 import danielcortes.xyz.data.SQLiteConnectionHolder;
 
 import java.sql.Connection;
@@ -42,8 +41,7 @@ public class SQLiteTipoIngresoDAO extends TipoIngresoDAO {
     @Override
     public List<TipoIngreso> findAll() {
         List<TipoIngreso> tiposIngresoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from tipos_ingreso");
             ResultSet rs = ps.executeQuery();
 
@@ -61,8 +59,7 @@ public class SQLiteTipoIngresoDAO extends TipoIngresoDAO {
     @Override
     public List<TipoIngreso> findById(int id) {
         List<TipoIngreso> tiposIngresoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from tipos_ingreso where id = ?");
             ps.setInt(1, id);
             ResultSet rs = ps.executeQuery();
@@ -81,8 +78,7 @@ public class SQLiteTipoIngresoDAO extends TipoIngresoDAO {
     @Override
     public List<TipoIngreso> findByNombre(String nombre) {
         List<TipoIngreso> tiposIngresoList = new ArrayList<>();
-        try {
+        try (Connection conn = connectionHolder.getConnection()) {
-            Connection conn = connectionHolder.getConnection();
             PreparedStatement ps = conn.prepareStatement("select * from tipos_ingreso where nombre = ?");
             ps.setString(1, nombre);
             ResultSet rs = ps.executeQuery();
@@ -91,7 +87,6 @@ public class SQLiteTipoIngresoDAO extends TipoIngresoDAO {
 
             rs.close();
             ps.close();
-            conn.close();
         } catch (SQLException e) {
             e.printStackTrace();
         }

src/danielcortes/xyz/models/tipo_ingreso/TipoIngresoDAO.java

@@ -33,12 +33,17 @@ import java.util.List;
 
 public abstract class TipoIngresoDAO {
     protected ConnectionHolder connectionHolder;
+
     public abstract List<TipoIngreso> findAll();
+
     public abstract List<TipoIngreso> findById(int id);
+
     public abstract List<TipoIngreso> findByNombre(String nombre);
 
     public abstract boolean insertTipoIngreso(TipoIngreso tipoEgreso);
+
     public abstract boolean updateTipoIngreso(TipoIngreso tipoEgreso);
+
     public abstract boolean deleteTipoIngreso(TipoIngreso tipoEgreso);
 
     List<TipoIngreso> tiposIngresoFromResultSet(ResultSet rs) throws SQLException {

src/danielcortes/xyz/utils/NaturalOrderComparator.java

@@ -22,7 +22,7 @@
 
 package danielcortes.xyz.utils;
 
-import java.util.*;
+import java.util.Comparator;
 
 public class NaturalOrderComparator implements Comparator {
     int compareRight(String a, String b) {

src/danielcortes/xyz/utils/Pair.java

@@ -35,8 +35,13 @@ public class Pair<L,R> {
         this.right = right;
     }
 
-    public L getLeft() { return left; }
+    public L getLeft() {
-    public R getRight() { return right; }
+        return left;
+    }
+
+    public R getRight() {
+        return right;
+    }
 
     @Override
     public boolean equals(Object o) {

src/danielcortes/xyz/views/MonthSelectDialog.java

@@ -31,12 +31,11 @@ import danielcortes.xyz.views.components.YearSpinnerModel;
 
 import javax.swing.*;
 import java.awt.*;
-import java.awt.event.*;
+import java.awt.event.KeyEvent;
+import java.awt.event.WindowAdapter;
+import java.awt.event.WindowEvent;
 import java.time.LocalDate;
 import java.util.ArrayList;
-import java.util.Calendar;
-import java.util.Date;
-import java.util.GregorianCalendar;
 
 public class MonthSelectDialog extends JDialog {
     private JPanel contentPane;
@@ -90,7 +89,7 @@ public class MonthSelectDialog extends JDialog {
 
     public LocalDate getMonth() {
         int year = Integer.valueOf((String) yearSpinner.getValue());
-        int month = this.months.indexOf((String) this.monthCombo.getSelectedItem()) + 1;
+        int month = this.months.indexOf(this.monthCombo.getSelectedItem()) + 1;
 
         LocalDate monthDate = LocalDate.of(year, month, 1);
         return monthDate;

src/danielcortes/xyz/views/TipoEgresoSelectDialog.java

@@ -26,15 +26,14 @@ package danielcortes.xyz.views;
 
 import com.intellij.uiDesigner.core.GridConstraints;
 import com.intellij.uiDesigner.core.GridLayoutManager;
-import com.intellij.uiDesigner.core.Spacer;
-import danielcortes.xyz.models.egreso.Egreso;
-import danielcortes.xyz.models.egreso.SQLiteEgresoDAO;
 import danielcortes.xyz.models.tipo_egreso.SQLiteTipoEgresoDAO;
 import danielcortes.xyz.models.tipo_egreso.TipoEgreso;
 
 import javax.swing.*;
 import java.awt.*;
-import java.awt.event.*;
+import java.awt.event.KeyEvent;
+import java.awt.event.WindowAdapter;
+import java.awt.event.WindowEvent;
 import java.util.List;
 
 public class TipoEgresoSelectDialog extends JDialog {

src/danielcortes/xyz/views/components/NumberFormatedTextField.java

@@ -29,10 +29,7 @@ import org.mariuszgromada.math.mxparser.Expression;
 import javax.swing.*;
 import java.awt.event.FocusEvent;
 import java.awt.event.FocusListener;
-import java.awt.event.KeyAdapter;
-import java.awt.event.KeyEvent;
 import java.text.NumberFormat;
-import java.text.ParseException;
 
 /**
  * Crea un JTextField que formatea automaticamente su texto como un integer el cual se puede obtener
@@ -68,7 +65,7 @@ public class NumberFormatedTextField extends JTextField {
      * Llama a readValue por un bug seguramente relacionado con el focus listener:
      * - No actualizaba el valor al momento de hacer requestfocus a otro componente, probablemente porque no alcanza
      * a realizarse la accion antes que ocurra la siguiente
-     *
+     * <p>
      * Fuerza a que se lea el valor en el textfield antes de retornarlo
      */
     public int getValue() {