-3

Java初心者です。
値を繰り返し入力し合計値が100を超える、または、10回入力が行われた場合に入力を止めそれまでの値を出力するプログラムを作成しています。

Eclipseでループ処理と配列を使って書いているんですが、whileで条件を指定したところ、
以下の警告が出てしまいます。
・演算子 < は引数の型 Scanner, int で未定義です
スコープの外で定義していると思うのですが、
エラーが出てしまう理由がよくわからないので、
解決策を教えてください。
お願いします。

    int s[] = new int[100];
    int input = 0;
    Scanner scan;
    // 値をキーボードから入力
    for (int i = 0; i < 10; i++) {
        System.out.println("整数を入力してください");
        scan = new Scanner(System.in);
        s[i] = scan.nextInt();
    }

    // 入力値が100を超える、または入力が10回行われたら入力ストップ
    while (scan < 10 && s < 100) {

    }

    // 入力された数値を全て出力
}
1
  • 2
    「学校の課題」を全部こちらのQAとして終わらせるつもりでしょうか…。規則としてどうこうではありませんが、好まれる行動ではありませんし、もう少しご自分で考える時間を多く持たれないと全く実力に結びつきませんが…。とりあえず前質問の回答のどれかがその質問を解決したのであれば、一番役だった回答に「承認」の操作をお願いします。解決できないのであれば、コメント等で回答者に知らせるようにしてください。
    – OOPer
    Commented 2017年8月15日 8:32

2 件の回答 2

2

あなたのコードには以下のようなまずい点があります。

  • 指定条件で「入力を止め」ないといけないのに、あなたのコードはfor (int i = 0; i < 10; i++) {...}で、必ずループを10回行なってしまった後で、while(...) {...}の中に「入力を止め」るための条件を書こうとしています。

  • あなたのコードの中でscanScanner型のインスタンスを表しています。これがあなたの意向を忖度して、時に「入力回数」を表すなんてことはありません。従ってscanint型の値である10と比較することもできません。

    「演算子 < は引数の型 Scanner, int で未定義です」と言うエラーは、「Scanner型の値とint型の値を比較できるような比較演算子<は存在しない」ことを表しているのであって、Scanner型の何かが未定義だといっているのではありません。

  • 別エラーで隠されているようですが、s < 100も正しい式ではありません。あなたのコードではsは「int型の配列」を表します。Javaの配列では配列名sとだけ書いてその合計を表すような機能はありません。合計値が必要であれば、別途計算してやる必要があります。


そこら辺を修正した私なりのコードをお示ししておきます。

public static void main(String[] args) {
    // 配列sの初期化
    int[] s = new int[10];
    //### 合計値を表す変数を持つ
    int total = 0;
    //### どこまで入力されたかを示す変数を持つ
    int lastIndex = -1;
    System.out.println("整数を入力してください"); //### プロプトの表示は最初の1回で良いはず
    Scanner scan = new Scanner(System.in); //### Scannerのインスタンスもループごとに作る必要はない
    // 入力を10(=配列のサイズ)回受け付ける
    for( int i = 0; i < s.length; ++i ) { //### 配列全体を回すときは i < s.length が常套句
        try {
            s[i] = scan.nextInt();
            total += s[i]; //### 合計値を更新
            //### どこまでを合計に加えたかを覚えておく
            lastIndex = i;
            //### 合計値が100を超えると入力ストップ
            //### (「入力が10回行われたら」は、for()で実現できている
            if( total > 100 ) {
                break;
            }
        } catch( Exception e ) {
            //### 問題の条件にはないが、整数値が入力できない場合も入力終了
            break;
        }
    }
    scan.close();
    // 入力された数値を全て出力
    for( int i = 0; i <= lastIndex; ++i ) { //### `lastIndex`番目の値も有効なので`<=`になっている
         System.out.println(s[i]);
    }
}

操作・出力例(コード整形してあるのが出力部分)

整数を入力してください
10
20
30
40
50

10
20
30
40
50

※「100を超えたら」と言う表現ではぴったり100は「超え」てはいない、との判定が普通です。「それまでの値」に入力を超えることになった最後の値を含めるのかどうかは微妙ですが、上のコードでは「含める」動作になります。(私がこの課題を出して丸がもらえなかったら、出題が曖昧なのが悪いとごねるでしょう。)

操作・出力例2(出題指定外の条件による停止)

整数を入力してください
10
20
30
end

10
20
30

最新のJavaコードとするのであれば、前質問でKohei TAMURAさんが示されたように、try-with-resourcesを使う方がよりいい のだろうとは思うのですが、使用されるJavaのバージョンが不明ですので、少し古いJavaでも動くように書いてあります。(老Javaプログラマーが慣れている書き方をした、と言うのもありますが。)

少し細かめにあれこれコメントをつけてあります。十分理解されないまま課題レポートにコピペして終了として、次の課題に移るのではなく、しっかり理解していただいて、次の課題ではご自身で解ける部分を少しでも増やすようにしていただけるよう願っております。

0
int s[] = new int[10];
int input = 0;

// 配列sをクリア
for(int i = 0; i < 10; i++) {
  s[i] = 0;
}

Scanner scan;
// 値をキーボードから入力
for (int i = 0; i < 10; i++) {
    System.out.println("整数を入力してください");
    scan = new Scanner(System.in);
    s[i] = scan.nextInt();

    int sum = 0;
    for(int j = 0; j < 10; j++) {
      sum += s[i];
    }
    if (sum > 100) break;
}

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。