如何避免 java.util.ConcurrentModificationException

Question

我在这个应用程序中使用 Hibernate。 我正在尝试将数据从数据库调用到 jTable。 当数据库为空时，代码正在编译但是当我将数据添加到 mysql 表程序时抛出 java.util.ConcurrentModificationException。

 public FrmMain() {
    initialize();
    getData();
 }

 public void getData() {
    
    model = (DefaultTableModel) tblBookList.getModel();
    
    List<Book> books = new ArrayList<>(bookManager.getBook());
    
    ListIterator<Book> listIterator = books.listIterator();
      
    while(listIterator.hasNext()) {
      
      Book book = listIterator.next();
      
      Object[] row = { book.getName(), book.getAuthor(), book.getPublisher(),
      book.getPage(), book.getTranslator(), book.getPublishYear(), book.getType()
      };
      
      model.addRow(row);
      
    }
    
}

我的数据访问代码。 也许问题就在这里

public class MySqlBookDal implements IBookDal {

SessionFactory factory = new Configuration()
        .configure("mysqlHibernate.cfg.xml")
        .addAnnotatedClass(Book.class)
        .buildSessionFactory();

Session session = factory.getCurrentSession(); 

public List<Book> select() {
    
    List<Book> books = new CopyOnWriteArrayList<Book>();
    
    try {
        
        session.beginTransaction();
        
        books = session.createQuery("from Book").getResultList();
        
        for(Book book: books) {
            books.add(book);
        }
        
        session.getTransaction().commit();
        
    } finally {
        factory.close();
    }
    
    return books;
}

有人可以帮助我吗？ 堆栈跟踪：

ERROR: Connection leak detected: there are 1 unclosed connections upon 

shutting down pool jdbc:mysql://localhost:3306/book?useUnicode=true&useLegacyDatetimeCode=false&serverTimezone=Turkey
java.util.ConcurrentModificationException
    at java.base/java.util.ArrayList$Itr.checkForComodification(ArrayList.java:1013)
    at java.base/java.util.ArrayList$Itr.next(ArrayList.java:967)
    at com.github.bookManagementSystem.dataAccess.MySqlBookDal.select(MySqlBookDal.java:32)
    at com.github.bookManagementSystem.business.BookManager.getBook(BookManager.java:20)
    at com.github.bookManagementSystem.FrmMain.getData(FrmMain.java:98)
    at com.github.bookManagementSystem.FrmMain.<init>(FrmMain.java:90)
    at com.github.bookManagementSystem.FrmMain$1.run(FrmMain.java:76)
    at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:316)
    at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:770)
    at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
    at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:715)
    at java.base/java.security.AccessController.doPrivileged(AccessController.java:391)
    at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
    at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:740)
    at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
    at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
    at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
    at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
    at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
    at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

Answer 1

您正在用另一个列表替换books的值，然后循环尝试将新列表books附加到自身中 - 因此出现 ConcurrentModificationException。

假设 getResultList() 返回一个 List ，您需要做的就是将结果直接附加到书籍而无需重新分配：

books.addAll(session.createQuery("from Book").getResultList());

// books = session.createQuery("from Book").getResultList();
// for(Book book: books) {
//         books.add(book);
// }

Answer 2

尝试使用CopyOnWriteArrayList而不是ArrayList 。 这将允许并发修改。 但是请注意，如果您经常更改列表，这不是很有效。 但是，如果您主要是阅读，那应该没问题。

Answer 3

抛出ConcurrentModificationException是因为您将元素添加到正在迭代的同一个列表中。 这里：

for(Book book : books) {
    books.add(book);
}

这段代码是错误的。

您可以返回 Hibernate 返回的List实例，也可以创建一个新列表。

您不需要使用CopyOnWriteArrayList ！ 这是一个非常昂贵的数据结构。 您的错误与同时修改和访问列表的不同线程无关。 此外，您不需要针对数据库启动显式事务，因为您只是在阅读。 整个select方法可以完美地简化为：

public List<Book> select() {
    return new ArrayList<>(session.createQuery("from Book").getResultList());
}

你也不需要关闭factory 。

Answer 4

@DuncG 可以解决您当前的问题。

假设您通过 start.spring.io 下载所需的依赖项，您的代码可能会使用 Spring Boot 简化为以下内容：

import org.springframework.data.jpa.repository.JpaRepository;
import org.springframework.stereotype.Repository;

@Repository
public interface BookRepository extends JpaRepository<Book, Long> {
}

在您的服务类中：

@Autowired //or use constructor injection
BookRepository BookRepository;

public List<Book> findAll() {
    return bookRepository.findAll();
}

JpaRepository继承了findAll()方法以及 CRUD 存储库的其他非常常见的行为。 有关更多详细信息，请参阅文档。