Search in a table in Mysql with Java

Question

I’m having trouble trying to get something on the table, I’m new so I did it the way I know.

Button code:

livros.setPesquisarLivro(txtLivro.getText());
    try {
        modelo.setNumRows(0);

        for (ObjetoLivro c : livroDAO.pesquisarLivro(livros)) {
            modelo.addRow(new Object[]{
                c.getNomeLivro(),
                c.getAutor(),
                c.getGenero()});
                c.getAlunoLivro();
        }

    } catch (Exception e) {
    }

Research code:

public ArrayList<ObjetoLivro> pesquisarLivro(ObjetoLivro pesquisar) throws SQLException {
    ResultSet rs = ConnectionFactory.getStatement().executeQuery("SELECT IDLIVRO,NOMELIVRO,AUTOR,GENERO,NOMEALUNO FROM LIVRO WHERE NOMELIVRO LIKE  '%"+pesquisar+"%'");
    ArrayList<ObjetoLivro> livros = new ArrayList<ObjetoLivro>();
    while (rs.next()) {
        ObjetoLivro livros2 = new ObjetoLivro();
        livros2.setIdLivro(rs.getInt(1));
        livros2.setNomeLivro(rs.getString(2));
        livros2.setAutor(rs.getString(3));
        livros2.setGenero(rs.getString(4));
        livros2.setAlunoLivro(rs.getString(5));
        livros.add(livros2);
    }
    return livros;
}

Code of ConnectionFactory:

package util;
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.Statement;
import java.util.logging.Level;
import java.util.logging.Logger;

public class ConnectionFactory {

private static Connection connection = null;
private static Statement statement;

static {
    try {
        Class.forName("com.mysql.jdbc.Driver");

        connection = DriverManager.getConnection("jdbc:mysql://localhost:3306/biblioteca",
                 "root", "");
        statement = connection.createStatement();

    } catch (Exception ex) {
        Logger.getLogger(ConnectionFactory.class.getName()).log(Level.SEVERE, null, ex);
    }
}

public static Statement getStatement() {
    return statement;
}
}

Code of ObjetoLivro:

package Model;
public class ObjetoLivro {
private String nomeLivro,autor,genero,alunoLivro,pesquisarLivro;
private int idLivro;
public String getPesquisarLivro() {
    return pesquisarLivro;
}

public void setPesquisarLivro(String pesquisarLivro) {
    this.pesquisarLivro = pesquisarLivro;
}

public String getNomeLivro() {
    return nomeLivro;
}

public void setNomeLivro(String nomeLivro) {
    this.nomeLivro = nomeLivro;
}

public String getAutor() {
    return autor;
}

public void setAutor(String autor) {
    this.autor = autor;
}

public String getGenero() {
    return genero;
}

public void setGenero(String genero) {
    this.genero = genero;
}

public int getIdLivro() {
    return idLivro;
}

public void setIdLivro(int idLivro) {
    this.idLivro = idLivro;
}

public String getAlunoLivro() {
    return alunoLivro;
}

public void setAlunoLivro(String alunoLivro) {
    this.alunoLivro = alunoLivro;
} 
}

The return of the search code is:

[]

And my table shows nothing, after clicking the search button.

Answer 1

The problem is that, in the section below, you are concatenating the class instance itself ObjetoLivro instead of the book name:

WHERE NOMELIVRO LIKE '%"+pesquisar+"%'");
//Deveria ser
WHERE NOMELIVRO LIKE '%"+pesquisar.getNomeLivro()+"%'");

Also, the ideal would be for you to use a PreparedStatement instead of a Statement, since the first has advantages over the second as you can see here.

With a PreparedStatement your code would look like this:

Connectionfactory:

public class ConnectionFactory {
    //O resto do código

    public static PreparedStatement createPreparedStatement(String sql) throws SQLException {
        connection.prepareStatement(sql);
    }
}

Research code:

public ArrayList<ObjetoLivro> pesquisarLivro(ObjetoLivro pesquisar) throws SQLException {
    PreparedStatement ps = ConnectionFactory.createPreparedStatement("SELECT IDLIVRO,NOMELIVRO,AUTOR,GENERO,NOMEALUNO FROM LIVRO WHERE NOMELIVRO LIKE ?");
    ps.setString(1, "%" + pesquisar.getNomeLivro() + "%");

    ResultSet rs = ps.executeQuery();

    ArrayList<ObjetoLivro> livros = new ArrayList<ObjetoLivro>();
    while (rs.next()) {
        ObjetoLivro livro = new ObjetoLivro();
        livro.setIdLivro(rs.getInt(1));
        livro.setNomeLivro(rs.getString(2));
        livro.setAutor(rs.getString(3));
        livro.setGenero(rs.getString(4));
        livro.setAlunoLivro(rs.getString(5));
        livros.add(livro);
    }
    return livros;
}

Answer 2

The first problem with your code, as others have pointed out, is this:

'%"+pesquisar+"%'"

What you wanted was this:

'%"+pesquisar.getNomeLivro()+"%'"

Another possibility is to exchange this:

public ArrayList<ObjetoLivro> pesquisarLivro(ObjetoLivro pesquisar) throws SQLException {

That’s why:

public List<ObjetoLivro> pesquisarLivro(String pesquisar) throws SQLException {

That’s because I believe you want a method that:

"from a title or part of a book title to be researched, return all books that have that title".

That’s a little different than a method that:

"from a book that has some title, return all books that contain the title of the book provided within its own titles".

However your code still has other problems:

• You should be wearing the Try-with-Resources. To understand why and how to use it, see that question.

• Your code has a injection of SQL.

To solve these problems, here’s how your ConnectionFactory and its research method:

package util;
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.SQLException;

public class ConnectionFactory {

    public static Connection connect() throws SQLException {
        return DriverManager.getConnection("jdbc:mysql://localhost:3306/biblioteca", "root", "");
    }
}

public List<ObjetoLivro> pesquisarLivro(String nomeLivro) throws SQLException {
    List<ObjetoLivro> livros = new ArrayList<>();
    String sql = "SELECT IDLIVRO, NOMELIVRO, AUTOR, GENERO, NOMEALUNO FROM LIVRO WHERE NOMELIVRO LIKE ?";
    try (
        Connection conn = ConnectionFactory.connect();
        PreparedStatement ps = conn.prepareStatement(sql);
    ) {
        ps.setString(1, "%" + nomeLivro + "%");
        try (ResultSet rs = ps.executeQuery()) {
            while (rs.next()) {
                ObjetoLivro livros2 = new ObjetoLivro();
                livros2.setIdLivro(rs.getInt(1));
                livros2.setNomeLivro(rs.getString(2));
                livros2.setAutor(rs.getString(3));
                livros2.setGenero(rs.getString(4));
                livros2.setAlunoLivro(rs.getString(5));
                livros.add(livros2);
            }
        }
    }
    return livros;
}