Se limpio el metodo de TipoIngreso

Ademas se corrigio un bug que introduci al cambiar la forma en que el
wraper funcionaba

src/danielcortes/xyz/controllers/EgresosController.java

@@ -78,7 +78,7 @@ public class EgresosController extends BaseController {
    */
   private void fillTipoEgresoCombo() {
     JComboBox<TipoEgresoToStringWrapper> tipoCombo = view.getTipoCombo();
-    for (TipoEgreso tipoEgreso : DAOManager.getTipoEgresoDAO().findAll()) {
+    for (TipoEgreso tipoEgreso : DAOManager.getTipoEgresoDAO().getAll()) {
       tipoCombo.addItem(new TipoEgresoToStringWrapper(tipoEgreso));
     }
   }

src/danielcortes/xyz/controllers/EstadoResultadoController.java

@@ -172,11 +172,11 @@ public class EstadoResultadoController extends BaseController {
 
   private void fillGastosGenerales() {
     TipoEgreso facturaGastosGenerales = DAOManager.getTipoEgresoDAO()
-        .findByNombre("Factura Gastos Generales").orElse(TipoEgreso.EMPTY);
+        .getByNombre("Factura Gastos Generales").orElse(TipoEgreso.EMPTY);
     TipoEgreso gastoGeneralConBoleta = DAOManager.getTipoEgresoDAO()
-        .findByNombre("Gasto General Con Boleta").orElse(TipoEgreso.EMPTY);
+        .getByNombre("Gasto General Con Boleta").orElse(TipoEgreso.EMPTY);
     TipoEgreso gastoGeneralSinRespaldo = DAOManager.getTipoEgresoDAO()
-        .findByNombre("Gasto General Sin Respaldo").orElse(TipoEgreso.EMPTY);
+        .getByNombre("Gasto General Sin Respaldo").orElse(TipoEgreso.EMPTY);
 
     int cuentaCorrienteFactura = this.estadoResultado.getCuentaCorrienteFactura();
     int cuentaCorrienteBoleta = this.estadoResultado.getCuentaCorrienteBoleta();
@@ -201,7 +201,7 @@ public class EstadoResultadoController extends BaseController {
   }
 
   private void fillGastosOperacionales() {
-    TipoEgreso tipoPagoPartime = DAOManager.getTipoEgresoDAO().findByNombre("Pago Partime").orElse(TipoEgreso.EMPTY);
+    TipoEgreso tipoPagoPartime = DAOManager.getTipoEgresoDAO().getByNombre("Pago Partime").orElse(TipoEgreso.EMPTY);
 
     int costoVenta = this.estadoResultado.getCostoVenta();
     int remuneraciones = this.estadoResultado.getRemuneraciones();

src/danielcortes/xyz/controllers/IngresosController.java

@@ -82,7 +82,7 @@ public class IngresosController extends BaseController {
    */
   private void fillTipoIngresoCombo() {
     JComboBox<TipoIngresoToStringWrapper> tipoCombo = this.view.getTipoCombo();
-    for (TipoIngreso tipo : DAOManager.getTipoIngresoDAO().findAll()) {
+    for (TipoIngreso tipo : DAOManager.getTipoIngresoDAO().getAll()) {
       tipoCombo.addItem(new TipoIngresoToStringWrapper(tipo));
     }
   }

src/danielcortes/xyz/informes/InformeEstadoResultado.java

@@ -171,7 +171,7 @@ public class InformeEstadoResultado {
 
     partime.createCell(0).setCellValue("Partime");
     partime.createCell(1).setCellValue(DAOManager.getEgresoDAO().getTotalEgresoMesPorTipo(this.mes,
-        DAOManager.getTipoEgresoDAO().findByNombre("Pago Partime").orElse(TipoEgreso.EMPTY)));
+        DAOManager.getTipoEgresoDAO().getByNombre("Pago Partime").orElse(TipoEgreso.EMPTY)));
 
     bonosPersonal.createCell(0).setCellValue("Bonos Personal");
     bonosPersonal.createCell(1).setCellValue(this.estadoResultado.getBonosPersonal());
@@ -222,11 +222,11 @@ public class InformeEstadoResultado {
 
   private void fillGastosGeneralesData() {
     TipoEgreso facturaGastosGenerales = DAOManager.getTipoEgresoDAO()
-        .findByNombre("Factura Gastos Generales").orElse(TipoEgreso.EMPTY);
+        .getByNombre("Factura Gastos Generales").orElse(TipoEgreso.EMPTY);
     TipoEgreso gastoGeneralConBoleta = DAOManager.getTipoEgresoDAO()
-        .findByNombre("Gasto General Con Boleta").orElse(TipoEgreso.EMPTY);
+        .getByNombre("Gasto General Con Boleta").orElse(TipoEgreso.EMPTY);
     TipoEgreso gastoGeneralSinRespaldo = DAOManager.getTipoEgresoDAO()
-        .findByNombre("Gasto General Sin Respaldo").orElse(TipoEgreso.EMPTY);
+        .getByNombre("Gasto General Sin Respaldo").orElse(TipoEgreso.EMPTY);
 
     int startId = 29;
 

src/danielcortes/xyz/informes/InformeLibroDeVentas.java

@@ -38,11 +38,11 @@ public class InformeLibroDeVentas{
     IngresoDAO ingresoDAO = DAOManager.getIngresoDAO();
     TipoIngresoDAO tipoIngresoDAO = DAOManager.getTipoIngresoDAO();
 
-    TipoIngreso boletaFiscal = tipoIngresoDAO.findByNombre("Boletas Fiscales").get(0);
+    TipoIngreso boletaFiscal = tipoIngresoDAO.getByNombre("Boletas Fiscales").orElse(TipoIngreso.EMPTY);
-    TipoIngreso boletaManual = tipoIngresoDAO.findByNombre("Boletas Manuales").get(0);
+    TipoIngreso boletaManual = tipoIngresoDAO.getByNombre("Boletas Manuales").orElse(TipoIngreso.EMPTY);
-    TipoIngreso boletaExenta = tipoIngresoDAO.findByNombre("Boleta Exenta").get(0);
+    TipoIngreso boletaExenta = tipoIngresoDAO.getByNombre("Boleta Exenta").orElse(TipoIngreso.EMPTY);
-    TipoIngreso factura = tipoIngresoDAO.findByNombre("Facturas").get(0);
+    TipoIngreso factura = tipoIngresoDAO.getByNombre("Facturas").orElse(TipoIngreso.EMPTY);
-    TipoIngreso guias = tipoIngresoDAO.findByNombre("Guias").get(0);
+    TipoIngreso guias = tipoIngresoDAO.getByNombre("Guias").orElse(TipoIngreso.EMPTY);
 
     while (currentDate.isBefore(endDatePlusOne)) {
       Caja caja = cajaDAO.getByFecha(currentDate).orElse(Caja.EMPTY);

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

@@ -65,7 +65,7 @@ public class SQLiteEgresoDAO implements EgresoDAO {
           while (rs.next()) {
             //Confio en que la base de datos me dara un id existente
             @SuppressWarnings("OptionalGetWithoutIsPresent")
-            TipoEgreso tipoEgreso = tipoEgresoDAO.findById(rs.getInt("tipo_egreso_id")).get();
+            TipoEgreso tipoEgreso = tipoEgresoDAO.getById(rs.getInt("tipo_egreso_id")).get();
 
             //Confio en que la base de datos me dara un id existente
             @SuppressWarnings("OptionalGetWithoutIsPresent")
@@ -101,7 +101,7 @@ public class SQLiteEgresoDAO implements EgresoDAO {
             TipoEgresoDAO tipoEgresoDAO = DAOManager.getTipoEgresoDAO();
             //Confio en que la base de datos me dara un id existente
             @SuppressWarnings("OptionalGetWithoutIsPresent")
-            TipoEgreso tipoEgreso = tipoEgresoDAO.findById(rs.getInt("tipo_egreso_id")).get();
+            TipoEgreso tipoEgreso = tipoEgresoDAO.getById(rs.getInt("tipo_egreso_id")).get();
 
             CajaDAO cajaDAO = DAOManager.getCajaDAO();
             //Confio en que la base de datos me dara un id existente
@@ -141,7 +141,7 @@ public class SQLiteEgresoDAO implements EgresoDAO {
             TipoEgresoDAO tipoEgresoDAO = DAOManager.getTipoEgresoDAO();
             //Confio en que la base de datos me dara un id existente
             @SuppressWarnings("OptionalGetWithoutIsPresent")
-            TipoEgreso tipoEgreso = tipoEgresoDAO.findById(rs.getInt("tipo_egreso_id")).get();
+            TipoEgreso tipoEgreso = tipoEgresoDAO.getById(rs.getInt("tipo_egreso_id")).get();
 
             Egreso egreso = new Egreso();
             egreso.setId(rs.getInt("id"));
@@ -176,7 +176,7 @@ public class SQLiteEgresoDAO implements EgresoDAO {
           while (rs.next()) {
             //Confio en que la base de datos me dara un id existente
             @SuppressWarnings("OptionalGetWithoutIsPresent")
-            TipoEgreso tipoEgreso = tipoEgresoDAO.findById(rs.getInt("tipo_egreso_id")).get();
+            TipoEgreso tipoEgreso = tipoEgresoDAO.getById(rs.getInt("tipo_egreso_id")).get();
             //Confio en que la base de datos me dara un id existente
             @SuppressWarnings("OptionalGetWithoutIsPresent")
             Caja caja = cajaDAO.getById(rs.getInt("caja_id")).get();

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

@@ -64,8 +64,9 @@ public class SQLiteIngresoDAO implements IngresoDAO {
           CajaDAO cajaDAO = DAOManager.getCajaDAO();
 
           while (rs.next()) {
-            TipoIngreso tipoIngreso = tipoIngresoDAO.findById(rs.getInt("tipo_ingreso_id")).get(0);
+            //Confio en que la base de datos me dara un id existente
-
+            @SuppressWarnings("OptionalGetWithoutIsPresent")
+            TipoIngreso tipoIngreso = tipoIngresoDAO.getById(rs.getInt("tipo_ingreso_id")).get();
             //Confio en que la base de datos me dara un id existente
             @SuppressWarnings("OptionalGetWithoutIsPresent")
             Caja caja = cajaDAO.getById(rs.getInt("caja_id")).get();
@@ -106,7 +107,9 @@ public class SQLiteIngresoDAO implements IngresoDAO {
         try (ResultSet rs = ps.executeQuery()) {
           TipoIngresoDAO tipoIngresoDAO = DAOManager.getTipoIngresoDAO();
           while (rs.next()) {
-            TipoIngreso tipoIngreso = tipoIngresoDAO.findById(rs.getInt("tipo_ingreso_id")).get(0);
+            //Confio en que la base de datos me dara un id existente
+            @SuppressWarnings("OptionalGetWithoutIsPresent")
+            TipoIngreso tipoIngreso = tipoIngresoDAO.getById(rs.getInt("tipo_ingreso_id")).get();
 
             Ingreso ingreso = new Ingreso();
 
@@ -141,7 +144,9 @@ public class SQLiteIngresoDAO implements IngresoDAO {
           CajaDAO cajaDAO = DAOManager.getCajaDAO();
 
           if (rs.next()) {
-            TipoIngreso tipoIngreso = tipoIngresoDAO.findById(rs.getInt("tipo_ingreso_id")).get(0);
+            //Confio en que la base de datos me dara un id existente
+            @SuppressWarnings("OptionalGetWithoutIsPresent")
+            TipoIngreso tipoIngreso = tipoIngresoDAO.getById(rs.getInt("tipo_ingreso_id")).get();
 
             //Confio en que la base de datos me dara un id existente
             @SuppressWarnings("OptionalGetWithoutIsPresent")

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

@@ -46,7 +46,7 @@ public class SQLiteTipoEgresoDAO implements TipoEgresoDAO {
   }
 
   @Override
-  public List<TipoEgreso> findAll() {
+  public List<TipoEgreso> getAll() {
     List<TipoEgreso> tipoEgresoList = new ArrayList<>();
     String query = "select * from tipos_egreso";
     try (Connection conn = connectionHolder.getConnection()) {
@@ -56,6 +56,7 @@ public class SQLiteTipoEgresoDAO implements TipoEgresoDAO {
             TipoEgreso tipoEgreso = new TipoEgreso();
             tipoEgreso.setId(rs.getInt("id"));
             tipoEgreso.setNombre(rs.getString("nombre"));
+            tipoEgresoList.add(tipoEgreso);
           }
         }
       }
@@ -66,7 +67,7 @@ public class SQLiteTipoEgresoDAO implements TipoEgresoDAO {
   }
 
   @Override
-  public Optional<TipoEgreso> findById(int id) {
+  public Optional<TipoEgreso> getById(int id) {
     TipoEgreso tipoEgreso = null;
     String query = "select * from tipos_egreso where id = ?";
     try (Connection conn = connectionHolder.getConnection()) {
@@ -87,7 +88,7 @@ public class SQLiteTipoEgresoDAO implements TipoEgresoDAO {
   }
 
   @Override
-  public Optional<TipoEgreso> findByNombre(String nombre) {
+  public Optional<TipoEgreso> getByNombre(String nombre) {
     TipoEgreso tipoEgreso = null;
     String query = "select * from tipos_egreso where nombre = ?";
     try (Connection conn = connectionHolder.getConnection()) {

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

@@ -52,9 +52,9 @@ import java.util.List;
 import java.util.Optional;
 
 public interface TipoEgresoDAO {
-  List<TipoEgreso> findAll();
+  List<TipoEgreso> getAll();
 
-  Optional<TipoEgreso> findById(int id);
+  Optional<TipoEgreso> getById(int id);
 
-  Optional<TipoEgreso> findByNombre(String nombre);
+  Optional<TipoEgreso> getByNombre(String nombre);
 }

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

@@ -7,6 +7,26 @@ public class TipoEgresoToStringWrapper extends TipoEgreso {
     this.tipoEgreso = tipoEgreso;
   }
 
+  @Override
+  public String getNombre() {
+    return tipoEgreso.getNombre();
+  }
+
+  @Override
+  public void setNombre(String nombre) {
+    tipoEgreso.setNombre(nombre);
+  }
+
+  @Override
+  public int getId() {
+    return tipoEgreso.getId();
+  }
+
+  @Override
+  public void setId(int id) {
+    tipoEgreso.setId(id);
+  }
+
   @Override
   public String toString() {
     return this.tipoEgreso.getNombre();

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

@@ -24,6 +24,7 @@
 
 package danielcortes.xyz.models.tipo_ingreso;
 
+import danielcortes.xyz.data.ConnectionHolder;
 import danielcortes.xyz.data.SQLiteConnectionHolder;
 import java.sql.Connection;
 import java.sql.PreparedStatement;
@@ -31,31 +32,33 @@ import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.util.ArrayList;
 import java.util.List;
-import java.util.logging.Level;
+import java.util.Optional;
 import java.util.logging.Logger;
 
-public class SQLiteTipoIngresoDAO extends TipoIngresoDAO {
+public class SQLiteTipoIngresoDAO implements TipoIngresoDAO {
 
   private static final Logger LOGGER = Logger.getLogger(SQLiteTipoIngresoDAO.class.getName());
+  private ConnectionHolder connectionHolder;
 
   public SQLiteTipoIngresoDAO() {
     this.connectionHolder = new SQLiteConnectionHolder();
   }
 
   @Override
-  public List<TipoIngreso> findAll() {
+  public List<TipoIngreso> getAll() {
     List<TipoIngreso> tiposIngresoList = new ArrayList<>();
-    try (Connection conn = connectionHolder.getConnection()) {
     String query = "select * from tipos_ingreso";
-      PreparedStatement ps = conn.prepareStatement(query);
+    try (Connection conn = connectionHolder.getConnection()) {
-      ResultSet rs = ps.executeQuery();
+      try (PreparedStatement ps = conn.prepareStatement(query)) {
-
+        try (ResultSet rs = ps.executeQuery()) {
-      LOGGER.log(Level.FINE, "QUERY: {0}", query);
+          while (rs.next()) {
-
+            TipoIngreso tipoIngreso = new TipoIngreso();
-      tiposIngresoList = this.tiposIngresoFromResultSet(rs);
+            tipoIngreso.setId(rs.getInt("id"));
-
+            tipoIngreso.setNombre(rs.getString("nombre"));
-      rs.close();
+            tiposIngresoList.add(tipoIngreso);
-      ps.close();
+          }
+        }
+      }
     } catch (SQLException e) {
       e.printStackTrace();
     }
@@ -63,45 +66,45 @@ public class SQLiteTipoIngresoDAO extends TipoIngresoDAO {
   }
 
   @Override
-  public List<TipoIngreso> findById(int id) {
+  public Optional<TipoIngreso> getById(int id) {
-    List<TipoIngreso> tiposIngresoList = new ArrayList<>();
+    TipoIngreso tipoIngreso = null;
-    try (Connection conn = connectionHolder.getConnection()) {
     String query = "select * from tipos_ingreso where id = ?";
-      PreparedStatement ps = conn.prepareStatement(query);
+    try (Connection conn = connectionHolder.getConnection()) {
+      try (PreparedStatement ps = conn.prepareStatement(query)) {
         ps.setInt(1, id);
-      ResultSet rs = ps.executeQuery();
+        try (ResultSet rs = ps.executeQuery()) {
-
+          if (rs.next()) {
-      LOGGER.log(Level.FINE, "QUERY: {0} | values: {1}", new Object[]{query, id});
+            tipoIngreso = new TipoIngreso();
-
+            tipoIngreso.setId(rs.getInt("id"));
-      tiposIngresoList = this.tiposIngresoFromResultSet(rs);
+            tipoIngreso.setNombre(rs.getString("nombre"));
-
+          }
-      rs.close();
+        }
-      ps.close();
+      }
     } catch (SQLException e) {
       e.printStackTrace();
     }
-    return tiposIngresoList;
+    return Optional.ofNullable(tipoIngreso);
   }
 
   @Override
-  public List<TipoIngreso> findByNombre(String nombre) {
+  public Optional<TipoIngreso> getByNombre(String nombre) {
-    List<TipoIngreso> tiposIngresoList = new ArrayList<>();
+    TipoIngreso tipoIngreso = null;
-    try (Connection conn = connectionHolder.getConnection()) {
     String query = "select * from tipos_ingreso where nombre = ?";
-      PreparedStatement ps = conn.prepareStatement(query);
+    try (Connection conn = connectionHolder.getConnection()) {
+      try (PreparedStatement ps = conn.prepareStatement(query)) {
         ps.setString(1, nombre);
-      ResultSet rs = ps.executeQuery();
+        try (ResultSet rs = ps.executeQuery()) {
-
+          if (rs.next()) {
-      LOGGER.log(Level.FINE, "QUERY: {0} | values: {1}", new Object[]{query, nombre});
+            tipoIngreso = new TipoIngreso();
-
+            tipoIngreso.setId(rs.getInt("id"));
-      tiposIngresoList = this.tiposIngresoFromResultSet(rs);
+            tipoIngreso.setNombre(rs.getString("nombre"));
-
+          }
-      rs.close();
+        }
-      ps.close();
+      }
     } catch (SQLException e) {
       e.printStackTrace();
     }
-    return tiposIngresoList;
+    return Optional.ofNullable(tipoIngreso);
   }
 
 }

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

@@ -49,6 +49,12 @@
 package danielcortes.xyz.models.tipo_ingreso;
 
 public class TipoIngreso {
+  public final static TipoIngreso EMPTY;
+
+  static {
+    EMPTY = new TipoIngreso();
+    EMPTY.setNombre("EMPTY");
+  }
 
   private int id;
   private String nombre;

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

@@ -24,37 +24,13 @@
 
 package danielcortes.xyz.models.tipo_ingreso;
 
-import danielcortes.xyz.data.ConnectionHolder;
-import java.sql.ResultSet;
-import java.sql.SQLException;
-import java.util.ArrayList;
 import java.util.List;
-import java.util.logging.Level;
+import java.util.Optional;
-import java.util.logging.Logger;
 
-public abstract class TipoIngresoDAO {
+public interface TipoIngresoDAO {
+  List<TipoIngreso> getAll();
 
-  private static final Logger LOGGER = Logger.getLogger(SQLiteTipoIngresoDAO.class.getName());
+  Optional<TipoIngreso> getById(int id);
 
-  protected ConnectionHolder connectionHolder;
+  Optional<TipoIngreso> getByNombre(String nombre);
-
-  public abstract List<TipoIngreso> findAll();
-
-  public abstract List<TipoIngreso> findById(int id);
-
-  public abstract List<TipoIngreso> findByNombre(String nombre);
-
-  List<TipoIngreso> tiposIngresoFromResultSet(ResultSet rs) throws SQLException {
-    ArrayList<TipoIngreso> tiposIngresoList = new ArrayList<>();
-    while (rs.next()) {
-      TipoIngreso tipoIngreso = new TipoIngreso();
-      tipoIngreso.setId(rs.getInt("id"));
-      tipoIngreso.setNombre(rs.getString("nombre"));
-
-      LOGGER.log(Level.FINE, "Se a creado: {0]", tipoIngreso);
-
-      tiposIngresoList.add(tipoIngreso);
-    }
-    return tiposIngresoList;
-  }
 }

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

@@ -1,15 +1,34 @@
 package danielcortes.xyz.models.tipo_ingreso;
 
 public class TipoIngresoToStringWrapper extends TipoIngreso {
+  private TipoIngreso tipoIngreso;
 
   public TipoIngresoToStringWrapper(TipoIngreso tipoIngreso) {
-    this.setId(tipoIngreso.getId());
+    this.tipoIngreso = tipoIngreso;
-    this.setNombre(tipoIngreso.getNombre());
   }
 
+  @Override
+  public int getId() {
+    return tipoIngreso.getId();
+  }
+
+  @Override
+  public void setId(int id) {
+    tipoIngreso.setId(id);
+  }
+
+  @Override
+  public String getNombre() {
+    return tipoIngreso.getNombre();
+  }
+
+  @Override
+  public void setNombre(String nombre) {
+    tipoIngreso.setNombre(nombre);
+  }
 
   @Override
   public String toString() {
-    return this.getNombre();
+    return this.tipoIngreso.getNombre();
   }
 }

src/danielcortes/xyz/views/dialogs/TipoEgresoSelectDialog.java

@@ -106,7 +106,7 @@ public class TipoEgresoSelectDialog extends JDialog {
 
   private void createTipoEgresoCombo() {
     tipoEgresoCombo = new JComboBox<>();
-    for (TipoEgreso tipoEgreso : DAOManager.getTipoEgresoDAO().findAll()) {
+    for (TipoEgreso tipoEgreso : DAOManager.getTipoEgresoDAO().getAll()) {
       tipoEgresoCombo.addItem(new TipoEgresoToStringWrapper(tipoEgreso));
     }
   }