1 files changed, 32 insertions, 30 deletions

diff --git a/src/dao/JDBC.java b/src/dao/JDBC.java
index 714803e..324ff2d 100644
--- a/src/dao/JDBC.java
+++ b/src/dao/JDBC.java
@@ -5,9 +5,9 @@ import java.sql.DriverManager;
 import java.sql.SQLException;
 
 /**
- * connexion à la BDD avec JDBC
- * tous les paramètres de connexion et les setters sont statiques
- * le nom de la table sélectionnée est dans ModelCRUD 
+ * connexion avec JDBC pour utilisation par la classe fille: SQLexecutor,
+ * les paramètres de connexion et les setters sont statiques
+ *  
  */
 abstract public class JDBC
 {
@@ -20,25 +20,31 @@ abstract public class JDBC
         private static String host= "localhost";
         private static String strUrl;
         
+        /**
+         * retourne une instance de Connection avec JDBC
+         * @return instance de Connection
+         * @throws SQLException en cas d'erreur de connexion
+         * @throws ClassNotFoundException pour Class.forName()
+         */
         protected Connection getConnection() throws SQLException, ClassNotFoundException
         {
                 Class.forName(strClassName);
                 return DriverManager.getConnection(strUrl, login, password);
-//              try {
-//                      Class.forName(strClassName); // chargement du driver
-//                      return DriverManager.getConnection(strUrl, login, password);
-//              }
-//              catch(ClassNotFoundException e) {  
-//                      System.err.println("Driver non chargé !");
-//                      e.printStackTrace();
-//              }
-//              catch(SQLException e) {
-//                      e.printStackTrace();
-//              }
-//              return null;
         }
         
         // setters
+        /**
+         * génère l'url utilisée par le driver,
+         * à n'exécuter que lorsque tous les paramètres ont été renseignés (en particulier dbname, login et password),
+         * certains: host, dbms, strClassName et port ont des valeurs par défaut pour un serveur mysql sur localhost
+         */
+        public static void setStrUrl() {
+                if(dbname != null && login != null && password != null) {
+                        JDBC.strUrl = "jdbc:" + dbms + "://" + host + ":" + port + "/" +  dbname;
+                }
+                else System.out.print("il manque des paramètres nécéssaires à la connexion au driver");
+        }
+        
         public static void setHost(String host) {
                 JDBC.host = host;
         }
@@ -55,18 +61,25 @@ abstract public class JDBC
                         JDBC.port = "3306";
                 }
         }
-        private static void setStrUrl() {
-                JDBC.strUrl = "jdbc:" + dbms + "://" + host + ":" + port + "/" +  dbname;
-        }
         public static void setDbName(String dbName) {
                 JDBC.dbname = dbName;
         }
         public static void setLogin(String login) {
                 JDBC.login = login;
         }
-        public static void setMotdepasse(String motdepasse) {
+        public static void setPassword(String motdepasse) {
                 JDBC.password = motdepasse;
         }
+        
+        /**
+         * donner tous les paramètres de connexion en une fois
+         * il est également possible de les donner un par un avec les setters en finissant pas JDBC.setStrUrl()
+         * @param host adresse du serveur
+         * @param dbms obtenir le nom du driver et le numéro du port à partir du nom du SGBD
+         * @param dbname de la base de données
+         * @param login nom d'utilisateur
+         * @param password mot de passe
+         */
         public static void setInfos(String host, String dbms, String dbname, String login, String password)
         {
                 JDBC.dbname = dbname;
@@ -76,15 +89,4 @@ abstract public class JDBC
                 setDBMS(dbms);
                 JDBC.setStrUrl();
         }
-        
-//      public void close()
-//      {
-//              try {
-//                      this.conn.close(); // libérer le slot (même principe que l'attaque DDOS)
-//                      this.conn = null; // pour pouvoir réinstancier avec d'autres paramètres
-//              }
-//              catch (SQLException e) {
-//                      e.printStackTrace();
-//              }
-//      }
 }