Usuário só loga uma vez

Prezados,
Consegui resolver o problema simplismente não dando close na conexão depois de fazer as transações.
Dai lembrei que minha professora também nunca dava close no connection.
Tem algum problema não dar close no connection?
O fato de não dar close no connection não faz meu programa ficar vulnerável?

bem, se vc não fechar a conexao, sua máquiba vai ficar sEMPRE, com a conexão aberta,o que vai sobrecarregar a memória.
o melhor é: apos cada consulta, fechar a conexçao.
nova consulta: reabre, consualta, fecha e depois maniupla o resultado.

abraços

Pois é, existe uma “política” (digamos assim) para a conexões ao banco em aplicativos web que sugere vc abrir uma conexão POR REQUEST, e não por COMANDOS SQL.

Uma outra opção mais simples é o uso de DATASOURCES. E como vc configura quase tudo no ambiente, pouca coisa vc alteria no código em si.
O container ficaria encarregado de abrir e fechar as conexções para vc. Ele monta o famoso pool de conexões e te oferta um java.sql.Connection viável sempre que vc requisitar.

Na documentação do tomcat tem explicando…
http://tomcat.apache.org/tomcat-6.0-doc/jndi-datasource-examples-howto.html

[]s

Valeu gente boa!

Mas da pra fazer abrindo e fechando conexão em cada consulta ou update?

Bom, eu nunca usei Datasources pra valer, não tenho certeza pra te confirmar isso.
Mas, se não me engano, quando vc manda fechar a conexão na verdade vc tá solicitando ao container para fechá-lo quando convir.
É melhor vc dar uma olhada direito, posso está falando besteira, hehe

[]s

já pensou se o usuário digitar o nome dele assim:

'; delete from usuario where nome not like ’

fre_d:

já pensou se o usuário digitar o nome dele assim:

'; delete from usuario where nome not like '

É só usar prepared statement, e evitar o injection.

Para a conexão não ficar pesando na memoria no seu metodo estatico getConexão da classe Conexão vc pode usar um singleton (Padrão de projeto onde so retorna uma instancia.)

Mais ou menos assim o

public class Conexao{

 private Connection con = null;

 public static Connection getConnection(){
 
          if(con == null){
               
                   //Aqui vc faz uma nova conexão
           }
           return con;

 }


 public static void closeConnection(){
   
         con.close();
         con = null;

  }

}

Assim vc pode fechar e abrir a conexão de qualquer Servlet/JSP.

Felagund:

fre_d:

já pensou se o usuário digitar o nome dele assim:

'; delete from usuario where nome not like '

É só usar prepared statement, e evitar o injection.

Para a conexão não ficar pesando na memoria no seu metodo estatico getConexão da classe Conexão vc pode usar um singleton (Padrão de projeto onde so retorna uma instancia.)

Mais ou menos assim o

public class Conexao{

 private Connection con = null;

 public static Connection getConnection(){
 
          if(con == null){
               
                   //Aqui vc faz uma nova conexão
           }
           return con;

 }


 public static void closeConnection(){
   
         con.close();
         con = null;

  }

}

Assim vc pode fechar e abrir a conexão de qualquer Servlet/JSP.

Pois é, mas mesmo assim cuidado com a hora de fechar a conexão.

Fala gente boa.
uchoaaa: Estive conversando com um professor meu e ele me disse exatamente isso: que se eu fechar a conexão eu estou mandando o container fechá-la, assim eu teria que reiniciar o tomcat pra poder abrir outra conexão.

fre_d: Não saquei! Se ele digitar isso, como você faz a verificação de coneção cada vez que entra. Ele não vai logar, muito menos executar o statement. No meu caso eu consulto o usuário que tenta logar e retorna do banco se não bater senha e login não sai da tela de login. Recebendo erro.

Felagund: Grande eu estou tentando fazer assim mesmo, como você falou.

public static Connection getConexao(String nome, String senha) throws Exception {
		if(conexao == null){
			Class.forName("org.postgresql.Driver").newInstance();
			conexao = DriverManager.getConnection("jdbc:postgresql:escola",nome,senha);
		}//fim do if
		return conexao;
	}//fim do método getConexao

Mas quando eu fecho a conexão, não consigo criar outra se não reiniar o tomcat 6.0.
Então to fazendo dessa forma mas devolvendo um statement e não connection.

package conexao;

import java.sql.*;
public class Conexao {
	public static Connection conexao = null;
	private static Statement statement;
	
	public static Statement getConexao(String nome, String senha) throws Exception {
		if(conexao == null){
			Class.forName("org.postgresql.Driver").newInstance();
			conexao = DriverManager.getConnection("jdbc:postgresql:escola",nome,senha);
		}//fim do if
		statement = conexao.createStatement();
		
		return statement;
	}//fim do método getConexao
	
}//fim da classe

Valeu pessoal.
Qualquer sugestão “posta” aí.

Sim, sim… Faz sentido, não?!
Quando vc fecha sua conexão, ela não fica nula, fica apenas fechada(!). Ou seja, quando vc chamar getConnection() depois de um close() o seu IF aí nunca vai ser true.

O que vc pode fazer é verificar ser além de nula sua conexão está fechada. Mas daí daria quase na mesma: se fechar a conexao toda vez que executar um comando SQL vc vai ter uma instância para cara próxima chamada de getConexao()…

Ah, outra coisa, seu problema não é com login, é com a conexão com o banco…

[]s

:shock: :shock: :shock:
Cara! Desculpe a todos pelo tempo.
E muito obrigado.
Porra como eu não me liguei nisso. Fiz e nem percebi.
Sem palavras, valeu pessoal.

leohunther:

:shock: :shock: :shock:
Cara! Desculpe a todos pelo tempo.
E muito obrigado.
Porra como eu não me liguei nisso. Fiz e nem percebi.
Sem palavras, valeu pessoal.

Cara, se achas que resolvestes, explica aqui como e marca o título do post como resolvido…
Podes ajudar outros com dúvidas semelhantes.

[]s

leohunther:

Fala gente boa.

Estive conversando com um professor meu e ele me disse exatamente isso: que se eu fechar a conexão eu estou mandando o container fechá-la, assim eu teria que reiniciar o tomcat pra poder abrir outra conexão.

Isso ta errado. Da sim para fechar e abrir a conexão.
Meu problema era mesmo no “conection == null”.
Conexão fechada não é igual a conexão null. Então o código nunca voltava uma conexão.