２つの変数に数を代入することを繰り返す

Question

条件
①一番初めにコマンドラインに数字を入力
②コマンドラインに入力された数の回数だけ数を足したり引いたりする
(1)SETと入力された場合
入力フォーマット＝SET 1 50
変数aに５０を加える
(2)ADDと入力された場合
入力フォーマット＝ADD 50
変数bにaと引数の和を代入
(3)SUbと入力された場合
入力フォーマット＝SUB 90
変数bにaから引数の数を引いた値を代入
③最終的な変数a,bの値を代入

というわけなのですが
疑問点は
①変数a,bを
static int a=0;
static int b=0;
とした時にエラーが出る理由と
②このコードでyを条件付きで宣言したにもかかわらず、シンボルが見つからないというエラーが出る
理由がわかりません
僕としては配列の長さが３になるのはSETと最初に入力された時のみでその時だけyを作ればreadLineArray[2]もしっかりあるし、整数型に変換しているしおかしいはずがないのにと考えたのですが....

import java.io.BufferedReader;
import java.io.InputStreamReader;

public class Main {
    public static void main(String[] args) throws Exception {
        BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
        int a=0;
        int b=0;
        String line = br.readLine();
        int n = Integer.parseInt(line);
        for(int i=0;i<n;i++){
            String change = br.readLine();
            change = change.trim();
            String [] readLineArray = change.split(" ",3);
            int x = Integer.parseInt(readLineArray[1]);
            if(readLineArray.length >2){
            int y = Integer.parseInt(readLineArray[2]);
            }
            if(readLineArray[0].equals("SET")){
                if(x == 1){
                    a += y;
                }else{
                    b += y;
                }
            }
            if(readLineArray[0].equals("ADD")){
                b = a+x;
            }
            if(readLineArray[0].equals("SUB")){
                b = a-x;
                }
            }
        System.out.println(a+" "+b);;
    }
}

もしよろしかったらご意見お願いします

Answer 1

(1)変数a,bを
static int a=0;
static int b=0;
とした時にエラーが出る理由

メソッド内でstatic修飾した変数はローカルstatic変数となり、
Javaの言語仕様として許可されていません。
(C#等は許可されている様ですが…)

static変数は以下の様に宣言します。
これで他のクラスからもアクセスできる様になります。

public class Main {

	static int a;
	static int b;

	public static void main(String[] args) throws Exception {...}

} 

(2)このコードでyを条件付きで宣言したにもかかわらず、シンボルが見つからないというエラーが出る
理由がわかりません

エラーの原因箇所は下記コード部です。

            if(readLineArray.length >2){
            int y = Integer.parseInt(readLineArray[2]);
            }

「変数が有効かどうか」というのを変数の「スコープ」と表現します。
基本的なスコープの考えとして
「中括弧{}の中で宣言された変数はその中括弧の中だけで有効」となります。
これは
「メソッドの中括弧の中で宣言された変数は、
そのメソッドの中だけで有効。」であり、
if文の中括弧の中で宣言してしまうと、if文の中括弧の中だけで有効とされます。
else パートで宣言しても同様です。

なので修正方法としては以下の様になると思います。

            int y;
            /* 初期化が必要な場合は int y = 0; など */
            if(readLineArray.length >2){
                y = Integer.parseInt(readLineArray[2]);
            }

Answer 2

1. a,bを宣言しているのがメソッドの中だからです。

staticはクラス変数でしか宣言できません。
2. x,yはfor文の中で宣言しており、forを抜けると破棄されるからです。

Comments

[modonokouki]

1,解決できましたありがとうございます
2,解決できたのですが質問が一つあります
for文の外でx,yを初期化して宣言したことで成功したのは良いのですがなぜfor文の中で初期化して宣言するといけないのかという理由がわからないです
すごい細かい質問ですが、これから間違えたくないのでお願いなのですが教えていただけると幸いです

[swordone]

失礼、yはif文の中だったのですね。
これはスコープの問題です。{}の中で宣言した変数は、その{}の外では使えません。
今回のif文の場合で説明するなら、このif文は必ずしも通るとは限りません。通らなかった場合、その後出てくるyって何？となるわけです。