ArrayList添加元素后循环失效及removeMovie()返回null问题求助

嘿，我来帮你排查这两个头疼的问题！先聚焦你提到的两个核心痛点：removeMovie()始终返回null，还有ArrayList添加元素后循环功能异常。咱们一个个来捋清楚：

Java里的List.remove()方法（不管是按对象还是按索引），如果找不到目标元素，都会返回null（针对对象类型）。出现这个问题，大概率是以下两个原因之一：

• Movie类没正确重写equals()和hashCode()方法：
  List判断两个对象是否相等，默认是比较对象的内存引用，而不是你关心的电影属性（比如ID、片名）。如果你是通过新建一个Movie对象（比如new Movie("星际穿越")）去调用remove()，哪怕属性和集合里的电影一样，因为引用不同，List会认为找不到这个元素，直接返回null。
  解决办法：给Movie类重写这两个方法，基于电影的唯一标识（比如ID+片名，确保不会重复）。示例代码：

  import java.util.Objects;
  
  public class Movie {
      private String id;
      private String title;
      // 其他属性、构造器、getter/setter
  
      @Override
      public boolean equals(Object o) {
          if (this == o) return true;
          if (o == null || getClass() != o.getClass()) return false;
          Movie movie = (Movie) o;
          return Objects.equals(id, movie.id) && Objects.equals(title, movie.title);
      }
  
      @Override
      public int hashCode() {
          return Objects.hash(id, title);
      }
  }
  

• removeMovie()的实现逻辑有问题：
  如果你是自己遍历集合查找元素，可能遍历逻辑出错了（比如条件判断写错，或者没正确移除元素）。比如正确的按ID移除的实现应该是这样：

  public Movie removeMovie(String targetMovieId) {
      Iterator<Movie> iterator = moviesAvailable.iterator();
      while (iterator.hasNext()) {
          Movie currentMovie = iterator.next();
          if (targetMovieId.equals(currentMovie.getId())) {
              iterator.remove(); // 用Iterator的remove方法避免索引混乱
              return currentMovie; // 返回被成功移除的对象
          }
      }
      return null; // 确实找不到目标电影才返回null
  }
  

  如果你之前是直接用moviesAvailable.remove(targetMovie)，但没重写equals，就会出现找不到元素返回null的情况。

这个问题常见的原因也有几个，你可以逐一排查：

• 添加元素的代码没被执行到：
  比如你把add操作放在了某个条件分支里（比如if判断），但条件没满足，导致元素根本没被加到集合里。可以在add之后加一句System.out.println("当前可用电影数量：" + moviesAvailable.size())，确认集合大小是不是符合预期。

• 循环的集合不对：
  比如你往moviesAvailable里加了元素，但循环的是moviesRented集合，那自然看不到新添加的元素。仔细检查循环的变量名是不是和添加元素的集合一致。

• 循环方式导致的问题：
  如果你在循环中修改了集合（比如add/remove元素），用普通的for循环（按索引遍历）会导致索引混乱，出现漏循环或者数组越界的情况。这种情况下建议用Iterator遍历，或者用增强for循环（只读遍历，不能修改集合）：

  // 只读遍历，适合查看元素
  for (Movie movie : moviesAvailable) {
      System.out.println("当前电影：" + movie.getTitle());
  }
  
  // 支持修改集合的遍历（比如删除元素）
  Iterator<Movie> iter = moviesAvailable.iterator();
  while (iter.hasNext()) {
      Movie m = iter.next();
      // 如果需要删除：iter.remove();
      System.out.println("当前电影：" + m.getTitle());
  }
  

如果能提供完整的removeMovie()代码和循环相关的代码片段，我能更精准地帮你定位问题，但先按上面的常见原因排查，应该能解决大部分问题~

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