3

Olá, estou com seguinte problema, eu tenho uma função que busca no banco e retorna um objeto. Tenho uma função "for" que chama esse metodo quase que 300 vezes, devido a essa quantidade de chamadas o mysql está retornando o seguinte erro.

Algo deu errado ao fazer uma ação envolvendo formulações. com.mysql.jdbc.exceptions.jdbc4.MySQLNonTransientConnectionException: Data source rejected establishment of connection,  message from server: "Too many connections"

O codigo abaixo. Não faço ideia do que me resta fazer para impedir que esse erro ocorra, já que estou fechando o Statement, Resultset e a conexão em cada consulta. Poderiam me ajudar com isso? Obrigado.

 private Formulacao retornaFormulacaoDoItem(Item item) throws Exception {
    Connection cx = br.com.projeto.Conexao.Conexao.abrir();
    try {

        String comando = "SELECT * FROM Formulacao WHERE fk_item = ?";
        PreparedStatement ps = cx.prepareStatement(comando);
        ps.setInt(1, item.getIdItem());
        ResultSet rs = ps.executeQuery();
        Formulacao f = new Formulacao();
        ArrayList list = new ArrayList();
        NutrienteDAO nutri = new NutrienteDAO();
        int i = 0;
        while (rs.next()) {
            if (rs.getString("operador") != null) {
                list.add(rs.getString("operador"));
            } else if (rs.getString("fk_nutriente") != null) {
                list.add(rs.getString("fk_nutriente"));
            } else {
                list.add(nutri.retornaUmNutriente(rs.getInt("fk_nutriente")).getQuantidade());
            }
        }
        String[] re = new String[list.size()];
        for (int j = 0; j < list.size(); j++) {
            re[j] = list.get(j).toString();
        }
        f.setCalculo(re);
        ps.close();
        rs.close();
        return f;
    } catch (Exception e) {
        throw new Exception("Algo deu errado ao retornar os dados de formula do item" + e);
    } finally {
        cx.close();
    }
}

Classe de conexão com banco.



import java.sql.Connection;
import java.sql.DriverManager;

/**
 *
 * @author jpsa
 */
public class Conexao {
    private static final String USUARIO = "root";
    private static final String SENHA = "123456";
    private static final String URL = "jdbc:mysql://localhost:3306/miner";
    private static final String DRIVER = "com.mysql.jdbc.Driver";

    // Conectar ao banco
    public static Connection abrir() throws Exception {
        // Registrar o driver
        Class.forName(DRIVER);
        // Capturar a conexão
        Connection conn = DriverManager.getConnection(URL, USUARIO, SENHA);
        // Retorna a conexao aberta
        return conn;
    }
}


18
  • 3
    Você está fazendo uma conexão por consulta? Normalmente você faz a conexão fora do loop, faz o que tem que fazer e fecha depois. Dependendo do caso, dá pra deixar a conexão aberta durante a vida do aplicativo, mas depende da escala em que vai ser usado.
    – Largato
    11/05/2016 às 16:50
  • 2
    Na verdade, não se trata de boa prática ou não (não existe nenhuma razão para algo ser chamado de "boa prática" ou "má prática" em programação, isso é uma lenda furada). No caso, é errado mesmo, e a razão verdadeira é o que o Caffé explicou depois da parte da boa prática :)
    – Largato
    11/05/2016 às 17:02
  • 1
    @Bacco Só pra não ir muito longe, atendo-se apenas a este exemplo específico, não é errado como você falou. É apenas má prática - afinal compila, passa no teste automatizado, e até funciona pra uma quantidade menor de registros, e poderia nunca dar erro. Então não é necessariamente errado, mas é necessariamente uma má prática, que eventualmente pode se revelar (como acabou se revelando). Acreditar na baboseira de que "não existe boa prática" não é uma boa prática :D
    – Caffé
    11/05/2016 às 17:20
  • 2
    @caffé não é questão de fé, é que na prática, tudo que é chamado de boa prática ou má prática precisa de uma explicação, então o termo é apenas ruído. E como é um termo que em nada acrescenta de informação palpável, acaba sendo usado absurdamente mais por quem não tem como se explicar. Vira uma muleta, e seu uso continuado acaba refletindo um problema maior do usuário. Que o meu termo "errado" não seja o melhor, mas "má prática" também não diz nada. Melhor falar que é "problemático por motivo x, y, z", que é o que você explicou depois, pondo o foco na razão do problema, e não no termo-coringa.
    – Largato
    11/05/2016 às 17:26
  • 2
    @Caffé fica a cargo dos leitores optar por seguir suas recomendações ou as minhas, acho que é desnecessário eu prolongar a discussão aqui, insisto no meu ponto com uma razão que vai muito além da sua pessoa, aproveite quem quiser. Mas se você acha que do seu jeito é melhor pras pessoas, siga em frente ué. MInha experiência diz que não, mas também diz que não tem sentido ficarmos discutindo (até pq não é você exatamente que eu quero convencer de nada, e acredito que a recíproca seja verdadeira). Assim, me dou por satisfeito com o que eu disse anteriormente.
    – Largato
    11/05/2016 às 17:33

1 Resposta 1

1

Olá,

Olhando apenas o método, está correto. Você faz a consulta e fecha os ResultSet e PreparedStatement em seguida. Eu apenas colocaria os closed() no finally, junto com a conexão.

Como o pessoal diz nos comentários.. O problema esta em você chamar isso "300" vezes.. pq quando pedimos para fazer um closed nos objetos, esse closed não é imediato. Pode levar alguns milisegundos, e nesse meio termo, você continua bombardeando a base pedindo mais conexões.

A solução seria usar a mesma conexão, então tente diminuir as chamadas ao getConnection... fazer algo como manter uma única conexão e usar essa para fazer as 300 chamadas:

public static Connection abrir() throws Exception {
    Connection conn = null;
    if (conn != null) {
      Class.forName(DRIVER);
      conn = DriverManager.getConnection(URL, USUARIO, SENHA);
    }
    return conn;
}

Um teste que costumo fazer é monitorar na base as conexões existentes, isso costuma deixar bem claro o que acontece.

É errado deixar assim.. mas apenas para teste, comente o cx.close(), mesmo que a conexão fique aberta o tempo todo, você consegue testar e confirmar onde esta o problema.. então tentar deixar fora do for esse passo.

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .