JAVAに関してシンプルな書き方で変更したいです

Question

前提・実現したいこと

JAVAのソースを僕は下記のように書きましたが、
さらにシンプルな書き方があれば教えてください！！！

発生している問題・エラーメッセージ

下記の問題に関してです。

「サイコロを200回振りました．それぞれの出た目（1～6）の回数を記録し，「＊」で表示しする」

該当のソースコード

import java.util.Scanner;
import java.util.Random;

public class Test {
  public static void main(String[] args) {
    
    int[] counts = new int[7];

    for (int i = 0; i <=200; i++) {
      int a = DieA.roll();
      
      counts[a]++;
    }
    
    for (int i = 1; i < counts.length; i++) {
      System.out.printf("%d : ", i);
      for (int j = 0; j < counts[i]; j++) {
        System.out.print("*");
      }
      System.out.println();
    }
  }
}
class DieA {
  public static int roll() {
    double r = Math.random() * 6; 
    int randInt = (int) r;       
    return randInt + 1;         
  }
}

試したこと

シンプルな書き方で変更したい

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[BluOxy]

どこの処理がシンプルに書けそうだと思いますか。

Answer 1

もうひと工夫欲しいところだけど、思いつかない。
参考: StreamでGroup By処理を行いたい(group-by-count, group-by-sum, group-by-max) - Qiita

import java.util.Random;
import java.util.stream.Collectors;
import java.util.function.Function;

public class Test {
	public static void main(String[] args) {
		new Random()
			.ints(200, 1, 7)
			.boxed()
			.collect(Collectors.groupingBy(Function.identity(), Collectors.counting()))
			.entrySet()
			.stream()
			.map(e -> e.getKey() + " : " + "*".repeat(e.getValue().intValue()))
			.forEach(System.out::println);
	}
}

Comments

[swordone]

これ、順番保証ができないのでは…

[raccy]

順番はバラバラでおかしくないはずなのですが、何故かソートされます。collectのドキュメントを読んでもよくわかりません。順番保証するなら、sorted()をかましたほうが良いかもしれませんね。

あとは、非常に低確率ですが、一回も出なかった目が表示されないというのもあります。これはどうすれば良いのか…こうなると、boxed()せずにIntStreamのままやったほうが良かったのかも知れません。

[TN8001]

ばかばかしいですがこれはどうですか^^;

IntStream.concat(IntStream.range(1, 6 + 1), new Random().ints(200, 1, 6 + 1)).boxed().collect(Collectors.groupingBy(Function.identity(), Collectors.counting())).entrySet().stream().map(e -> e.getKey() + " : " + "*".repeat(e.getValue().intValue() - 1)).forEach(System.out::println);

[xebme]

200回試行の乱数の作り方ですがFibonacci Hashingを使うとよいかもしれません。
final int HASH_INCREMENT = 0x61c88647;
Stream.iterate(HASH_INCREMENT,i -> HASH_INCREMENT + i).limit(200).map(e->Math.abs(e)%6+1).collect(Collectors.groupingBy(Function.identity(),TreeMap::new,Collectors.counting())) .entrySet().forEach(System.out::println);
https://stackoverflow.com/questions/38994306/what-is-the-meaning-of-0x61c88647-constant-in-threadlocal-java/38994612

Answer 2

for (int i = 0; i <=200; i++)
これだと1回多くループしてしまいます。
また、わざわざDieAクラスを作る必要もないかと。

public class Test {
    public static void main(String[] args) {

        int[] counts = new int[7];

        for (int i = 1; i <= 200; i++) {            //i = 0をi = 1に変更
            int a = (int) (Math.random() * 6 + 1);　//DieAクラスを削除

            counts[a]++;
        }

        for (int i = 1; i < counts.length; i++) {
            System.out.printf("%d : ", i);
            for (int j = 0; j < counts[i]; j++) {
                System.out.print("*");
            }
            System.out.println();
        }
    }
}

Answer 3

さらにシンプルな書き方があれば教えてください！！！

public class Test {
	public static void main(String[] s) {
		int[] k = new int[6];
		for (int i = 0; i < 200; i++)
			k[(int)(Math.random() * 6)]++;
		for (int i = 0; i < 6; i++)
			System.out.println(i+1 + ": " + "*".repeat(k[i]));
	}
}

追記
あなたの Java のバージョンは何ですか？
Java 11 以降でないと Stringクラスの repeatメソッドが使えないので、
次のようにしてみました。

class Test {
	public static void main(String[] args) {
		String s = new String(new char[200]).replaceAll(".", "*");
		int[] k = new int[6];
		for (int i = 0; i < s.length(); i++)
			k[(int)(Math.random() * 6)]++;
		for (int i = 0; i < 6; i++)
			System.out.println(i+1 + ": " + s.substring(0, k[i]));
	}
}

追記2
substring の代わりに printf を使ってもできます。

-            System.out.println(i+1 + ": " + s.substring(0, k[i]));
+            System.out.printf("%d: %."+k[i]+"s\n", i+1, s);

Answer 4

なんかガチな書き方が多いので、学生さま向けのライトな書き方をしてみたくなりました。
とはいえ、Javaは真面目に書いたこと無いので、"ちょっと違う"という点があれば優しく指摘下さい。

まず、解説をコメントしてたら長くなったのでコメント抜きで・・・

public class Main {
    static final int MAX_EYES = 6;

    public static void main(String[] args) {
        var counts = new int[MAX_EYES];

        for (var i = 0; i < 200; i++) {
            var eyes =  rollTheDice();
            counts[eyes]++;
        }

        for (var i = 0; i < MAX_EYES; i++) {
            var count = counts[i];
            var bar = repeat('*', count);
            var str = String.format("%d: %s", i + 1, bar);
            System.out.println(str);
        }
    }
    
    public static int rollTheDice(){
        return (int) (Math.random() * MAX_EYES);
    }

    public static String repeat(char c, int count) {
        var sb = new StringBuilder();
        for(var i = 0; i < count; i++) {
            sb.append(c);
        }
        return sb.toString();
    }
}

コメント入りです。

public class Main {
    // サイコロの目の数は６面とは限らない
    // 定数として外だしする
    static final int MAX_EYES = 6;

    public static void main(String[] args) {
        // 自明な型はvarで問題ない
        // 添字が0から始まるのは違和感があるが余分に取るとメモリを
        // 消費するので慣れた人にはそちらのほうが気持ちが悪い
        var counts = new int[MAX_EYES];

        // 200回回す場合のループはこのように書く場合が多い
        // 0から始まった方が添え字に使いやすい
        for (var i = 0; i < 200; i++) {
            // サイコロを振るってことを関数名で説明する
            // 賽の目ってeyesって言うこと知らなかった
            var eyes =  rollTheDice();
            counts[eyes]++;
        }

        // ループの回数が配列から取れる場合はいわゆるforeachを使う
        // 記述がスッキリするだけでなく変数の長さの変更についていける
        // ループの書き方は、かなりのパターンがあるので使い分けること
        for (var i = 0; i < MAX_EYES; i++) {
            // 一般的に、1行で書けることも分けた方が読みやすくなる
            var count = counts[i];
            // 文字を繰り返すの部分は関数化するとスッキリする
            var bar = repeat('*', count);
            var str = String.format("%d: %s", i + 1, bar);
            System.out.println(str);
        }
    }
    
    // 1行でも関数化することによって読みやすくなるコードはある
    public static int rollTheDice(){
        // さっき行を分けたのと逆に１行にするパターン
        // 戻り値が変数のみになっていないのはなれると多い書き方
        // 良い書き方かどうかは判らないがなれると良い
        return (int) (Math.random() * MAX_EYES);
    }

    // 指定された文字を繰り返す関数　<-　コメントもソースのうち
    // 一文字ずつ出力するのも悪くないけど、まとめた形にしてくれる関数は便利
    public static String repeat(char c, int count) {
        // ループなどで不特定の回数の文字列の連結はStringBuilder推奨
        // メモリを余分に使用しない
        // あと、変数名は特にない場合は頭文字をとってsbとする
        var sb = new StringBuilder();
        for(var i = 0; i < count; i++) {
            sb.append(c);
        }
        return sb.toString();
    }
}

Comments

[momon-ga]

ロジックとは関係ないですが

> 実際の用例（特にイギリス英語）では「1個のさいころ」をa diceと呼ぶことは珍しくない。しかし正式には「diceはdieの複数形。単数形としてdiceを使うのは間違い」とされる。アメリカ英語ではdieとdiceを使い分ける傾向が強い。

元の質問者さんのこだわりを感じたので ＜ DieA.roll

あと、サイコロの目はeyeを使わないかなぁ。調べるとpipsやspotsというのが一般的のような気がします。

[iwamoto_takaaki]

英語は弱いのでコメントいただけるのはありがたい。
DieA.rollってそういう意味だったんですね。
有難うございます。

Answer 5

私もpaiza.ioでへろへろと。

import java.util.*;

public class Main {
    public static void main(String[] args) throws Exception {
        // Your code here!
        Random r = new Random();
        int res[] = new int[6];
        for (int i = 0; i < 200; ++i) {
            res[r.nextInt(6)]++;
        }
        
        for (int i = 0; i < 6; ++i) {
            System.out.print(String.valueOf(i+1) + ": ");
            for (int j = 0; j < res[i]; ++j) {
                System.out.print("*");
            }
            System.out.println();
        }
    }
}

Answer 6

※Java11で記述しています

import java.security.SecureRandom;
import java.util.List;
import java.util.stream.Collectors;
import java.util.stream.IntStream;

/**
 * @author a-pz
 *
 */
public class DiceSample {

	public static void main(String[] args) {
		DiceSample sample = new DiceSample();
		sample.start(10);

	}

	public void start(int sampleCount) {
		List<Integer> dices = dice(sampleCount);
		IntStream.rangeClosed(1, 6).boxed().forEach(target -> {
			int hit = (int)dices.stream().filter(val -> val == target).count();
			System.out.println(target + " " + "*".repeat(hit));
		});

	}

	List<Integer> dice(int sampleCount) {
		SecureRandom r = new SecureRandom();
		return r.ints(sampleCount, 1, 7).boxed().collect(Collectors.toList());
	}

}

Answer 7

十分シンプルだと思いますが。
既出ですが、repeatのところだけ変更すれば十分だと思います。
あとは余計な一時変数を削除するだけでスッキリします。
※200回のループの条件が違うので修正

public class Test {
	public static void main(String[] args) {

		int[] counts = new int[7];

		for (int i = 0; i < 200; i++) {	// 200回なら =< でなく <
			counts[DieA.roll()]++;
		}

		for (int i = 1; i < counts.length; i++) {
			System.out.printf("%d : %s%n", i, "*".repeat(counts[i]));
		}

	}
}

class DieA {
	public static int roll() {
		return (int)(Math.random() * 6) + 1;
	}
}

Answer 8

「さいころを振る」メソッドとしてDieA.rollメソッドがあるようですが、1箇所でしか使っておらず、それほど行数もないので、メソッド化する意味はあまりなさそうです。そしてこの程度の処理ならば1文で書けます。
また、さいころの目が16であるため、配列の16を使うために7要素で宣言しているようですが、「さいころの目-1」としてみなせば0~5の6要素で事足ります。
また、現在のコードでは200回ではなく、201回の繰り返しになります。
以上まででコードを書くと次のようになります。

import java.util.Scanner;
import java.util.Random;

public class Test {
  public static void main(String[] args) {

    int[] counts = new int[6];

    for (int i = 0; i < 200; i++) {
      int a = (int)(Math.random() * 6);
      counts[a]++;
    }

    for (int i = 0; i < counts.length; i++) {
      System.out.printf("%d : ", i + 1);
      for (int j = 0; j < counts[i]; j++) {
        System.out.print("*");
      }
      System.out.println();
    }
  }
}

シンプルの意味合いにもよりますが、これ以上シンプルに書くのは困難だと思われます。

Answer 9

Java 11 で書きました。
問題文を理解しておらず、的外れなコードを書いていたため修正しています。失礼しました。

import java.util.Map;
import java.util.Random;
import java.util.stream.Collectors;
import java.util.stream.Stream;

public class Test {
    public static void main(String[] args) {
        var rnd = new Random();
        Map<Integer, Long> counts = Stream.generate(() -> rnd.nextInt(6) + 1)
            .limit(200)
            .collect(Collectors.groupingBy(n -> n, Collectors.counting()));
        
        for (var entry : counts.entrySet()) {
            int n = entry.getKey();
            int count = entry.getValue().intValue();
            System.out.printf("%d : %s\n", n, "*".repeat(count));
        }
    }
}

一応ほとんどの場合は上記のコードで期待通りになると思います。
無視できるほどの低確率だとは思いますが（200回中一度も1が出ないなどの）偏りが出た場合、抜ける可能性がありますね…。

Comments

[退会済みユーザー]

元ソースは出た回数を記号の繰り返しで出力にみえるが

[htsign]

質問者様とは出力が異なりますが、お題には反していないつもりでした。
一応回数を出力するものも付記しました。

[htsign]

根本的に勘違いしていました。修正します。