Java

@ligun 様、回答ありがとうございます。

＞＞そもそもIndicator側だけでエラー処理するのではだめなのでしょうか？
これは、クラスを分けない方がいい、という意味でしょうか？（初心者なのでうまく意味を受け取れていなかったらすいません m(> <)m）
ここに載せているコードは本来のコードを簡潔に書いたもので、本来はもっと長いコードとなっています。なのでクラスを分けた方がいいかなと思い「読み込み」と「表示」で分けているのですが、確かに一緒でもいいような気がしますね。

＞＞どれも要件からは微妙に外れてしまいそうですが
すいません、ここの文章も私の勉強不足故、どういう意味か理解できませんでした m(> <)m

＞＞個人的にはreadFileというメソッド名と実装がマッチしてないのが気持ち悪いです。
readFile()はファイルを読み込んでいるメソッドなのでそう命名したのですが、どうおかしいでしょうか。教えてくださると幸いです m(> <)m

勉強不足故変なことを聞いていたらすいません m(> <)m

readFile()はファイルを読み込んでいるメソッドなのでそう命名したのですが、どうおかしいでしょうか。教えてくださると幸いです m(> <)m

すみません、こちらは自分の勘違いでした。
// ここにファイルの内容をリストにするコードを書き込む。 のコメントを「ファイル内容を表示する」と勘違いしたことによるものでした。
質問を見直したところ自分がコメントした内容と同じくリスト生成と表示は分離されていました。
混乱させてしまい申し訳ございません。

これは、クラスを分けない方がいい、という意味でしょうか？（初心者なのでうまく意味を受け取れていなかったらすいません m(> <)m）

設計ややりたい内容にもよるのであくまで自分の場合ですが、データを取得するのも表示するのも同じデータに対する操作なので、データを扱うクラスが合ってそれに対するloadメソッドとdisplayメソッドがあれば十分かなと思いました。

Javaしばらく書いていないのとIDE通してないので多分色々と間違ってそうですが、下記のような感じでどうでしょうか？
（詳しい人にマサカリ投げられそう）

public class CsvData {
    private List<List<String>> data;
    private CsvData() {}

    static CsvData load(String path) throws IOExecption {
        var csvData = new CsvData();
        //CSVファイルを読んでdataに格納する
　　　　//...
        
　　　　//生成したオブジェクトを返す
        return csvData;
    }

    void display() {
        //dataを出力する処理
    }
}

すいません、ここの文章も私の勉強不足故、どういう意味か理解できませんでした m(> <)m

これは単に元々の質問の要求ですとFileReader側だけで処理したいとのことでしたが、自分の案だとIndicator側で処理してたり出力処理の場所を変更してたりする提案をしたので、厳密に言えば要求にあっていないなということでした。

@Verclene 様、回答ありがとうございます。
クラス名のFileReaderについてですが、本来区別できるよう命名していたのですが情報保護のため名前をQiita用に変える際に失念しており標準ライブラリと同じ名前にしてしまっていました、失礼しました m(> <)m
ご指摘ありがとうございます。

お察しの通り、オブジェクト指向的な書き方の習得に難航しております。。。
「例外をそのままthrowsしてしまう」というのはエラーを起こすという意味でしょうか？
ユーザーが使う以上エラーは基本的にcatchするべきなのではと思うのですがそんなことはないのでしょうか？（初心者故的外れなことを言っていればすいません、、）

今の所、TableDataReaderのときにcatchにならなければ最後にフィールドのArrayListに格納するようにしており、catchに引っ掛かればArrayListに格納ができずNullになるので、Indicatorの時にif文でArrayListがNullの場合returnを返すようにして対応しております。

「例外をそのままthrowsしてしまう」というのはエラーを起こすという意味でしょうか？
ユーザーが使う以上エラーは基本的にcatchするべきなのではと思うのですがそんなことはないのでしょうか？

Javaにはthrows宣言というものがありまして,用途は調べていただければと思いますが,要はこのメソッドはこのエラーを吐くかもしれんから使用者の責任で対処しろという宣言ができます.

まずここでいう「使用者」とはユーザーのことではなく,メソッドから見てそれを呼び出す側のことをやむを得ずそう呼称しています.詳しく解説するより,抽象化について理解が深まってからのほうがイメージしやすいと思われますので,細かい話は割愛しますが,「どこで異常に対処するかによってもクラスの役割が変わってきてしまう」という話だと思ってください.

ソフトウェアのユーザー目線の話は一旦忘れておいていいです.そこは別の設計の話になりますので,あんまり細かいこと言うと初学者にはまだしんどいですし,ポートフォリオ程度どうせログ吐いて終わりですんで…