java - if-else 在 for-each 循环中

Question

我是编程新手。我有这段代码：

public void findFilm ( String nameofFilm ) {  
   int index = -1;
   int i = 0;
   for( Film aFilm : list ) {  
      if( aFilm.gettitle().equals( nameofFilm )) {
         index = i;
         i++;
         break ;
      } 
   }
   if( index >= 0 ) {
      aFilm.print();
   }
   else {
      System.out.println(
         "The film " + nameofFilm + " does not belong to the collection" );
      } 
   }

更具体地说，我想搜索列表并查找它是否包含与我插入的名称相同的电影。有任何想法吗 ？非常感谢。

Answer 1

aFilm仅在 for 循环中是本地的，您需要在 for 循环中打印它。

public void findFilm ( String nameofFilm ) {  
   for( Film aFilm : list ) {  
      if( aFilm.gettitle().equals(nameofFilm) ) {
         aFilm.print();
         return;
      } 
   }
   System.out.println(
      "The film " + nameofFilm + " does not belong to the collection");
}

Answer 2

该aFilm变量在 for 循环中定义。因此，它的范围仅限于循环，并且变量不能在循环之外使用。你可以这样重写循环：

Film foundFilm = null;
for (Film aFilm : films) {
    if (aFilm.getTitle().equals(nameOfFilm)) {
        foundFilm = aFilm;
        break;
    }
}
if (foundFilm == null) {
    ...
}
else {
    ...
}

不需要任何索引或递增计数器。

而且，为了使代码更简洁，您可以将第一部分提取到它自己的方法中，然后像这样重写它：

Film foundFilm = findFilmByTitle(films, filmTitle);
if (foundFilm == null) {
    ...
}
else {
    ...
}

Answer 3

您可以选择一种功能方法，例如通过Guava：

public boolean containsFilm(List<Film> list, final String nameOfFilm) {
    Film film = Iterables.find(list, new Predicate<Film>() {
        @Override
        public boolean apply(Film film) {
            return film.getTitle().equals(nameOfFilm);
        }
    }, null);
    return film != null;
}

此外，我建议防御性地检查null（电影、标题等）。我已经离开检查以保持示例简单。

Answer 4

使用像地图这样的数据索引：

SortedMap< String, Film > filmsByTitle = new TreeMap<>();
...
if( ! filmsByTitle.contains( newFilm.getTitle()) {
   filmsByTitle.put( newFilm.getTitle(), newFilm );
}