ArrayList не хранит объекты

Question

Здравствуйте, у меня возникли проблемы с распечаткой элементов в моем ArrayList. Я могу распечатать его в моем методе PatronBorrow, но в PatronList и PatronReturn он ничего не печатает. Может ли кто-нибудь сказать мне, что не так с кодом? Спасибо всем большое

package proj1; 

import java.util.ArrayList; 
import java.util.List; 


public class Patron { 

private int id; 
private Book book; 
private List<Book> books; 

public Patron(int id){ 
    this.id = id; 
    books = new ArrayList<Book>(); 
} 

public int getID(){ 
    return id; 
} 

public List<Book> getBooks(){ 
    return books; 
} 

public void PatronBorrow(String b){ 
    book = new Book(b); 
    books.add(book); 
    System.out.println("Patron " + id + " has borrowed " + book.getTitle()); 
} 

public void PatronReturn(String b){ 
    for(Book book : books){ 
     if(book.getTitle().equals(b)){ 
      books.remove(book); 
      System.out.println("Patron " + id + " has borrowed " +  book.getTitle()); 
     } 
    } 
} 

public void PatronList(){ 
    for(Book b : books){ 
     System.out.println("Patron " + id + " has borrowed " + books.size() + " item(s)"); 
     System.out.println(b); 
    } 
} 

}

package proj1; 

import java.io.FileInputStream; 
import java.io.FileNotFoundException; 
import java.util.Scanner; 

public class Project1 { 

public static boolean isNumeric(String str){ 
    for(char c : str.toCharArray()){ 
     if(Character.isDigit(c)){ 
      return true; 
     } 
    } 
    return false; 
} 

public static void main(String[] args){ 

    String command; 
    String line; 
    Patron patron; 
    int patronID; 
    String title; 
    String newTitle; 
    String infile = args[0]; 

    if (args.length != 1){ 
     throw new IllegalArgumentException("Enter in file name"); 
    } 

    try{ 
     Scanner file = new Scanner(new FileInputStream(infile)); 
     while(file.hasNext()){ 
      command = file.next(); 
      if(isNumeric(command)){ 
       patronID = Integer.parseInt(command); 
       patron = new Patron(patronID); 
       command = file.next(); 
       if(command.equals("borrow")){ 
        title = file.nextLine(); 
        newTitle = title.substring(2, title.length() - 1); 
        patron.PatronBorrow(newTitle); 
       }else if(command.equals("return")){ 
        title = file.nextLine(); 
        newTitle = title.substring(2, title.length() - 1); 
        patron.PatronReturn(newTitle); 
       }else if(command.equals("list")){ 
        patron.PatronList(); 
       } 
      }else{ 

      } 
     } 

    }catch (FileNotFoundException e) { 
     System.out.println("File not found" + e.getMessage()); 
     System.exit(0); 
    } 


} 

}

Answer 1

В цикле, который использует Patron класс, вы создаете новый (пустой) Patron каждый раз.

Что вы хотите сделать, если хотите переключиться между покровителями, есть Map<Integer, Patron> patrons или тому подобное в вашей основной функции. Вместо того, чтобы создавать каждый раз new Patron(patronID), извлеките его из patrons и создайте только один (и сохраните его на карте), если его еще нет.

(Насколько ваш Patron класса, хотя, вы можете обнаружить, если вы делаете некоторые реальные испытания этого класса, который PatronReturn часто вызывает исключение при удалении книги. ConcurrentModificationException, а именно, потому что вы удаляете из списка вы повторяете в то время. Просто голова.)