Ik had een sollicitatie gesprek en moest een stukje code schrijven. Nou gaf de werkgever een opdracht (heb gefaald) in Java maar gaf aan dat mijn SQL opdrachten onveilig zouden zijn.

Waarom is dat en hoe doe ik het beter?

Mijn code:

import java.util.List;
import javax.sql.DataSource;
import org.springframework.jdbc.core.JdbcTemplate;

public class KlantJDBCTemplate implements KlantDAO {
private DataSource dataSource;
private JdbcTemplate jdbcTemplateObject;

public void setDataSource(DataSource dataSource) {
this.dataSource = dataSource;
this.jdbcTemplateObject = new JdbcTemplate(dataSource);
}
public void create(String naam, Integer leeftijd) {
String SQL = "insert into Klant (naam, leeftijd) values (?, ?)";
jdbcTemplateObject.update( SQL, naam, leeftijd);
System.out.println("Created Record naam = " + naam + " leeftijd = " + leeftijd);
return;
}
public Klant getKlant(Integer id) {
String SQL = "select * from Klant where id = ?";
Klant Klant = jdbcTemplateObject.queryForObject(SQL,
new Object[]{id}, new KlantMapper());

return Klant;
}
public List<Klant> listKlants() {
String SQL = "select * from Klant";
List <Klant> Klants = jdbcTemplateObject.query(SQL, new KlantMapper());
return Klants;
}
public void delete(Integer id) {
String SQL = "delete from Klant where id = ?";
jdbcTemplateObject.update(SQL, id);
System.out.println("Deleted Record with ID = " + id );
return;
}
public void update(Integer id, Integer leeftijd){
String SQL = "update Klant set leeftijd = ? where id = ?";
jdbcTemplateObject.update(SQL, leeftijd, id);
System.out.println("Updated Record with ID = " + id );
return;
}
}