élément dupliqué dans hashset (après avoir implémenté correctement la méthode comparaison/comparaison)

Question

J'écris actuellement un programme qui lit les lignes d'un fichier et crée des jetons. Les jetons sont ensuite sauvegardés dans treeset. Je ne veux pas de doublons dans l'ensemble, j'ai donc implémenté Comparator avec une méthode de comparaison correcte. Mais je ne suis toujours pas capable de comprendre pourquoi un élément est répété dans l'arborescence.

fichier d'entrée: (13 lignes)

VEG,SMALL,Corn,MOZZARELLA 
VEG,LARGE,Capsicum,CHEDDAR 
VEG,MEDIUM,Olives,COTTAGE 
VEG,MEDIUM,Garlic,MOZZARELLA 
VEG,LARGE,Peppers,CHEDDAR  
NONVEG,MEDIUM,Beef,CHEDDAR 
NONVEG,LARGE,Chicken,COTTAGE 
NONVEG,SMALL,Pork,MOZZARELLA 
NONVEG,LARGE,Mutton,CHEDDAR  
NONVEG,SMALL,Mutton,MOZZARELLA 
VEG,SMALL,Corn,MOZZARELLA 
NONVEG,MEDIUM,Beef,CHEDDAR 
NONVEG,SMALL,Mutton,MOZZARELLA 

mon code:

public Set<Pizza> populateData(String FileName) { 

    Set<Pizza> set = new TreeSet(new Comparator<Pizza>(){ 

     @Override 
     public int compare(Pizza o1, Pizza o2) { 

      if(o1.getPizzType().compareTo(o2.getPizzType())==0) 
      { 
       if(o1.getCheeseType().compareTo(o2.getCheeseType())==0) 
       { 
        if(o1.getSize().compareTo(o2.getSize())==0) 
        { 
         if(o1.getTopping().compareTo(o2.getTopping())==0) 
         { 
          return 0; 
         } 
        } 
       } 
      } 
      return 54; 
     } 


    }); 


    File file = new File(FileName); 
    Scanner scanner=null; 

    try { 
     scanner=new Scanner(new FileInputStream(file)); 
     Pizza pizza=null; 

     while(scanner.hasNext()) 
     { 
      String line=scanner.nextLine(); 
      String words[]=new String[4]; 
      words=line.split(","); 

      pizza=new Pizza(); 

      if(words[0].trim().length()>0) 
      { 
       pt=PizzaType.valueOf(words[0].trim()); 
       pizza.setPizzType(PizzaType.valueOf(words[0].trim())); 
      } 
      if(words[1].trim().length()>0) 
      { 
       sz=(Size.valueOf(words[1].trim())); 
       pizza.setSize(Size.valueOf(words[1].trim())); 
      } 
      if(words[2].trim().length()>0) 
      { 
       tp=(words[2].trim()); 
       pizza.setTopping(words[2].trim()); 
      } 
      if(words[3].trim().length()>0) 
      { 
       ct=(CheeseType.valueOf(words[3].trim())); 
       pizza.setCheeseType(CheeseType.valueOf(words[3].trim())); 
      } 


      set.add(pizza); 

     } 

    } catch (FileNotFoundException e) { 
     // TODO Auto-generated catch block 
     e.printStackTrace(); 
    } 

    return set; 
} 

(sortie imprimée le long d'une longueur de chaîne et hashcode)

sortie: (11 lignes) {premier et la dernière ligne est en double: P}

VEG SMALL Corn MOZZARELLA 22 589431969 
VEG LARGE Capsicum CHEDDAR 23 1252169911 
VEG MEDIUM Olives COTTAGE 22 2101973421 
VEG MEDIUM Garlic MOZZARELLA 25 685325104 
VEG LARGE Peppers CHEDDAR 22 460141958 
NONVEG MEDIUM Beef CHEDDAR 23 1163157884 
NONVEG LARGE Chicken COTTAGE 25 1956725890 
NONVEG SMALL Pork MOZZARELLA 25 356573597 
NONVEG LARGE Mutton CHEDDAR 24 1735600054 
NONVEG SMALL Mutton MOZZARELLA 27 21685669 
VEG SMALL Corn MOZZARELLA 22 2133927002 

Résultats attendus: (10 lignes)

VEG SMALL Corn MOZZARELLA 
VEG LARGE Capsicum CHEDDAR 
VEG MEDIUM Olives COTTAGE 
VEG MEDIUM Garlic MOZZARELLA 
VEG LARGE Peppers CHEDDAR 
NONVEG MEDIUM Beef CHEDDAR 
NONVEG LARGE Chicken COTTAGE 
NONVEG SMALL Pork MOZZARELLA 
NONVEG LARGE Mutton CHEDDAR 
NONVEG SMALL Mutton MOZZARELLA 

Answer 1

Comparator contract oblige à retourner négatif si o1 < o2, positif si o1 > o2, et zéro si o1 == o2.

Ce qui signifie que votre code doivent ressembler à

public int compare(Pizza o1, Pizza o2) { 

    int compvalue = o1.getPizzType().compareTo(o2.getPizzType()); 
    if(compvalue == 0) 
    { 
     compvalue = o1.getCheeseType().compareTo(o2.getCheeseType()); 
     ...etc.. 
     if(compvalue == 0) 
     ... 
    } 
    return compvalue; 
}