在优先级队列中使用比较器进行排序,但没有正确排序

时间:2012-07-09 03:14:15

标签: java

PriorityQueue<Player> playerQueue = new PriorityQueue<Player>(30,
                new Comparator<Player>( ) {
                    // overriding the compare method
                    public int compare(Player i, Player j) {
                        return (int)i.playerPrice() < (int)j.playerPrice() ? -1 : ((int)i.playerPrice() == (int)j.playerPrice()) ? 0 : 1;
                    }
                }
          );

        for(Player a:XMLParserViaSax.getListOfPlayers()){
            System.out.println(a.getName()+ "-->"+a.playerPrice());
        }

        for(Player player:XMLParserViaSax.getListOfPlayers()){
            playerQueue.add(player);
        }
        //playerQueue.addAll(XMLParserViaSax.getListOfPlayers());

        System.out.println("**************************");
        for(Player j:playerQueue){
            System.out.println(j.getName()+ "-->"+j.playerPrice());
        }

我正在尝试按玩家价格对玩家进行排序,我没有任何变量作为玩家类别中的玩家价格.playerPrice()是一种计算玩家价格的方法。现在当我尝试对这些玩家进行排序时对象给我一个随机的顺序。

我是否需要在我的播放器类中拥有一个playerprice私人会员? “ 玩家类是这样的:

public class Player {

    private String name;
    private int battingStrength;
    private int bowlingStrength;
    private int fieldingStrength;
    private int keepingStrength;
    private int jerseyNumber;

    public int playerStrength() {
    return AuctionPlayerHelper.calculateStrength(this);
    }

    public boolean canKeepWickets() {
    return (keepingStrength == 10);
    }

    public boolean isForeignPlayer(){
    return AuctionPlayerHelper.isForeignPlayer(this.jerseyNumber);
    }

    public double playerPrice() {
     return AuctionPlayerHelper.calculatePrice(this);

    }

    //getters,setters
}

1 个答案:

答案 0 :(得分:1)

来自PriorityQueue API

的注释
  

方法iterator()中提供的迭代器不保证以任何特定顺序遍历优先级队列的元素。如果需要有序遍历,请考虑使用Arrays.sort(pq.toArray())。

因此,正如您所做的那样遍历队列将不会处于“顺序”。

当你从队列中删除头部时,例如当你在队列上调用poll()时,优先顺序就会变得很明显。