Skip to main content
se añadieron 2836 caracteres en el cuerpo
Origen Enlace

El codigo debería serEditado: 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

conpublic =class conexion.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
PreparedStatement getData           String usuario="admin_g1";
            String password="a7586";
            //inicializamos objeto conecta, del tipo connection
            conecta = conDriverManager.prepareStatementgetConnection("SELECTservidor, IDUsuariousuario, Nicknamepassword);

        }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, Passwordpero se ejecuta si o si
        }

    }

    public static PreparedStatement preparedStatement(Connection con ,String Emailsql) FROMthrows usuario"SQLException {
        return con.prepareStatement(sql);
    }
}

El codigo debería ser:

con = conexion.conecta();

PreparedStatement getData = con.prepareStatement("SELECT IDUsuario, Nickname, Password, Email FROM usuario");

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);
    }
}
se añadieron 69 caracteres en el cuerpo
Origen Enlace

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)

El codigo debería ser:

con = conexion.conecta();

PreparedStatement getData = con.preparedStatementprepareStatement("SELECT IDUsuario, Nickname, Password, Email FROM usuario");

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.

El codigo debería ser:

con = conexion.conecta();

PreparedStatement getData = con.preparedStatement("SELECT IDUsuario, Nickname, Password, Email FROM usuario");

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)

El codigo debería ser:

con = conexion.conecta();

PreparedStatement getData = con.prepareStatement("SELECT IDUsuario, Nickname, Password, Email FROM usuario");
Origen Enlace

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.

El codigo debería ser:

con = conexion.conecta();

PreparedStatement getData = con.preparedStatement("SELECT IDUsuario, Nickname, Password, Email FROM usuario");