標準入力から最小最大値を求める

Question

コンパイルがうまくいきません。
どこが間違っているか教えてください。
宜しくお願い致します。

【問題】
標準入力されている数字の最大と最小を求める

【標準入力されている数字】
9
12
3
6
10

import java.util.*;


class Main {
    public static void main(String[] args) {
        // 自分の得意な言語で
        // Let's チャレンジ！！

        Scanner sc1 = new Scanner(System.in);
        int a = sc1.nextInt();
        
         Scanner sc2 = new Scanner(System.in);
        int b = sc2.nextInt();
        
         Scanner sc3 = new Scanner(System.in);
        int c = sc3.nextInt();
        
         Scanner sc4 = new Scanner(System.in);
        int d = sc4.nextInt();
        
         Scanner sc5 = new Scanner(System.in);
        int e = sc5.nextInt();
        
       int array[] = {a, b, c, d, e };
        int min = 0;
        for(int i = 0; i < array.length; i++) {
            if(min > array[i]) {
                min = array[i];
            }
        }
        System.out.println("最小値 : " + min);
    }
}

Comments

[LouiS0616]

これPaizaの問題ですよね？Paizaの規約は確認されましたか。

[ganbaruyo]

arigatougozaimasu

[ganbaruyo]

私そちらのサイト使っていないので。私が選んだ問題ではないのでそのようなことを言われてもわかりません。

[LouiS0616]

それでは、問題の出典はどちらでしょうか。

Answer 1

入力値を保持する必要なし

Scanner sc = new Scanner(System.in);
int min = Integer.MAX_VALUE;
int max = Integer.MIN_VALUE;
while (sc.hasNext()) {
    int value = sc.nextInt();
    if (value < min) {
        min = value;
    }
    if (value > max) {
        max = value;
    }
}
System.out.println("min=" + min + ", max=" + max);
//min=3, max=12

Comments

[ganbaruyo]

簡単におさまるのですね。参考になりました。本当にありがとうございます

Answer 2

こんにちは、一番の原因はmin = 0と初期化しているところです。
入力した数値がすべて0より大きいのでmin = 0になっているのです。
最小値minを入力した値の最初の数値などにしておくと正常に動くと思います。

ほかに気になったこと

(i)Scannerを5回も生成する必要はありません。一つで十分です。
.Scanner sc = new Scanner(System.in);

(ii)変数をa,b,c,d,eにすると5回もsc.nextIntも書いていますが
最初から入力する変数を格納する配列を準備するとfor文で済みます。

import java.util.Scanner;

public class Main {
    public static void main(String[] args) throws Exception {
        
        Scanner sc = new Scanner(System.in);
        int []data = new int[5];
        for(int i = 0; i < data.length; i++){
            data[i] = sc.nextInt();
        }
        int min = data[0];
        
        for(int i = 0; i < data.length; i++){
            if(min > data[i]){
                min = data[i];
            }
        }
        System.out.println(min);
        
    }
}

別解を言うと

.配列やArrayListを使うと格納された数値をソートすることで最小値を得る方法もあります。

詳しくは配列やListをソートをご覧ください。

(i)配列を使う場合

import java.util.*;

public class Main {
    public static void main(String[] args) throws Exception {
        
        Scanner sc = new Scanner(System.in);
        ArrayList <Integer> array = new ArrayList<>();
        
        int []data = new int[5];
        
        for(int i = 0; i < 5; i++){
            data[i] = sc.nextInt();
        }
        Arrays.sort(data);

        int min = data[0];
        System.out.println(min);
        
    }
}

(ii)ArrayListを使う場合

import java.util.*;

public class Main {
    public static void main(String[] args) throws Exception {
        
        Scanner sc = new Scanner(System.in);
        ArrayList <Integer> array = new ArrayList<>();
        
        for(int i = 0; i < 5; i++){
            int n = sc.nextInt();
            array.add(n);
        }
        Collections.sort(array);

        int min = array.get(0);
        System.out.println(min);
        
    }
}

Comments

[ganbaruyo]

詳しくありがとうございます。独学で始めてくじけそうになっているところ、このように教えてくれる方がいると本当に助かります。
参考になります。

Answer 3

• そもそも本当に「コンパイルがうまくいかない」のか？実行時エラーではなくて？
• なんでScannerを5回も生成しているのか

せっかくなので、Java8からの新クラスを紹介
IntSummaryStatistics

public static void main(String[] args) {
    IntSummaryStatistics summary = new IntSummaryStatistics();
    try (Scanner sc = new Scanner(System.in)) {
        while (sc.hasNextInt()) {
            summary.accept(sc.nextInt());
        }
    }
    System.out.println("最大値: " + summary.getMax());
    System.out.println("最小値: " + summary.getMin());
}

Comments

[ganbaruyo]

ありがとうございます。質問に丁寧に答えていただいてうれしいです。参考にさせていただきました

Answer 4

if(min > array[i]) {

最初に、min = 0 としていますが、
入力値が 全て 0以上のため、 0より小さい値はありません。

Comments

[ganbaruyo]

きちんと考えながらプログラムを書かないとと思いました、ありがとうございました。

Answer 5

... どこが間違っているか ...

エラーメッセージが、教えてくれているはずです。

追記：

動作するようにしてみました。
MainA.java

import java.util.Scanner;

public class MainA {
    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);
        int a = sc.nextInt();
        int b = sc.nextInt();
        int c = sc.nextInt();
        int d = sc.nextInt();
        int e = sc.nextInt();

        int array[] = { a, b, c, d, e };
        int min = array[0];
        for (int i = 0; i < array.length; i++) {
            if (min > array[i]) {
                min = array[i];
            }
        }
        System.out.println("最小値 : " + min);
    }
}

実行例

コンパイルすると、warning が出ます。
ぜひ、解消させてみてください。

Comments

[ganbaruyo]

何度も訂正してもできないので質問していますありがとうございます

[katoy]

追記しました。

[ganbaruyo]

詳しくありがとうございます。コンパイルでエラーになった時にもきちんと対応できないとだめですね。学びました