LinkedHashMapをvalueクラスのフィールドでソートするにはどうすればよいですか？

Question

次の行を使用してLinkedHashMapをソートしますが、すべての項目がソートされているわけではありません。

LinkedHashMap<String,PatternData> statisticsMap; 
// fill in the map ... 

LinkedHashMap<String,PatternData> sortedStatisticsMap=new LinkedHashMap<String,PatternData>();  // Sort it by patternData's average 

ArrayList<PatternData> statisticsMapValues=new ArrayList<PatternData>(statisticsMap.values()); 
Collections.sort(statisticsMapValues,Collections.reverseOrder());    // Sorting it (in reverse order) 

patternData last_i=null; 
for (PatternData i : statisticsMapValues)          // Now, for each value 
{ 
    if (last_i==i) continue;               // Without dublicates 
    last_i=i; 

    for (String s : statisticsMap.keySet())           // Get all hash keys 
    if (statisticsMap.get(s)==i)             // Which have this value 
    { 
     sortedStatisticsMap.put(s,i); 
    } 
} 


class PatternData implements Comparable<PatternData> 
{ 
    float sum=0,average; 
    int totalCount=0; 
    Vector<String> records=new Vector<String>(); 

    public PatternData() { } 

    public void add(float data) 
    { 
    sum+=data; 
    totalCount++; 
    average=sum/totalCount; 
    } 

    public void add(float data,String record) 
    { 
    add(data); 
    records.add(record); 
    } 

    float getAverage() { return average; } 

    public int compareTo(patternData o) { return (int)(average-o.average); } 
} 

Answer 1

あなたはint型、範囲を返すときの平均-o.averageの間で-1と1は常に0

一つの解決策は単純にあなたのcompareTo機能を変更されて返されます。

return Float.compare(average, o.average); 

Answer 2

整数を使用して浮動小数点数をソートしています。整数は丸められません。それらは切り捨てられます。また、実際にソートを行っている方法を考えれば、代わりにTreeHashMapを使用することを検討してください。

（とちょうどnitpickする、Javaの規則は、メソッドや変数名に小文字を使用しています）