尝试从返回null的hashmap值数组生成hashmap键数组

Question

我已经创建了一个方法，它接受一个hashmap作为输入，将值转换成一个数组，使用气泡排序对数组进行排序，然后我希望它使用排序后的值从初始hashmap中选择前N个索引键。我知道这不是最好的方法，但这是好的（我写了这个方法作为一个替代）。

该方法返回null，因为maxItems为空，并且“System.out.println（Entry.GetKey（））；”不会将任何内容打印到控制台。

谢谢你的帮助！

public String[]getMaxListAlt（int n,HashMap itemCount）{

    itemCount.values().toArray();

    Integer[] maxArray = itemCount.values().toArray(new Integer[0]);

    int length = maxArray.length;
    for (int i = 0; i < length-1; i++)
        for (int j = 0; j < length-i-1; j++)
            if (maxArray[j] > maxArray[j+1])
            {
                int temp = maxArray[j];
                maxArray[j] = maxArray[j+1];
                maxArray[j+1] = temp;
                
            }

    String[] maxItems = new String[n];
    int maxIndex = n-1;
    System.out.println(maxIndex);


    for (int i=0; i >= n-1; i++) {
        for (Map.Entry<String, Integer> entry : itemCount.entrySet()) {
            if (entry.getValue().equals(maxArray[i])) {
                System.out.println(entry.getKey());
                maxItems[i] = entry.getKey();
            }
        }
    }

        for (int counts : maxArray) {

        System.out.println(counts);

    }

    return maxItems;
}

Answer 1

如果我理解你正在尝试做什么，这里有另一种方法，你可能会感兴趣。然而，看起来你可能做错了几件事。

1. 您的冒泡排序是按升序排序的，我想您不想这样做。
2. 在循环entryset时，您会得到重复的值，因为您总是得到具有特定数字的第一个值。但这可能是您已有的内容的重复。

下面的流式处理是这样工作的。

• 它首先对字符串数组进行流式传输
• 然后使用groupingby
创建基于映射的频率计数
• 然后它将entryset流式传输，并根据值（计数）按降序排序
• 然后将条目数限制为n并将键映射到流
• 并返回数组中的这些键。

创建一些测试数据

Random r = new Random();
String[] v = r.ints(100, 'a', 'z' + 1)
        .mapToObj(c -> (char) c + "").toArray(String[]::new);

int n = 10;

流式处理

Map<String, Long> itemCount = Arrays.stream(v)
        .collect(Collectors.groupingBy(a -> a, Collectors.counting()));
        
String[] result = itemCount.entrySet()
        .stream()
        .sorted(Entry.<String,Long>comparingByValue().reversed())
        .limit(n).map(Entry::getKey).toArray(String[]::new);

for (int i = 0; i < result.length; i++) {
   System.out.println(result[i] + " --> " + itemCount.get(result[i]));
}

打印类似于

h --> 7
q --> 7
t --> 7
f --> 6
a --> 5
e --> 5
m --> 5
x --> 5
z --> 5
d --> 4

注意，如果需要，第一个流可以通过流entrySet()直接流入第二个流。在这种情况下，您返回数组，却看不到实际的映射。

如果要保留原始地图

这里是一个带有注释的修改，如果你想使用你的原始地图和方法。

public static String[] getMaxListAlt(int n,
        Map<String, Long> itemCount) {
    
    
    // copy entries to a list for sorting.
    List<Entry<String,Long>> entries = new ArrayList<>(itemCount.entrySet());
    
    // sort the entries based on value in  reverse order.
    entries.sort(Entry.<String,Long>comparingByValue().reversed());
    
    // now create the array and just grab the top n items from the sorted
    // entries list.
    String[] maxItems = new String[n];
    for (int i = 0; i < n; i++) {
        maxItems[i] = entries.get(i).getKey();
    }
    
    return maxItems;
} 

Answer 2

您的for-循环没有运行，因为您有一个键入-int i=0；i>=n-1；但您的i永远不能大于您的n-1。相反：

for (int i=0; i <= n-1; i++) {
    for (Map.Entry<String, Integer> entry : itemCount.entrySet()) {