2017-02-24 40 views
0

我的数据作业&算法类告诉我们创建一个系统任务工作顺序程序。它要求我们执行各种操作,比如向数组添加一个新的SystemTask对象,删除任务并列出/打印已排序的对象数组。JAVA:Object Array只打印最后一个元素

我已经完成了大部分编码工作,但是我一直在打印出我的对象数组。看来无论我添加到对象数组中的最后一个数据是什么,它都会成为我数组中所有元素的数据。

一些指令:

  • 我的工作订单()构造函数必须将4默认SystemTask对象在数组中。
  • listSystemTask()方法列出数组中所有SystemTask对象(按顺序),前面有一个数字。

注意:请不要告诉我使用List或任何先行java 方法。我的家庭作业有其局限性。

SystemTask类

public class SystemTask{ 

    private static String month, message; 
    private static int day, hour, minute; 

    //constructor methods 
    public SystemTask(){ 
    month = "Jan"; 
    day = 1; 
    hour = 00; 
    minute = 00; 
    message = "Task Here"; 
    } 

    public SystemTask(String mon, int d, int h, int mnt, String msg){ 
    month = mon; 
    day = d; 
    hour = h; 
    minute = mnt; 
    message = msg; 

    /*setMonth(mon); 
    setDay(d); 
    setHour(h); 
    setMinute(mnt); 
    setMessage(msg);*/ 
    } 

    //get and set methods 
    public static String getMonth(){ 
    return month; 
    } 
    public static int getDay(){ 
    return day; 
    } 
    public static int getHour(){ 
    return hour; 
    } 
    public static int getMinute(){ 
    return minute; 
    } 
    public static String getMessage(){ 
    return message; 
    } 
    public static void setMonth (String mon){ 
    if(mon.length()!= 3) 
     System.out.println("Invalid 3 Letter Code. Try Again."); 
    else 
     month = mon; 
    } 
    public static void setDay(int d){ 
    if(d<1 || d>31) 
     System.out.println("Invalid Day. Try Again."); 
    else 
     day = d; 
    } 
    public static void setHour(int h){ 
    if(h<0 || h>23) 
     System.out.println("Invalid Hour. Try Again."); 
    else 
     hour = h; 
    } 
    public static void setMinute(int mnt){ 
    if(mnt<0 || mnt>59) 
     System.out.println("Invalid Minute. Try Again."); 
    else 
     minute = mnt; 
    } 
    public static void setMessage(String msg){ 
    if(msg.length()<1 || msg.length()>40) 
     System.out.println("Invalid Message. Try Again."); 
    else 
     message = msg; 
    } 

    @Override 
    public String toString(){ 
    return getMonth()+" "+getDay()+", "+String.format("%02d",getHour()) 
      +":"+String.format("%02d",getMinute())+" "+getMessage(); 
    } 
} 

工单类

public class WorkOrders { 

private final SystemTask[] ary = new SystemTask[20]; //declare private 20 objects array 
//public static int index, empty; 

public static void main(String args[]){ 
    WorkOrders object = new WorkOrders();  //create a WorkOrders object 
    object.run();        //call a run method 
} 

public WorkOrders(){ 
    // for(int i = 0;i<ary.length;i++){ 
    //  ary[i] = new SystemTask(); 
    // } 

    /*SystemTask t1 = new SystemTask("Mar", 4, 21, 30, "Backup Users"); 
    ary[0] = t1; 
    SystemTask t2 = new SystemTask("Apr", 1, 17, 0, "Upgrade Hard Drives"); 
    ary[1] = t2; 
    SystemTask t3 = new SystemTask("May", 6, 10, 45, "Virus Scan"); 
    ary[2] = t3; 
    SystemTask t4 = new SystemTask("Jun", 3, 9, 15, "Database Backup"); 
    ary[3] = t4;*/ 

    ary[0] = new SystemTask("Mar", 4, 21, 30, "Backup Users"); 
    ary[1] = new SystemTask("Apr", 1, 17, 0, "Upgrade Hard Drives"); 
    ary[2] = new SystemTask("May", 6, 10, 45, "Virus Scan"); 
    ary[3] = new SystemTask("Jun", 3, 9, 15, "Database Backup"); 
} 

public void run(){ 
    char choice; 
    do{ 
     System.out.println("SYSTEM WORKORDER PROGRAM:"); 
     System.out.println("A)dd SystemTask"); 
     System.out.println("D)elete SystemTask"); 
     System.out.println("L)ist SystemTask"); 
     System.out.println("E)xit"); 
     System.out.print("Select an option: "); 
     choice = UserInput.getChar(); 

     switch (choice) { 
      case 'a': 
      case 'A': 
       //addSystemTask(); 
       break; 
      case 'd': 
      case 'D': 
       //deleteSystemTask(); 
       break; 
      case 'l': 
      case 'L': 
       listSystemTask(); 
       break; 
      case 'e': 
      case 'E': 
       System.out.println("You quit the program."); 
       System.out.println("\nThanks for using System Workorder!"); 
       System.exit(0); 
       break; 
      default: 
       System.out.println("\'" + choice + "\' does not exist. Try    again."); 
       break; 
     } 
     System.out.println(); 
    }while(choice !='E'|| choice !='e'); 
} 

public void listSystemTask(){ 
    System.out.println("\nALL LISTED TASKS"); 
    for(int c=0;c<ary.length;c++){ 
     if(ary[c]!=null) 
      System.out.println(c+1 + ": " + ary[c].toString()); 
    } 
}  

还有一种UserInput类,我做了。我没有包括它,因为它仅仅用于我的键盘输入。我省略了一些与我的问题无关的代码/方法。我只关心打印数组中的对象。

这里的问题是,这是我所得到的输出,当我打印的清单:

SYSTEM WORKORDER PROGRAM: 
A)dd SystemTask 
D)elete SystemTask 
L)ist SystemTask 
E)xit 
Select an option: L 

ALL LISTED TASKS 
1: Jun 3, 09:15 Database Backup 
2: Jun 3, 09:15 Database Backup 
3: Jun 3, 09:15 Database Backup 
4: Jun 3, 09:15 Database Backup 

我不知道哪里出了问题的来源。我怀疑它是在构造函数中初始化数组,但我似乎无法修复它。或者也许在其他地方?

+0

的可能的复制[为什么我的ArrayList中包含的最后一个项目的N份添加到列表中?](http://stackoverflow.com/questions/19843506/why-does-my-arraylist-contain- n-copies-of-the-last-item-added-to-the-list) –

+0

顺便说一下,你不能将对象添加到数组中。数组有固定大小。所以,你宁愿“设置”到阵列 –

回答

3

的问题是与SystemTask

public class SystemTask { 
    private static String month, message; 
    private static int day, hour, minute; 

您已声明的所有成员变量为static。将会有多个SystemTask的实例,但它们都将它们的值存储在相同的static字段中。它正在打印“最后一个SystemTask”,因为它是最后一个要创建的,覆盖前一个设置的值。

只需删除该static声明...

public class SystemTask { 
    private String month, message; 
    private int day, hour, minute; 

您还定义了所有的访问方法为static。从这些中删除static,以允许它们访问对象作用域变量。

public static String getMonth() { 
    return month; 
    } 

成为...

public String getMonth() { 
    return month; 
    } 
+0

非常感谢。这工作。我想我需要学习更多的时候使用'static'关键字。 –

+0

@DavidDredddeJesus使用静态的一般规则是非常RAIRLY :)他们创建了被认为是反模式的“单身人士”。如果你认为你需要一个单身人士,通常有办法避免它们,或者减少你实际需要的人数。网络上有很多关于单身人士的争论和利弊。 – slipperyseal