Question

Buenas compañeros, estoy intentando hacer conexión mediante JSP a MySQL con Tomcat y Apache.

Tengo este código que no se si es el correcto, el driver ya me lo he descargado para ecplise y esta incluido en el path (o eso creo), pero no me termina de conectar y no se cual es el problema. La conexión se que funciona puesto que desde PHP si acceso a la base de datos.

package conexion;

/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;

/**
 *
 * @author Jorge Félix
 */
public class ConexionDB {
    public static Connection conex;
    static Statement stm;
    public static ResultSet rs;

    public ConexionDB() throws ClassNotFoundException, SQLException {
        Class.forName("com.mysql.jdbc.Driver");
        String servidor="jdbc:mysql://localhost/carrito";
        String usuario="root";
        String pass="";
        conex=(Connection)DriverManager.getConnection(servidor, usuario, pass);
        }

    public Connection getConex() {
            return conex;
    }

    public static void setConex(Connection conex) {
            ConexionDB.conex = conex;
    }

    public void desconectar() throws SQLException {
            if (!conex.isClosed()) {
                    conex.close();
            }
    }
}

¿Alguna solución? Gracias :)

Answer 1

Bueno, ya lo he conseguido...he seguido este video tutorial y me ha aclarado las ideas:

https://www.youtube.com/watch?v=8G8_JnPmTEg espero sirva para todos aquellos que se atasquen con lo mas básico como me ha pasado a mi :)

Muchas gracias a todos :)

Answer 2

Te faltó especificar el puerto que por lo general para MySQL es 3306. Siempre cuando tengas problemas, trata de ser más explícito, dinos si se lanzó una excepción, etc.

Answer 3

Hola Yesod para hacer eso es mejor que utilices PreparedStatement que esta preparada para evitar inyeccion sql, ademas al final de la consulta debes cerrar la conexión,

public class ConexionDB {       

        private static final String DB_DRIVER = "com.mysql.jdbc.Driver";
    private static final String DB_CONNECTION = "jdbc:mysql://localhost:3306/nombre_bbdd";
    private static final String DB_USER = "user";
    private static final String DB_PASSWORD = "pass";
private static Connection getDBConnection() {
        Connection dbConnection = null;
        try {
            Class.forName(DB_DRIVER);
        } catch (ClassNotFoundException e) {
        }
        try {
            dbConnection = DriverManager.getConnection(
                             DB_CONNECTION, DB_USER,DB_PASSWORD);
            return dbConnection;
        } catch (SQLException e) {
        }
        return dbConnection;
    }
}
public class Consultas {

/*esta funcion devuelve null si no se encuentra ningun resultado */
public ResultSet Autenticacion(String user, String pass) throws SQLException {
                Connection dbConnection = null;
        PreparedStatement preparedStatement = null;
                if (username == null) {
            username = "";
        }
        if (password == null) {
            password = "";
        }
        username = username.trim();
                String query ="Select * from Usuarios where Usuario= ? and Password= ?";
                try {
            dbConnection = Connection.getDBConnection();
            preparedStatement =           dbConnection.prepareStatement(query);
            preparedStatement.setString(1, user);
                        preparedStatement.setString(2, pass);
            // execute select SQL stetement
            ResultSet resultSet = preparedStatement.executeQuery();
            return resultSet;
        } catch (SQLException e) {
        } finally {
            if (preparedStatement != null) {
                preparedStatement.close();
            }

            if (dbConnection != null) {
                dbConnection.close();
            }
        }
        return null;
         }
}

Edito en respuesta a tu comentario:

Buenas el codigo que te pase esta probado, lo que pasa que hay bastantes cosas que pueden estar fallando en el codigo, ademas no lo has copiado bien, aparte de eso la excepcion que te lanza es un NullPointer en la linea
47 de la clase Consultas:

conexion.Consultas.Clientes(Consultas.java:47)

te aconsejo que depures un poco mas el error mostrando los datos de las variables por consola para ver que valores va tomando y ademas imprimiendo y controlando las SQL Exception en los bloques catch para controlar si se estan produciendo ya que ahora mismo los bloques catch estan vacios y si se producen excepciones ni te das cuenta.
Aparte de eso has copiado mal esto:

String cadSQL ="Select * from usuarios where DNI='"+username+"' and Password='"+password+"'";

No utilices la concatenación de string para pasarle los parámetros a un preparedStatment, para eso se utiliza la interrogación

String query ="Select * from Usuarios where Usuario= ? and Password= ?";

y con esto los vas añadiendo

preparedStatement.setString(1, username);
        ConexionDB.preparedStatement.setString(2, password);

Ademas no es muy buena practica guardar los resultados en la clase que hace la conexión a la base de datos pero bueno eso es otro tema

Answer 4

Según tu código, dice que la linea 47 del archivo ConsultasDb...

if (ConexionDB.dbConnection != null)

Lanza una excepción de puntero nulo que hace clara referencia a que te falto instanciar un objeto. Por lo que veo, el problema puede ser 1 cosa: En algún momento, del ciclo de vida de la aplicacion, ConexionDB perdió su instancia y cuando quiere llamar a su atributo, lanza la excepción que te da.