Comportement étrange d'un .equals

**cleecky** · 05/11/2011, 23h06

Bonjour a tous et à toute !

Je viens vers vous car je cherche à développer un petit jeu en java et je me retrouve avec un comportement vraiment étrange... ou alors je passe à côté de quelques chose ^^

Je vous explique, j'ai une méthode qui évalue une réponse passé en paramètre avec une liste de mot clef...
J'ai choisit un HashSet pour l'ensemble contenant les mot clef mais n'étant qu'une débutante il se peut que ce choix ne soit aucunement justifié.

Ceci dit voici la méthode (bien plus parlant) :

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
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
 
/* Methode qui test si la phrase passé en paramétre comporte des correspondances avec les mots contenu dans le tableau de la class */
	private boolean testMotClef(String rep){
 
		int tot = 0;							// nb de mot clef trouvé das la réponse fournit par l'utilisateur
 
		int debut = -1;
		int fin = -1;
		int dernier_espace = 0;
		Pattern p = Pattern.compile(" ");
		Matcher m = p.matcher(rep);
		boolean existe = false;
 
		// lancement de la recherche
		while(m.find()) {
			if ( !existe )
			{
 
				if  ( debut != -1 && dernier_espace == 0 )								// Si une position est enregistré c'est que nous sommes dans le second passage et +
				{
					// j'ai la position de fin et de debut du mot
					String mot = rep.substring(debut, m.start()+1);
					debut = m.start()+1;								// debut du mot suivant dans la réponse fournit par l'utilisateur
 
					// Le mot trouvé est il présent dans le tableau de mot clef?
					Iterator<String> it = mot_clef.iterator();			// iterateur pour parcourir le hashSet
 
					// on parcours l'ensemble
					while ( it.hasNext() )
					{
						String o = (String) it.next();
						System.out.println("o : "+o);
						System.out.println("mot : "+mot);
						if ( mot.equalsIgnoreCase(o) )							// Si les deux mots concordent
						{
							System.out.println("Le mot correspond :"+o);
							// On a trouvé un mot
							tot += 1;
							// Le nb de mot clef est il atteind?
							if ( tot >= this.nb_mc){
								// Si ca correspond c'est gagné
								existe = true;
								System.out.println(nb_mc+" occurence ont était trouvé ! c'est gagné...");
								break;
							}
						}
					} 
					if ( rep.lastIndexOf(" ") == m.start() ){
						dernier_espace = 1;
					}
------------------------------------------------------
				}else if  ( debut == -1 )			// On est au premier espace de la phrase	 => 	fin du premier mot
				{	
					// on peut tester de 0 a debut pour recuperer le premier mot
					fin = m.start()+1;
					debut = m.start()+1;				// début du second mot de la réponse fournit
 
					// On récupère le mot
					String mot = rep.substring(0, fin);
 
					// Le mot trouvé est il présent dans le tableau de mot clef?
					Iterator<String> it = mot_clef.iterator();				// iterateur pour parcourir le hashSet
					while ( it.hasNext() )									// Pour chaque éléments de l'ensemble
					{
						String o = (String) it.next();						// o reçoit les mot clef les un aprés les autres
						System.out.println("o : "+o);
						System.out.println("mot : "+mot);
						System.out.println(o.equalsIgnoreCase(mot));
						if ( o.equals(mot) )								// et teste leurs concordance avec le mot extrait de la réponse
						{
							System.out.println("Le mot correspond :"+o);
							tot += 1;										// On incremente le tot de correspondance
						}
					}
					// ------------------------------------------------------
				}
				if ( dernier_espace == 1 )							// Il s'agit du dernier mot contenu dans la réponse de l'utilisateur
				{
					// je connait la position de fin => la longueur de la phrase
					fin = rep.length();
 
					String mot = rep.substring(m.start()+1, fin ); 				// de l'espace +1 jusqu'a la fin
					// Le mot trouvé est il présent dans le tableau de mot clef?
					Iterator<String> it = mot_clef.iterator();			// iterateur pour parcourir le hashSet
					while ( it.hasNext() )
					{
						String o = (String) it.next();
						if ( o.equals(mot) )							// Si les deux mots concordent
						{
							System.out.println("Le mot correspond :"+o);
							// On a trouvé un mot
							tot += 1;
							System.out.println("il y a "+tot+" correspondance trouvé a ce stade");
							// Le nb de mot clef est il atteind?
							if ( tot >= this.nb_mc){
								// Si ca correspond c'est gagné
								existe = true;
								System.out.println(nb_mc+" occurence ont était trouvé ! c'est gagné...");
							}
						}
					}
				}
			}
		}
 
		return existe;
	}

Je suis désolé si c'est un peu lourd... mais je tenais à tout vous montrer afin que vous compreniez bien le problème.

En effet grâce aux 'alert' je voit bien que les correspondances sont trouvé mais equals renvoie false dans tous les cas sauf le dernier

voici le débogage :

Code :

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

1
2
3
4
5
6
 
// constructeur + ajout des mot clef à l'ensemble
...
carteJeu1.lireQuestion();
carteJeu1.lireReponse();
carteJeu1.evaluerReponse("Sarkozy s'apelle Carla Bruni");

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
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
 
Comment s'appelle la femme du président de la France?
Carla Bruni Sarkozy
 
Le premier mot est :Sarkozy 
o : sarkozy
mot : Sarkozy 
false
 
o : carla
mot : Sarkozy 
false
 
o : Sarkozy
mot : Sarkozy 
false
 
o : bruni
mot : Sarkozy 
false
 
o : Bruni
mot : Sarkozy 
false
 
o : Carla
mot : Sarkozy 
false
 
o : sarkozy
mot : s'apelle 
o : carla
mot : s'apelle 
o : Sarkozy
mot : s'apelle 
o : bruni
mot : s'apelle 
o : Bruni
mot : s'apelle 
o : Carla
mot : s'apelle 
 
o : sarkozy
mot : Carla 
o : carla
mot : Carla 
o : Sarkozy
mot : Carla 
o : bruni
mot : Carla 
o : Bruni
mot : Carla 
o : Carla
mot : Carla 
Le mot correspond :Bruni
il y a 1 correspondance trouvé a ce stade

Voila j’espère avoir été clair... mais en gros ma question est ^^ pourquoi le equals renvoie false alors que la correspondance existe?

**thelvin** · 06/11/2011, 00h48

Bon... J'ai pas l'énergie pour parler de tout ce qui ne va pas là-dedans. Et du coup, j'admets que sans améliorer tout ça, c'est difficile de montrer un programme plus court quand on ne sait pas d'où vient le problème.

Parlons seulement du problème qui t'intéresse :
Mets des [] autour des mots que tu débugues. Comme ça :

Code :

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

System.out.println("mot : [" + mot + "]");

Ça devrait te donner ta réponse.

**Mopmop** · 06/11/2011, 02h35

Envoyé par thelvin

Bon... J'ai pas l'énergie pour parler de tout ce qui ne va pas là-dedans.

Allez, on va dire que moi je l'ai

.

Pour simplifier ton code, utilise la méthode "contains(String s)" de la classe String...
Du coup ta fonction se simplifie en :

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
 
   private boolean testMotClef(String rep){
 
        int tot = 0;// nb de mots clef trouvés dans la réponse fournie par l'utilisateur
        Iterator<String> it = mot_clef.iterator();// iterateur pour parcourir le hashSet
 
        // on parcours l'ensemble des mots clés
        while ( it.hasNext() )
            if ( rep.contains(it.next()) )    // Si la réponse contient le mot clé 
            {                        
                tot++ ;
                // Le nb de mot clef est il atteint?
                if ( tot >= this.nb_mc)
                    return true ; 
            }       
        //Si on arrive là, c'est que le nombre de mots clés n'a pas été atteint
        return false;
    }

Et en ajoutant un peu de "toLowerCase()", tu peux ignorer la casse...

Et autre "problème" de ton code : le passage par la variable "existe" avec des break n'est pas nécessaire (c'est même caca), des return true / return false aux bons endroits c'est tellement mieux !

Voilà voilà donc pour résumer tu t'es (beaucoup) emmerdée pour pas grand chose

... la dure loi d'être débutante

A+

**tchize_** · 06/11/2011, 08h41

Je rajoute les conseils suivants:
pour couper une chaîne de caractères autour d'une séquence de caractères, utilisez

String[] elements = machaine.split(expression).

Donc dans votre cas
String mots = phrase.split(" ");

Ensuite, pour trouver un élément dans un hashset, inutile de le parcourir, utilisez sa méthode "contains". A noter que ce n'est pas utilisable pour "equalsIgnoreCase". Si vous voulez un comportement spécifique, englobez vos String dans un objet personnalisé. Exemple optimisable:

Code :

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

1
2
3
4
5
6
7
8
public class MaString{
    private final String contenu;
    public MaString(String contenu){this.contenu=contenu;}
    public boolean equals (Object o) {
        return (o instanceof MaString) && ((MaString)o).contenu.equalsIgnoreCase(contenu);
    }
    public int hashCode(){return contenu.toUpperCase().hashCode();}
}

**cleecky** · 07/11/2011, 00h08

Bonsoir,

Un grand merci à vous tous pour vos réponse !

Je me rend compte quand je code que je me complique la vie mais je ne voyait pas comment procéder autrement.
Là j'ai de quoi faire !
Je vais appliquer tous vos conseils et sachez que je vous suis vraiment très reconnaissante de toutes ses explications claire et complète.

Je croit que mon problème est résolu

Amicalement.
Cleecky

**Mopmop** · 07/11/2011, 00h45

Envoyé par cleecky

Je me rend compte quand je code que je me complique la vie

Comme je disais... la dure loi du débutant

.

Content d't'avoir aidée.

A+

**cleecky** · 07/11/2011, 10h25

Re !

J'ai mis en pratique vos solutions, Mopmop ta solution marche parfaitement bien ! C'est quand même un peu frustrant de voir ce petit bout de code marcher à la perfection quand je vois le temps que j'ai passé pour réussir à tester tous les mots avec ma technique foireuse...

Ceci dit je suis très contente d'avoir découvert cette méthode .contains() qui va me permettre de simplifier tout mon code et j’espère de gagner du temps pour la suite de mes développements.

tchize_ je te remercie aussi pour la piste de l'objet personnalisé je ne sait pas trop si je vais oser me lancer dedans...
Si choisit de créer une classe MaString pour englober mes String j'imagine que je serai dans l'obligation de la modéliser dans mon diagramme UML?!?

En tout cas encore Merci et bonne continuation à tous !

**tchize_** · 07/11/2011, 11h13

Envoyé par cleecky

Si choisit de créer une classe MaString pour englober mes String j'imagine que je serai dans l'obligation de la modéliser dans mon diagramme UML?!?

Non, java fonctionne très bien, même sans diagramme UML.

Je rajouterais que vousloir tout documenter dans l'UML, c'est vouloir se tirer une balle dans le pieds. Avec l'expérience, tu verra qu'un diagramme UML pour donners des ordre ou parler à des devs, c'est bien pratique, mais pour documenter l'existant c'est problématique: c'est jamais en phase avec le code, on fini toujours par oublier d'y ajouter / supprimer des trucs et au bout de 6 mois le diagramme ne sert plus à rien

**Robin56** · 07/11/2011, 11h36

Envoyé par tchize_

Non, java fonctionne très bien, même sans diagramme UML.

Je rajouterais que vousloir tout documenter dans l'UML, c'est vouloir se tirer une balle dans le pieds.

C'est sur oui. Par contre, pour débuter, je pense qu'il n'est pas forcément inintéressant de "se tirer une balle dans le pied".

Par la suite, tu retrouveras tout ce que vient de mentionner tchize_. Mais je t'invite tout de même à t'inculquer une certaine rigueur. C'est pratique je trouve pour débuter. Surtout qu'au commencement, on a toujours une vision macro un peu flou de notre application.

Edit : LEET \o/ ... (ah bah plus maintenant )

**cleecky** · 07/11/2011, 13h14

Je ne suis pas fan de l'UML mais comme ce projet est à rendre je vais être obligé de fournir un diagramme avec ^^

Du coup c'est vrai que pour l'instant ma technique de travail ne doit pas être optimal mais j'ai quand même réalisé un diagramme avant de me lancer dans le code. Ceci dit je pense que l’appellation diagramme n'est valide que pour moi

J'ai essayé de poser mes idées et l'architecture des classes mais j'ai du mal avec les liens (dépendance et tous le reste). Ce qui fait que je travail pour l'instant avec mon semi-UML et j'envisage d'aller voir du coté d'EclipseUML pour obtenir quelques choses de propre à présenter une fois mon application terminé.

Je vous vois déjà faire la moue devant vos écran

**tchize_** · 07/11/2011, 13h17

en général le petites classes d'aide (comme celle présentée qui ne sert qu'à un algorithme de tri, à un endroit ponctuel), je ne les mes jamais dans un diagramme. Le diagramme doit représenter la manière dont travaillent les composant principaux d'une application ou d'une section de l'application. En général, un diagramme ne devrait pas excéder une dixaine de classes selon moi

**yaraco** · 07/11/2011, 17h25

Si ça peut t'intéresser, j'ai pu utiliser ArgoUML (gratuit) il n'y a pas longtemps, également pour documenter un TP pour l'école.

Je ne sais pas ce qu'il vaut concernant tous les diagrammes qu'on peut se coltiner en UML, mais pour mon diagramme de classes il a bien fait l'affaire.

**cleecky** · 07/11/2011, 19h37

Si ça peut t'intéresser, j'ai pu utiliser ArgoUML (gratuit)

On m'en avait déjà parlé... Je l'est essayé lorsque j'ai commencé mon projet mais il m'a modéliser un truc louche qui ne ressemblait pas du tout à ce que j'avais fait et sans aucune dépendance entre les classes

Mais merci pour l'info ^^ je retenterai le coup une fois le projet finit ^^

**tchize_** · 07/11/2011, 22h39

c'est toi qui est censé modéliser le classes, pas argoUML.

Pour moi, le meilleur outil ca reste le grand tableau blanc et le feutres

**Robin56** · 07/11/2011, 23h25

Envoyé par cleecky

Je ne suis pas fan de l'UML mais comme ce projet est à rendre je vais être obligé de fournir un diagramme avec ^^

Envoyé par tchize_

Pour moi, le meilleur outil ca reste le grand tableau blanc et le feutres

<Professeur> Alors vous me rendez tous votre projet Javadoc + Source + UML pour le 1er Décembre au format ZIP
<Eleve> Je fais comment avec mon tableau blanc moi ?

**Mopmop** · 07/11/2011, 23h28

Envoyé par tchize_

Pour moi, le meilleur outil ca reste le grand tableau blanc et le feutres

Pauv prof, s'il a 30 élèves qui lui rendent un tableau ça va lui faire lourd

**thelvin** · 07/11/2011, 23h48

Envoyé par Robin56

<Professeur> Alors vous me rendez tous votre projet Javadoc + Source + UML pour le 1er Décembre au format ZIP
<Eleve> Je fais comment avec mon tableau blanc moi ?

Marre-toi mais dans ma boîte, nous utilisons souvent le tableau blanc pour décrire les choses, puis, pour ne pas perdre l'information qui y est stockée, nous le prenons en photo, faisons une retouche rapide de contraste, et utilisons ça parmi nos ressources de documentation.

J'ai de plus en plus envie d'un "tableau blanc tablette graphique" qui enregistrerait ce qu'on y dessine en temps réel sous forme numérique, pour éviter de se farcir des photos.

(Accessoirement, j'ai aussi des tableaux blancs chez moi quand je bricole, mais ça on peut envisager que c'est juste moi. Et Sheldon Cooper. Et Leonard Hofstadter.)

**Drakiss** · 08/11/2011, 10h14

Envoyé par thelvin

J'ai de plus en plus envie d'un "tableau blanc tablette graphique" qui enregistrerait ce qu'on y dessine en temps réel sous forme numérique, pour éviter de se farcir des photos.

Je te comprend, mais une tablette graphique est souvent un peu cher (et pour cet usage, un peu trop puissant), alors que j'ai fait l'acquisition d'un stylo numérique il y as peu, et ça me simplifie la vie pour moins de 100€.
Je ne ferai de pub pour personne ici, mais je trouve ça effectivement vraiment pratique. Après il existe des applications notamment sur certains smartPhones qui te permettent d'enregistrer et de contraster directement un paperBoard ou autre support et qui sont aussi vraiment pas mal, mais c'est toujours délicat de sortir son phone en réunion pour prendre une photo, tous le monde pense à la confidentialité

.

**tchize_** · 08/11/2011, 13h29

Envoyé par Robin56

<Professeur> Alors vous me rendez tous votre projet Javadoc + Source + UML pour le 1er Décembre au format ZIP
<Eleve> Je fais comment avec mon tableau blanc moi ?

On a pas inventé les appareils photos pour rien. C'est bien pratique une photo 5Mpixels pour conserver un graphe UML, une séance de brainstorming ou de mind mapping

edit: si on continue sur le sujet, je crois que je vais déplacer une partie de la discussion dans le forum conception

Comportement étrange d'un .equals

avec Java

Discussions similaires

Partager

Partager