在java中傳遞數組中的代碼意外輸出

Question

我一直在java傳遞數組中遇到問題。代碼是：

package input_output; 

import static java.lang.System.out; 
import java.util.Scanner; 

public class InputOutput { 
    private static Scanner sc; 
    public static void main(String []args){ 
     sc = new Scanner(System.in); 
     out.print("Enter the length of arrays :\t"); 
     int n = sc.nextInt(); 
     Employee[] emp = new Employee[n]; 
     for(int i=0;i<n;i++){ 
      out.print("\nEnter name and age of " + (i+1) + " employee :\t"); 
      emp[i] = new Employee(); 
      emp[i].setName(sc.nextLine()); 
      sc.nextLine(); 
      emp[i].setAge(sc.nextInt()); 
     } 

     Operation operate = new Operation(emp,n); 

     operate.printOnScreen(); 

    } 
} 

class Operation{ 
    Employee []emp; 
    public Operation(Employee[] emp,int n){ 
     this.emp=emp; 
     for(Employee e: this.emp) 
      e = new Employee(); 
    } 

    public void printOnScreen() { 
     for(Employee e : emp){ 
      e = new Employee(); 
      out.println("Name:\t" + e.getName() + "\t\tAge:\t" + e.getAge() + "\n"); 
     } 
    } 
} 

僱員類是一個簡單的bean，包含年齡和名稱的getter和setter方法。

輸出來：

名稱：空
年齡：0

什麼是我的錯？

---

現在我已經改變了構造函數

public Operation(Employee[] emp, int n){ 
     this.emp=emp; 
} 

和刪除行：

e = new Employee(); 

從PrintOnScreen()

，輸出是：

名稱：
年齡：21

Answer 1

你不是在下面的代碼從陣列輸出數據：

public void printOnScreen() 
{ 
    for(Employee e : emp) 
    { 
     e = new Employee(); // <<< THIS IS WRONG, AS e IS ALREADY SET BY LOOP!! 
     out.println("Name:\t" + e.getName() + "\t\tAge:\t" + e.getAge() + "\n"); 
    } 
} 

此代碼創建的循環的每次迭代新e = >你不是從數組中輸出條目，而是新創建的條目。該代碼應閱讀

public void printOnScreen() 
{ 
    for(Employee e : emp) 
    { 
     out.println("Name:\t" + e.getName() + "\t\tAge:\t" + e.getAge() + "\n"); 
    } 
} 

---

此外，下面的代碼沒有意義：在this.emp

public Operation(Employee[] emp,int n){ 
    this.emp=emp; 
    for(Employee e: this.emp) 
     e = new Employee(); 
} 

爲什麼你遍歷所有的員工創建是Employee新實例存儲在哪裏？另外，不使用n參數。此代碼應閱讀：新員工的

public Operation(Employee[] emp) 
{ 
    this.emp=emp; 
} 

Answer 2

雖然打印數據，要打印的數據，而不是您創建的早期員工對象。

Answer 3

不確定，但我認爲Array不是在Java中用「=」複製內容。更改類操作

this.emp=emp; 
for(Employee e: this.emp) 
    e = new Employee(); 

到

for(int i=0; i<n; i++) { 
    this.emp[i].setName(emp.getName()); 
    this.emp[i].setAge(emp.getAge()); 
} 

然後改變

operate.printOnScreen(); 

到

operate.printOnScreen(emp); 

和

public void printOnScreen() { 
    for(Employee e : emp){ 
     e = new Employee(); 
     out.println("Name:\t" + e.getName() + "\t\tAge:\t" + e.getAge() + "\n"); 
    } 
} 

到

public void printOnScreen(Employee emp) { 
    for(Employee e : emp){ 
     out.println("Name:\t" + e.getName() + "\t\tAge:\t" + e.getAge() + "\n"); 
    } 
} 

Answer 4

你的第二個問題：（空name場）

的問題是sc.nextInt()電話。它從輸入流中讀取一個整數，但在那裏留下標記carriage return和line feed。接下來的sc.nextLine()調用將讀取這些符號，而不是其他的。這樣所有名字都填上\r\n。

您可以修復到方式：

public static void main(String []args){ 
    sc = new Scanner(System.in); 
    out.print("Enter the length of arrays :\t"); 
    int n = sc.nextInt(); 
    sc.nextLine(); // <-- new 
    Employee[] emp = new Employee[n]; 
    for(int i=0;i<n;i++){ 
     out.print("\nEnter name and age of " + (i+1) + " employee :\t"); 
     emp[i] = new Employee(); 
     emp[i].setName(sc.nextLine()); 
     //sc.nextLine(); // <-- removed 
     emp[i].setAge(sc.nextInt()); 
     sc.nextLine(); // <-- new 
    } 
    //.. 
} 

或者

public static void main(String []args){ 
    sc = new Scanner(System.in); 
    out.print("Enter the length of arrays :\t"); 
    int n = Integer.parseInt(sc.nextLine()); // <-- changed 
    Employee[] emp = new Employee[n]; 
    for(int i=0;i<n;i++){ 
     out.print("\nEnter name and age of " + (i+1) + " employee :\t"); 
     emp[i] = new Employee(); 
     emp[i].setName(sc.nextLine()); 
     //sc.nextLine(); // <-- removed 
     emp[i].setAge(Integer.parseInt(sc.nextLine())); // <-- changed 
    } 
    //.. 
} 

如果解決您的第二個問題，請接受Thorsten Dittmar小號anwser你的主要問題。