[Java] クラス内で作成したオブジェクト（二次元配列）を引数なしで出力するvoid型の関数

Question

前提

行列の値が保存されているテキストファイルを読み込み（テキストファイルは入力ストリームで指定、3つのファイルがある）、その内容を二次元配列に格納するプログラムを作成しています。（最終的に、二次元配列を用いて行列積を計算する）

テキストファイルは

3 3
0.1 0.2 0.3
0.4 0.5 0.6
0.7 0.8 0.9

のように、1行目に行、列のサイズ、2行目以降が行列の内容となっています。

public Matrix read(String filename)にて
・1行ずつ読み込み、値をスペースで分割して取り出す処理
・取り出した値を二次元配列に格納
する処理までは実行できました。

また、

public class Matrix{
    private int row, col;
    private double[][] m;
}
public Matrix(int M, int N)
public Matrix read(String filename)

の記述は条件で指定されているため、変更できません。

実現したいこと

Matrix read(String filename)クラス内で作成した二次元配列を引数なしで出力するvoid型の関数(Print())を作成したいです。プログラムではAのみ作成していますが、このあと、2つ目のファイルから配列Bを作成し、出力、3つ目も同様に作成・出力したいです。

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

実行しても配列の中身が出力されません。
引数を用いない条件で任意の配列を出力する方法がわからないため、main文での

A.print();

も正しいのかわかっていません。

該当のソースコード

import java.io.*;
import java.io.FileNotFoundException;
import java.io.BufferedReader;


public class Matrix{
    private int row, col;
    private double[][] m;

    public Matrix(){
    }

    public Matrix(int M, int N){
    }

/*ファイルから行列を読み込む(Matrixクラスのオブジェクトを作成*/
    public Matrix read(String filename){
            Matrix mat=new Matrix();
            //mat.m = new double[row][col];
            mat.m = new double[3][3];//try文の外で書くと配列のサイズが得られないので、とりあえず3*3で指定←try文の中で書くべき？
        try {
            BufferedReader br = new BufferedReader(new FileReader(filename));
            String[] rowcol = br.readLine().split(" ");//1行目読み込み(行と列のサイズが記述されている)
            row = Integer.parseInt(rowcol[0]);//行のサイズ
            col = Integer.parseInt(rowcol[1]);//列のサイズ
            System.out.println(row);//確認用出力
            System.out.println(col);//確認用出力
            //mat.m = new double[row][col];
            
            for (int row_read = 0; row_read < row; row_read++) {//読み込む行を指定
                String[] record = br.readLine().split(" "); //(row_read)行の文字列を読み込み、空白で分割
                for (int col_read = 0; col_read < col; col_read++){//読み込む列を指定
                    //System.out.println(Double.parseDouble(record[col_read]));//確認用出力(row_read行col_read列の値を出力)
                    mat.m[row_read][col_read] = Double.parseDouble(record[col_read]);//二次元配列に格納する
                }
            }
            

        }

        catch (FileNotFoundException e) {
			e.printStackTrace();
            System.exit(1);
        }   
        catch (IOException e) { 
            e.printStackTrace();
            System.exit(1); 
        } 
        catch (NumberFormatException e) { 
			e.printStackTrace(); 
            System.exit(1);
        }

        return mat;
    }

    public void print(){
            for (int i = 0; i < row; i++) {
            for (int j = 0; j < col; j++){
                System.out.println(" " + m[i][j]);
            }    
            System.out.println();
        }
        System.out.println();
    }
    public static void main(String[] args){

        try {
            Matrix mat=new Matrix();
            Matrix A=mat.read(args[0]);//args[0]から作成した配列を配列Aとする
            A.print();//配列Aを出力
        
        }
        catch (Exception e) { e.printStackTrace(); }
    }
}

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

Java バージョン8

Answer 1

read()内のrow, colをmatのrow, col (=mat.row, mat.col)にしてください

Comments

[ozwk]

ちなみに`read`がインスタンスメソッドである意味がないので、
問題のつくりがよろしくないか、問題文の解釈がおかしい可能性があります。

[karintage]

表題の件については解決しました！ありがとうございます！
`read`の件は問題で指定されているため、自分も困っている最中です。

表題の件から外れてしまいますが、'read'内4行目でdouble[3][3]としている処理を、mat.rowとmat.colを使って表すことはできるのでしょうか。
可能でしたら教えていただきたいです。

表題の件とは異なるため、返信がない場合でも24:00になりましたらベストアンサーとさせていただきます。

[ozwk]

tryの中でコメントアウトされている行で表せてます

[karintage]

ありがとうございます！
コメントアウトの行でサイズを指定できていました！
try文の前と中でmat.m=new doubleとしているので、二度手間のように感じるのですが、これは仕方ないのでしょうか？

[ozwk]

tryの前のものは不要です

> 二度手間のように感じるのですが

いい感性をお持ちです。
そこから一歩踏み出して、「では消したらどうなるのか」と考えて実践してみるとよいかと。

[karintage]

おっしゃる通り、tryの前のものは不要でした。
いろいろと試してみることを意識します！
非常にわかりやすく回答してくださり、ありがとうございました！