Discussion :

[Kasko]

Bonjour,
J'ai un problème au niveau de récupération d'une valeur.Au fait je voulais récupérer la valeur maximale d'un champs.
Voilà ce que j'ai fait :

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
public List<Proces> MAXProces(HttpServletRequest request) throws DAOException {
		Connection connexionBase = null;
	       PreparedStatement preparedStatement = null;
	       ResultSet resultat = null;
	       List<Proces> plugranPro = new ArrayList<Proces>();
	       try{
	    	   connexionBase = daoFactory.getConnection();
	    	   preparedStatement = connexionBase.prepareStatement("select max(LibelleProces) as Proplugrand from proces;");
	    	   resultat = preparedStatement.executeQuery();
                   Proces proces = new Proces()
	    	   while(resultat.next()){
	    		   plugranPro.add(proces.setNumProces(resultat.getLong("NumProces")));
	    		   plugranPro.add(proces.setLibelleProces(resultat.getString("LibelleProces")));
	    		   plugranPro.add(proces.setDateProces(resultat.getDate("DateProces")));
 
	    	       HttpSession session = request.getSession();
 
	    		 String nbreMAX = resultat.getString("Proplugrand ");//je récupère la valeur trouvée
 
	    	       HttpSession session = request.getSession();
	    		   session.setAttribute("nombreMaximal", nbreMAX );//je place cette valeur en session pour la récupérer dans ma Servlet et en fin l'afficher dans ma JSP.
	    	   }
	       }catch(SQLException s){
	    	   throw new DAOException("Aucun n'enregistrement trouvé.",s);
	       }finally{
	    	   fermeturesSilencieuses(resultat,preparedStatement,connexionBase);
	       }
		return 	plugranPro;
	}

Merci de votre aide.

[eulbobo]

Et c'est quoi ton soucis?

J'en vois plusieurs potentiels, mais j'aimerai bien savoir ce qui coince chez toi.

Ce que je vois comme erreurs :
- tu as un ; dans ta requête
- tu essayes de récupérer des champs qui n'existent pas

Ce que je vois comme mauvaises pratiques
- tu place ta valeur dans une variable de session (tu as vraiment besoin de la garder pendant toute la durée de la session?)
- tu place la valeur dans une variable de session PENDANT que tu es en train de boucler sur le resultat.next() (même si ici on a au mieux 1 résultat)
- tu récupères un max d'une valeur que j'imagine numérique que tu récupères en tant que chaine de caractères
- tu lèves une DOAException en disant "aucun enregistrement trouvé" en cas de SQLException. Sauf que ça peut être autre chose, et que le fait de n'avoir aucun enregistrement est une erreur fonctionnelle qui devrait être gérée autrement que par exception
- tu passes un objet HTTPRequest uniquement pour avoir accès au HTTPSession (pourquoi ne pas passer directement le HTTPSession?)
- ta méthode s'appelle MAXProces (un nom de méthode ne doit pas commencer par une majuscule)

Ce que je vois qui n'a pas de sens
- ta méthode s'appelle MAXProces et elle renvoie une liste (qui ne contiendra rien, CF plus haut)
- ta méthode renvoie quelque chose ET met un nombre en session plutôt que de faire ce qu'elle doit faire : renvoyer le nombre maximal dans ta table proces
elle devrait avoir la signature suivante

Code :

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

public Number getMaxProces(HttpServletRequest request) throws DAOException

[Pill_S]

- tu passes un objet HTTPRequest uniquement pour avoir accès au HTTPSession (pourquoi ne pas passer directement le HTTPSession?)

... et pourquoi passer un objet web dans une couche si basse????

pour moi il est là le WTF: hors contexte web, tu ne peux pas réutiliser le dao (couplage fort, pas de séparation des responsabilités, toussa...)

[eulbobo]

... et pourquoi passer un objet web dans une couche si basse????

pour moi il est là le WTF: hors contexte web, tu ne peux pas réutiliser le dao (couplage fort, pas de séparation des responsabilités, toussa...)

Je suis même pas allé assez loin en considérant que ça pourrait avoir un sens de mettre l'élément en session, mais j'en parle plus bas : la méthode devrait juste renvoyer le max.. Et donc ne pas avoir d'arguments

[Kasko]

Mon soucis c'est de selectionner et afficher dans ma JSP la valeur maximale du champ "LibelleProces" et ce champ est de type "varchar".Si vraiment vous savez comment je dois m'y prendre merci de me guider.

[eulbobo]

Je veux bien aider (d'où mes remarques plus haut), mais il faut nous dire là où ça coince chez toi.

Je pense que tu dois avoir une erreur de type SQLException qui est liée à la présence d'un ; dans ta requête.
Pour le reste, c'est des conseils généraux de bonne pratique.


Sinon, si tu veux vraiment la valeur max d'un libellé (donc une chaîne de caractère, donc classé par ordre ascii et pas par ordre numérique), je pense que tu vas avoir un soucis


Parce que ça va te donner ça comme ordre :
1
10
11
12
13
14
15
16
17
18
19
2
20
21

[Kasko]

Au fait quand j'affiche la variable dans laquelle je récupère la valeur "MAX" je ne vois rien dan ma JSP du coup je comprend que cette variable est "null".En exécutant ma requête dans MYSQL ça marche c'est donc peut être la manière dont récupère la valeur qui n'est pas bonne.Pour ce qui est du nom de la méthode je ne pense pas que c'est un problème puisque j'ai des méthodes qui commencent bien par une lettre majuscule et ça fonctionne correctement.Merci d'avance!!!

[eulbobo]

Au fait quand j'affiche la variable dans laquelle je récupère la valeur "MAX" je ne vois rien dan ma JSP du coup je comprend que cette variable est "null".En exécutant ma requête dans MYSQL ça marche c'est donc peut être la manière dont récupère la valeur qui n'est pas bonne.Pour ce qui est du nom de la méthode je ne pense pas que c'est un problème puisque j'ai des méthodes qui commencent bien par une lettre majuscule et ça fonctionne correctement.Merci d'avance!!!

Sans le code de la JSP, je ne peux rien dire... Mais à priori, si tu essayes d'afficher un élément null, ça affiche null, pas rien du tout.

Et je confirme que la méthode pour récupérer la valeur n'est pas bonne (je ne vais pas répéter toutes les raisons ici, autre que ce qui rentre dans la partie bonne pratiques).

Pour faire court : enlève le ; dans ta requête !


Mais ça ne résoudra qu'un seul problème : celui qui fait que ça ne te renvoie rien (parce que tu as une exception que tu dois traiter silencieusement quelque part)

[Kasko]

Merci eulbobo;
j'ai pu résoudre le problème de récupération maintenant mon soucis c'est de pouvoir récupérer les 2 derniers chiffres de l'année du plus grand procès et prendre aussi les 2 derniers chiffres de l'année en cours.