一亩三分地论坛

 找回密码
 获取更多干货,去instant注册!

扫码关注一亩三分地公众号
查看: 781|回复: 2
收起左侧

[算法题] PriorityQueue使用求助

[复制链接] |试试Instant~ |关注本帖
raphtao07 发表于 2015-12-12 07:13:02 | 显示全部楼层 |阅读模式

注册一亩三分地论坛,查看更多干货!

您需要 登录 才可以下载或查看,没有帐号?获取更多干货,去instant注册!

x
之前一直使用Prioriry Queue没出现过什么问题。这次给一个Map Queue按照规则进行排序,重写了一下Comparator。

使用的时候发现队列的排序功能无效,每次peek只能peek出最后一个放进队列的元素。

在Comparator里添加log之后发现map1 和 map2是一样的。也就是每次都在和自己比大小。

求大神帮忙解答问题出在哪









PriorityQueue<HashMap<String, Integer>> queueOfContents = new PriorityQueue<HashMap<String, Integer>>(new Comparator<HashMap<String, Integer>>(){
                    @Override
                    public int compare(HashMap<String, Integer> map1, HashMap<String, Integer> map2){
                            System.out.println(map1);
                            System.out.println(map2);
                    double refVal1 = 0.0;
                    double refVal2 = 0.0;
                    location local = firstLocation;
                    while(local != null){
                            if(map1.get(local.locationId) != null) {
                                    refVal1 += locationMap.get(local.locationId) * contentMap.get(map1.get(local.locationId));
                            }
                            if(map2.get(local.locationId) != null) {
                                    refVal2 += locationMap.get(local.locationId) * contentMap.get(map2.get(local.locationId));
                            }
                            local = local.next;
                    }

                    if (refVal1 > refVal2) {
                            return -1;
                    } else if (refVal1 == refVal2) {
                            return 0;
                    }
                    else {
                            return 1;
                    }
                           
                    }
            });

ryb 发表于 2015-12-12 09:30:13 | 显示全部楼层
这段代码感觉没什么问题, map1和map2地址相同说明你可能之前push到queue里的都是同一个map,也就是说你每次都在操作一个hashmap. 你每次放进去有new HashMap吗?

评分

1

查看全部评分

回复 支持 反对

使用道具 举报

 楼主| raphtao07 发表于 2015-12-12 10:50:03 | 显示全部楼层
ryb 发表于 2015-12-12 09:30
这段代码感觉没什么问题, map1和map2地址相同说明你可能之前push到queue里的都是同一个map,也就是说你每 ...

确实发现问题不在priority queue上,我的queue后来检查放进去的是n个空map, 我再检查检查
回复 支持 反对

使用道具 举报

本版积分规则

请点这里访问我们的新网站:一亩三分地Instant.

Instant搜索更强大,不扣积分,内容组织的更好更整洁!目前仍在beta版本,努力完善中!反馈请点这里

关闭

一亩三分地推荐上一条 /5 下一条

手机版|小黑屋|一亩三分地论坛声明 ( 沪ICP备11015994号 )

custom counter

GMT+8, 2016-12-11 12:41

Powered by Discuz! X3

© 2001-2013 Comsenz Inc. Design By HUXTeam

快速回复 返回顶部 返回列表