Tester un attribut servlet

Bonjour je besoin d'aide pour un code je veux quand je modifier les attributs de la classe fournisseur tester si le nom existe déjà voici le code que j'ai utilisé :

Citation:

---

if (action.equals("edit")) {// edit Fournisseur
Session session = HibernateUtil.currentSession();
session.beginTransaction();
String nom;
nom=request.getParameter("nom");
String hql = "from Fournisseur F where F.nom='"+ nom + "'";
//String hql = "from Fournisseur F where F.nom='"+ nom + "' limite 1";
Query query = session.createQuery(hql);
Fournisseur user = (Fournisseur) query.setMaxResults(1);
session.getTransaction().commit();

if (id_fournisseur != null) {
Fournisseur fournisseur = (Fournisseur) sessionDb.load(Fournisseur.class, id_fournisseur);
try {
fournisseur.setNom(request.getParameter("nom"));
fournisseur.setPays(request.getParameter("pays"));
fournisseur.setVille(request.getParameter("ville"));
fournisseur.setAdresse(request.getParameter("adresse"));
fournisseur.setTelephone(request.getParameter("telephone"));
fournisseur.setMail(request.getParameter("mail"));
fournisseur.setFax(request.getParameter("fax"));


sessionDb.update(fournisseur);
sessionDb.flush();
sessionDb.connection().commit();
sessionDb.close();
} catch (HibernateException e) {
e.printStackTrace();

} catch (SQLException e) {

e.printStackTrace();
}
response.sendRedirect("fournisseurs?op=index&id="+fournisseur.getId());
}

---

Bonjour

Quel est le problème exactement ? ou l'erreur ?
Est-ce que la mise à jour ne se fait pas ?

Juste quelques remarques::
1)
Avant de construire la string hql,il faut peut être s'assurer que la variable nom n'est pas vide ou null

Code:

---

1 2 3 4

String nom; nom=request.getParameter("nom"); String hql = "from Fournisseur F where F.nom='"+ nom + "'";

---

2)

Dans le code

Code:

---

1 2 3

Fournisseur user = (Fournisseur) query.setMaxResults(1); session.getTransaction().commit();

---

je pense que le commit ne sert à rien puisque que c'est un select

3) d'où vient le

Code:

---

1 2	id_fournisseur

---

4) et enfin est-ce que tu es sûr qu'il est n'est pas null et que tu rentres dans ta condition

Code:

---

1 2 3 4

if (id_fournisseur !=null){ // }

---

salut merci ISMA j'ai trouvé la solution, j'ai ajouté unique key dans la table furnisseur et dans ma requête j'ai ajouté l'id de fournisseur comme suit :

Code:

---

String hql = "from Fournisseur F where F.nom='"+ nom + "'and F.id !='"+ id_fournisseur + "'";

---

Tant mieux! mais ça ne doit pas t'empêcher de prendre en compte les remarques (tester si le nom est null ou pas etc ....)
n'oublies pas le tag résolu

salut pour répondre à vos questions le problème que j'ai c'est que je ne peut pas modifier seulement un champs d'un fournisseur il se croit que ce fournisseur existe donc il ne faut pas modifier leurs champs.

1)j'ai pas fais ce test car c'est obligatoirement de remplir le champ nom.

2)j'ai pas fais attention en tous cas j'ai changé

Code:

---

query.setMaxResults(1);

---

PAR

Code:

---

query.uniqueResult();

---

3)le id_fournisseur vient de ma table fournisseur c'est l'id

Code:

---

1 2 3 4 5

try { id_fournisseur = Integer.parseInt(request.getParameter("id")); } catch (NumberFormatException e) { System.out.println(e); }

---

4)oui dans ma table fournisseur j'exige que l'id ne doit pas être null c'est un clé primaire.

j'espère que j'ai bien expliquer et j'ai bien répondu à vos questions. merci de vos remarques.

Juste une question : comment vous avez fait pour avoir rendu le champs nom obligatoire ?

j'utilise le javascript comme suit:

Code:

---

1 2 3 4 5 6 7 8

function validate(){ var reference=document.pdt.reference; if ((reference.value==null)||(reference.value=="")){ alert("Entrer un référence"); reference.focus(); return false; } }

---

Ok mais ça ne suffit pas !
Il faut toujours faire deux validations :
La première est à faire côté client avec le Javascript (=> C'est ce que tu as fait)
Par contre il faut faire une deuxième validation côté serveur ! Tu peux le faire soit dans ta servlet dans une méthode private, soit dans une classe utilitaire

Tu vas me dire pourquoi il faut le faire des deux côtés (client et serveur) : voici deux bonnes raisons :

- la première est que dans ton navigateur tu peux désactiver le Javascript ( et contrairement à ce qu'on peut penser ,c'est quelque chose qui arrive)
- la deuxième est que même sir ton javascript est activé une personne malintentionnée peut faire de l'injection javascript

voilà bon courage
et n'oublies pas de mettre le tag résolu pour les autres

merci j'ai essayé de le faire dans ma servlet voici ce que j'ai mis mais je ne pense pas que c'est un bon code

Code:

---

1 2 3 4 5 6 7 8 9

String reference,designation,taille; reference=request.getParameter("reference"); designation=request.getParameter("designation"); taille=request.getParameter("taille");   if(reference.equals("")||designation.equals("")||taille.equals("")) { response.sendRedirect("produits?op=new&id="+produit.getId()); }

---

juste une question j'ai pas compris cette phrase une personne malintentionnée peut faire de l'injection javascript
comment il peut faire l'injection du javascript.

1) n'oublies pas de tester le cas null et pas que la chaine vide
2) et si c'est vide ou null par exemple tu mets un message d'erreur dans ta request

Code:

---

1 2	request.setAttribute("error_message","Le nom n'est pas renseigné");

---

et dans ta JSP tu affiches le message d'erreur si il en a un

3) pour l'injection javascript il y a plein d'exemple sur le net et les moyens de contourner .mais un exemple simple est que à la place du nom la personne met le code javascript alert('tata');
A chaque fois que tu affiches ta page ton javascript alert va s'afficher

et l'autre injection est SQL
Imagines que à la place du nom la personne entre dans le champs

Code:

---

INSERT INTO USER (...)

---

merci infiniment ok je vais essayer, pour la jsp je mets ce code pour récupérer le message

Code:

---

1 2 3

<p style="background-color: red; padding: 5px;"> <B><I><FONT COLOR="#0000FF"><%=request.getAttribute("message")%></FONT></I></B> </p>

---

mais le problème que j'ai c'est que ce message s'affiche toujours dans la page jsp et une autre chose si je veux tester 10 champs par exemple si ils sont vides ou bien de les tester si ils sont numérique, comment je peut le faire? merci

Plusieurs choses

tu devrais faire un message d'erreur pour chaque champs
Ensuite tu mets tout ça dans une liste par exemple..

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

List<String> listError = new ArrayList<String>(); String nom = request.getParameter("nom"); String telephone = request.getParameter("telephone");   if (null == nom ||"".equals(nom)){ listError.add("Veuillez renseigner le nom"); //ou autre message }   //et tu fais ça pour chaque champs //je te conseille de faire ça dans une méthode private qui prend en //paramètre ta requête et qui renvois une liste de string if (listError.size()>0){ request.setAttribute("error_messages",listError); // + forward vers ta JSP }else{ //autres traitements : }

---

Pour les entiers, ou autre champs avec un format, il n'y a pas mieux que les expressions régulières.Mais pour ça il faut tout un chapitre
Il y a plein de tutoriels à ce sujet ...

Dans ta JSP en taglib ou en scriplets (ou autres)

Code:

---

1 2 3 4 5 6 7 8 9 10 11

<%if (request.getAttribute("error_messages") != null){ for(String mess : ((List<String>)request.getAttribute("error_messages") ){ <p style="background-color: red; padding: 5px;"> <B><I><FONT COLOR="#0000FF"><%=mess %></FONT></I></B> </p> } %>     <%}%>

---

A peu de chose près ça peut ressembler à ça
Si tu veux n'hésites pas à utiliser les tagslibs à la place des scriplets, c'est mieux

merci infiniment je vais tester ça.

merci ISMA je suis intéressé d'utiliser les tagLibs mais je ne sais pas comment faire pourriez vous me diriger?

Est-ce que tu as résolu ton problème sans taglib ?

sinon pour les taglib, il y a plein de tutoriel, notamment sur ce site
On ne peut pas expliquer ça en un post

bonjour merci isma j'ai testé ce que vous m'avez proposer sans utilisé les TagLibs mais quand je mets ce code dans ma jsp j'ai un erreur

Code:

---

1 2 3 4 5 6 7 8 9 10

<%if (request.getAttribute("error_messages") != null){ for(String mess : ((List<String>)request.getAttribute("error_messages") )){ <p style="background-color: red; padding: 5px;"> <B><I><FONT COLOR="#0000FF"><%=mess %></FONT></I></B> </p> } %> <%}%>

---

il ne comprend pas ce que j'ai mis en rouge.

j'ai trouvé une solution voici le code

Code:

---

1 2 3 4 5 6 7 8 9 10 11

<h1>Nouveau Patient</h1> <%if (request.getAttribute("error_messages") != null){ for(String mess : ((List<String>)request.getAttribute("error_messages") )){ %> <p style="background-color: red; padding: 5px;"><B><I><FONT COLOR="#0000FF"><%=mess %></FONT></I></B></p> <% } %> <% } %>

---

le problème que j'ai maintenant c'est que je veux pas afficher le message dés le lancement de la page je veux le cacher et dés que je clique sur ajouter et le champ n'est pas rempli le message s'affiche .
j'espère que j'ai bien exprimé mon besoin merci

Bonjour

la partie d'erreur dans la JSP ne s'affiche que lorsqu'il y a des erreurs,
car les erreurs sont stockés dans la request. Et c'est la servlet qui se charge de mettre la liste d'erreur dans la request

La cinématique est la suivante :
Servlet d'affichage du formulaire => renvoi vers la page => submit du formulaire => récupérations des données dans la servlet => traitements sur les données

Si dans la servlet tu ne mets pas ta liste de message dans ta request elle ne s'affiche pas

Qu'est ce que tu as comme affichage actuellement ?

merci infiniment,quand je lance ma page jsp le message

Code:

---

Veuillez renseigner le nom

---

s'affiche . regardez la pièce jointe s'il vous plaît

Le problème doit sûrement venir de la servlet où le traitement des erreurs doit être mal placé
Est-ce que tu peux poster le code de la servlet stp ?