Créer des requêtes SQL avec des méthodes comme en PHP avec QueryBuilder

Version imprimable

Bonsoir,

Je souhaiterai faire en Java des requêtes SQL avec des méthodes comme en PHP, je vous montre un exemple en PHP :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
$qb = $this->createQueryBuilder('a');
        $qb->select('
            a.id,
            a.annee, 
            a.prix, 
            ma.marque, 
            mo.modele, 
            p.emplacement AS photos
        ');
 
        $qb->join('a.marque', 'ma')
           ->join('a.modele', 'mo')
           ->join('a.carburant', 'c')
           ->join('a.photos', 'p')
           ->where('p.estPrincipal = 1');
        if( ! empty($marqueId))
           $qb->andWhere('ma.id = :marqueId')
               ........................................
        return $qb->getQuery()->getResult();

Est-ce que l'on peut faire de telles requêtes en JAVA et faire aussi des conditions comme je l'ai fait dans le code ci-dessus ?

07/01/2023, 04h00
marc.collin

regarde du côté de spring data jpa / spring data jdbc
dépendant si tu veux utiliser un orm ou non
07/01/2023, 13h14
hbx360

Peut-être qu'un professionnel Java pourrai me montrer un exemple de code correspondant à celui que j'ai donné en PHP ?
07/01/2023, 19h21
thelvin

Hello, il n'y a pas d'équivalent très connu en Java.

D'un point de vue de ce qui se fait en Java, cet exemple PHP fait les choses à moitié, et on peut se demander pourquoi. Logique : si on fait des requêtes SQL on fait des requêtes SQL, et si on veut abstraire la syntaxe SQL alors on l'abstrait vraiment, on oblige pas l'utilisateur à remplacer le fait de taper JOIN par le fait d'appeler des fonctions appelées JOIN.

- Pour faire l'équivalent en étant au plus près de SQL, on écrit la requête telle que, à cela près qu'on y injecte les valeurs au lieu de les écrire directement, pour faciliter l'écriture et empêcher les injections SQL.
- Pour faire l'équivalent avec abstraction de SQL, on définit des classes qui représentent les données contenues dans les tables, et on décrit quelles colonnes sont jointables. Le select est un appel à get() ou à find() que le système écrit lui-même.

Dans les deux cas, ça fonctionne trop différemment pour montrer "la même chose". Ce n'est pas la même chose.

Il n'y a pas de raison de faire un équivalent sous forme de méthode pour des choses aussi basiques (sans compter qu'il semble manquer des éléments à cet exemple).
Pour illustrer mon propos (qui pourrait paraître un peu brutal), voici ce que je ferais.
En utilisant JDBC (l'api bas niveau de l'accès aux bases de données en java), le plus simple est de construire une chaîne de caractère représentant la requête à exécuter, quelque chose comme ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 StringBuilder sb = new StringBuilder(); sb.append("select a.id, a.annee, a.prix, ma.marque, mo.modele, p.emplacement AS photos"); sb.append(" from schema.table1 a"); sb.append(" join schema.table2 b on a.id = b.id_table1"); sb.append(" where 1=1"); if (monParametreId != null) { sb.append(" and a.id = ?") } PreparedStatement pstmt = connection.prepareStatement(sb.toString()); pstmt.setInt(1, monParametreId); ResultSet rs = pstmt.executeQuery(); while (rs.next()) { ... on traite les colonnes renvoyées par la requête } // on ferme toutes les ressources de la requête
Simple, dynamique, facile à comprendre.
Avec JPA, ce serait à peine différent.
Petit détail, ajouter systématiquement where 1=1 évite d'avoir à se poser la question pour chaque paramètre conditionnel si c'est le premier (donc il faudrait mettre where) ou un suivant (donc on peut mettre and ou or).

Après, rien n'empêche de se faire quelques méthodes utilitaires pour construire sous condition certaines parties de la requête.
J'utilise des méthodes de ce genre pour simplifier l'écriture :

Code:

QueryUtils.addQueryParameter(sb, CondiditionEnum.AND, columnName, columnAlias, monParametreId);

Pour faire simple, la méthode teste le paramètre, s'il est null, la commande ne fait rien, s'il ne l'est pas, il ajouter à "sb" l'expression de la condition.
En parallèle, il y en a une autre pour fixer la valeur du parametre

Code:

QueryUtils.setQueryParameter(pstmt, index, monParametre);

Pareil que pour l'autre, si le paramètre est null, il ignore le paramètre.

(c'est une illustration, mon code est un peu plus compliqué que ça mais l'idée est là).

3 pièce(s) jointe(s)

Je n'avais pas mis toute la méthode mais juste une partie, voici la méthode complète, donc en PHP :

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
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
 
 /**
     * Undocumented function
     *
     * @param string $page
     * @param integer $resultPerPages
     * @param integer $isPagination
     * @param string|null $marqueId
     * @param string|null $modele
     * @param string|null $carburant
     * @param integer|null $min_year
     * @param integer|null $max_year
     * @param integer|null $min_kms
     * @param integer|null $max_kms
     * @param integer|null $min_price
     * @param integer|null $max_price
     * @return array[]
     */
    public function findAdResearched(
        string $page, 
        int $resultPerPages,
        int $isPagination,
        ?string $marqueId, 
        ?string $modele, 
        ?string $carburant, 
        ?int $min_year,
        ?int $max_year, 
        ?int $min_kms, 
        ?int $max_kms, 
        ?int $min_price, 
        ?int $max_price
    ): array
    {
        $qb = $this->createQueryBuilder('a');
        $qb->select('
            a.id,
            a.annee, 
            a.prix, 
            ma.marque, 
            mo.modele, 
            p.emplacement AS photos
        ');
 
        $qb->join('a.marque', 'ma')
           ->join('a.modele', 'mo')
           ->join('a.carburant', 'c')
           ->join('a.photos', 'p')
           ->where('p.estPrincipal = 1');
        if( ! empty($marqueId))
           $qb->andWhere('ma.id = :marqueId')
              ->setParameter("marqueId", $marqueId);
        if( ! empty($modele))
           $qb->andWhere('mo.modele LIKE :modele')
              ->setParameter("modele", $modele);
        if( ! empty($carburant))
           $qb->andWhere('c.carburant LIKE :carburant')
              ->setParameter("carburant", $carburant);
        if( ! (empty($min_year) && empty($max_year)))
           $qb->andWhere('a.annee BETWEEN :min_year AND :max_year')
              ->setParameter('min_year', $min_year)
              ->setParameter('max_year', $max_year);
        if( ! (empty($min_price) && empty($max_price)))
            $qb->andWhere('a.prix BETWEEN :min_price AND :max_price')
            ->setParameter("min_price", $min_price)
            ->setParameter("max_price", $max_price);
        if( ! (empty($min_kms) && empty($max_kms)))
           $qb->andWhere('a.kilometrage BETWEEN :min_kms AND :max_kms')
              ->setParameter("min_kms", $min_kms)
              ->setParameter("max_kms", $max_kms);
              $qb->orderBy('a.createdAt', 'DESC');
        if($isPagination){
            $qb->setFirstResult(($page-1) * $resultPerPages);
            $qb->setMaxResults($resultPerPages);
        }
 
        return $qb->getQuery()->getResult();  
    }

Ce code récupère les informations envoyé par le formulaire donc j'ai plusieurs liste marques, modèles, kilomètres, prix. Et on gère les cas où l'utilisateur n'a pas sélectionné un champ dans une liste.

@OButterlin Merci pour ton exemple de code.

@thelvin, dans l'application Java que j'avais fait pendant mon stage de formation il y avait aussi des listes je vous montre en image :

Pièce jointe 632330

Pièce jointe 632329

Donc ici, c'est un peu près la même approche que sur mon application PHP Symfony pour vérifier et récupérer les information du formulaire.
Voilà comment j'ai fait dans mon code pour traité les champs non sélectionnés (je met la méthode en entier elle est un peu longue) :

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
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
 
private List<Predicate> ajouterFiltres(Map<String, String> map, Root<Appel> appel, List<Predicate> predicates, CriteriaBuilder cb) throws ParseException{
		// DEBUT FILTRE A.
		if(map.containsKey("mySelect1")) {
 
		    if(  map.get("AV1") != null &&  map.get("AV1Bis") != null){
 
		        predicates.add(
	        		cb.between(appel.get("dateHeure"), 
    				new SimpleDateFormat("yyyy-MM-dd HH:mm:ss").parse(map.get("AV1") + " 00:00:00"), 
    				new SimpleDateFormat("yyyy-MM-dd HH:mm:ss").parse(map.get("AV1Bis") + " 23:59:59"))
	        	);
 
		        /*String AV1 = map.get("AV1") + " 00:00:00";
		        DateTimeFormatter formatterAV1 = DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss");
		        LocalDateTime startDateAV1 = LocalDateTime.parse(AV1, formatterAV1);
 
		        String AV1Bis = map.get("AV1Bis") + " 23:59:59";
		        DateTimeFormatter formatterAV1Bis = DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss");
		        LocalDateTime endDateAV1Bis = LocalDateTime.parse(AV1Bis, formatterAV1Bis );
 
		        Date startDate_AV1 = Date.from(startDateAV1.atZone(ZoneId.systemDefault()).toInstant());
		        Date endDate_AV1Bis  = Date.from(endDateAV1Bis.atZone(ZoneId.systemDefault()).toInstant());
 
		        predicates.add(cb.between(appel.get("dateHeure"), startDate_AV1, endDate_AV1Bis));*/
		    }
		    if(  map.get("AV2") != null){
		        predicates.add(cb.equal(appel.get("agence"), Integer.parseInt(map.get("AV2"))));
		    }	    	// Note the parse method throws an exception if it fails so we have to throw it. 
		    if(  map.get("AV3") != null){
		        predicates.add(cb.equal(appel.get("suiviPar"), Integer.parseInt(map.get("AV3"))));
		    }
		    if(  map.get("AV4") != null){
		        predicates.add(cb.equal(appel.get("typeAppelant"), Integer.parseInt(map.get("AV4"))));
		    }
		    if(  map.get("AV5") != null){
		        predicates.add(cb.like(appel.get("nom"), map.get("AV5")));
		    }
		    if(  map.get("AV6") != null){
		        predicates.add(cb.like(appel.get("prenom"), map.get("AV6")));
		    }
		    if(  map.get("AV7") != null){
		        predicates.add(cb.like(appel.get("mobile"), map.get("AV7")));
		    }
		    if(  map.get("AV8") != null){
		        predicates.add(cb.like(appel.get("telephone"), map.get("AV8")));
		    }
		    if(  map.get("AV9") != null){
		        predicates.add(cb.like(appel.get("email"), map.get("AV9")));
		    }
		    if(  map.get("AV10") != null){
		        predicates.add(cb.like(appel.get("objet"), map.get("AV10")));
		    }
		    if(  map.get("AV11") != null){
		        predicates.add(cb.equal(appel.get("annonce"), Integer.parseInt(map.get("AV11"))));
		    }
		    if(  map.get("AV12") != null){
		        predicates.add(cb.equal(appel.get("origine"), Integer.parseInt(map.get("AV12"))));
		    }
		    if(  map.get("AV14") != null){
		        predicates.add(cb.like(appel.get("nomDuMandat"), map.get("AV14")));
		    }
		    if(  map.get("AV15") != null){
		        predicates.add(cb.equal(appel.get("recuPar"), Integer.parseInt(map.get("AV15"))));
		    }
		    if(  map.get("AV16") != null){
		        predicates.add(cb.equal(appel.get("estAccuse"), 1));
		    }
		    if(  map.get("AV17") != null){
		        predicates.add(cb.like(appel.get("suivre"), map.get("AV17")));
		    }
		}
 
		// DEBUT FILTRE B
		System.out.println("\n\n");
    	System.out.println("mySelect2 : " + map.get("mySelect2"));
    	System.out.println("\n\n");
		if(map.containsKey("mySelect2")) {
		    if(  map.get("BV1") != null &&  map.get("BV1Bis") != null){
		    	predicates.add(cb.between(appel.get("dateHeure"),
	    				new SimpleDateFormat("yyyy-MM-dd HH:mm:ss").parse(map.get("BV1")+" 00:00:00"), 
	    				new SimpleDateFormat("yyyy-MM-dd HH:mm:ss").parse(map.get("BV1Bis") + " 23:59:59")));
		      //predicates.add(cb.between(appel.get("dateHeure"), map.get("AV1"), map.get("AV1Bis"))); //Génère une erreur de type format.
		    }
		    if(  map.get("BV2") != null){
		        predicates.add(cb.equal(appel.get("agence"), Integer.parseInt(map.get("BV2"))));
		    }
		    if(  map.get("BV3") != null){
		        predicates.add(cb.equal(appel.get("suiviPar"),  Integer.parseInt(map.get("BV3"))));
		    }
		    if(  map.get("BV4") != null){
		        predicates.add(cb.equal(appel.get("typeAppelant"),  Integer.parseInt(map.get("BV4"))));
		    }
		    if(  map.get("BV5") != null){
		        predicates.add(cb.like(appel.get("nom"), map.get("BV5")));
		    }
		    if(  map.get("BV6") != null){
		        predicates.add(cb.like(appel.get("prenom"), map.get("BV6")));
		    }
		    if(  map.get("BV7") != null){
		        predicates.add(cb.like(appel.get("mobile"), map.get("BV7")));
		    }
		    if(  map.get("BV8") != null){
		        predicates.add(cb.like(appel.get("telephone"), map.get("BV8")));
		    }
		    if(  map.get("BV9") != null){
		        predicates.add(cb.like(appel.get("email"), map.get("BV9")));
		    }
		    if(  map.get("BV10") != null){
		        predicates.add(cb.like(appel.get("objet"), map.get("BV10")));
		    }
		    if(  map.get("BV11") != null){
		        predicates.add(cb.equal(appel.get("annonce"), Integer.parseInt(map.get("BV11"))));
		    }
		    if(  map.get("BV12") != null){
		        predicates.add(cb.equal(appel.get("origine"), Integer.parseInt(map.get("BV12"))));
		    }
		    if(  map.get("BV14") != null){
		        predicates.add(cb.like(appel.get("nomDuMandat"), map.get("BV14")));
		    }
		    if(  map.get("BV15") != null){
		        predicates.add(cb.equal(appel.get("recuPar"), Integer.parseInt(map.get("BV15"))));
		    }
		    if(  map.get("BV16") != null){
		        predicates.add(cb.equal(appel.get("estAccuse"), 1));
		    }
		    if(  map.get("BV17") != null){
		        predicates.add(cb.like(appel.get("suivre"), map.get("BV17")));
		    }
		}
 
		// DEBUT FILTRE C
		System.out.println("\n\n");
    	System.out.println("mySelect3 : " + map.get("mySelect3"));
    	System.out.println("\n\n");
		if(map.containsKey("mySelect3")) {
		    if(  map.get("CV1") != null &&  map.get("CV1Bis") != null){
		        predicates.add(cb.between(appel.get("dateHeure"), 
	    			new SimpleDateFormat("yyyy-MM-dd HH:mm:ss").parse(map.get("CV1")+" 00:00:00"), 
	    			new SimpleDateFormat("yyyy-MM-dd HH:mm:ss").parse(map.get("CV1Bis") + " 23:59:59"))
		        );
		    }
		    if(  map.get("CV2") != null){
		        predicates.add(cb.equal(appel.get("agence"), Integer.parseInt(map.get("CV2"))));
		    }
		    if(  map.get("CV3") != null){
		        predicates.add(cb.equal(appel.get("suiviPar"), Integer.parseInt(map.get("CV3"))));
		    }
		    if(  map.get("CV4") != null){
		        predicates.add(cb.equal(appel.get("typeAppelant"), Integer.parseInt(map.get("CV4"))));
		    }
		    if(  map.get("CV5") != null){
		        predicates.add(cb.like(appel.get("nom"), map.get("CV5")));
		    }
		    if(  map.get("CV6") != null){
		        predicates.add(cb.like(appel.get("prenom"), map.get("CV6")));
		    }
		    if(  map.get("CV7") != null){
		        predicates.add(cb.like(appel.get("mobile"), map.get("CV7")));
		    }
		    if(  map.get("CV8") != null){
		        predicates.add(cb.like(appel.get("telephone"), map.get("CV8")));
		    }
		    if(  map.get("CV9") != null){
		        predicates.add(cb.like(appel.get("email"), map.get("CV9")));
		    }
		    if(  map.get("CV10") != null){
		        predicates.add(cb.like(appel.get("objet"), map.get("CV10")));
		    }
		    if(  map.get("CV11") != null){
		        predicates.add(cb.equal(appel.get("annonce"), Integer.parseInt(map.get("CV11"))));
		    }
		    if(  map.get("CV12") != null){
		        predicates.add(cb.equal(appel.get("origine"), Integer.parseInt(map.get("CV12"))));
		    }
		    if(  map.get("CV14") != null){
		        predicates.add(cb.like(appel.get("nomDuMandat"), map.get("CV14")));
		    }
		    if(  map.get("CV15") != null){
		        predicates.add(cb.equal(appel.get("recuPar"), Integer.parseInt(map.get("CV15"))));
		    }
		    if(  map.get("CV16") != null){
		        predicates.add(cb.equal(appel.get("estAccuse"), 1));
		    }
		    if(  map.get("CV17") != null){
		        predicates.add(cb.like(appel.get("suivre"), map.get("CV17")));
		    }
		}
		if(map.get("CP") != null && ! map.get("CP").equals("3")) {
			predicates.add(cb.equal(appel.get("estTermine"), map.get("CP")));			
		}
 
		return predicates;
	}

Quand j'ai montré sa pour traité les valeurs du formulaire mon maître de stage (qui soit dit en passant avait plus de 40 ans et ingénieur donc avait une parfaite maître du langage Java) à été super sympa de pas me mettre des coups de pied au cul :mrgreen:

Voici la conversation que j'ai eu avec lui sur Discord :

Mon maitre de stage : tu as essayé d'utiliser des fonctions lambda sur le criteriaBuilder comme tu l'avais montré en PHP ?

Moi : oui je pense, je te montre un morceau de code.

Mon maître de stage : excellent, vas-y colle

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
 
private List<Predicate> ajouterFiltres(Map<String, String> map, Root<Appel> appel, List<Predicate> predicates, CriteriaBuilder cb) throws ParseException{
		// DEBUT FILTRE A.
		if(map.containsKey("mySelect1")) {
 
		    if(  map.get("AV1") != null &&  map.get("AV1Bis") != null){
		        predicates.add(cb.between(appel.get("dateHeure"), new SimpleDateFormat("yyyy-MM-dd HH:mm:ss").parse(map.get("AV1")+" 00:00:00"), new SimpleDateFormat("yyyy-MM-dd HH:mm:ss").parse(map.get("AV1Bis") + " 23:59:59")));
		    }
		    if(  map.get("AV2") != null){
		        predicates.add(cb.equal(appel.get("agence"), Integer.parseInt(map.get("AV2"))));
		    }
		    if(  map.get("AV3") != null){
		        predicates.add(cb.equal(appel.get("suiviPar"), Integer.parseInt(map.get("AV3"))));
		    }
		    if(  map.get("AV4") != null){
		        predicates.add(cb.equal(appel.get("typeAppelant"), Integer.parseInt(map.get("AV4"))));
		    }
		    if(  map.get("AV5") != null){
		        predicates.add(cb.like(appel.get("nom"), map.get("AV5")));
		    }
		    if(  map.get("AV6") != null){
		        predicates.add(cb.like(appel.get("prenom"), map.get("AV6")));
		    }
		    if(  map.get("AV7") != null){
		        predicates.add(cb.like(appel.get("mobile"), map.get("AV7")));
		    }
		    if(  map.get("AV8") != null){
		        predicates.add(cb.like(appel.get("telephone"), map.get("AV8")));
		    }
		    if(  map.get("AV9") != null){
		        predicates.add(cb.like(appel.get("email"), map.get("AV9")));
		    }
		    if(  map.get("AV10") != null){
		        predicates.add(cb.like(appel.get("objet"), map.get("AV10")));
		    }
		    if(  map.get("AV11") != null){
		        predicates.add(cb.equal(appel.get("annonce"), Integer.parseInt(map.get("AV11"))));
		    }
		    if(  map.get("AV12") != null){
		        predicates.add(cb.equal(appel.get("origine"), Integer.parseInt(map.get("AV12"))));
		    }
		    if(  map.get("AV14") != null){
		        predicates.add(cb.like(appel.get("nomDuMandat"), map.get("AV14")));
		    }
		    if(  map.get("AV15") != null){
		        predicates.add(cb.equal(appel.get("recuPar"), Integer.parseInt(map.get("AV15"))));
		    }
		    if(  map.get("AV16") != null){
		        predicates.add(cb.equal(appel.get("estAccuse"), 1));
		    }
		    if(  map.get("AV17") != null){
		        predicates.add(cb.like(appel.get("suivre"), map.get("AV17")));
		    }
		}

Moi : ça marche mais après faudrait faire des tests en profondeur.

Mon maître de stage : attends je te fais une capture

Pièce jointe 632332

Moi : Je savais pas que ça existait ou en tout cas ça ma échapé. Ok merci.

Voilà donc pour mon maître de stage il y a la possibilité de faire comme en PHP avec semble t-il des fonctions lambda mais là je maîtrise pas.

Après j'ai vue qu'il y avait CriteriaBuilder : je vous met un lien https://docs.oracle.com/javaee/6/api...eriaQuery.html mais le site date de 2011 donc je sais pas si c'est périmé ou pas CriteriaBuilder. J'ai regardé dans la Java Doc et j'ai trouvé ça : https://javaee.github.io/javaee-spec...iaBuilder.html mais dans ce lien je ne vois pas le select alors que dans le premier lien il y est, c'est sans doute du fait que Java EE 6 est périmé.

Que pensez-vous de CriteriaBuilder est-ce que c'est ça que je dois utilisé pour la gestion des formulaires et donc faire comme en PHP ?

11/01/2023, 20h43
marc.collin

avec les predicates, critera, c'est une facon par exemple de faire une recherche avancé et c'est la façon que j'utilise habituellement
13/01/2023, 09h26
hbx360

D'accord merci.
13/01/2023, 15h07
thelvin

On notera qu'il y a pas de signe de fonctions lambdas dans ces bouts de code.

Il est vrai qu'un usage habituel des fonctions lambdas c'est le map/filter/reduce d'une collection de données, et que faire du map/filter/reduce comme ça c'est une des fonctionnalités/intérêts de SQL.
Mais quand on prend les données d'une base de données, c'est le moteur de la base de données qui doit effectuer le map/filter/reduce, sinon il va sélectionner et transmettre toutes les données d'origine de la base de données au programme principal, ce qui est très inefficace.

Comme donc ils doivent être effectués par le moteur de base de données, ils ne peuvent pas être exprimés en Java puisque le moteur ne connaît pas Java. Et donc on ne peut pas utiliser de lambdas pour ça.
13/01/2023, 15h13
hbx360

D'accord alors mon Maître de stage c'est trompé ou le fait avec des lambdas je sais pas, mais avoué que le code en Java pour récupérer les info du formulaire est vachement lourd !
13/01/2023, 16h45
thelvin

Le fait est que les frameworks de bdd Java ne sont pas faits pour des formulaires de ce genre où on peut construire la requête qu'on veut avec un formulaire. Normalement on construit des requêtes avec du code informatique, pas avec un formulaire. Mais bon, ok, le code PHP montré s'en sort mieux pour faire ça quand on le fait quand même.

N'empêche que tu gères bien plus de trucs avec ton code Java qu'avec ton code PHP. Tu ferais la même chose dans tous les langages, ce serait déjà nettement plus lourd dans les langages non-Java.

Avec quelques méthodes utilitaires, on peut faire ce genre de traitement assez facilement, voici un exemple que je fais (sur la base de ce que j'ai expliqué plus haut mais là en JPA) de query où les paramètres ne sont pas connus à l'avance (donc ton cas avec un formulaire)
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 // "code" et "libelle" sont les paramètres dans cet exemple QueryLazyResult<SuiteDonnee> result = new QueryLazyResult<SuiteDonnee>(); StringBuilder sb = new StringBuilder(); sb.append("select s from SuiteDonnee s where 1=1"); QueryUtils.addQueryParameter(sb, QueryRelation.AND, "s.code", "code", QueryFunction.UPPER, QueryOperation.LIKE, code); QueryUtils.addQueryParameter(sb, QueryRelation.AND, "s.libelle", "libelle", QueryFunction.UPPER, QueryOperation.LIKE, libelle); sb.append(" order by s.code"); Query query = getEntityManager().createQuery(sb.toString()); QueryUtils.setQueryParameter(query, "code", QueryUtils.replaceJoker(code)); QueryUtils.setQueryParameter(query, "libelle", QueryUtils.replaceJoker(libelle)); if (first > 0) { query.setFirstResult(first); } if (pageSize > 0) { query.setMaxResults(pageSize); } result.setList(query.getResultList());
En orange, les fonctions utilitaires prennent en charge la présence ou non du paramètre. S'il n'est pas renseigné, la ligne est ignorée.
En vert, c'est pour la pagination. Si la question ne se pose pas, ça fait autant de lignes d'économisées.

Il va de soi que j'aurais pu ajouter d'autres méthodes utilitaires pour gérer la pagination mais bon, je ne suis pas à la recherche du code ayant le moins de caractères...

Dans l'exemple de ton maître de stage, je ne vois pas de condition pour les critères, donc on n'est pas exactement dans le même cas et si on conditionnait, ça rallongerait le code pour en arriver à quelque chose d'à peine plus court.
Le "truc" en JPA qui ressemblerait le plus à ce qu'il t'a montré serait via CriteriaBuilder, mais force est de constater que Hibernate Criteria est largement plus simple à utiliser que l'équivalent JPA.
Exemple Hibernate
Code:

1 2 3 4 5 6 session.createCriteria(Employee.class) .add(Restrictions.like("name", "%i%")) .addOrder(Order.desc("id")) .setFirstResult(0) .setMaxResults(2) .list();
De mon point de vue (et lié à mon expérience), je trouve que le modèle JPA à base d'entity n'est pas (toujours) adapté à des query optimisés, je préfère utiliser l'api bas niveau (JDBC) ou les requêtes natives pour ça.
Le problème avec les requêtes natives ou JDBC est l'externalisation de la requête si le code est supposé fonctionner sur plusieurs bases de données. Mais d'un point de vue performance, il n'y a pas photo ! (Sauf bien sûr des requêtes basiques sur une table).
A titre d'information, j'avais fait passer le temps d'exécution d'une requête de ce genre (à partir d'un formulaire) de 25s à 5s en remplaçant du code JPA entity par une requête native... mais bon, c'était une grosse requête complexe.

Bref, en résumé, ce n'est pas Java le problème mais plutôt que les développeurs n'ont pas forcément jugé utile de coder l'équivalent de l'outil PHP que tu références.
Mais rien ne t'empêche de le coder toi même ;)

23/03/2023, 09h04
hbx360

Merci pour vos réponses, je met le thread en résolue.