ファイルから行列配列を作成しようとしましたが、プログラムは同じ行列を2つ返しますか？

Question

さて、このプログラムの目的は、ファイルで満たされた2つの行列を加算することです。私のプログラムは最初の配列を作成して印刷しますが、2番目の配列を塗りつぶして印刷しようとすると、最初の配列を塗りつぶして再印刷します。私は理由を正当化することができますが、それを修正する方法についてはわかりません。あなたは以下のコードで表示されますよう

はここだけの行と列を決定するためにされているファイル

3 4 

2 1 7 -10 
0 5 -3 12 
1 7 -2 -5 

0 1 2 3 
4 5 6 7 
8 9 0 1 

3および4です。だから、最初の配列/行列は次のようになります。ここでは

2 1 7 -10 
0 5 -3 12 
1 7 -2 -5 

そして第二に

0 1 2 3 
4 5 6 7 
8 9 0 1 

は私のコードです：

public class Matrices { 

    public static int[][] readMatrix(int rows, int columns, String file) throws FileNotFoundException { 
     Scanner fileReader = new Scanner(new FileInputStream(file)); 
     rows = fileReader.nextInt(); 
     columns = fileReader.nextInt(); 
     int[][] result = new int[rows][columns]; 

     for (int i = 0; i < rows; i++) { 
      for (int j = 0; j < columns; j++) { 
       result[i][j] = fileReader.nextInt(); 

      } 
     } 
     return result; 
    } 

    public static void printMatrix(int[][] matrix) { 
     int rows = matrix.length; 
     int columns = matrix[0].length; 

     for (int i = 0; i < rows; i++) { 
      for (int j = 0; j < columns; j++) { 
       System.out.print(matrix[i][j] + " "); 
      } 
      System.out.println(); 
     } 
    } 
} 

そして、私のドライバー：

public class MatricesDriver { 
    public static void main(String[] args)throws FileNotFoundException { 
     Scanner keyboard = new Scanner(System.in); 
     System.out.println("Enter name of file: "); 
     String filename = keyboard.next(); 
     File file = new File(filename); 
     int rows = 0; 
     int columns = 0; 

     int[][] a = Matrices.readMatrix(rows, columns, filename); 
     int[][] b = Matrices.readMatrix(rows, columns, filename); 

     Matrices.printMatrix(a); 
     System.out.println(); 
     Matrices.printMatrix(b); 
    } 
} 

コード最初のマトリックスの塗りつぶしと印刷には問題ありませんが、第2のマトリックスでも同じことをすることはできません。出力は次のとおりです。

Enter name of file: 
data/MAtrices.txt 
2 1 7 -10 
0 5 -3 12 
1 7 -2 -5 

2 1 7 -10 
0 5 -3 12 
1 7 -2 -5 

もう一度最初の行列が出力されます。同じファイルを使用して2つの別々の行列を作成するにはどうすればよいですか？

Answer 1

各行が正しい行番号から読み取られていることを確認する必要があります。あなたは、行/列番号の記録を保持している場合ので、あなたは設定されます：

class Matrices { 
private Scanner fileReader; 
private int rows; 
private int columns; 

public Matrices(String file) throws FileNotFoundException { 
    this.fileReader = new Scanner(new FileInputStream(file)); 
    rows = fileReader.nextInt(); 
    columns = fileReader.nextInt(); 
} 

public int[][] readMatrix() throws FileNotFoundException { 

    int[][] result = new int[rows][columns]; 

    for (int i = 0; i < rows; i++) { 
     for (int j = 0; j < columns; j++) { 
      result[i][j] = fileReader.nextInt(); 

     } 
    } 
    return result; 
} 

public static void printMatrix(int[][] matrix) { 
    for (int[] anArray : matrix) { 
     for (int anInt : anArray) { 
      System.out.print(anInt+ " "); 
     } 
     System.out.println(); 
    } 
} 
} 

とmain方法はなる：

public static void main(String[] args) throws FileNotFoundException { 
    Scanner keyboard = new Scanner(System.in); 
    System.out.println("Enter name of file: "); 
    String filename = keyboard.next(); 

    Matrices matrixReader = new Matrices(filename); 
    int[][] a = matrixReader.readMatrix(); 
    int[][] b = matrixReader.readMatrix(); 

    Matrices.printMatrix(a); 
    System.out.println(); 
    Matrices.printMatrix(b); 
} 

とOUTPUTは次のようになります。

2 1 7 -10 
0 5 -3 12 
1 7 -2 -5 

0 1 2 3  
4 5 6 7 
8 9 0 1 

Answer 2

2つの行列を読み込んで返すように読み込みメソッドを変更する必要があります。

今すぐファイルを開いて、最初のエントリだけを読み込み、それを返します。 2番目の行列のデータを読み取るのではなく、現在のコードが単に停止するため、最初の行列のみが表示されます。

ファイルフォーマットを拡張して、マトリックスの数も含めることができます。そして、入力ファイルに与えられたすべての情報を読むだけで、行列の配列を返す必要があります。

しかし、正直なところ、あなたのコードは今のところかなり良いです、私はむしろ複数のファイルを使用し、それぞれのファイルに正確に1つの行列を保持することに行きます。