Problème vérification existance d'un identifiant

Bonjour à tou(te)s,

Je développe une petite appli' sur une création de visiteurs. Je génère un identifiant en récupérant la première lettre de son nom suivi de deux chiffres aléatoires compris entre 0 et 9. Jusque là, rien de difficile. Vient ensuite le test de l'identifiant pour vérifier qu'il n'existe pas déjà.

Pour ce test, j'ai une méthode "existeVisiteur" :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

private boolean existeVisiteur(String id){ boolean trouve = false; int i = 0; //listeV récupère tous les visiteurs existants dans la BDD listeV = AccesBDD.getLesVisiteurs(); while(i<listeV.size() || trouve == false){ if(id.equals(listeV.get(i).getId())) trouve = true; i++; } return trouve; }

---

Je l'appel de cette façon là :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

try { //affectation des données à des variables String id = textFieldId.getText(); if (existeVisiteur(id) == false) afficheMessage("Cet identifiant est disponible"); else afficheMessage("Cet identifiant n'est pas disponible") } catch (Exception e) { afficheMessage("Problème de saisie."); }

---

Lorsque l'identifiant est déjà utilisé, il me retourne bien "Cet identifiant n'est pas disponible", en revanche lorsque celui-ci n'existe pas, je passe dans le catch.

La connexion entre mon appli Java et la base MySQL est fait en hibernate. (Je n'ai pas le choix)

Je ne vois pas d'où ça peut venir, si quelqu'un pouvait m'aider :(

Merci d'avance

Salut,

dans :

Code:

---

1 2 3 4 5

while(i<listeV.size() || trouve == false){ if(id.equals(listeV.get(i).getId())) trouve = true; i++; }

---

Il faut que tu mettes && au lieu de || parce que sinon ta boucle va continuer à boucler et tenter d'accéder à un élément en dehors de ta liste et tu te retrouves avec un IndexOutofBoundException ou un truc similaire.


Tchao!

Salut,

pour tester l'existence de ton id, tu devrais plutôt mettre l'id à tester en paramètre de ta requête, ca éviterait que si il y a des milliers de visiteurs (voire des millions), ton prog boucle sur des milliers de tests, de transporter une réponse gigantesque (ça va prendre beaucoup de mémoire pour rien et pour peu que ta base soit distante et ça va prendre des plombes)

si l'id existe tu as une réponse, sinon tu n'en a pas

et si en plus tu fait un count dans ta requête, tu récupères 0 ou 1 selon que ça existe ou pas.

reste le problème de l'accès concurrent ! je ne sais pas s'il est prévu que plusieurs utilisateurs puissent se connecter simultanément à ta base pour créer des utilisateurs. statistiquement il est peu probable qu'il y ait le même id créé en même temps, mais ce n'est pas impossible.

@toutgrego Un gros merci pour ta réponse, ça fonctionne ... Aucune idée pourquoi j'y ai jamais pensé ! :roll:

@joel.drigo J'ai fais cette requête en m'inspirant de celle qui récupère tous les visiteurs. Elle me parait bonne mais lorsque j'exécute, je passe directement dans le catch.

Ma requête :

Code:

---

1 2 3 4 5

public static List<Visiteur> getLeVisiteurs(String id) { return s.createQuery("select count(*) from Visiteur where id= '" + id).list();   }

---

Comment je l'appel :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

try{ String id = textFieldId.getText(); List<Visiteur> leVisiteur= AccesBDD.getLeVisiteurs(id); if (leVisiteur.size()==0) afficheMessage("Cet identifiant est disponible"); else afficheMessage("Cet identifiant n'est pas disponible"); } catch (Exception e) { afficheMessage("Problème de saisie."); }

---

Bonjour,

Un select count(*) retourne un entier mais pas de liste de visiteur.
N'insère pas directement des valeurs dans une requête.

Code:

---

1 2 3 4 5

public static Integer getNbVisiteur(String id) { return (Integer) s.createQuery("select count(id) from Visiteur where id= ?").setString (0, id).uniqueResult();   }

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11

try{ String id = textFieldId.getText(); if (AccesBDD.getNbVisiteur(String id) ==0 ) afficheMessage("Cet identifiant est disponible"); else afficheMessage("Cet identifiant n'est pas disponible"); } catch (Exception e) { e.printStackTrace(); // il nous raconte des histoires intéressants }

---

A+.

Que fait cette partie de requête exactement ? Je me doute que "id" dans les parenthèses du setString correspond à la variable qui "remplacera" le ?, mais le reste je ne comprends pas le reste ^^"

Code:

---

where id= ?").setString (0, id).uniqueResult();

---

J'ai mis tout le code que tu m'as donné (peut-être une erreur de ma part?) dans mon code et j'ai ce message qui apparait :

Code:

---

1 2 3

java.lang.ClassCastException: java.lang.Long cannot be cast to java.lang.Integer at com.utilitaire.AccesBDD.getNbVisiteur(AccesBDD.java:33) at com.vue.AjoutVisiteur$5.actionPerformed(AjoutVisiteur.java:269)

---

Citation:

---

java.lang.ClassCastException: java.lang.Long cannot be cast to java.lang.Integer

---

C'est que la requête retourne un type Long et on a tenté de le caster en Integer.

Code:

---

return ((Long) s.createQuery("select count(id) from Visiteur where id= ?").setString (0, id).uniqueResult()).intValue;

---

A+.

Nickel ! Ça fonctionne ! =)

Un énorme merci à vous tous :ccool:

Citation:

---

Envoyé par ShowLess

Que fait cette partie de requête exactement ? Je me doute que "id" dans les parenthèses du setString correspond à la variable qui "remplacera" le ?, mais le reste je ne comprends pas le reste ^^"

Code:

---

where id= ?").setString (0, id).uniqueResult();

---

---

Oui, tout est expliqué sur le javadoc.

A+.