ヒストグラムのプログラムをつくっていますがデータのカウントが上手くいきません

Question

得点の10点刻みの分布をつくっています。乱数のプログラムから乱数を読みこむ形です。乱数のプログラムとはパイプでつなぐので、そこは問題なくできています。
以下は私がつくったプログラムです。

import java. util.Scanner;
public class Histogram{
    public static void main(String[]args){
	Scanner stdin=new Scanner (System.in);
	int [] counter=new int[11];
	int p=0;
	 counter[p/10]++;

	while(stdin.hasNext()){
            counter[p/10]=stdin.nextInt();
	     }
	
	for(int x=0;x<11;x++){
	    if(x==10){
		System.out.printf("%3d",100);
	    }else{
		System.out.printf("%3d-%2d",x*10,x*10+9);
	    }
	    for(int i=0;i<counter[x];i++){
		System.out.printf("%3d",counter[x]);
	    }
	    System.out.println();
	}
    }
}

出力結果は、
100点の人が何人、90-99の人が何人、というような感じにしたいと思っています。
配列で読み込んで数えあげるのが上手くいかなくて困っています。おかしな点がありましたら教えていただけると助かります。宜しくお願いします。

Answer 1

こういう感じになるでしょうか。

package sample;

import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.Optional;

public class Histogram {
	
	// 入力値
	List<Integer> points = new ArrayList<>();
	
	// ヒストグラム値
	Map<Integer,Integer> distribution = new HashMap<>();
	
	// 値を設定する
	public void setValues(List<Integer> points) {
		this.points.addAll(points);
	}
	
	// 10点ごとのヒストグラムにした結果を返す。キー名は仮に2桁目以降の値。
	public Map<Integer,Integer> getDistribution() {
		points.stream().forEach(point -> {
			int dist = point.intValue() / 10;
			int count = Optional.ofNullable(distribution.get(dist)).orElse(0);
			distribution.put(dist, ++count);
		});
		
		return distribution;
	}
}

テストケースは次の通り：

package test;

import static org.junit.Assert.*;

import java.util.Arrays;
import java.util.Map;

import org.junit.Before;
import org.junit.Test;

import sample.Histogram;

/**
 * @author A-pZ
 *
 */
public class HistogramTest {
	
	Histogram histogram;
	
	/**
	 * @throws java.lang.Exception
	 */
	@Before
	public void setUp() throws Exception {
		histogram = new Histogram();
	}

	@Test
	public void test() {
		histogram.setValues(Arrays.asList(0,10,20,30,40,50,60,70,80,90,100));
		
		Map<Integer,Integer> dist = histogram.getDistribution();
		
		assertEquals(new Integer(1) , dist.get(0));
		assertEquals(new Integer(1) , dist.get(1));
		assertEquals(new Integer(1) , dist.get(2));
		assertEquals(new Integer(1) , dist.get(3));
		assertEquals(new Integer(1) , dist.get(4));
		assertEquals(new Integer(1) , dist.get(5));
		assertEquals(new Integer(1) , dist.get(6));
		assertEquals(new Integer(1) , dist.get(7));
		assertEquals(new Integer(1) , dist.get(8));
		assertEquals(new Integer(1) , dist.get(9));
		assertEquals(new Integer(1) , dist.get(10));
	}
	
	@Test
	public void test2() {
		histogram.setValues(Arrays.asList(10,10,10,10,10,20,21,23,25,27,30));
		
		Map<Integer,Integer> dist = histogram.getDistribution();
		
		assertEquals(null , dist.get(0));
		assertEquals(new Integer(5) , dist.get(1));
		assertEquals(new Integer(5) , dist.get(2));
		assertEquals(new Integer(1) , dist.get(3));
		assertEquals(null , dist.get(4));
		assertEquals(null , dist.get(5));
		assertEquals(null , dist.get(6));
		assertEquals(null , dist.get(7));
		assertEquals(null , dist.get(8));
		assertEquals(null , dist.get(9));
		assertEquals(null , dist.get(10));
	}
	
}

Answer 2

ヒストグラム作成する部分について
int p=0;
counter[p/10]++;

while(stdin.hasNext()){
        counter[p/10]=stdin.nextInt();
     }

↓
//counterの各要素を0で初期化しておく（ソースコード略）
//以下の処理でヒストグラム作成
while(stdin.hasNext()){
counter[stdin.nextInt() / 10]++;
}

後は表示部も何か変だと思うけど、そこは作った人の考えがわからないと指摘できないです。