1 files changed, 23 insertions, 25 deletions

diff --git a/src/dao/SQLexecutor.java b/src/dao/SQLexecutor.java
index 8f7f130..430c1c0 100644
--- a/src/dao/SQLexecutor.java
+++ b/src/dao/SQLexecutor.java
@@ -5,13 +5,20 @@ import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
 
+/**
+ * exécuter des requêtes SQL:
+ * connexion avec JDBC => préparation de la requête => exécution
+ * => copie du ResultSet dans une ArrayList d'HashMap => fermeture de la connexion 
+ */
 public class SQLexecutor extends JDBC
 {
+        // constructeur par défaut
+        
         /**
          * Exécute une requête SQL et retourne le résultat si applicable.
          * @param sql requête avec des ?
-         * @param params Object... permet d'ajouter autant de paramètres qu'on veut et ça fait un tableau
-         * @return Un ResultSet si la requête est un SELECT, sinon null
+         * @param params nombre libre de paramètres, "Object..." permet de créer un tableau à partir des paramètres
+         * @return ResultSet si la requête est un SELECT, null sinon
          * @throws SQLException en cas d'erreur SQL
          * @throws ClassNotFoundException pour DriverManager.getConnection
          */
@@ -24,26 +31,19 @@ public class SQLexecutor extends JDBC
             try {
                 connection = getConnection();
                 boolean isSelect = sql.trim().toLowerCase().startsWith("select");
-        
+                
+                statement = connection.prepareStatement(sql);
                         if (params.length > 0)
                         {
-                                int option;
-                                if(isSelect)
-                                {
-                                        option = ResultSet.TYPE_SCROLL_INSENSITIVE;
-                                        // navigation possible dans le ResultSet: next(), previous(), absolute();
-                                        // ResultSet non modifiable en cas d'accès concurrent en mémoire
-                                }
-                                else
-                                {
-                                        option = PreparedStatement.NO_GENERATED_KEYS;
-                                        // optimisation en ne retournant pas les clé générés lors d'un INSERT (RETURN_GENERATED_KEYS)
-                                }
-                                statement = connection.prepareStatement(sql, option);
-                                        //isSelect ? ResultSet.TYPE_SCROLL_INSENSITIVE : PreparedStatement.NO_GENERATED_KEYS);
+// if(isSelect)
+// => ResultSet.TYPE_SCROLL_INSENSITIVE;
+// navigation possible dans le ResultSet: next(), previous(), absolute();
+// ResultSet non modifiable en cas d'accès concurrent en mémoire
+// => PreparedStatement.NO_GENERATED_KEYS;
+// optimisation en ne retournant pas les clé générés lors d'un INSERT (RETURN_GENERATED_KEYS)
+// statement = connection.prepareStatement(sql, isSelect ? ResultSet.TYPE_SCROLL_INSENSITIVE : PreparedStatement.NO_GENERATED_KEYS);
+                                
                                 this.setParameters(statement, params);
-                        } else {
-                                statement = connection.prepareStatement(sql);
                         }
                         
                         if(isSelect) {
@@ -51,17 +51,15 @@ public class SQLexecutor extends JDBC
                                 ResultObject result = new ResultObject(resultSet);
                                 resultSet.close();
                                 return result;
-                    } else {
+                    }
+                        else {
                                 statement.executeUpdate();
                                 return null;
                     }
-                } finally {
+                }
+            finally {
                         statement.close();
             connection.close();
-//                  if (resultSet == null) {
-//                      statement.close();
-//                  connection.close();
-//              }
             }
         }