Discussion :

[xeron33]

Bonjour à tous;
J'ai besoin de créer une méthode qui compare 2 objets :

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
 
 public static int comparePersonne(Object o1,Object o2) 
      { 
             Personne p1 = null,p2 = null; 
             Article a = null;
             Livre l = null;
              if (o1 instanceof Personne & o2 instanceof Personne) 
              { 
                      p1=(Personne)o1; 
                      p2=(Personne)o2; 
              } 
              else 
            	  if (o1 instanceof Article & o2 instanceof Livre) 
            	  {
            		  a=(Article)o1; 
                      l=(Livre)o2; 
            	  }
               else
               {
                      return Classable.ERREUR; 
              } 
              if (p1.getNom().compareTo(p2.getNom())<0) 
              { 
            	  return Classable.INFERIEUR; 
              } 
              if (p1.getNom().compareTo(p2.getNom())>0) 
              { 
                      return Classable.SUPERIEUR; 
              }
   X             else
   X           {
   X           return Classable.EGAL;
   X           }
 
              if (a.getref().compareTo(l.getref())<0)
              {
            	  return Classable.INFERIEUR; 
              }
              if (a.getref().compareTo(l.getref())>0)
              {
            	  return Classable.SUPERIEUR; 
              }
              else
              {
                return Classable.EGAL;
              }  
 
 
}

Cette méthode telle qu'elle est écrite renvoie une erreur de compilation :"Unreachable code"
En supprimant les lignes ou j'ai mis X l'erreur disparait et je n'arrive pas à comprendre pourquoi
Si quelqu'un a une idée
MERCI

[OButterlin]

Vu du point de vue du compilateur, p1 est soit supérieur à p2, soit inférieur à p2 soit égal... pourquoi voudrais-tu qu'il considère ce qui suit ?
Tes tests sont mal placés, tu devrait remonter les tests dans les conditions sur les types correspondants.
Autre chose, ce n'est pas & mais && pour une condition (le & est un ET binaire)

[xeron33]

Vu du point de vue du compilateur, p1 est soit supérieur à p2, soit inférieur à p2 soit égal... pourquoi voudrais-tu qu'il considère ce qui suit ?
Tes tests sont mal placés, tu devrait remonter les tests dans les conditions sur les types correspondants.
Autre chose, ce n'est pas & mais && pour une condition (le & est un ET binaire)

***************************************************
Merci pour ton aide . OK mais pourquoi ça marche ça ? :

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
 
 public static int comparePersonne(Object o1,Object o2) 
      { 
             Personne p1 = null,p2 = null; 
             Article a = null;
             Livre l = null;
              if (o1 instanceof Personne && o2 instanceof Personne) 
              { 
                      p1=(Personne)o1; 
                      p2=(Personne)o2; 
              } 
              else 
            	  if (o1 instanceof Article && o2 instanceof Livre) 
            	  {
            		  a=(Article)o1; 
                      l=(Livre)o2; 
            	  }
               else
               {
                      return Classable.ERREUR; 
              } 
              if (p1.getNom().compareTo(p2.getNom())<0) 
              { 
            	  return Classable.INFERIEUR; 
              } 
              if (p1.getNom().compareTo(p2.getNom())>0) 
              { 
                      return Classable.SUPERIEUR; 
              }
              else
              {
              return Classable.EGAL;
              }
}

Merci A+

[OButterlin]

et bien parce que p1 est soit supérieur à p2, soit il est inférieur, et en dernier ressort, il est égal... au-delà de ça, il n'y a plus de possibilité.

Comme dit, le problème vient du fait que tu n'as pas associé tes tests dans les conditions sur les types. Du coup, pour le compilateur, il se fiche (à ce niveau) de savoir que p1 et p2 sont null (ou peuvent l'être), ça ce sera un plantage à l'exécution...

[xeron33]

et bien parce que p1 est soit supérieur à p2, soit il est inférieur, et en dernier ressort, il est égal... au-delà de ça, il n'y a plus de possibilité.

Comme dit, le problème vient du fait que tu n'as pas associé tes tests dans les conditions sur les types. Du coup, pour le compilateur, il se fiche (à ce niveau) de savoir que p1 et p2 sont null (ou peuvent l'être), ça ce sera un plantage à l'exécution...

******************************************
Merci pour ta réponse mais je ne vois pas ce que tu veux dire car si tu regardes la fonction il y a deux tests identiques un pour p1 et p2 et l'autre pour a et l :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
if (p1.getNom().compareTo(p2.getNom())<0) 
              { 
            	  return Classable.INFERIEUR; 
              } 
              if (p1.getNom().compareTo(p2.getNom())>0) 
              { 
                      return Classable.SUPERIEUR; 
              }
              else
              {
              return Classable.EGAL;
              }

et

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
if (a.getref().compareTo(l.getref())<0)
              {
            	  return Classable.INFERIEUR; 
              }
              if (a.getref().compareTo(l.getref())>0)
              {
            	//  return Classable.SUPERIEUR; 
              }
              else
              {
                return Classable.EGAL;
              }

et le fait de supprimer le 1er else :

Code :

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

1
2
3
4
5
 
 /*        else
              {
              return Classable.EGAL;
              }*/

permet de corriger l'erreur ... ca veut dire que l'on ne peut pas faire 2 tests identiques sur une meme fonction pour des variables différentes ?
La j e ne comprends pas ...

[OButterlin]

Je ne sais pas comment te le dire mais bon, un dernier essai :

Tu testes p1 avec p2 en premier donc :

- soit p1 est supérieur à p2
- soit p1 est inférieur à p2
- soit p1 est égal à p2

Quel autre cas selon toi peut-il exister ?

Il n'y en a pas !
==> ce qui suit ne peut pas être atteint

Si tu mettais ce test dans une condition "si p1 et p2 sont différents de NULL" alors on pourrait continuer avec a et l mais comme tu ne l'as pas fait, le compilateur te dit (à juste titre) que les lignes qui suivent ne peuvent être atteintes.