Java:try文でオブジェクトを作ろうとしたのですが...

Question

前提・実現したいこと

前提として自分はjava初心者なので、おかしな所があったとしても、
温かい目で見ていただけると幸いです。

本題です。CSVファイルの中身のデータを読み込むプログラムを作ろうとしたのですが、try文の中で、
BufferedReader br = new BufferedReader(new FileReader(file));
このオブジェクトを作成しようとしたらエラーが出てしましました。
エラーを見る限りだと、
br.readLine();
のbrオブジェクトが無いと言われてるような気がするのですが、
なぜbrオブジェクトが作られないのか、さっぱりわかりません。
try文の中で作るだけでは足りないのでしょうか？
※yyy.scvはjavaファイルと同じディレクトリに配置してあります。

なぜこのエラーが出るのか、どなたか教えていただけないでしょうか。

発生している問題・エラーメッセージ

xxx.java:23: error: cannot find symbol                                                                      
            br.readLine();                                                                                         
            ^                                                                                                      
  symbol:   variable br                                                                                            
  location: class xxx                                                                                       
1 error

該当のソースコード

import java.io.BufferedReader;
import java.io.FileReader;
import java.io.File;
import java.io.FileNotFoundException;

class xxx
{
    public static void main(String[] args)
    {
        File file = new File("./yyy.csv");
        try
        {
            BufferedReader br = new BufferedReader(new FileReader(file));
        }
        catch(FileNotFoundException a)
        {
            System.out.println("ファイルが開けませんでした");
        }
        if(file.exists())
        {
            System.out.println("hit file");
            br.readLine();
            
        }
        else
        {
            System.out.println("not file");
        }
    }
}

試したこと

いろいろ対策法は考えたのですが、どれもだめでした。
try文の中で処理を全部書くしかないのでしょうか？
それだと、コードが見にくいためできるだけやりたくはないのですが...
お手上げ状態です。お助けください。

補足情報（FW/ツールのバージョンなど）

javaバージョン
openjdk version "11.0.11" 2021-04-20

Comments

[K_3578]

インデントをしっかりしてみればわかるんじゃないですか？

[neconekocat]

原因はtryの中で宣言した変数を外で呼べるわけ無いじゃんって話ではあるんですが、
> コードが見にくいためできるだけやりたくはないのですが
ここの意味が分かりません。try-catchが何のために存在しているか理解されていますか？

[ganmodoki_]

ご回答ありがとうございます。
オブジェクトの作成だけをtry文の中に入れたほうが見やすいと思ったのですが
それはできないということですね。

Answer 1

try-catchで括った場合、tryの中で宣言した変数のスコープはtryの中に限定されます。
tryの外で参照しようとすると"cannot find symbol"でエラーになります。

try文の中で処理を全部書くしかないのでしょうか？

BufferedReaderのcloseを考えるとtry-with-resourcesを使用すべきでしょう。

https://qiita.com/NagaokaKenichi/items/124f0e14ce5dfcbd0b6b

Comments

[ganmodoki_]

ご回答ありがとうございます。
他の方の回答も読ませていただいた結果、try文の中にすべて入れるしか解決方法は無いのですね。
超初心者的なことに回答していただいてありがとうございます。

[neconekocat]

細かいことですがtry-with-です。

[Luice]

失礼しました。修正しておきます。

Answer 2

変数のスコープの問題かと。
BufferedReader brはtryのブロックの中で宣言されているため、その中でしか活きません。
外で宣言するか、そもそもファイルが開けなければexists()もfalseになるので、全部tryの中に入れてしまってはどうでしょうか。

try文の中で処理を全部書くしかないのでしょうか？

それだと、コードが見にくいためできるだけやりたくはないのですが...

見にくくはないかと。
catchでどうするか次第ですが、
やはりtryの中に対象とする処理（今回はファイル操作）
がないのはロジックとしてもおかしくなるので、
個人的な見にくい見やすいという感覚ではなく
「処理のまとまり」として考えられては？

階層が深ければtrycatchをその場で書かずとも
throwsで上位にエラーハンドリングを任せるという手もありますが、
まだそこまでのアプリケーションではなさそうですし、
まとまりを意識した実装のほうが先ですね。

Comments

[ganmodoki_]

ご回答ありがとうございます。
スコープ関連の話はなんとなくしか理解できていないのですが、try文の中で作ったオブジェクトはtry文の外には持っていけないのですね。
try文については有効範囲のルールはないのだと思っていました。

[m.ts10806]

{}
↑囲まれた中がブロックなのでそんな特例はありません。
回答に書いたように宣言（初期値ullとか）を外に書くことでも解決できますが、
FileNotFoundExceptionが起きたときにnullのままとなるので、その場合の処理も必要となり無用なコードが1行以上増えます。
外部からの入力やデータを受け付ける以上、どのエラーがいつ起きるかはシステム側では事前に全て把握することはできませんので、やはりtryは全体にしておいても良いくらいです。
Exceptionも特定のものだけでなく、最上位のExceptionもcatchしておくべきと思います。

[ganmodoki_]

そうなのですね。勉強になります。
import文と同じようにtry-catchも必要な文だけ書くものだと思っていました。
ご指摘のとおりにしてみたいと思います。
ありがとうございます。

[m.ts10806]

>import文と同じようにtry-catchも必要な文だけ書くものだと思っていました。

そう思うことになった根拠を知りたいです。
importはライブラリや機能の読み込みのためのもの、
try-catchは異常処理（想定しない例外の捕捉）のためのもの
で、全く方向性の違う機能です。

[ganmodoki_]

すみません、根拠は特にないです。
何となくそう思っただけなので、確かにその通りですね。

[neconekocat]

必要な文だけ書く、という日本語はあっていますよ。
try内における必要な文とはファイルをcloseするまでの全ての処理ですが。

[ganmodoki_]

とても勉強になりました。
初投稿でドキドキしましたが、やはり質問してよかったです。

[m.ts10806]

あぁ、たしかに
今回は「必要な文の範囲」が把握できてなかっただけですね。