電子書籍の厳選無料作品が豊富!

int型の配列の各要素に1~10の乱数を代入し、各要素の値を縦向きの*のグラフで表示するプログラムを作っているのですが、結果がランダムででるので、自分の書いたプログラムが正しいのかわかりません。ソースを載せますので合っているのか間違っているか教えて下さい。もし間違っているならどこが間違いなのか教えていただけると嬉しいです。よろしくお願いします。

●ソース
import java.util.Random;
import java.util.Scanner;

class Graph {

public static void main(String[] args){
Random rand = new Random();
Scanner stdIn = new Scanner(System.in);

System.out.print("要素数:");
int n = stdIn.nextInt();
int a[] = new int[n];

for (int i = 0; i < n; i++)
a[i] = 1 + rand.nextInt(10);

for (int i = 1; i <= 10; i++){
for (int j = 0; j < n; j++)
if (a[j] <= i)
System.out.print("* ");
else
System.out.print(" ");
System.out.println();
}
}
}

●実行例

要素数:12
*
*
* *
* * * * *
* * * * * * *
* * * * * * * *
* * * * * * * *
* * * * * * * *
* * * * * * * * *
* * * * * * * * * * * *

A 回答 (2件)

こういう場合はプログラムの一部分を関数として切りだしてみれば良いんです。


今回の例なら、int配列とその長さを受け取ってグラフを表示する関数を作りましょう。
そうすればその関数の動作をチェックすることで、プログラムが正しいかどうかがわかります。
    • good
    • 1

単純に a[] の配列の中身を数値で出力すれば自分で確認できるでしょ?

    • good
    • 0

お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!