質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.61%

乱数から上限と下限を設定して間の値を表示したい

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 656

score 9

前提・実現したいこと

1~99の乱数100個から上限と下限を入力して間の値を昇順で表示したいです。

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

Arrays.sortで昇順からi番目のを表示してしまうと重複した値が原因でずれてしまい、うまく上限と下限が反映しなくなる。

実行例
乱数一覧
34 33 46 17 56 73 71 20 17 81 
28 54 75 36 76 4 7 78 81 58 
80 5 82 3 39 8 48 42 81 81 
45 51 72 31 94 20 59 37 86 77 
74 30 59 78 16 97 61 94 7 22 
85 75 7 27 93 4 38 72 92 70 
99 5 63 31 52 96 45 93 41 63 
9 79 2 48 21 58 31 83 32 69 
66 6 6 21 28 33 47 6 76 45 
52 83 15 43 49 85 41 5 5 67 

抽出する下限と上限を入力してください
下限
16
上限
21

16と21の間のデータは昇順で
15 16 17 17 20 20 
の6件です

このように15が表示されてしまったり、21が表示されなくなります。

該当のソースコード

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

public class RandomNumber {
    public static void main(String[] args) {
        Random ran = new Random();
        // 要素数100の配列を宣言
        int[] lists = new int[100];
        System.out.print("乱数一覧");
        for (int i = 0; i < lists.length; i++) {
            int Number = ran.nextInt(99) + 1;
            lists[i] = Number;
        }
        // 改行と空白を付与し、一覧を表示
        for (int i = 0; i < lists.length; i++) {
            if (i % 10 == 0) {
                System.out.println("");
            }
            System.out.print(lists[i] + " ");
        }
        // 上限と下限を入力する
        Scanner sc = new Scanner(System.in);

        System.out.println("\n");
        System.out.println("抽出する下限と上限を入力してください");
        System.out.println("下限");
        int min = sc.nextInt();
        System.out.println("上限");
        int max = sc.nextInt();
        // 昇順にソートする
        Arrays.sort(lists);

        System.out.printf("%n%dと%dの間のデータは昇順で%n", min, max);

        // 昇順の件数をカウントする変数を宣言
        int count = 0;
        for (int i = min; i <= max; i++) {
            System.out.print(lists[i] + " ");
            count++;
        }

        System.out.printf("%nの%d件です%n%n", count);
        sc.close();
    }
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • dodox86

    2021/02/02 18:21

    > Arrays.sortで昇順からi番目のを表示してしまうと重複した値が原因でずれてしまい、うまく上限と下限が反映しなくなる。

    それは当然そうやっているのでそうなっているとしか。最初に自力でminとmax の位置を探しに行く必要があります。

    キャンセル

回答 2

+1

ソートした後で、min 以上で max 以下のものだけを表示すればよいのでは?

        for (int i = 0; i < lists.length; i++) {
            if (lists[i] >= min) {
                if (lists[i] > max) break;
                System.out.print(lists[i] + " ");
                count++;
            }
        }

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

checkベストアンサー

0

最初に100個乱数の入った配列を作っていますが、これに重複した値が入った場合が考慮されてないことが原因かと思います。
上限と下限ではなく、配列の要素「下限の値」~配列の要素「上限の値」までが出力されてます。
昇順に並べ替えても、重複した値が一つでもあれば順番は狂います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2021/02/02 19:46

    回答ありがとうございます。
    ご指摘を元に書き直してみたのですが、何故かまだあらぬ値が出てしまいます。
    ----------------------------------------------------------------------------------
    ソースコード:
    // 昇順の件数をカウントする変数を宣言
    int count = 0;
    for(int i = 0; i < lists.length; i++) {
    if(min <= lists[i] && lists[i] <= max) {
    Arrays.sort(lists);
    System.out.print(lists[i] + " ");
    count++;
    }
    }
    System.out.printf("%nの%d件です%n%n", count);
    ------------------------------------------------------------------------
    実行例:
    乱数一覧
    42 67 68 99 87 9 16 75 42 31
    72 11 12 69 38 75 84 9 63 89
    67 80 14 86 58 20 70 12 6 74
    76 78 75 12 81 43 39 26 78 20
    63 79 8 55 82 46 3 11 65 23
    60 67 9 90 38 1 69 62 77 21
    13 67 78 71 29 2 34 71 97 63
    15 21 48 98 81 44 26 74 18 16
    16 29 71 36 40 33 87 2 28 7
    63 48 74 19 33 58 61 91 40 5

    抽出する下限と上限を入力してください
    下限
    90
    上限
    99

    90と99の間のデータは昇順で
    3 90 91 97 98 99
    の6件です

    キャンセル

  • 2021/02/02 20:17

    配列全てに対して、ifの条件を満たしているかどうか、というのは合ってます。
    ただ、先に配列をソートしておく必要があることと、条件を満たしたときにソートするのは間違ってます。
    あと、この方法だとやはり配列に重複した値が入った場合の考慮がされないかと思います・・・ので、重複した値が入らないように工夫する必要があるかと思います。

    キャンセル

  • 2021/02/02 23:16

    Set<Integer> linkedHash = new LinkedHashSet<Integer>();を利用して解決いたしました。
    ありがとうございました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る