Mysql Connection Close?

[dennisT]

Hi, ich habe diese Ausschnitt, wo ich mal eine Frage zu habe. Die Verbindung zu Mysql schließe ich ja mit .close(). Frage: Brauche ich das hier an der Stelle (über dem "return;")?

try{
          Connection con = DriverManager.getConnection("jdbc:mysql://localhost/...","root","");
          Statement stmt = con.createStatement();
          ResultSet rslt = stmt.executeQuery("select * from ... where ... = '"+...+"'");
          if(rslt.next() == false){
            stmt.close();
            con.close();
            return;
          }
          stmt.close();
          con.close();
        }
        catch (Exception f){}

 

[Thief]

ich würds so machen:

public void xxxx()
{

   try{
   }catch(){
   }finally{
     con.close(); 
   }
}

das wird dann immer ausgeführt bevor die methode verlassen wird.

 

[dennisT]

Kann man machen, nur aus interesse, ist das denn richtig, wie ich das gemacht habe?

 

[maki]

COnnection#close hat eine SQLException in der Signatur, also eine sog. checked Exception, soll heissen: Da muss entweder noch ein try/catch drumrum oder ein throws ist angesagt.

 

[Nicer]

public void xxx() throws Exception 
{

(...)

try
{
stmt.close();
con.close();
}
catch (Exception e)
{
e.printStackTrace()
}
}

so sollte das gehn

 

[dennisT]

COnnection#close hat eine SQLException in der Signatur, also eine sog. checked Exception, soll heissen: Da muss entweder noch ein try/catch drumrum oder ein throws ist angesagt.

Nochmal unter der if Abfrage ein Try/Catch?

 

[Nicer]

siehe oben

 

[dennisT]

So meintest du, oder?

try{
          Connection con = DriverManager.getConnection("jdbc:mysql://localhost/...","root","");
          Statement stmt = con.createStatement();
          ResultSet rslt = stmt.executeQuery("select * from ... where ... ");
          if(rslt.next() == false){
            try{
              stmt.close();
              con.close();
            }
            catch(Exception f){
              f.printStackTrace();
            }
            return;
          }
          stmt.close();
          con.close();
        }
        catch (Exception f){}

 

[Nicer]

so in etwa , funktioniert das so ?

 

[dennisT]

so in etwa , funktioniert das so ?

Ich kann keine Fehler feststellen. Weißt du ob man irgendwie rausfinden kann, ob die Verbindung noch offen ist, sprich close nicht funktionierte?

 

[Nicer]

versuch einfach mal nach dem closen noch irgendwas aus dem SQL ausgeben kannst , zB einfach nen System.out.println und gib da einen String aus der DB mit oder so.

wenn du dann eine SQLException , NullPointerException oder eine FileNotFoundException bekommst dann ist der close erfolgreich.

 

[eRaaaa]

LOL :lol:

.... btw: es gibt auch die Methode isClosed() :toll:

 

[dennisT]

vjava.sql.SQLException: No operations allowed after statement closed.

Dann sollte es ja geklappt haben. Danke

 

[Nicer]

*rot wird*

ich bauer ^^

jo , mit isClosed(); lässt sich das natürlich viel leichter feststellen ^.^ mach dann halt eine If Abfrage und mach bei true einfach eine Ausgabe dassdu siehst dass es true ist

edit : grade gesehn es hat auch mit meinem umständlichen vorschlag geklappt ^^
bitte dann noch als erledigt makieren

 

[maki]

Um jedes close müsste eigentlich ein eigener try/catch Block, auch um sicherzustellen dass die nachfolgeneden (close) Operationen noch abgearbeitet werden, und nebenbei bemerkt, [c]Exception[/c] zu fangen/zu werfen ist wohl das schlechteste was man machen kann in 99,9% der Fälle so wie hier

Korrekter JDBC Code ist "hässlich" und unintuitiv, man kann aber viel kapseln, siehe die ganzen Frameworks für Peristenz Lösungen.