ブロック内の変数を外で使用したい

Question

2桁の整数同士の足し算問題を５回繰り返し、最後にまとめて答えと正誤評価を表示させたいです。
for文内の変数ansをブロック外で使用できれば解決すると思うのですが、その方法がわかりません。
開発環境
macOS, Sublime Text

import java.io.*;

class Practice3
{
	public static void main(String[] args) throws IOException
	{
		int[][] kazu = new int[2][5];

		kazu[0][0] = (int)(Math.random() * 99);
		kazu[0][1] = (int)(Math.random() * 99);
		kazu[0][2] = (int)(Math.random() * 99);
		kazu[0][3] = (int)(Math.random() * 99);
		kazu[0][4] = (int)(Math.random() * 99);
		kazu[1][0] = (int)(Math.random() * 99);
		kazu[1][1] = (int)(Math.random() * 99);
		kazu[1][2] = (int)(Math.random() * 99);
		kazu[1][3] = (int)(Math.random() * 99);
		kazu[1][4] = (int)(Math.random() * 99);

		for(int i=0; i<5; i++){

		System.out.print(kazu[0][i] + "+" + kazu[1][i] + "=");

		BufferedReader br =
		 new BufferedReader(new InputStreamReader(System.in));

		String num = br.readLine();
		int ans = Integer.parseInt(num);
	    }

		System.out.println("答え");

	    for(int j=0; j<5; j++){

        	if(ans == kazu[0][j] + kazu[1][j]){
        		System.out.println(kazu[0][j] + "+" + kazu[1][j] + "=" + (kazu[0][j] + kazu[1][j]) + " 正解");
        	}
        	else{
        		System.out.println(kazu[0][j] + "+" + kazu[1][j] + "=" + (kazu[0][j] + kazu[1][j]) + " 不正解");
        	}
        }
    }
}

Answer 1

• ansは配列にして、for文の外側で宣言する
• kazuの値は出題するfor文の中で生成する
• 入力した答えは配列の要素ansiに格納する
• 答え合わせのfor文の問題文は共通なのでif文の前で表示する

import java.io.*;

class randomcheck {
    public static void main(String[] args) throws IOException {
        int[][] kazu = new int[2][5];
        int[] ans = new int[5];

        for (int i = 0; i < 5; i++) {
            kazu[0][i] = (int) (Math.random() * 100);
            kazu[1][i] = (int) (Math.random() * 100);
            System.out.print(kazu[0][i] + "+" + kazu[1][i] + "=");

            BufferedReader br = new BufferedReader(new InputStreamReader(System.in));

            String num = br.readLine();
            ans[i] = Integer.parseInt(num);
        }

        System.out.println("答え");

        for (int i = 0; i < 5; i++) {
            System.out.print(kazu[0][i] + "+" + kazu[1][i] + "=" + (kazu[0][i] + kazu[1][i]));
            if (ans[i] == kazu[0][i] + kazu[1][i]) {
                System.out.println(" 正解");
            } else {
                System.out.println(" 不正解");
            }
        }
    }
}

Comments

[foifoi]

とてもわかりやすい解答ありがとうございます。
ベストアンサーにしたいです。

Answer 2

こんな方法もありかな、と。

import java.io.*;
import java.util.Random;

class Main {
    public static void main(String[] args) throws IOException {
        var rand = new Random();

        final var numOfQuestions = 5;
        var messages = new String[numOfQuestions];

        try(
            var inputStreamReader = new InputStreamReader(System.in);
            var bufferedReader = new BufferedReader(inputStreamReader);
        ) {
            for(int i = 0; i < numOfQuestions; ++i) {
                int x = rand.nextInt(100);
                int y = rand.nextInt(100);
                messages[i] = String.format("%d+%d=%d", x, y, x+y);
                
                System.out.printf("%d+%d=?", x, y);
                int ans = Integer.parseInt(bufferedReader.readLine());

                if(ans == x + y) {
                    messages[i] += " 正解";
                }
                else {
                    messages[i] += " 不正解";
                }
            }
        }

        System.out.println("答え");
        for(var message: messages) {
            System.out.println(message);
        }
    }
}

乱数の生成や正誤の判定、果てはメッセージまでもループの中で作ってしまう方法です。
それ以外にもさまざま改造しています。参考までに。

Comments

[foifoi]

初心者の自分には全くわかりません・・笑
これから勉強します！

[LouiS0616]

正誤判定を先に行っちゃうと楽だよというのが回答の主眼なので、それ以外の改良は全部おまけです。
Javaに慣れてきたころにもう一度見てみると得られるものもあるかと思います。

Answer 3

ブロックの内部で宣言した変数は、ブロックの中でしか使えません。
ブロックの外で使う変数は、ブロックの外で宣言してください。
もしくは、ブロックの外で宣言した変数へと値を移してください。

なお、現在の状態だと、ansは一つのデータしか入りませんので、記録されているのは、
最後に入力した値だけになります。
5つのデータを入力したい場合には、ansも配列にして複数の値を扱えるようにする必要があります。

ansも配列にする必要があること、理解できました。

for文を使わずに地道に5つの計算をさせてみます。

forを使うことは問題ありません。むしろ配列とforは相性が良いですし。
(とりあえず、forを作る前の下準備の練習として一度展開してみるってのはありですが)

forを使う場合、ansはどのように扱えばよいのでしょうか。

forを使うかどうかにはあまり関係がありません。

変数にはスコープという寿命があるため、ブロックの中で宣言した変数は、ブロックの中でしか使えません。
これは、使った変数が想定外の場所に悪影響を与えるのを防止するために、変数の影響範囲を抑えるための機能です。

逆に言えば、変数は使いたい位置で使えるスコープが確保できるように宣言を行います。

たとえば、以下のように書けば、変数xのスコープはブロックの中だけですので、その外で使おうとすればエラーになります。

{
  int x = 3;
}
print(x); //エラー

なので、以下のように、ブロックの外で変数xを宣言することで、変数xのスコープはブロックの外になるので、
ブロックの外でも使うことができるようになります。

int x;
{
  x = 3;
}
print(x); 

---

もう一つ余談ですが、
Math.random()は、0以上1未満(0は含む、1は含まない)の乱数を返すメソッドですので、
(int)(Math.random() * 99)とした場合、0以上99未満(99は含まない)の整数になりますので、
結果は、0～98になります。

Comments

[foifoi]

ansも配列にする必要があること、理解できました。
for文を使わずに地道に5つの計算をさせてみます。

[foifoi]

forを使う場合、ansはどのように扱えばよいのでしょうか。
いろいろと質問してしまい、申し訳ございません。

[foifoi]

forを使わずに書いたら成功しました。
ありがとうございます。