Discussion :

[Addjira]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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);
 
                          //...
}

[professeur shadoko]

idée comme ça en passant utilise MessageFormat:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[andry.aime]

Bonjour,

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

A+.

[tchize_]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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();
//...

[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,

[tchize_]

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

[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 ).

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,

[Addjira]

Merci pour toutes ces indications je vais arrenger tous sa.