ArrayList中最后添加的对象覆盖所有元素，已排查非静态字段及对象创建

嘿，我之前也踩过几乎一模一样的坑！你说每次循环都新建对象但还是被覆盖，结合你提到的国际象棋框架里的protected字段，大概率是你新建的对象其实共享了同一个内部引用类型的实例——哪怕外层对象是新的，里面的核心字段（比如board）指向的还是同一个框架实例，最后一次修改自然会覆盖所有列表元素的状态。

可能的核心原因 #

• 框架实例被复用而非重建：你可能在循环中一直修改同一个Chessboard（或Move）实例，然后把它赋值给新创建的自定义对象。虽然外层对象是new出来的，但它们的board字段都指向同一个底层框架实例，最后一次修改会同步影响所有列表元素。
• 构造逻辑的隐性共享：如果你的自定义对象构造函数里，不小心引用了循环外定义的全局/外层作用域变量，也会导致所有新对象共享同一个实例。比如：

  // 错误示例：循环外复用框架实例
  Chessboard sharedBoard = Chessboard.getChess(fen);
  while(...) {
      MyChessMove move = new MyChessMove();
      move.setBoard(sharedBoard); // 所有move的board都指向同一个实例
      sharedBoard.makeMove(...); // 修改会影响所有列表里的对象
      list.add(move);
  }
  

具体解决办法 #

1. 每次循环创建框架实例的深拷贝
   优先看看框架是否提供了复制实例的方法，比如clone()或复制构造函数：

   // 正确示例：每次循环新建框架实例
   while(...) {
       // 复制现有棋盘状态，创建全新实例
       Chessboard newBoard = new Chessboard(currentBoard); 
       // 或者 Chessboard newBoard = currentBoard.clone();
       
       MyChessMove move = new MyChessMove();
       move.setBoard(newBoard);
       newBoard.makeMove(...); // 修改的是独立的新实例
       list.add(move);
   }
   

   如果框架没提供深拷贝方法，因为字段是protected，你可以通过子类继承来手动复制字段：

   public class MyChessboard extends Chessboard {
       public MyChessboard(Chessboard original) {
           // 手动复制protected字段（注意引用类型字段需要深拷贝）
           this.move = original.move; // 如果move是不可变类型可直接赋值，否则也要拷贝
           this.board = deepCopyBoard(original.board); // 自己实现棋盘数组的深拷贝
           this.color = original.color;
       }
       
       private int[][] deepCopyBoard(int[][] original) {
           int[][] copy = new int[original.length][];
           for(int i=0; i<original.length; i++) {
               copy[i] = Arrays.copyOf(original[i], original[i].length);
           }
           return copy;
       }
   }
   

2. 验证实例独立性
   循环中添加对象后，打印对象及其内部字段的hashCode，确认是否共享实例：

   list.add(move);
   System.out.println("Move实例哈希: " + move.hashCode() 
                     + " | 内部Board哈希: " + move.getBoard().hashCode());
   

   如果所有Board的哈希值都相同，那百分百是共享了同一个实例，重点解决这个问题即可。

3. 确保所有状态字段在循环内初始化
   检查自定义对象的所有引用类型成员，确保它们都是在循环内部创建或复制的，避免使用循环外的全局变量。

内容的提问来源于stack exchange，提问作者randomdude