Discussion :

[ShowLess]

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 :

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

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 :

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

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

[toutgrego]

Salut,

dans :

Code :

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

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!

[joel.drigo]

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.

[ShowLess]

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

@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 :

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

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 :

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

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

[andry.aime]

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 :

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

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 :

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

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+.

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

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

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 :

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

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)

[andry.aime]

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 :

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

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

A+.

[ShowLess]

Nickel ! Ça fonctionne ! =)

Un énorme merci à vous tous

[andry.aime]

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 :

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

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

Oui, tout est expliqué sur le javadoc.

A+.