[java初心者]「テキストファイルから行列データを読み込み二次元配列に保存したものを表示する」プログラムでのエラー

Question

追記2
教えていただいたことを参考に
getterやsetterを書き加えました
まだ正しく記述できておらず、コンパイルエラーが出ている状態のため
修正の方法についてお聞きしたいです

追記1
最初に質問した内容(上のプログラムについて)は回答していただいた方のおかげで解決しました。
続けて質問させていただくために修正後のプログラムを最初のプログラムの下に追加させていただきました。
修正後のプログラムには以下のようなエラーが表示されました。
ーーーーーーーーーーーーーーーーー
Matrix.java:39: エラー: シンボルを見つけられません
matrix.print();
^
シンボル: メソッド print()
場所: タイプdouble[][]の変数 matrix
Matrix.java:74: エラー: 不適合な型: Matrixをdouble[][]に変換できません:
A = y.read(args[0]);
^
Matrix.java:76: エラー: 不適合な型: Matrixをdouble[][]に変換できません:
B = z.read(args[1]);
ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー

※以下最初の質問の内容

「テキストファイルから行列データを読み込み二次元配列に保存したものを表示する」プログラムを作成、
コンパイルしたところ
以下のようなエラーメッセージが表示されました。
ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー
Matrix.java:26: エラー: '.class'がありません
while (int M=0 < row && (line = br.readLine()) != null){
^
Matrix.java:26: エラー: ';'がありません
while (int M=0 < row && (line = br.readLine()) != null){
^
ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー
これはどのような原因が考えられるでしょうか？

テキストファイルの例
3 3 ←行と列の数を表す
0.11 0.03 0.1 ←以下行列の要素
0.2 0.22 0.66
0.4 0.56 0.09

getRow、getColumメソッドにより指定されたファイルから1行読み込み、行と列の数を取得する
readメソッドにより以降の行も順に読んでいき、二次元配列に保存する
printメソッドにより表示を行う
といった流れのプログラムです

JDK8を使用しています

//最初の質問に関するプログラム
import java.io.*;
public class Matrix{
	private int row,col;
	public Matrix();
	public Matrix(int M,int N){
		this.row=M;
		this.col=N;
	}
	private double[][] matrix = new double [row][col];
	public int getRow(String filename){
		BufferedReader br = new BufferedReader(new FileReader(fileName));
		String[] M = br.split("\s");
		int m = Integer.parseInt(M[0]);
		return m;
	}
	public int getColumn(String filename){
		BufferedReader br = new BufferedReader(new FileReader(fileName));
		String[] N = br.split("\s");
		int n = Integer.parseInt(N[1]);
		return n;
	}
	public Matrix read(String filename){
		try (BufferedReader br = new BufferedReader(new FileReader(filename))){
			String[] rowcol = br.readLine().split(" ");
			String line = null;
			while (int M=0 < row && (line = br.readLine()) != null){ 
				String[] record = line.split(" "); 
				for (int N = 0; N < col ; N++) { 
					matrix[M][N] = Double.parseDouble(record[N]);
				} 
				M++; 
			}
			matrix.print();
		}
		catch (FileNotFoundException e) {
			e.printStackTrace(); 
		} 
		catch (IOException e) { 
			e.printStackTrace(); 
		} 
		catch (NumberFormatException e) { e.printStackTrace(); 
		}
	}
	public void print(){ 
		for (double[] record : matrix) {
			for (double d : record) {
				System.out.print(d + " ");
			}
			System.out.println();
		}
	}
	public static void main(String[] args){
		if (args.length != 2) {
		System.out.println("need two files"); System.exit(1); 
		}
		int arow,acol,brow,bcol;
		Matrix x = new Matrix();
		arow=x.getrow(args[0]);
		acol=x.getColumn(args[0]);
		brow=x.getrow(args[1]);
		bcol=x.getColumn(args[1]);
		double [ ][ ] A = new double [arow][acol];
		double [ ][ ] B = new double [brow][bcol];
		double [ ][ ] C = new double [acol][brow];
		Matrix x = new Matrix(arow,acol);
		Matrix A = x.read(args[0]);
		Matrix x = new Matrix(brow,bcol);
		Matrix B = x.read(args[1]);
	}
}

//次の質問に関するプログラム
import java.io.*;
import java.io.FileInputStream;
import java.io.InputStreamReader;
public class Matrix{
	private int row,col;
	public Matrix(){
	}
	public Matrix(int M,int N){
		this.row=M;
		this.col=N;
	}
	private double[][] matrix = new double [row][col];
	public int getRow(String filename){
		BufferedReader br = new BufferedReader(new FileReader(filename));
		String rline = br.readLine();
		String[] R = rline.split("\s");
		int m = Integer.parseInt(R[0]);
		return m;
	}
	public int getColumn(String filename){
		BufferedReader br = new BufferedReader(new FileReader(filename));
		String cline = br.readLine();
		String[] C = cline.split("\s");
		int n = Integer.parseInt(C[1]);
		return n;
	}
	public Matrix read(String filename){
		try (BufferedReader br = new BufferedReader(new FileReader(filename))){
			String[] rowcol = br.readLine().split(" ");
			String line = null;
			int M=0;
			while (M < row && (line = br.readLine()) != null){ 
				String[] record = line.split(" "); 
				for (int N = 0; N < col ; N++) { 
					matrix[M][N] = Double.parseDouble(record[N]);
				} 
				M++; 
			}
			matrix.print();
		}
		catch (FileNotFoundException e) {
			e.printStackTrace(); 
		} 
		catch (IOException e) { 
			e.printStackTrace(); 
		} 
		catch (NumberFormatException e) { 
		e.printStackTrace(); 
		}
	}
	public void print(){ 
		for(int i = 0; i<row; i++){
			for(int j = 0; j<col; j++){
				System.out.println(matrix[i][j]);
			}
		}

	}
	public static void main(String[] args){
		if (args.length != 2) {
		System.out.println("need two files");
		System.exit(1); 
		}
		int arow,acol,brow,bcol;
		Matrix x = new Matrix();
		arow=x.getRow(args[0]);
		acol=x.getColumn(args[0]);
		brow=x.getRow(args[1]);
		bcol=x.getColumn(args[1]);
		double [ ][ ] A = new double [arow][acol];
		double [ ][ ] B = new double [brow][bcol];
		double [ ][ ] C = new double [acol][brow];
		Matrix y = new Matrix(arow,acol);
		A = y.read(args[0]);
		Matrix z = new Matrix(brow,bcol);
		B = z.read(args[1]);
	}
}

//getterやsetterを利用した形
import java.io.*;
import java.io.FileInputStream;
import java.io.InputStreamReader;
public class Matrix{
	private int row,col;
	public Matrix(){
	}
	public Matrix(int M,int N){
		this.row=M;
		this.col=N;
	}
	private double[][] matrix = new double [row][col];
	public int getRow(String filename){
		BufferedReader br = new BufferedReader(new FileReader(filename));
		String rline = br.readLine();
		String[] R = rline.split("\s");
		int m = Integer.parseInt(R[0]);
		return m;
	}
	public int getColumn(String filename){
		BufferedReader br = new BufferedReader(new FileReader(filename));
		String cline = br.readLine();
		String[] C = cline.split("\s");
		int n = Integer.parseInt(C[1]);
		return n;
	}
	public Matrix read(String filename){
		try (BufferedReader br = new BufferedReader(new FileReader(filename))){
			String[] rowcol = br.readLine().split(" ");
			String line = null;
			int M=0;
			while (M < row && (line = br.readLine()) != null){ 
				String[] record = line.split(" "); 
				for (int N = 0; N < col ; N++) { 
					matrix[M][N] = Double.parseDouble(record[N]);
				} 
				M++; 
			}
			print();
		}
		catch (FileNotFoundException e) {
			e.printStackTrace(); 
		} 
		catch (IOException e) { 
			e.printStackTrace(); 
		} 
		catch (NumberFormatException e) { 
		e.printStackTrace(); 
		}
	}
	public void print(){ 
		for(int i = 0; i<row; i++){
			for(int j = 0; j<col; j++){
				System.out.println(matrix[i][j]);
			}
		}
	}
	public void setmatrix(double[][] MAT) {
	MAT=this.matrix;
	}
	public double[][] getmatrix(){
		return MAT;
	}
	public void main(String[] args){
		if (args.length != 2) {
		System.out.println("need two files");
		System.exit(1); 
		}
		int arow,acol,brow,bcol;
		Matrix x = new Matrix();
		arow=x.getRow(args[0]);
		acol=x.getColumn(args[0]);
		brow=x.getRow(args[1]);
		bcol=x.getColumn(args[1]);
		double [ ][ ] A = new double [arow][acol];
		double [ ][ ] B = new double [brow][bcol];
		double [ ][ ] C = new double [acol][brow];
		Matrix y = new Matrix(arow,acol);
		y.read(args[0]);
		y.setmatrix(A);
		A=matrix;
		Matrix z = new Matrix(brow,bcol);
		z.read(args[1]);
		z.setmatrix(B);
		B=matrix;
	}
}

Comments

[dodox86]

追記される新たな質問を読んで：（※回答した当事者ではないのですが、気になったので）
最初の質問に関してはいただいた回答で解決したと思うのですけど、それを解決することで新たな問題が出て、全て解決するまでこの質問は終わらないということなのでしょうか。それはちょっとマナー的にどうなのか、と思いますがいかがでしょうか。

Answer 1

行列データの読み込みは書いたので、
読み込んだ 2つの行列データの掛け算は自分で書いてください。

import java.io.*;

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

	public Matrix() {}

	public Matrix(Matrix a, Matrix b) {
		if (a.col != b.row) return;
		row = a.row;
		col = b.col;
		matrix = new double[row][col];
		// ここで matrix[i][j] に a * b の結果を入れる
	}

    public void read(String filename) {
        try (BufferedReader br = new BufferedReader(new FileReader(filename))){
            String[] rowcol = br.readLine().split(" ");
			row = Integer.parseInt(rowcol[0]);
			col = Integer.parseInt(rowcol[1]);
			matrix = new double[row][col];
            String line = null;
            for (int m = 0; m < row; m++) {
            	line = br.readLine();
                String[] record = line.split(" "); 
                for (int n = 0; n < col; n++)
                    matrix[m][n] = Double.parseDouble(record[n]);
            }
        }
        catch (FileNotFoundException e) {
            e.printStackTrace(); 
        } 
        catch (IOException e) { 
            e.printStackTrace(); 
        } 
        catch (NumberFormatException e) { 
			e.printStackTrace(); 
        }
    }

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

    public static void main(String[] args){
        if (args.length != 2) {
			System.out.println("need two files");
			System.exit(1); 
        }

        Matrix a = new Matrix();
		a.read(args[0]);
		a.print();

        Matrix b = new Matrix();
		b.read(args[1]);
		b.print();

		Matrix c = new Matrix(a, b);
		c.print();
    }
}

追記
行列の掛け算をコンストラクタで実行するのではなく、
a.multiply(b) で結果の Matrix を返してくれるようにしてみました。

import java.io.*;

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

	public Matrix() {}

	public Matrix(int row, int col) {
		this.row = row;
		this.col = col;
		matrix = new double[row][col];
	}

    public void read(String filename) {
        try (BufferedReader br = new BufferedReader(new FileReader(filename))){
            String[] rowcol = br.readLine().split(" ");
            row = Integer.parseInt(rowcol[0]);
            col = Integer.parseInt(rowcol[1]);
            matrix = new double[row][col];
            for (int m = 0; m < row; m++) {
                String[] record = br.readLine().split(" "); 
                for (int n = 0; n < col; n++)
                    matrix[m][n] = Double.parseDouble(record[n]);
            }
        }
        catch (FileNotFoundException e) {
            e.printStackTrace(); 
        } 
        catch (IOException e) { 
            e.printStackTrace(); 
        } 
        catch (NumberFormatException e) { 
            e.printStackTrace(); 
        }
    }

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

    public Matrix multiply(Matrix b) {
        if (col != b.row) return null;
		Matrix c = new Matrix(row, b.col);
		for (int i = 0; i < row; i++)
			for (int j = 0; j < col; j++) {
				c.matrix[i][j] = 0;
				for (int k = 0; k < b.row; k++)
					c.matrix[i][j] += matrix[i][k] * b.matrix[k][j];
			}
		return c;
    }

    public static void main(String[] args){
        if (args.length != 2) {
            System.out.println("need two files");
            System.exit(1); 
        }

        Matrix a = new Matrix();
        a.read(args[0]);
        a.print();

        Matrix b = new Matrix();
        b.read(args[1]);
        b.print();

        Matrix c = a.multiply(b);
        c.print();
    }
}

Comments

[soft_cream]

せっかくプログラムを頂いたのにごめんなさい
本文に記載しているメソッドのプロトタイプやメソッドの呼び出し方などは
仕様として指定されているので変更ができません
ですが参考にはさせていただこうかと思います
ありがとうごさいます

Answer 2

多分ですけど、『whileの条件式内で変数を宣言しているのが原因』じゃないかなと。

問題の部分を見ると、

while (int M=0 < row && (line = br.readLine()) != null){
   ...
}

となっていますね。

ブロックの数とかは合っていそうですが、
ただ、『普通はwhile内で宣言はしないのでは』ないでしょうか。

試しに、Paiza IOで、

import java.util.*;

public class Main {
    public static void main(String[] args) throws Exception {
        // Your code here!
        
       while( int n = 0 < 5 ){
           System.out.println( "OK");
       }
    }
}

とやると同じようなエラーメッセージが出てきました。

構文もよく見ると、";" (セミコロン)がありません。

変数宣言は、

int a;

または

int a = 1;

のように宣言するはずです。

もしかして、for文と勘違いしていませんか?

---

[追記1]

Matrix.java:39: エラー: シンボルを見つけられません
matrix.print();
^
シンボル: メソッド print()

当たり前です。

肝心の 39行目を見ると、

matrix.print();

とありますが、このmatrixとはなんでしょうか?

ローカル変数(readメソッド内にある変数)としても存在しません。

でも、よく見ると、フィールドとして二次元配列があります。
( private double[][] matrix = new double [row][col]; )

ですが、これはあくまで『二次元配列』です。

二次元配列はあくまで配列の二次元版です。

配列が持っていないメンバ( printメソッド )を動かそうとしているから、

『おいおい、にーちゃん(あるいはねーちゃん)、そんなもんねーよ?』と突っ込まれているのです。

そもそも、可読性が悪すぎる。

今回のエラーは、『可読性が悪いことが原因』になっています。

クラス名 Matrix で、これのオブジェクトが matrix.

で、内部で保持している二次元配列も matrix となっている。

これでは、気を抜いたら書いた本人ですら『混乱』します。

(あれ、どっちのだったけ……って感じで)

Matrix.java:74: エラー: 不適合な型: Matrixをdouble[][]に変換できません:
A = y.read(args[0]);
^

これに関しては、定義を確認してください。

まず、『A』とはなんでしょうか。

どうやら double [ ][ ] A のようですね。

つまり double型の二次元配列。

『y』とはなんでしょうか。

どうやら、Matrix y のようですね。

それでは、y.read( なんちゃら ) とはなんでしょうか。

Matrixクラスの
public Matrix read(String filename)
ですね。

定義を確認してください。

つまり、日本語で無理に書けば、

double型二次元配列 = Matrixクラスのオブジェクト.read(引数);

の状態になっています。

つまり、『本来はMatrixのオブジェクトを返しているのに、受け手がなぜかdouble型二次元配列で受け取ろうとしている』状態です。

ある意味、『椅子を10個のグラスに注ごうとしているようなもの』です。

そもそも入るわけがないです。

このように、『コンパイルエラーにはヒントが隠されています』。

コンパイルエラーは怒声でも暴言でも罵倒でもハラスメントでもなく、
コンパイラ等からのメッセージです。

なので恐れずに、メッセージを読んで対処しましょう。

---

[追記2]

ほぼ脳死でメソッドを使うときは　.(メソッド)

基礎からやってください。

入門書やサイト等にあるのは『基礎』であり、
別の言い方にすると『道具』です。

プログラミング言語があるからソフトとかを作る…
のではなく、

『とある問題を解決するための道具として』プログラミング言語があるのです。

元々は軍事利用で、たとえば、弾道計算や暗号解読とかを目的にしたのです。

『プログラムは魔法でもなんでもなく、人間が現実世界でやっている手順を逐一指示されながら処理しているだけほ代物である』です。

基礎をやるのは、料理で言えば
『包丁の使い方』や『材料が揃っているかの確認』とかに相当します。

なので、基礎を理解しましょう。

readメソッドに関してなのですが...

直接は無理です。

ですが、Matrixクラス側で、いわゆるgetter/setterに相当するやつを追加すれば、
戻り値として受け取るのはMatrixオブジェクトですが、
後から配列や二次元配列に変換できます。

まずは基礎からやりましょう。

---

[追記3]

まず、getterとかのエラーの前に『オブジェクト指向とは何か』を学びましょう。

オブジェクト指向(以降、OOP)の基本概念は、
『データ(= フィールド)と処理(= メソッド)をひとまとめにしたオブジェクト』なるものを中心に見る発想法です。

C言語のようなオブジェクト指向以前の言語では
データと処理は別々でした。

それらをひとまとめにすることで、
『オブジェクトだけが対象データとその処理方法を知っている』状態にでき、
それによって、『オブジェクトに対象データの処理や管理を任せる』ということができます。

クラスやら継承やらはこの概念を表現するためのものです。

オブジェクトだけが知っているという状態にするため、
フィールドは基本的にprivateにするのです。

でも、それではフィールドを直接見ることはできません。

なので『苦肉の策』としていわゆるgetterとかを使うのです。

クラスはコンパイラとかに「こーゆーデータ構造(どういうデータを持っているかとか)があるんでよろしく〜♪」
のように伝えるためのものです。

staticなメソッドはC言語の関数と一緒です。

オブジェクトがする処理方法というより、
単なる関数としてのものに近いです。

なので、今回の場合、
staticなgetterで別のオブジェクトのデータを引っ張り出そうとしているからエラーになっています。
(厳密に言えば違うが、イメージ的にはそういう感じ)

Comments

[soft_cream]

回答していただいたお二人の意見を参考にしてプログラムを修正しコンパイルしたところ
隠れていたいくつかのエラーが表示されたため、
最初のプログラムを以下のように修正させていただきました。
その中の3つのエラーの修正の仕方が分からないため
また意見を頂けないでしょうか
最初の問題については解決しました。ありがとうございます
ーーーーーーーーーーーーーーーーーーーーーーー
Matrix.java:39: エラー: シンボルを見つけられません
matrix.print();
^
シンボル: メソッド print()
場所: タイプdouble[][]の変数 matrix
Matrix.java:74: エラー: 不適合な型: Matrixをdouble[][]に変換できません:
A = y.read(args[0]);
^
Matrix.java:76: エラー: 不適合な型: Matrixをdouble[][]に変換できません:
B = z.read(args[1]);

[soft_cream]

import java.io.*;
import java.io.FileInputStream;
import java.io.InputStreamReader;
public class Matrix{
private int row,col;
public Matrix(){
}
public Matrix(int M,int N){
this.row=M;
this.col=N;
}
private double[][] matrix = new double [row][col];
public int getRow(String filename){
BufferedReader br = new BufferedReader(new FileReader(filename));
String rline = br.readLine();
String[] R = rline.split("\s");
int m = Integer.parseInt(R[0]);
return m;
}
public int getColumn(String filename){
BufferedReader br = new BufferedReader(new FileReader(filename));
String cline = br.readLine();
String[] C = cline.split("\s");
int n = Integer.parseInt(C[1]);
return n;
}
public Matrix read(String filename){
try (BufferedReader br = new BufferedReader(new FileReader(filename))){
String[] rowcol = br.readLine().split(" ");
String line = null;
int M=0;
while (M < row && (line = br.readLine()) != null){
String[] record = line.split(" ");
for (int N = 0; N < col ; N++) {
matrix[M][N] = Double.parseDouble(record[N]);
}
M++;
}
matrix.print();
}
catch (FileNotFoundException e) {
e.printStackTrace();
}
catch (IOException e) {
e.printStackTrace();
}
catch (NumberFormatException e) {
e.printStackTrace();
}
}
public void print(){
for(int i = 0; i<row; i++){
for(int j = 0; j<col; j++){
System.out.println(matrix[i][j]);
}
}

}
public static void main(String[] args){
if (args.length != 2) {
System.out.println("need two files");
System.exit(1);
}
int arow,acol,brow,bcol;
Matrix x = new Matrix();
arow=x.getRow(args[0]);
acol=x.getColumn(args[0]);
brow=x.getRow(args[1]);
bcol=x.getColumn(args[1]);
double [ ][ ] A = new double [arow][acol];
double [ ][ ] B = new double [brow][bcol];
double [ ][ ] C = new double [acol][brow];
Matrix y = new Matrix(arow,acol);
A = y.read(args[0]);
Matrix z = new Matrix(brow,bcol);
B = z.read(args[1]);
}
}

[BeatStar]

ここにコードを書かない方がいいです。
インデントが無視されて読みづらいです。

本文に追記として書くか、別の問題と考えるのなら別のスレとして立ててください。
(今回は本文に追記でいいと思う)

[soft_cream]

申し訳ありません。
質問に追記してプログラムを記述しました。

[BeatStar]

忘れていましたが、『元のプログラムはそのままで』お願いします。
(質問の意図がずれる為)

…あ、書いてありますね。

ただ、どちらの方なのかわかりにくいため、何かしらの方法でわかりやすくした方が無難かもしれません。

[soft_cream]

返信ありがとうございます。
printメソッドに関して修正させていただきました。
ほぼ脳死でメソッドを使うときは　.(メソッド)　でいいのかな、なんて考えてしまっていました。
エラーメッセージに関しても
初めから無理だと決めつけていました。

[soft_cream]

readメソッドに関してなのですが
readメソッドの型はMatrixのままで
readメソッドにより読み込んだ二次元配列をAに代入することは出来ないでしょうか
A = y.read(args[0]);
↑この「A」の部分を
Matrix x = new Matrix();で定義した「x」に変えて
xを二次元配列に変換する
事などできないでしょうか

[soft_cream]

ごめんなさい、何言ってるか意味不明ですよね
java習いたてでまだ理解ができていない状態です

[BeatStar]

追記しました。

[soft_cream]

本文の質問にgetterやsetterを加えたプログラムを追記しました
setterでprivateなmatrixをMATに代入し
getterで利用する
といった考えで書いたところ以下のようなエラーが表示されました
Matrix.java:62: エラー: シンボルを見つけられません
return MAT;
^
シンボル: 変数 MAT
場所: クラス Matrix

[soft_cream]

考え方自体が間違っているのでしょうか
もし記述ミスなどあれば教えてもらいたいです

getterやsetterについて参考にできるサイトなどふぁあれば教えてもらいたいです
習い始めたばかりで基礎ができていないのも全くその通りなので
java自体について知ることのできるサイトなども教えてもらえないでしょうか

Answer 3

僕が普段使う方法を記しました。
質問の回答にはなっていないかもしれませんが一応参考までに記しておきます。

try{
	BufferedReader bufferedReader = new BufferedReader(new FileReader(/*ファイル名*/));
	
	//二次元配列の長さ
	int x = Integer.parseInt(bufferedReader.readLine());
	int y = Integer.parseInt(bufferedReader.readLine());
	
	String[] elements_line = new String[y];
	String elements = new String[y][x];
	
	//とりあえず一列ずつ読み込み
	for(int i=0;i<y;i++){
		elements_line[i] = bufferedReader.readLine();
	}
	bufferedReader.close();//変数bufferedReaderはもう使わないので
	
	//一列ずつ読み込んだものを一つずつにする
	for(int ii=0;ii<y;ii++){
		for(int i=0;i<x;i++){
			image_ids[ii][i]=String.valueOf(elements_line[ii].charAt(i));
		}
	}
}catch(Exception e){
	e.printStackTrace();
}

エラーだけしか見ていませんが質問の回答します。while (int M=0 < row && (line = br.readLine()) != null){ の中にある
int M=0が悪さをしていると思うのでただの0にした方がよいかと思います。

while (int M=0 < row && (line = br.readLine()) != null){ 

から

while (0 < row && (line = br.readLine()) != null){ 

Comments

[soft_cream]

回答していただいたお二人の意見を参考にしてプログラムを修正しコンパイルしたところ
隠れていたいくつかのエラーが表示されたため、
最初のプログラムを以下のように修正させていただきました。
その中の3つのエラーの修正の仕方が分からないため
また意見を頂けないでしょうか
最初の問題については解決しました。ありがとうございます
ーーーーーーーーーーーーーーーーーーーーーーー
Matrix.java:39: エラー: シンボルを見つけられません
matrix.print();
^
シンボル: メソッド print()
場所: タイプdouble[][]の変数 matrix
Matrix.java:74: エラー: 不適合な型: Matrixをdouble[][]に変換できません:
A = y.read(args[0]);
^
Matrix.java:76: エラー: 不適合な型: Matrixをdouble[][]に変換できません:
B = z.read(args[1]);

[soft_cream]

import java.io.*;
import java.io.FileInputStream;
import java.io.InputStreamReader;
public class Matrix{
private int row,col;
public Matrix(){
}
public Matrix(int M,int N){
this.row=M;
this.col=N;
}
private double[][] matrix = new double [row][col];
public int getRow(String filename){
BufferedReader br = new BufferedReader(new FileReader(filename));
String rline = br.readLine();
String[] R = rline.split("\s");
int m = Integer.parseInt(R[0]);
return m;
}
public int getColumn(String filename){
BufferedReader br = new BufferedReader(new FileReader(filename));
String cline = br.readLine();
String[] C = cline.split("\s");
int n = Integer.parseInt(C[1]);
return n;
}
public Matrix read(String filename){
try (BufferedReader br = new BufferedReader(new FileReader(filename))){
String[] rowcol = br.readLine().split(" ");
String line = null;
int M=0;
while (M < row && (line = br.readLine()) != null){
String[] record = line.split(" ");
for (int N = 0; N < col ; N++) {
matrix[M][N] = Double.parseDouble(record[N]);
}
M++;
}
matrix.print();
}
catch (FileNotFoundException e) {
e.printStackTrace();
}
catch (IOException e) {
e.printStackTrace();
}
catch (NumberFormatException e) {
e.printStackTrace();
}
}
public void print(){
for(int i = 0; i<row; i++){
for(int j = 0; j<col; j++){
System.out.println(matrix[i][j]);
}
}

}
public static void main(String[] args){
if (args.length != 2) {
System.out.println("need two files");
System.exit(1);
}
int arow,acol,brow,bcol;
Matrix x = new Matrix();
arow=x.getRow(args[0]);
acol=x.getColumn(args[0]);
brow=x.getRow(args[1]);
bcol=x.getColumn(args[1]);
double [ ][ ] A = new double [arow][acol];
double [ ][ ] B = new double [brow][bcol];
double [ ][ ] C = new double [acol][brow];
Matrix y = new Matrix(arow,acol);
A = y.read(args[0]);
Matrix z = new Matrix(brow,bcol);
B = z.read(args[1]);
}
}

[soft_cream]

本文に追記する形で修正後のプログラムを記述させていただきました

[cretter]

>Matrix.java:39: エラー: シンボルを見つけられません
matrix.print();をただのprint();にしてください。
matrixは二次元配列の変数です。
次に
>public Matrix read(String filename){
は戻り値がMatrixなのに対して変数Aはdoubleの二次元配列です。
それと、readメソッドにはreturn文がありません。

[soft_cream]

返信ありがとうございます。
printメソッドに関して修正させていただきました。

[soft_cream]

申し訳ありません、return matrix;の文を追加させていただきました。

readメソッドの型はMatrixのままで
readメソッドにより読み込んだ二次元配列をAに代入することは出来ないでしょうか
A = y.read(args[0]);
↑この「A」の部分を
Matrix x = new Matrix();で定義した「x」に変えて
xを二次元配列に変換する
事などできないでしょうか

[soft_cream]

もしかすると、下の部分を
double [ ][ ] A = new double [arow][acol];
double [ ][ ] B = new double [brow][bcol];
double [ ][ ] C = new double [acol][brow];
↓
Matrix A = new Matrix(arow,acol);
Matrix B = new Matrix(brow,bcol);
Matrix c = new Matrix(acol,brow);
A.read();
B.read();

のようにすれば
A、Bにそれぞれ行列の要素が代入されるでしょうか

書き忘れていましたが
C行列はこの後さらに「AとBの積を計算する」メソッドを追加するつもりで
C行列はその時の積を代入するためのものです

[cretter]

クラス変数A,Bに配列の要素が代入されているというよりも
クラス変数A,B内の二次元配列の変数として保存されていますよ
一応書きますがクラス変数の中から変数を取得するためには「クラス変数.変数名」で使うことができます
今回の場合、クラス変数Aの中にあるmatrixの二次元配列が使いたけれは「A.matrix[/*数字*/][/*数字*/]」で使うことができます