変数が初期化されていないというエラーが出て困っています

解決済

回答 6

投稿

  • 評価
  • クリップ 0
  • VIEW 7,581

prog1717nm

score 52

整数データが1行に1つ計500個書かれているテキストファイルdata_5_1000_500.txtのデータの最大値、最小値、平均値を求めるプログラムCalcFile.javaを書きたいです。ちなみに500個の整数は乱数です。
以下はプログラムです.
import java.util.Scanner;

public class CalcFile{ 
public static void main(String[] args) {

Scanner stdin = new Scanner(System.in); 
int max =Integer.MIN_VALUE,x; 
int min =Integer.MAX_VALUE;
double ave;
int  numbers;
int sum;
int i;

while(stdin.hasNext()) { 
x =stdin.nextInt(); 
if(x>max) max = x; 
if(x<min) min = x;
}

for(i=0;i<args.length;i++){
sum=+i;
ave=sum/500;}

System.out.printf("max=%d\n", max); 
System.out.printf("min=%d\n", min);
System.out.printf("ave=%d\n", ave);


}

コンパイルすると、エラー: 変数aveは初期化されていない可能性があります
となりました。
どこをどう直したらいいか分からなくて困っています。
aveは求めたい結果なのになぜこのエラーが出るのか分かりません。
プログラムの誤りを直して欲しいです....!
よろしくお願いします。

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 6

+1

コードを整形した状態かつコンパイルエラーの発生しない状態にしたものが以下になります。

public class CalcFile {

    /**
     * @param args
     */
    public static void main(String[] args) {
        Scanner stdin = new Scanner(System.in);
        int max = Integer.MIN_VALUE, x;
        int min = Integer.MAX_VALUE;
        double ave = 0.0d;
        int numbers = 0;
        int sum = 0;
        int i = 0;

        while (stdin.hasNext()) {
            x = stdin.nextInt();
            if (x > max)
                max = x;
            if (x < min)
                min = x;
        }

        for (i = 0; i < args.length; i++) {
            sum = +i;
            ave = sum / 500;
        }

        System.out.printf("max=%d\n", max);
        System.out.printf("min=%d\n", min);
        System.out.printf("ave=%d\n", ave);
    }

}

いくつか気になる点を。

  • Scanner stdinは処理が終了したら閉じなければなりません。
  • intやdoubleを宣言したあと、何らかの初期を入れておかずに分岐処理へ渡す場合は、コンパイルエラーになります。
  • 変数 numbers はどこにも使われていません。
  • for (i = 0; i < args.length; i++) { の部分で、argsを指定していますが、これはこのクラスのmainメソッド起動時に渡される引数です。
  • ave = sum / 500;と書かれていますが、現在のコードでは500個入力されているかをチェックするロジックはないため、すべて500で割り算してしまいます。入力のあった個数で割るのが正しいでしょう。
  •   System.out.printf("ave=%d\n", ave);の行は、aveの値がdouble値のため、%dではフォーマットエラーになります。

なお、while (stdin.hasNext()) {で数値入力の終わらせ方は、CTRL+CないしはCTRL+Zです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

checkベストアンサー

0

コードが間違っているように見えます。

iは読み込んだデータ数、sumは入力値の合計とします。

while(stdin.hasNext()) {  
  x =stdin.nextInt();  
  if(x>max) max = x;  
  if(x<min) min = x;
  sum += x;
  i++;
}

ave=sum/i;

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

初期化していない変数をプログラム内で使用することは危険なためコンパイルエラーになります。
これから求めるとかは関係ありません。0を代入するといいでしょう。

また、コードはコード専用の記述方法があるのでそちらを使用してください。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/06/18 10:38

    0を代入したらエラーは消えました。ありがとうございます。
    でも平均値だけは出力されません。多分for文のところが間違っているんだと思います。
    どこが間違っているか教えていただけないでしょうか?

    キャンセル

  • 2016/06/18 10:41

    argsの長さが0なんだと思います。

    一つの投稿に複数の質問を乗せるのはよくないですよ。

    キャンセル

  • 2016/06/18 12:45

    すみません。今度から気をつけます。

    キャンセル

0

値が入らない可能性のある変数aveを出力しようとしているからです。

for文の中でaveに値を入れているようですが、もしfor文の中に入らなかったらaveは空っぽですよね。

ということで単純に解決するのであればaveを初期化しましょう。
宣言の時に double ave = 0.0とでもしておけば、コンパイルエラーは消えるはずです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/06/18 10:39

    0を代入したらエラーは消えました。ありがとうございます。
    でも平均値だけは出力されません。多分for文のところが間違っているんだと思います。
    どこが間違っているか教えていただけないでしょうか?

    キャンセル

  • 2016/06/18 10:49

    少なくとも出力されないのは、printfの中身がおかしいからだと思います。
    %dじゃなくて、%fでは? 

    キャンセル

  • 2016/06/18 12:46

    とりあえず出力はされました。
    大変助かりました。ありがとうございました。

    キャンセル

0

args.lengthが0の場合は、ave が初期化されません。
正攻法としてはargs.lengthが0の場合を分けるのがいいと思います。

例えば、
if (args.length == 0) {
// args.lengthが0である旨のメッセージを出力して、終了
}
else {
// 今と同じ処理
}
です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

ave への代入は for ループ中にあります。
for ループの中味は実行されない可能性があるので、このエラーが発生します。

そもそも、ave の計算が誤っています。
sum が決定してから、 sum を データ数で割ったものを ave にするのが正しいです。
プログラムを書きなおしてみました。

import java.util.Scanner;

public class CalcFileX {
    public static void main(String[] args) {

        Scanner stdin = new Scanner(System.in);
        int max = Integer.MIN_VALUE;
        int min = Integer.MAX_VALUE;
        int sum = 0;
        int count = 0;
        double ave = 0.0;

        while (stdin.hasNext()) {
            int x = stdin.nextInt();
            min = Math.min(x, min);
            max = Math.max(x, max);
            sum += x;
            count += 1;
        };
        if (count ==  0) {
            System.out.printf("入力データがありません。\n");
        } else {
            ave = 1.0 * sum / count;
            System.out.printf("max=%d\n", max);
            System.out.printf("min=%d\n", min);
            System.out.printf("ave=%f\n", ave);
        }
    }
}

最大、最小は Math.max, Math.min を使ってみました。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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