配列値をある配列から別の配列に変更する

Question

私はCSクラスのJava割り当てを行っています。 ユーザー定義の配列から別の配列に値を転送する際に問題が発生しています。 配列を別のメソッドに転送できますが、配列の値を2番目の配列のパラメータに一致させることができません。 ありがとうございます。

import java.util.*; 
    import java.io.IOException; 
    public class HW5_1{ 
    public static void transpose(int mn[][]) throws IOException 
    { 
    Scanner keyboard = new Scanner(System.in); 
    int i, j; 
    int nm[][] = new int [5][3]; 
    for(i = 0; i < 5; i++){ 
    for(j = 0; j < 3; j++){ 
    nm[i][j]= mn[j][i]; 
    } 
    System.out.println(nm[i][j]+ " " + "/n");} 
    } 
    public static void main(String[] args) throws IOException{ 

    Scanner keyboard = new Scanner(System.in); 

    System.out.println("How many rows?"); 
    int row = keyboard.nextInt(); 

    System.out.println("How many columns?"); 
    int column = keyboard.nextInt(); 

    //Create array 
    int [][] mn = new int[row][column]; 

    //Create variables 
    int i,j = 0; 

    //For loops to receive input 
    for(i = 0; i < row; i++) 
    { 
    for (j = 0; j < column; j++) 
    { 
     System.out.println("Enter the numbers: "); 
     mn[i][j] = keyboard.nextInt(); 
    } 
    } 
    transpose(mn); 
    //Moves array to transpose method 
    } 
} 

私はこの答えになるだろう：

java.lang.ArrayIndexOutOfBoundsException: 3 
    at HW5_1.transpose(HW5_1.java:19) 
    at HW5_1.main(HW5_1.java:47) 
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
    at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) 
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) 
    at java.lang.reflect.Method.invoke(Unknown Source) 
    at edu.rice.cs.drjava.model.compiler.JavacCompiler.runCommand(JavacCompiler.java:267) 

Answer 1

を私はあなたがのためにループ内であなたのSystem.out.println(nm[i][j]+ " " + "/n");をしたい-methodあなたtransposeで考えます。また、あなたの方法では不要なものがたくさんあります。例えば。 throws宣言だけでなく、keyboard変数も含まれます。

次に、それぞれ行として5と3のような静的宣言があります。これは、入力配列から値を動的に読み取ることによってスキップすることができます。

最後に、iとjという変数をループ外に置く必要はなく、内部でスコープを設定することができます。

だから、クリーンアップの方法は次のようになります。

public static void transpose(int mn[][]){ 
    int nm[][] = new int[mn[0].length][mn.length]; // use the values from mn 
    for(int i = 0; i < 5; i++){ 
     for(int j = 0; j < 3; j++){ 
      nm[i][j] = mn[j][i]; 
      System.out.println(nm[i][j] + " " + "/n"); 
     } 
    } 
} 

Answer 2

をあなたの新しい配列の固定サイズがあります。

int nm[][] = new int [5][3]; 

は、元の配列の次元を使用して変数にそれを行います。

public static void transpose(int mn[][]) throws IOException { 
    Scanner keyboard = new Scanner(System.in); 
    int i, j; 
    int nm[][] = new int [mn.length][mn[0].length]; 
    for(i = 0; i < mn.length; i++){ 
     for(j = 0; j < mn[0].length; j++){ 
      nm[i][j]= mn[j][i]; 
     } 
     System.out.println(nm[i][j]+ " " + "/n"); 
    } 
}