Walter Jesus Santiago Gonzalez
Walter Jesus Santiago Gonzalez
Edson Enrique Torres Macassi
Jesús Patricio Pereira Matus
José Luis Ramírez Minaya
Emanuel Munera
Juan Sebastián Joya Rodríguez
Luis Hernando Soto Mesa
Walter Jesus Santiago Gonzalez
Jose Yasbet Alvarez Godinez
ANDRES ALFONSO MIRA MEJIA
Jesús Patricio Pereira Matus
Richard Alfredo Gómez Navarro
Cid Emmanuel Esquivel González
Deymer Hernández
Tara Campos Padilla
Alvaro Poma
Jose Carrillo Zenteno
Faryd Alejandro Peñuela Gonzàlez
Orlando Macias
Walter Jesus Santiago Gonzalez
Buen día,
Considero que en el método delete(Integer id) es necesario realizar la validación si existe o no el empleado que se quiere eliminar:
Mi solución con la validación:
@Override
public void delete(Integer id) throws SQLException {
if(getById(id) != null){
try(PreparedStatement myStamt = getConnection().prepareStatement("DELETE FROM employees WHERE id= ?")){
myStamt.setInt(1,id);
myStamt.executeUpdate();
System.``out``.println("Empleado eliminado");
}
}else{
System.``out``.println("Empleado a eliminar no existe");
}
}
Espero sea de ayuda para todos.
Saludos y éxitos.
@Override public void delete(Integer id) throws SQLException { if(getById(id) != null){ try(PreparedStatement myStamt = getConnection().prepareStatement("DELETE FROM employees WHERE id= ?")){ myStamt.setInt(1,id); myStamt.executeUpdate(); System.out.println("Empleado eliminado"); } }else{ System.out.println("Empleado a eliminar no existe"); } }
Yo no tengo la tabla de empleados, pero si deseas validar si existe o no creo que debes consultar en la bd, en mi caso no estoy usando un integer, estoy usando un string, algo asi lo hice lo mismo debe aplicar para eliminar.
Humildemente me gustaria hacer lo que ustedes hacen, enseñar programacion al resto para dar a conocer lo que es el mundo de la programacion para el que este leyendo esto, no es facil nada es facil en esta vida pero si tuviera que dar mi recomendacion para entender este curso es ver fundamentos de java y ver el lenguaje SQL para entender lo del CRUD (create, read, update, delete)
Por que en el if(employee.getId()!=null) no marca un error? a mi me indica que no puedo comparar con null ya que es un valor de tipo int
Hola José, seguramente es porque estás usando el tipo primitivo int, mientras que la profe está usando wrappers, en este caso la clase Integer. Los wrappers nos sirven para este tipo de casos puesto que los primitivos no al no ser clases, no tienen métodos en sí, mientras que los wrappers nos dan más libertad de operación con estos datos.
Intenta cambiando el dato id por tipo Integer
@emanuel.munera1 Llevo años haciendolo y no entendia el por qué. Te ganas un puntazo
@Override public void save(Employee employee) throws SQLException { String sql; // si contiene un valor numerico mayor que cero , entonces existe un id y es una actualizacion if(employee.getId()!= null && employee.getId()>0){ sql = "UPDATE empleados SET nombre = ?, apellidos = ?, correo = ?, salario = ? WHERE id = ?"; }else{ sql = "INSERT INTO empleados(nombre,apellidos,correo,salario) VALUES(?,?,?,?)"; } try(PreparedStatement myStamt = getConnection().prepareStatement(sql)){ myStamt.setString(1,employee.getNombre()); myStamt.setString(2,employee.getApellidos()); myStamt.setString(3,employee.getCorreo()); myStamt.setInt(4,employee.getSalario()); if (employee.getId() != null && employee.getId()>0) { myStamt.setInt(5,employee.getId()); } myStamt.executeUpdate(); }catch (SQLException e) { e.printStackTrace(); } }
Te invito a visualizar mi solución al reto en los comentarios de la clase pasada utilizando el método getById(Integer id) que realmente nos asegura que si existe el empleado para poder actualizar en la base de datos, saludos.
Yo cree un nuevo metodo para el update
final static String SQL_UPDATE= "UPDATE employees set first_name=?, pa_surname=?, ma_surname=?, email=?, salary=? where id=?"; @Overridepublic void updateById(final Employee employee) throws SQLException { Employee currentEmployee = this.findById(employee.getId()); if (currentEmployee == null) { System.out.printf("Employee with id %d not found.%n", employee.getId()); return; } try (PreparedStatement statement = DatabaseConnection.getInstance().prepareStatement(SQL_UPDATE)){ statement.setString(1, employee.getFirst_name()); statement.setString(2, employee.getPa_surname()); statement.setString(3, employee.getMa_surname()); statement.setString(4, employee.getEmail()); statement.setDouble(5, employee.getSalary()); statement.setInt(6, employee.getId()); int i = statement.executeUpdate(); if (i == 0) { throw new SQLException("Updating employee failed, no rows affected."); } System.out.println("Updated employee with name: " + employee.getFirst_name()); }}
✅
estaba pensando y creo que lo ideal para aplicar el update es recorrer el arraylist para ver si existe y validar con eso porque el cambio por lo que hace en el video siento que podria hacer cambios no deseados en la base de datos
Y para ingresar al else? como seria la logica para insertar un registro empleado?
Si le pasas como parametro un empleado sin ID automaticamente registras un nuevo empleado.
El if tambien se podría poner en la query:
String query = """
INSERT INTO public.users (id, first_name, last_name, email, last_connection, country, website, username, followers, following)
VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
ON CONFLICT (id) DO UPDATE SET
first_name = EXCLUDED.first_name,
last_name = EXCLUDED.last_name,
email = EXCLUDED.email,
last_connection = EXCLUDED.last_connection,
country = EXCLUDED.country,
website = EXCLUDED.website,
username = EXCLUDED.username,
followers = EXCLUDED.followers,
following = EXCLUDED.following;
""";
Si actualizas directamente seteando el valor de Id y el resto de valores, estás sobreescribiendo los datos del registro y puedes cometer errores no deseados, lo correcto es traer el registro por Id, modificar el dato y salvarlo:
Employee employee2 = repository.getById(6); employee2.setSalary(6400000); repository.save(employee2);
Creación de los métodos:Creación de los métodos:Creación de los métodos:````js Creación de los métodos:
@Override public void delete(Employee employee) throws SQLException { String query = "DELETE FROM employees WHERE id=?"; try(PreparedStatement stmt = getConnection().prepareStatement(query)){ if(findById(employee.getId())!=null){ stmt.setInt(1, employee.getId()); stmt.executeUpdate(); } } }
  @Override   public void delete(Employee employee) throws SQLException {   String query = "DELETE FROM employees WHERE id=?";   try(PreparedStatement stmt = getConnection().prepareStatement(query)){   if(findById(employee.getId())!=null){   stmt.setInt(1, employee.getId());   stmt.executeUpdate();   }   }   }
@Override
public void Eliminar(Integer Id) {
}
private PERSONA CrearPersona(ResultSet myRes) throws SQLException {
PERSONA e=new PERSONA();
e.setIdPersona(myRes.getInt("IdPersona"));
e.setNombrePersona(myRes.getString("NombrePersona"));
return e;
}
Hola
java.sql.SQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '= 6' at line 1
No entiendo por que me genera este error en el momento en que quiero actualizar el registro con id=6, al parecer es un error en la sintaxis de SQL, pero no logro saber en que estoy fallando
Hola Alejandro, el error que estas visualizando se puede presentar si quieres actualizar un id que no existe en tu base de datos o que no es correcto, o puede ser un error en el método verifica que el 'id' sea el correcto y que tu método save() este bien te dejo un ejemplo de mi método para que puedas comparar.
/** * Cuando nosotros no mandemos el id para save() será un CREATE en la base de datos * pero si mandamos el id entonces es un UPDATE * @param employee */ @Override public void save(Employee employee) throws SQLException { String sql; //id != null UPDATE of INSERT if (employee.getId()!=null && employee.getId()>0){ sql = "UPDATE employees SET first_name=?, pa_surname=?, ma_surname=?, email=?, salary=? WHERE id=?"; } else { sql = "INSERT INTO employees (first_name, pa_surname, ma_surname, email, salary) VALUES(?, ?, ?, ?, ?)"; } try (PreparedStatement myStamt = getConnection().prepareStatement(sql)){ myStamt.setString(1, employee.getFirst_name()); myStamt.setString(2, employee.getPa_surname()); myStamt.setString(3, employee.getMa_surname()); myStamt.setString(4, employee.getEmail()); myStamt.setString(5, String.valueOf(employee.getSalary())); if (employee.getId() != null && employee.getId()>0){ myStamt.setInt(6, employee.getId()); } myStamt.executeUpdate(); } catch (SQLException e){ e.printStackTrace(); } }
Te invito a visualizar mi solución al reto en los comentarios de la clase pasada utilizando el método getById(Integer id) que realmente nos asegura que si existe el empleado para poder actualizar en la base de datos, saludos.
