Fechando com o Finally

24 de julho de 2008 14 respostas

Naruto 24 de julho de 2008

boa tarde... como eu faria para fechar isso com o finally???

public boolean inserir(ContatosDTO contatosDTO){
		try{
			Connection con = Conexao.getConnection();
			PreparedStatement stmt = con.prepareStatement("Insert into contatos" +
					"(nome, data, endereco) values(?,?,?)");
			
		
			
			stmt.setString(1, contatosDTO.getNome());			
			stmt.setDate(2, contatosDTO.getData());			
			stmt.setString(3, contatosDTO.getEndereco());
			
			stmt.execute();
			stmt.close();
			System.out.println("Gravado com sucesso!");
			con.close();
			return true;
			
		}catch (SQLException e){
			e.printStackTrace();
			return false;
		}
	}

pois c ouver um erro ela não fechara o.O alguem pode me ajudar??
vlwwwwww

14 Respostas

fnandos 24 de jul. de 2008

finally{ con.close; return false; }

thingol 24 de jul. de 2008

Declare o objeto Connection fora do try, é lógico.
Mais ou menos assim:

Connection conn = null;
try {
...
} finally {
    try {if (conn != null) conn.close();}catch (SQLException ex) {}
}

flaviojunior 24 de jul. de 2008

public boolean inserir(ContatosDTO contatosDTO){
	Connection con = Conexao.getConnection();
	try{
		PreparedStatement stmt = con.prepareStatement("Insert into contatos" +
				"(nome, data, endereco) values(?,?,?)");
			
		
			
		stmt.setString(1, contatosDTO.getNome());			
		stmt.setDate(2, contatosDTO.getData());			
		stmt.setString(3, contatosDTO.getEndereco());
			
		stmt.execute();
		System.out.println("Gravado com sucesso!");			
		}catch (SQLException e){
			e.printStackTrace();
			return false;
		} finally{  
			stmt.close();
			con.close;  
		}  
		return true;
	}

Naruto 24 de jul. de 2008

nem consegui o.O

flaviojunior 24 de jul. de 2008

Naruto:
nem consegui o.O

o que aconteceu??? qual erro deu?

Naruto 24 de jul. de 2008

tp ele não reconheceu o

} finally{ stmt.close(); con.close; } return true;

depois do catch ele nem reconhece o stmt.close();
e nem o con.close;

flaviojunior 24 de jul. de 2008

Naruto:
tp ele não reconheceu o

} finally{ stmt.close(); con.close; } return true;

depois do catch ele nem reconhece o stmt.close();
e nem o con.close;

vc tentou igual o thingol falou?

Naruto 24 de jul. de 2008

igual a ele ainda não tinha feito igual ao seu deixa eu ver igual ao dele

Naruto 24 de jul. de 2008

foda que eu não consegui afzer do jeito dele… so meio ignorante nisso ainda

fenrir 24 de jul. de 2008

Todos os objetos que são criados dentro do try, não são visíveis fora dele (e o bloco catch e finally estão fora do try).

Portanto, você deveria fazer algo assim:

public boolean inserir(ContatosDTO contatosDTO){  
    Connection con = null;  
    PreparedStatement stmt = null;
    try{  
        con = Conexao.getConnection();  
        stmt = con.prepareStatement("Insert into contatos" +   "(nome, data, endereco) values(?,?,?)");  
        stmt.setString(1, contatosDTO.getNome());             
        stmt.setDate(2, contatosDTO.getData());           
        stmt.setString(3, contatosDTO.getEndereco());  

        stmt.execute();  
        System.out.println("Gravado com sucesso!");           
    }catch (SQLException e){  
        e.printStackTrace();  
        return false;  
    } finally{    
        if (stmt != null) {
            try {
                stmt.close();
            } catch (SQLException ignored) { }
        }
        if (con != null) {
            try {
                con.close();
            } catch (SQLException ignored) { }
        }
    }    
    return true;  
}

ViniGodoy 24 de jul. de 2008

Você poderia ler o que o thingol escreveu...

Outra forma é fazer isso:

public void inserir(ContatosDTO contatosDTO){   
   try{   
      Connection con = Conexao.getConnection();   
      PreparedStatement stmt = con.prepareStatement("Insert into contatos" +   
         "(nome, data, endereco) values(?,?,?)");   
      try {   
         stmt.setString(1, contatosDTO.getNome());             
         stmt.setDate(2, contatosDTO.getData());           
         stmt.setString(3, contatosDTO.getEndereco());   
               
         stmt.execute();   
         stmt.close();   
         System.out.println("Gravado com sucesso!");   
         return true;                  
      } finally {
         if (stmt != null)
            stmt.close();
         if (con != null)
            con.close();
      }
   }catch (SQLException e){   
      throw new RuntimeException(e);
   }
}

Naruto 24 de jul. de 2008

vlww a vcs dois vou estudar mais sobre isso o.O pis aida fiquei meio confuso de entender mas é isso mesmo que eu queria deu certim
vlwww

fenrir 24 de jul. de 2008

ViniGodoy:

public void inserir(ContatosDTO contatosDTO){   
   try{   
      Connection con = Conexao.getConnection();   
      PreparedStatement stmt = con.prepareStatement("Insert into contatos" +   
         "(nome, data, endereco) values(?,?,?)");   
      try {   
         stmt.setString(1, contatosDTO.getNome());             
         stmt.setDate(2, contatosDTO.getData());           
         stmt.setString(3, contatosDTO.getEndereco());   
               
         stmt.execute();   
         stmt.close();   
         System.out.println("Gravado com sucesso!");   
         return true;                  
      } finally {
         if (stmt != null)
            stmt.close();
         if (con != null)
            con.close();
      }
   }catch (SQLException e){   
      throw new RuntimeException(e);
   }
}

Dessa forma, se der erro na criação do PreparedStatement, a conexão com o banco continuará ativa, já que ela seria fechada apenas no try interno.

ViniGodoy 24 de jul. de 2008

Tem razão. Tem que declarar eles como nulo, e só inicia-los no try interno (é como eu normalmente fazia, antes do Spring):

public void inserir(ContatosDTO contatosDTO){     
   try{     
      Connection con = null;     
      PreparedStatement stmt = null;     
      try {     
         con = Conexao.getConnection();
         stmt = con.prepareStatement("Insert into contatos" +     
         "(nome, data, endereco) values(?,?,?)");

         stmt.setString(1, contatosDTO.getNome());               
         stmt.setDate(2, contatosDTO.getData());             
         stmt.setString(3, contatosDTO.getEndereco());     
                 
         stmt.execute();     
         stmt.close();     
         System.out.println("Gravado com sucesso!");     
         return true;                     
      } finally {   
         if (stmt != null)   
            stmt.close();   
         if (con != null)   
            con.close();   
      }   
   }catch (SQLException e){     
      throw new RuntimeException(e);   
   }   
}

Criado 24 de julho de 2008

Ultima resposta 24 de jul. de 2008

Respostas 14

Participantes 6

14 Respostas

Topicos relacionados