Estoy intentando generar un ArrayList
de users desde un DB
, el caso es que al ejecutar el PreparedStatement
me devuelve un NullPointer
y me dice que el objeto conexión es nulo, es algo que no me pasa en otras sentencias que tengo y funcionan.
Dejo el código del Array
y el código de la clase Conecta(Objeto Conect)
:
public class Usuario {
private int id;
private String nickName;
private String password;
private String Email;
private ConectaDB conexion;
public ArrayList<Usuario> getUsuarios(){
Connection con = null;
con = conexion.conecta();
ArrayList<Usuario> Usuarios = new ArrayList();
PreparedStatement getData = conexion.preparedStatement("SELECT IDUsuario, Nickname, Password, Email FROM usuario");
ResultSet data;
try{
data = getData.executeQuery();
while(data.next()){
id = data.getInt(1);
nickName = data.getString(2);
password = data.getString(3);
Email = data.getString(4);
Usuario usser = new Usuario(data.getInt(1), data.getString(2), data.getString(3), data.getString(4));
Usuarios.add(usser);
}
return Usuarios;
} catch(SQLException e){
} return Usuarios;
}
Y aquí está el código de la clase Conecta:
public class ConectaDB {
public static Connection conecta(){
Connection conecta=null;
try{
Class.forName("com.mysql.jdbc.Driver");
String servidor="jdbc:mysql://xxxxxx:xxxx/bbdd_g1"; //conectamos a nuestra propia maquina
String usuario="admin_g1";
String password="a7586";
//inicializamos objeto conecta, del tipo connection
conecta = (Connection)DriverManager.getConnection(servidor, usuario, password);
}catch(ClassNotFoundException e)
{
System.out.println(e.getMessage());
}
catch(Exception e) //capturar el resto de exceptions
{
System.out.println(e.getMessage());
}finally{
return conecta;//como el default de los case, pero se ejecuta si o si
}
}
public PreparedStatement preparedStatement(String x) {
throw new UnsupportedOperationException("Not supported yet."); //To change body of generated methods, choose Tools | Templates.
}
}
Gracias por la ayuda!
Statement
aquí en vez dePreparedStatement
? ¿Conoces para qué sirve cada uno? ¿Vas a seguir reutilizando la consulta que preparas? Si la respuesta es no, lo más óptimo aquí sería el uso de unStatement
, ya que tampoco tienes necesidad de proteger el código contra inyección en este caso concreto.