オブジェクトにArrayListを渡す

Question

こんにちは私はこのプログラムに問題があります。学生情報をタイプStudentのオブジェクトに格納する必要があります。保存情報：ラストネーム、グレード、および投票。投票はArrayListのIntegerに格納されています。 Studentという新しいオブジェクトを作成するたびにStudentというタイプのArrayListに追加すると（それは学校のすべての生徒を保存します）、既に入力した前のStudentオブジェクトに入力した新しい投票を追加し続けますArrayListに格納されます。

例：私は学生ArrayListにStudentを追加し、私は入力フレディ、5b及び123に与え、その後、私は学生ArrayListをチェックし、それは私がすでに追加されている学生含まれています：フレディ、5b及び123を、私は別のものを追加します私は入力ジョシュ、4tと1234を与える私はチェック

ArrayListで既に作成され、格納されているオブジェクトを変更しますか？どうすれば修正できますか？ここで

はコードです：

public class Student { 
    private String lastname; 
    private String grade; // example "4b" 
    private ArrayList<Integer> student_votes; 


    public Student(String lastname, String grade, ArrayList<Integer> student_votes) { 
     this.lastname=lastname; 
     this.grade=grade; 
     this.student_votes=student_votes; 
    } 

    public ArrayList getVotes() { 
     return student_votes; 
    } 

    public String getLastname() { 
     return lastname; 
    } 

    public String getGrade() { 
     return grade; 
    } 

    public String toString() { 
     return lastname+" "+grade+" "+getVotes(); 
    } 

    public void print_student (Student student) { 
     System.out.println(student); 
    } 

    public static void print_students(ArrayList<Student> students) { 
     for(Student s : students) { 
      System.out.print(s); 
     } 
     System.out.println(""); 
    } 

    public static void menu() { 
     System.out.println("\nPress 1 to add a student\nPress 2 to remove a student\nPress 3 to print the classroom\nPress 4 to exit"); 
    } 

    public static void main(String[] args) { 
     int choice, nv=0, i=0,average=0; 
     Boolean exit=false; 
     ArrayList<Student> students = new ArrayList<Student>(); 
     ArrayList<Integer> votes = new ArrayList<Integer>(); 
     String lastname = new String(); 
     String grade = new String(); 

     Scanner sc = new Scanner(System.in); 
     Scanner st = new Scanner(System.in); 
     do { 
      menu(); 
      choice=sc.nextInt(); 

      switch (choice) { 
       case 1: System.out.println("Enter your lastname:"); 
         lastname=st.nextLine(); 
         System.out.println("Enter your grade:"); 
         grade=st.nextLine(); 

         System.out.println("Enter the amount of votes"); 
         nv=sc.nextInt(); 



         for(i=0;i<nv;i++) { 
          System.out.println("Enter vote n:"+(i+1));     
          votes.add(sc.nextInt()); 
         } 

         students.add(new Student(lastname,grade,votes)); 
         System.out.println("student added!"); 

         break; 

       case 2: System.out.println("Enter student position: "); 
         nv = sc.nextInt(); 
         students.remove(nv-1); 
         break; 

       case 3: print_students(students); 
         break; 

       case 4: exit = true; 

      } 
     } while (exit==false); 
    } 
} 

Answer 1

あなたはすべての生徒の投票に同じArrayListを使用しています。すべての生徒が同じ票で同じ票を持っています（student-1にリストを追加すると2票と言います）。学生2と3票、その後両方の学生が今5票を獲得する）、次のように変更してください。

votes = new ArrayList<Integer>(); 
for(i=0;i<nv;i++) 
{ 
    System.out.println("Enter vote n:"+(i+1)); 
    votes.add(sc.nextInt()); 
} 

また、必要ではない2台のスキャナを使用しています.1台で十分です。

Answer 2

あなたは、各Studentに同じArrayListオブジェクトを使用しています。 Studentにはリストのコピーが含まれず、同じリストへのポインタが含まれています。任意の1つのポインタを介してリストに追加し続けると、そのようなすべてのポインタが影響を受けます。

生徒ごとにnew ArrayListを作成し、それをコンストラクタに渡す必要があります。

Answer 3

問題は、投票を入力するロジックにあります.1つのArrayListを使用して、すべての投票を収集します。これは、すべての学生に与えられます。

オブジェクトを渡すと、オブジェクトはコピーされませんが、オブジェクトへの参照が渡されます。ロジックでは、すべての生徒が、メインのvotes ArrayListと同じ参照を持ちます。これは、簡単に各学生のための新しいのArrayListを作成することによって解決することができます：

System.out.println("Enter the amount of votes"); 
nv=sc.nextInt(); 
final ArrayList<Integer> votes = new ArrayList<>(); 
for(i=0;i<nv;i++) 
{ 
    System.out.println("Enter vote n:"+(i+1)); 
    votes.add(sc.nextInt()); 
} 

これはあなたのためにそれを修正しますが、本当に量票を知っているだろうと、私の意見でのArrayListを使用しない点はありません。 コードはこれになりますが、学生は配列を受け入れるように変更する必要があります。

System.out.println("Enter the amount of votes"); 
nv=sc.nextInt(); 
final int[] votes = new int[nv]; 
for(i=0;i<nv;i++) 
{ 
    System.out.println("Enter vote n:"+(i+1)); 
    votes[i] = sc.nextInt(); 
}