Optimisation d'un requête dynamique

Version imprimable

Bonjour,

Comme dit le titre,
je viens de créer une requête mais le code de celle-ci me semble bien moche...
Je dois mal m'y prendre ou autre. Quelqu'un aurrait t'il des conseils SVP.

Je vous remercie d'avance.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
 
public ArrayList<Contrat> findListContrat(int numContrat, String nomContrat, String prenomContrat, Date dateNaissanceContrat, int codePostalContrat){
 
		// déclaration
		ArrayList<Contrat> listeContrat = new ArrayList<Contrat>(); 
		int argRequete = -1;
		String sql = "SELECT * FROM Contrat"; 
 
		// recupération du nombre de d'argument
		if (numContrat != 0)argRequete++;
		if (nomContrat != null)argRequete++;
		if (prenomContrat != null)argRequete++;
		if (dateNaissanceContrat != null)argRequete++;
		if (codePostalContrat != 0)argRequete++;
 
		// création de la requête
		if ((numContrat != 0) || (nomContrat != null) || (prenomContrat != null) || (dateNaissanceContrat != null) || (codePostalContrat != 0)){
			sql = sql + " WHERE";
		}
		if (numContrat != 0){
			sql = sql + " numContrat = " + numContrat;
			if (argRequete != 0){
				sql = sql + " AND";
				argRequete--;
			}
		}
		if (nomContrat != null){
			sql = sql + " nomContrat = '" + nomContrat + "'";
			if (argRequete != 0){
				sql = sql + " AND";
				argRequete--;
			}
		}
		if (prenomContrat != null){
			sql = sql + " prenomContrat = '" + prenomContrat + "'";
			if (argRequete != 0){
				sql = sql + " AND";
				argRequete--;
			}
		}
		if (dateNaissanceContrat != null){
			sql = sql + " dateNaissanceContrat = " + dateNaissanceContrat;
			if (argRequete != 0){
				sql = sql + " AND";
				argRequete--;
			}
		}
		if (codePostalContrat != 0){
			sql = sql + " codePostalContrat = " + codePostalContrat;
			if (argRequete != 0){
				sql = sql + "AND";
				argRequete--;
			}
		}
 
		System.out.println(sql);
 
                          //...
}

idée comme ça en passant utilise MessageFormat:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
Object[] args = {  numContrat!=0?numContrat: null, 
                           nomContrat,
                           prenomContrat,
                           // etc. etc.
                        } ;
 
for(int ix=0; ix < args.length, ix++ ) {
   // construire un pattern pour Message format
   // quand un object manque son numero n'est pas dans le pattern
   // on gère where et AND en fonction d'un booléen
}
  MessageFormat.format(pattern, args) ;
  //sachant que le pattern ne mentionne pas les numeros de objets manquants

29/01/2013, 12h55
andry.aime

Bonjour,

Il est à éviter de faire des concaténations de String, utilise un StringBuilder.

A+.

En gros je travaillerais avec un map + preparedStatement, c'est plus clair et ça évite l'injection sql

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
StringBuilder query = new StringBuilder();
Map<String,Object> parametres = new HashMap<String,Object>();
if (numContrat != 0)
 parametres.put("numContrat",(Integer)numContrat);
if (nomContrat != null){
 parametres.put("nomContrat",nomContrat);
//...
for (Map.Entry<String,Object> entry : parametres.entrySet()){
   if (query.length()>0)
     query.append(" AND ");
   query.append(entry.getKey).append(" = ? ");
}
String sqlQuery = null;
if (query.length()>0)
   sqlQuery = "SELECT * FROM Contrat WHERE "+query; 
else
   sqlQuery = "SELECT * FROM Contrat"; 
PreparedStatement ps = connection.prepareStatement(sqlQuery);
int i=0;
for (Map.Entry<String,Object> entry : parametres.entrySet())
   ps.setObject(++i);
}
ResultSet rs = ps.executeQuery();
//...

29/01/2013, 14h01
v4np13

Bonjour,

Pourquoi ne pas utiliser l'API criteria ? Ce serait beaucoup plus simple/sûr à gérer que des concaténations de Strings.

Cordialement,
29/01/2013, 14h17
tchize_

Citation:

Envoyé par v4np13

Bonjour,

Pourquoi ne pas utiliser l'API criteria ? Ce serait beaucoup plus simple/sûr à gérer que des concaténations de Strings.

Cordialement,

Criteria c'est du hibernate, je ne pense pas qu'il va installer tout le bordel d'hibernate juste pour faire une query sql :aie:
29/01/2013, 14h27
v4np13

Bah, ce sera plus simple à maintenir.

Là, il y a beaucoup de sources d'erreur possibles. Le code présenté s'apparente plus à du code spaghetti (il manque juste un goto :aie:).

Si sa query porte sur une table, le mapping sera très rapide puis son code business sera simplifié.

Autant partir sur de bonnes bases, je croirai, sans vouloir être méchant, relire du code d'il y a 10-20 ans...

Cordialement,
29/01/2013, 14h34
Addjira

Merci pour toutes ces indications je vais arrenger tous sa.