BitCell

by **valentin** » 25 Jan 2011, 21:29

Line number On/Off | Expand/Contract | Select all

import java.sql.*;
import java.util.Properties;
 
public class ConnectionManager
{
  private static final String dbClassName = "com.mysql.jdbc.Driver";
  private static String CONNECTION;
  public void ConnectionManager(final String database, final String username, final String password)    {
      CONNECTION = "jdbc:mysql://127.0.0.1/" + database;
      Class.forName(dbClassName);
      Properties connectionProperties = new Properties();
      connectionProperties.put("user", username);
      connectionProperties.put("password", password);
      Connection connect = DriverManager.getConnection(CONNECTION,connectionProperties);
  }
 public void closeConnection() {
     connect.close();
 }
 protected void finalize() {
     try {
         closeConnection();
     }
     finally {
         connect.close();
     }
 }
}
 
 

Unde ar trebui sa tratez exceptiile: in constructor sau atunci cand folosesc obiectul de tip ConnectionManager ?

Alte sugestii, probleme, optimizari pentru clasa de mai sus ?

by **smith** » 25 Jan 2011, 22:10

Dacă în constructorul tău ar putea să apară excepții (dintr-o cauză oarecare -imposibilitate de conectare etc.) atunci cred că ar trebui să fie transmise mai departe cu ajutorul keywordului throws. Când vei folosi obiecte de tipul ConnectionManager, la construcția lor vor putea să apară probleme, deci construcția unui obiect ConnectionManager cred că ar fi bine să fie într-un bloc try.
De asemenea cred că metoda finalize sau closeConnection ar trebui să verifice dacă conexiunea EXISTĂ și doar în cazul ăla să închidă conexiunea.
-dacă greșesc, să mă corecteze cineva.

LE:
Am băgat codul în eclipse și îmi apar niște probleme. În primul rând, nu înțeleg de ce constructorul tău returnează void.
Iată cum cred că ar trebui să arate codul:

Line number On/Off | Expand/Contract | Select all

import java.sql.*;
import java.util.Properties;
 
public class ConnectionManager{
    private static final String dbClassName = "com.mysql.jdbc.Driver";
    private static String CONNECTION;
    private Connection connect;
 
    public ConnectionManager(final String database, final String username,
            final String password) throws ClassNotFoundException, SQLException {
        CONNECTION = "jdbc:mysql://127.0.0.1/" + database;
        Class.forName(dbClassName);
        Properties connectionProperties = new Properties();
        connectionProperties.put("user", username);
        connectionProperties.put("password", password);
        Connection connect = DriverManager.getConnection(CONNECTION,
                connectionProperties);
    }
 
    public void closeConnection() throws SQLException {
        if (!connect.isClosed())
            connect.close();
    }
 
    protected void finalize() throws SQLException {
        try {
            closeConnection();
        } finally {
            //eliberare memorie obiect ?
        }
    

by **morpheus** » 25 Jan 2011, 22:17

In plus, nu ar trebui sa pui niciodata codul care inchide conexiunea catre baza de date in finalize().
Ar trebui sa fie intr-o metoda chemata de tine din codul client, recomandabil intr-un bloc finally.

by **valentin** » 26 Jan 2011, 19:43

^ si ^^: Multumesc mult !

BitCell

Tratarea exceptiilor - JDBC

Tratarea exceptiilor - JDBC

Re: Tratarea exceptiilor - JDBC

Re: Tratarea exceptiilor - JDBC

Re: Tratarea exceptiilor - JDBC

Who is online