开头 Java（直方图）

Question

我正在接受初学者 Java class，任务是使用以下 output 创建直方图程序：（100 和 10 是用户输入）。

多少号？ 100 有多少个间隔？ 10

Histogram
--------------------------------------------------------
  1 ****(4)
  2 ******(6)
  3 ***********(11)
  4 *****************(17)
  5 **************************(26)
  6 *************************(25)
  7 *******(7)
  8 ***(3)
  9 (0)
 10 *(1)
--------------------------------------------------------

我的代码给出以下 output 但是，任何人都可以帮我指出出了什么问题，非常感谢。

How Many Numbers? 10
How Many Intervals? 10

Histogram
--------------------------------------------------------
 1 **********(10)
 2 **********(10)
 3 **********(10)
 4 **********(10)
 5 **********(10)
 6 **********(10)
 7 **********(10)
 8 **********(10)
 9 **********(10)
 10 **********(10)

对于输入 100 和 10，我收到错误消息：

线程“main”中的异常 java.lang.ArrayIndexOutOfBoundsException: 10 at Lab6.main(Lab6.java:44) 我在下面标记了第 44 行。

哪些链接到此代码；

             intervalValue[j]++;

我不确定如何附加生成器 (.jar) 文件，它应该只是为我们生成随机 #'s。 再次感谢。

mport java.util.Scanner;

public class Lab6 {

    public static void main(String[] args) {

        int numbers, intervals;
        double intervalWidth;

        double max, mins, range;

        Scanner keyboard = new Scanner(System.in);

        System.out.print("How Many Numbers? ");

        numbers = keyboard.nextInt();

        System.out.print("How Many Intervals? ");

        intervals = keyboard.nextInt();

        double [] generate = new double[numbers];

        generate = randomGenerator(numbers);

        max = maximum(generate);

        mins = minimum(generate);

        range = max - mins;

        intervalWidth = range / intervals;

        int [] intervalValue = new int[intervals];

        for (int i=0; i < generate.length; i++) {

            for (int j = 0; j<generate.length; j++){
                double imin = mins+j*intervalWidth;
                double imax = max +j*(intervalWidth);
                if(generate[i] >= imin && generate[i] < imax)
                    intervalValue[j]++;         //LINE 44
            }
        }

        System.out.println("Histogram");

        System.out.println("--------------------------------" +
                           "------------------------");

        for (int a=0; a < intervalValue.length; a++) {

            System.out.print(" " + (a+1) + " ");

            for (int b=0; b < intervalValue[a]; b++) {
                System.out.print("*"); 
            }

            System.out.println("(" + intervalValue[a] + ")");
        }
    }

    private static double [] randomGenerator(int number) {
        double [] generate;

        generate = Generator.getData(number);

        return generate; 
    }

    private static double maximum(double [] a) {

        double max = a[0];

        for (int i = 1; i < a.length; i++) {        
            if (a[i] > max) {
                max = a[i];
            }         
        }

        return max;
    }

    private static double minimum(double [] a) {

        double mins = a[0];

        for (int i = 1; i < a.length; i++) {
            if (a[i] < mins) {
                mins = a[i];
            }
        }

        return mins;
    }
}

Answer 1

您需要从第43行的末尾删除分号：

if(generate[i] >= imin && generate[i] < imax);

分号会使您的if块为空。 结果，导致异常的行将无条件执行。

解决该问题后，以下提示可帮助您开始进一步调试代码：

---

看一下内部循环的终止条件：

for (int j = 0; j<generate.length; j++)

您正在外部循环中迭代生成的数字。 因此，您应该遍历内部循环中的时间间隔，以确定每个生成的数字所属的时间间隔。

---

看一下您用来确定循环中当前间隔范围的代码：

double imin = mins+j*intervalWidth;
double imax = max +j*(intervalWidth);

这些行需要修改以产生正确的值。 进行一些“笔和纸调试”以确定为什么它们当前存在错误。

---

我现在将其余内容留给读者作为练习。 稍后，我将再次检查是否需要任何进一步的帮助。

Answer 2

由于我在星期六很宽裕，因此尝试了一下，并重新编写了循环。

for (int j=0; j < generate.length; j++) {
    for(int i = 0; i < intervals; i++) {
        double imin = mins + i * intervalWidth;
        double imax = mins + (intervalWidth) * (i + 1);
        if(i == intervals - 1) imax = Double.POSITIVE_INFINITY;
        if(i == 0) imin = Double.NEGATIVE_INFINITY;

        if (generate[j] >= imin && generate[j] < imax) {
            intervalValue[i]++;
            break;
        }
    }
}

无限的东西是要捕捉直方图中的最小值和最大值。

Answer 3

这将为您提供一个非常漂亮（基本）的直方图。 试试吧

import java.util.HashMap;
import java.util.Map;

public class Histogram {
    public static void main(String[] args) {
        int[] age = { 25, 26, 33, 26, 27, 21, 26, 33, 21, 33, 21, 38, 19, 19};


        HashMap<Integer, Integer> m = new HashMap<Integer, Integer>();

        for (int i = 0; i < age.length; i++) {
            int c = 0;

            for (int j = 0; j < age.length; j++) {
                if (age[i] == age[j]) {
                    c++;
                }
            }
            m.put(age[i], c);

        }

        System.out.println("Histogram\n------------");
        for (Map.Entry<Integer, Integer> entry : m.entrySet()) {
            int key = entry.getKey();
            int value = entry.getValue();
            System.out.printf("%3d | ", key);
            for (int i = 0; i < value; i++) {
                System.out.print("=");
            }
            System.out.print(" " + value);
            System.out.println();
        }

    }

Outout：

Histogram
------------
 33 | === 3
 19 | == 2
 21 | === 3
 38 | = 1
 25 | = 1
 26 | === 3
 27 | = 1

我正在使用HashMap保留两个相关的值，即数组元素及其频率（它出现的频率）。

然后，嵌套循环将遍历数组中的每个元素，并使用c变量计算其频率。

之后，我用for-each循环和normal循环打印它

Answer 4

使用像eclipse这样的IDE，运行您的程序，查看第44行在哪里。 那是您获取ArrayIndexOutOfBoundsException的地方，并且需要确保不会失去联系。 Java中的数组在索引0处有第一项，因此长10项的数组将其编号为0、1、2、3、4、5、6、7、8、9。 如果发生在10点，您可能要迭代一个步骤。 到达那里时，请确保“ intervalValue [j]”中的j不为10。

在没有完全例外的情况下，源代码会在您粘贴的源代码中进行编译或更正行号。 没有更多的事情要做。 弄清楚这一点将帮助您获得与其余练习一样的知识。 试一试。

Answer 5

intervalValue[j]++; 

j上升到数字的数量，但是intervalValue是从间隔的数量分配的，如果这两个数字不相同，您将在这里看到错误。

Answer 6

您的数组异常似乎是由两次使用generate.length引起的-尽管您粘贴的代码似乎是从此编辑的。

但是，除此之外，问题还在于嵌套循环。 除了在第一个内部循环中使用错误的数组大小外，您还将沿着正确的方向前进。 然后，在if语句中-您想要增加该值，并且如果该值在正确的间隔内，则仅增加该值； 使用您当前的代码，您每次都会增加它们中的每一个，这将解释输出。

使用您自己的代码，并注释：

for (int i=0; i < generate.length; i++) {

  for (int j = 0; j<intervals; j++){ // could use intervalValues.length here; it's all preference
    double imin = mins+j*intervalWidth;
    double imax = mins +(intervalWidth)*(j+1);
    if(generate[i] >= imin && generate[i] < imax) 
        // for(int j1 = 0; j1 < intervalValue.length; j1++) <- this was causing the bad output
        /* I assume the j1 from your code was during a debug attempt. Changed back,
           since the preceeding loop has been removed */
        intervalValue[j]++; 


  }

}

Answer 7

你可以使用这个：

    int myArray[] = new int[]{1, 2, 1, 3, 3, 1, 2, 1, 5, 1};

    public static void main (String args[]) {
      HashMap<Integer, String> hash = new HashMap<>();
      hash.put(5, "");
      hash.put(4, "");
      hash.put(3, "");
      hash.put(2, "");
      hash.put(1, "");
      
      for (int i = 0; i < myArray.length; i++){
        hash.put(new Integer(myArray[i]), hash.get(myArray[i])+"*");
      }
      
      for(Integer key: hash.keySet()){
        System.out.println(key+": "+ hash.get(key));
      }
    }