Aunque no veamos la declaración del objeto conexion
, parece que te da NullPointer porque no estás accediendo al objeto con la conexión.
La conexión está en con
pero el preparedStatement lo sacas de conexion, que es una instancia de ConectaDB.
Editado: También debería ser prepareStatement (sin d en prepared)
Editado: Puedes usar el siguiente código. He preparado el metodo preparedStatement de conexión para hacer el prepareStatement, pero seguramente deberías elegir una de las siguientes estratégias.
Hacer un metodo estático que te prepare la query pasándole la conexión.
Hacer ConectaDB un objeto que guarde la conexión y sólo le pases la query.
Quitar el metodo prepareStatement de ConectaDB para no liarte con la instancia entre los dos objetos.
public class Usuario {
private int id; private String nickName; private String password; private String Email; private ConectaDB conexion; public Usuario(int id, String nickName, String password, String email) { this.id = id; this.nickName = nickName; this.password = password; Email = email; } public ArrayList<Usuario> getUsuarios() { Connection con = null; con = conexion.conecta(); ArrayList<Usuario> Usuarios = new ArrayList(); try { PreparedStatement getData = con.prepareStatement("SELECT IDUsuario, Nickname, Password, Email FROM usuario"); ResultSet data; 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
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 = 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 static PreparedStatement preparedStatement(Connection con ,String sql) throws SQLException {
return con.prepareStatement(sql);
}
}