javaでヒットアンドブロー

Question

javaでヒットアンドブローのプログラム描いてます。

大学の単位に関わるので教えて頂きたいです！

課題の条件として、以下の方法で重複しない４桁の数字を生成することが必要です。

1. 10個の配列に0~9までをいれる
2. 0~9までの乱数で1の位の数字を決める
3. 決めた数字を配列の初めの数字と入れ替える
4. 1~9までの乱数で10の位の数字にする
5. 決めた数字を配列の2番目の数字と入れ替える
6. 2~9までの乱数で100の位の数字にする
7. 決めた数字を配列の3番目の数字と入れ替える
8. 3~9までの乱数で1000の位の数字にする

いろいろ調べてはみたものの、どのサイトを見ても、同じ方法でやっている物はないように見えたので、質問させて頂きました。

こんな感じで作ったのですが、重複してしまいます。

	//10個の配列を作る
	int [] ans = {0,1,2,3,4,5,6,7,8,9};
	String [] name = {"0","1","2","3","4","5","6","7","8","9"}; 

	
	//千の位を持ってくる。
	Random rand1 = new Random ();
	int sen = rand1.nextInt(9);
	int a = ans[sen];
	//交換
	ans[sen]= ans[0];
	ans[0] = a;
	
	//１から9のansを取り出す
	int [] ans2 = new int [9];
	System.arraycopy(ans,1, ans2,0,9);
	//百の位を持ってくる。
	Random rand2 = new Random ();
	int hyaku = rand2.nextInt(8);
	int b = ans2[hyaku];
	//交換
	ans2[hyaku]= ans2[0];
	ans2[0] = b;
	
	
	//2から9のansを取り出す
	int [] ans3 = new int [8];
	System.arraycopy(ans2,1, ans3,0,8);
	//十の位を持ってくる。
	Random rand3 = new Random ();
	int zyuu = rand3.nextInt(7);
	int c = ans3[zyuu];
	//交換
	ans3[zyuu]= ans3[0];
	ans3[0] = c;
	
	
	//3から9のansを取り出す
	int [] ans4 = new int [7];
	System.arraycopy(ans3,1, ans4,0,7);
	//一の位を持ってくる。
	Random rand4 = new Random ();
	int iti = rand4.nextInt(6);
	int d = ans4[iti];
	//交換
	ans[iti]= ans4[0];
	ans4[0] = d;
	//4桁持ってくる
	System.out.println(ans[sen]+""+ans2[hyaku]+""+ans3[zyuu]+""+ans4[iti]);

よろしくお願い申し上げます。。

Comments

[jimbe]

コードは, 入力枠の上辺にある <code> を押すと出てくる ``` の間の "コード" という文字列を置き換える形で記入(コピペ)してください.
その際, "ここに言語を入力"という文字列を "java" に置き換えますと, java 言語のキーワード等のフォントが変わる等して見易くなります.
また, コードは実行できる状態で記述してください. 省略されますと, 回答側で実行するために追加したコードによっては, 再現しない場合があります.

なお, 「単位に関わる」からと急かされても回答側には関係ないことはご留意頂いたほうが宜しいかと思います.

> どのサイトを見ても、同じ方法でやっている物はないように見えた
コードの書き方は人それぞれです. 同じ授業を受けた同じレベルの方が同じ課題をやっても同じ方法となるとは限りません.
だからこそ, ただコードを書くだけでなく, 人のコードを読んで理解することもまたプログラミングには必要と思います.

[-Akane-]

ご指摘頂き有り難うございます。
teratailの使い方等未熟な部分が多くあり、ご迷惑をお掛けして申し訳ありません。
また、私の事情や浅学さについての客観的なご意見も頂き有り難うございます。
今後、質問等させて頂く際に参考にさせて頂きます。
この度は誠にありがとうございました。

[-Akane-]

ご回答頂いた皆様、本当に有り難うございます。
皆様のご意見を比較しながら、頑張ってみようと思います。
この度は誠にありがとうございました。

Answer 1

以下の方法で重複しない４桁の数字を生成すること

には配列のコピーは入っていません.
そもそもコピーしなくて済む方法が書かれています.
方法通りなら以下のようになるのではないでしょうか.

import java.util.*;

public class Main {
  public static void main(String[] args) throws Exception {
    Random rand = new Random();

    //1. 10個の配列に0~9までをいれる
    int [] ans = {0,1,2,3,4,5,6,7,8,9};

    //2. 0~9までの乱数で1の位の数字を決める
    int i = rand.nextInt(10); //0～9
    int iti = ans[i];
    //3. 決めた数字を配列の初めの数字と入れ替える
    ans[i] = ans[0];
    ans[0] = iti;

    //以下略
  }
}

Comments

[-Akane-]

ご意見を頂き有り難うございます。
参考にさせて頂きます。

[momon-ga]

jimbeさん以外のかたのコードは前提条件の
> 課題の条件として、以下の方法で重複しない４桁の数字を生成することが必要です。
を無視してるんですよね・・・
質問者は、わかってるのかしら？

[jimbe]

課題ということですから「どこまで回答するか」ですね.
質問に提示されたコードも条件を満たしているとは言い難いですので, まぁ「参考に」して頂けるようですし, 多少間違ったコードを提出されてもそれはそれで経験になるのではないでしょうか. (と前向きに^^;)

Answer 2

		Random r = new Random();
		int[] d = new int[4];
		for (int i = 0; i <d.length; i++) {
			d[3-i] = r.nextInt(10-i)+i;
			for (int j = 0; j < i; j++) {
				if (d[3-i] == d[3-j]) {
					i--;
					break;
				}
			}
		}
		System.out.println(Arrays.toString(d));

こんなかんじやろな、
1つ目のループ：桁数
2つ目のループ：重複チェック
あえて上記以外はコメントを入れないが読んで詰まったらコメントする（正し課題のため2月以降かな、理解できないのに単位とれるのはおかしいから）

Comments

[-Akane-]

ご意見を頂き有り難うございます。
参考にさせて頂きます。

Answer 3

課題ということなのでヒントだけ

ちょっと勘違いされているようですが、配列をコピーする必要はありません。
同じ配列ans[]に対して１～８の操作をする想定だと思います。
結果の表示は
System.out.println(ans[3] + "" + ans[2] + "" + ans[1] + "" + ans[0]);
このようになるはずです（私のほうが勘違いしていなければ^^;

あとint sen = rand1.nextInt(9);は、9にはなりません（0～8）
Randomのインスタンスは使いまわせますので、何度もnewする必要はありません。

Comments

[-Akane-]

ご意見を頂き有り難うございます。
参考にさせて頂きます。

Answer 4

もっと処理を単純化してはいかが？
Random() を毎回作る必要はないですよ。

import java.util.Random;

public class Main {
    public static void main(String[] args) {
        //10個の数列から4個の数をランダムに選ぶ
        int[] digits = {0,1,2,3,4,5,6,7,8,9};
        int[] ans = new int[4];
        Random rand = new Random();

        //千の位を選ぶ
        int sen = rand.nextInt(10);
        ans[0] = digits[sen];
        digits[sen] = digits[9];

        //百の位を選ぶ
        int hyaku = rand.nextInt(9);
        ans[1] = digits[hyaku];
        digits[hyaku] = digits[8];

        //十の位を選ぶ
        int zyuu = rand.nextInt(8);
        ans[2] = digits[zyuu];
        digits[zyuu] = digits[7];

        //一の位を選ぶ
        int iti = rand.nextInt(7);
        ans[3] = digits[iti];

        //選んだ4個の数を表示
        System.out.println("" + ans[0] + ans[1] + ans[2] + ans[3]);
    }
}

Comments

[退会済みユーザー]

1000の位は3-9で0ははいらないがこのコードはいるよな

[shiracamus]

質問のコードが int sen = rand1.nextInt(9); になってます。

[-Akane-]

ご意見を頂き有り難うございます。
参考にさせて頂きます。

[jimbe]

3～9 なのは配列のインデックスですので, 交換によって千の位に "0" が入ることはありえますし, そうなってはいけないという条件は無いように思います.

Answer 5

「フィッシャー-イェーツのシャッフル」を調べてみてください。