diff --git a/dist/Programa Caja.jar b/dist/Programa Caja.jar index 446e399..33bbd41 100644 Binary files a/dist/Programa Caja.jar and b/dist/Programa Caja.jar differ diff --git a/src/danielcortes/xyz/models/ingreso/SQLiteIngresoDAO.java b/src/danielcortes/xyz/models/ingreso/SQLiteIngresoDAO.java index a358bf2..8c0a1c0 100644 --- a/src/danielcortes/xyz/models/ingreso/SQLiteIngresoDAO.java +++ b/src/danielcortes/xyz/models/ingreso/SQLiteIngresoDAO.java @@ -44,7 +44,7 @@ import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; /** - * Objeto DAO que realiza las querys y mapeos necesarios del objeto EstadoResultado. En especifico + * Objeto DAO que realiza las querys y mapeos necesarios del objeto Ingreso. En especifico * esta implementacion se comunica con la base de datos SQLite */ public class SQLiteIngresoDAO implements IngresoDAO { diff --git a/src/danielcortes/xyz/models/tipo_egreso/SQLiteTipoEgresoDAO.java b/src/danielcortes/xyz/models/tipo_egreso/SQLiteTipoEgresoDAO.java index 0350481..17655d8 100644 --- a/src/danielcortes/xyz/models/tipo_egreso/SQLiteTipoEgresoDAO.java +++ b/src/danielcortes/xyz/models/tipo_egreso/SQLiteTipoEgresoDAO.java @@ -33,15 +33,30 @@ import java.sql.SQLException; import java.util.ArrayList; import java.util.List; import java.util.Optional; +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; + +/** + * Objeto DAO que realiza las querys y mapeos necesarios del objeto TipoEgreso. En especifico esta + * implementacion se comunica con la base de datos SQLite + */ public class SQLiteTipoEgresoDAO implements TipoEgresoDAO { + + private static final Logger log = LogManager.getLogger(SQLiteTipoEgresoDAO.class); private ConnectionHolder connectionHolder; public SQLiteTipoEgresoDAO() { this.connectionHolder = new SQLiteConnectionHolder(); } + /** + * Obtiene una lista todos los TipoEgreso existentes + * + * @return Una lista de tipoEgreso + */ @Override public List getAll() { + log.debug("Se intentara conseguir todos los tipoEgreso"); List tipoEgresoList = new ArrayList<>(); String query = "select * from tipos_egreso"; try (Connection conn = connectionHolder.getConnection()) { @@ -56,12 +71,22 @@ public class SQLiteTipoEgresoDAO implements TipoEgresoDAO { } } } catch (SQLException e) { + log.error("Error al intentar conseguir todos los tipoEgreso", e); } + log.debug("Se consiguieron " + tipoEgresoList.size() + " TipoEgreso"); return tipoEgresoList; } + /** + * Obtiene un TipoEgreso dado su id en la base de datos + * + * @param id Id pertenciente al TipoEgreso + * @return Un optional que contiene el TipoEgreso y puede estar vacio, dado que no es 100% seguro + * que el id entregado exista o siquiera sea valido. + */ @Override public Optional getById(int id) { + log.debug("Se intentara conseguir un TipoEgreso con id " + id); TipoEgreso tipoEgreso = null; String query = "select * from tipos_egreso where id = ?"; try (Connection conn = connectionHolder.getConnection()) { @@ -76,12 +101,22 @@ public class SQLiteTipoEgresoDAO implements TipoEgresoDAO { } } } catch (SQLException e) { + log.error("Error al intentar conseguir un TipoEgreso con id " + id, e); } + log.debug("Se consiguio el tipoEgreso " + tipoEgreso); return Optional.ofNullable(tipoEgreso); } + + /** + * Obtiene un TipoEgreso dado su nombre + * @param nombre Nombre del TipoEgreso buscado + * @return Un optional que contiene el TipoEgreso y puede estar vacio, dado que no es 100% seguro + * que el nombre entregado exista o siquiera sea valido. + */ @Override public Optional getByNombre(String nombre) { + log.debug("Se intentara conseguir un TipoEgreso con nombre" + nombre); TipoEgreso tipoEgreso = null; String query = "select * from tipos_egreso where nombre = ?"; try (Connection conn = connectionHolder.getConnection()) { @@ -96,7 +131,9 @@ public class SQLiteTipoEgresoDAO implements TipoEgresoDAO { } } } catch (SQLException e) { + log.error("Error al intentar conseguir un TipoEgreso con nombre " + nombre, e); } + log.debug("Se consigio el TipoEgreso " + tipoEgreso); return Optional.ofNullable(tipoEgreso); } }