当前位置: 代码迷 >> java >> Array.sort仅返回0个值。 大小正确的数组和数组已满
  详细解决方案

Array.sort仅返回0个值。 大小正确的数组和数组已满

热度:79   发布时间:2023-07-27 09:35:48.0

对不起,这些主题中的另一个主题,但是正在阅读那里的内容,没有任何内容回答我的问题。 我有一些非常基本的代码,它们仅接受5个值,对它们进行排序,并对3个中间值进行数学运算,最后输出一些结果。 不难,但是由于某种原因,我的Array.sort仅返回值0。我很确定数组中的所有点在排序之前都已填充,并且我的数组大小正确。 建议?

import java.util.*;
import java.util.Arrays;

    public class practice {

        public static void main(String[] args) {
            // TODO Auto-generated method stub
            Scanner scanner = new Scanner(System.in);
            int[] finals = new int[3];
            int[] scores = new int[5];
            int difficulty = 0;
            for (int x = 0; x < 3; x++) {
                System.out.println("Please input the scores for diver " + (x + 1));

                for (int y = 0; y < 5; y++) {
                    scores[x] = scanner.nextInt();
                    System.out.println(scores[x]);
                }
                Arrays.sort(scores);
                for (int j = 0; j < 5; j++) {
                    System.out.println(scores[x]);
                }
                System.out.println("Please input the difficult of diver " + (x + 1));
                difficulty = scanner.nextInt();
                finals[x] = (((scores[1] * scores[2] * scores[3]) / 3) * difficulty);
                System.out.println(finals[x]);
            }
            winner(finals);

        }

        public static void winner(int[] finals) {
            System.out.println(finals[0]);
            if (finals[0] > finals[2] && finals[0] > finals[1]) {
                System.out.println("Diver 1 is the winner of the olympics with a score of " + finals[0]);
            } else if (finals[1] > finals[2] && finals[1] > finals[0]) {
                System.out.println("Diver 2 is the winner of the olympics with a score of " + finals[1]);
            } else if (finals[2] > finals[0] && finals[2] > finals[1]) {
                System.out.println("Diver 3 is the winner of the olympics with a score of " + finals[2]);
            } else {
                System.out.println("There was a tie");
            }
        }

    }

这个,

for (int y = 0; y < 5; y++) {
    scores[x] = scanner.nextInt();
    System.out.println(scores[x]);
}

应该

for (int y = 0; y < scores.length; y++) {
    scores[y] = scanner.nextInt();
    System.out.println(scores[y]);
}

不要依赖硬编码的长度。 并注意您要迭代哪个变量。 您在这里犯了一个非常类似的错误

for (int j = 0; j < 5; j++) {
    System.out.println(scores[x]);
}

应该

for (int j = 0; j < scores.length; j++) {
    System.out.println(scores[j]);
}

您的问题有两个。

在您的for循环中

for (int y = 0; y < 5; y++) {
     scores[x] = scanner.nextInt();
     System.out.println(scores[x]);
}

您会继续将分数添加到scores数组的相同空间(在本例中为0)。 如果代码中有x,则应该将y的所有5个分数加到数组中,如下所示:

for (int y = 0; y < 5; y++) {
    scores[y] = scanner.nextInt();
    System.out.println(scores[y]);
}

在此代码中打印分数时,您犯了同样的错误,您再次仅打印了x,这是从您的外循环开始的,此时仍然相同(在本例中仍为0):

for (int j = 0; j < 5; j++) {
    System.out.println(scores[x]);
}

您的代码在此处应为:

for (int j = 0; j < 5; j++) {
    System.out.println(scores[j]);
}

这意味着您将随后遍历数组中的所有5个分数。

其余的一切看起来都不错,并且可以正常运行。

尽管最佳实践是在这两个循环的约束中使用scores.length而不是5。

  相关解决方案