我试图为我的编码挑战之一计算给定分数的排名,但是我对收到的输出感到惊讶。

我必须为给定的分数创建排行榜(假设分数以array的顺序排列)。我创建了一种方法,可以给出给定分数的排名,

 private static int[] assignCurrentRanks(int[] scores) {
        int[] currentRanks=new int[scores.length];
        currentRanks[0]=1;//by default the first score will be rank 1
        for(int i=1;i<currentRanks.length;i++) {
            currentRanks[i]=(scores[i]==scores[i-1])?currentRanks[i-1]:++currentRanks[i-1];
            System.out.println("Inside the loop : ["+i+"] "+scores[i]+" "+currentRanks[i]);
        }
        return currentRanks;
 }


输入示例:100,100,50,40,40,20,10
预期排名:1,1,2,3,3,4,5

我可以看到在循环过程中已正确分配了等级,当我在main方法中打印返回的数组不同之后,便打印了等级。

我知道逻辑存在问题,但我找不到。请帮忙!

以下是我的main方法,

public static void main(String[] args) {
        int[] scores=new int[]{100,100,50,40,40,20,10};
        int[] currentRanks=assignCurrentRanks(scores);

        System.out.println("Printing after the loop");
        for(int i=0;i<scores.length;i++)
            System.out.println("["+i+"]"+scores[i]+" "+currentRanks[i]);
}


我得到的结果是

//index 0 is already assigned with 1 by default
Inside the loop : [1] 100 1
Inside the loop : [2] 50 2
Inside the loop : [3] 40 3
Inside the loop : [4] 40 3
Inside the loop : [5] 20 4
Inside the loop : [6] 10 5
Printing after the loop
[0]100 1
[1]100 2
[2]50 3
[3]40 3
[4]40 4
[5]20 5
[6]10 5

最佳答案

++currentRanks[i-1];\\this increments the value of array element i-1

您增加循环内使用的优先级,从而导致意外行为。应将预期行为更改为:

currentRanks[i]=(scores[i]==scores[i-1])?currentRanks[i-1]:currentRanks[i-1] + 1;

10-07 17:50