Week3. DirectorsFilter

Author: SkyrocketStan

Week3/src/DirectorsFilter.java

@@ -1,5 +1,3 @@
-import java.util.Arrays;
-
 /**
  * A class for filter movies by directors
  *
@@ -15,8 +13,13 @@ public DirectorsFilter(String directors) {
 
   @Override
   public boolean satisfies(String id) {
-    String[] movieDirectors = MovieDatabase.getDirector(id).split(",");
+    String movieDirectors = MovieDatabase.getDirector(id);
     String[] filterDirectors = directors.split(",");
-    return Arrays.asList(movieDirectors).containsAll(Arrays.asList(filterDirectors));
+    for (String direcor : filterDirectors) {
+      if (movieDirectors.contains(direcor)) {
+        return true;
+      }
+    }
+    return false;
   }
 }

Week3/src/MovieRunnerWithFilters.java

@@ -46,16 +46,20 @@ public void printAverageRatings(int minimalRatings) {
     printRatingsList(averageRatings);
   }
 
-  private void printRatingsList(ArrayList<Rating> averageRatings) {
-    averageRatings.stream()
+  private void printRatingsList(ArrayList<Rating> averageRatingList) {
+    System.out.printf("Found %d movies%n", averageRatingList.size());
+    averageRatingList.stream()
         .sorted()
         .forEach(
-            rating ->
-                System.out.printf(
-                    "%-4s %s %s%n",
-                    rating.getValue(),
-                    MovieDatabase.getYear(rating.getItem()),
-                    MovieDatabase.getTitle(rating.getItem())));
+            rating -> {
+              String movieID = rating.getItem();
+              System.out.printf("%-4s %s%n", rating.getValue(), MovieDatabase.getTitle(movieID));
+              System.out.println("     Year: " + MovieDatabase.getYear(movieID));
+              System.out.println("     Time: " + MovieDatabase.getMinutes(movieID));
+              System.out.println("     Genre(s): " + MovieDatabase.getGenres(movieID));
+              System.out.println("     Director(s): " + MovieDatabase.getDirector(movieID));
+            });
+    System.out.println("-------");
   }
 
   /**
@@ -65,12 +69,8 @@ private void printRatingsList(ArrayList<Rating> averageRatings) {
    * @param year int Year of produce
    */
   public void printAverageRatingsByYear(int minimalRatings, int year) {
-    System.out.println("number of raters " + thirdRatings.getRaterSize());
-    System.out.println("number of movies " + MovieDatabase.size());
-    ArrayList<Rating> aveRating =
-        thirdRatings.getAverageRatingsByFilter(minimalRatings, new YearAfterFilter(year));
-    System.out.printf("found %d movies%n", aveRating.size());
-    printRatingsList(aveRating);
+    printRatingsList(
+        thirdRatings.getAverageRatingsByFilter(minimalRatings, new YearAfterFilter(year)));
   }
 
   /**
@@ -80,20 +80,8 @@ public void printAverageRatingsByYear(int minimalRatings, int year) {
    * @param genre Genre
    */
   public void printAverageRatingsByGenre(int minimalRatings, String genre) {
-    ArrayList<Rating> averageRatings =
-        thirdRatings.getAverageRatingsByFilter(minimalRatings, new GenreFilter(genre));
-    System.out.printf("found %d movies%n", averageRatings.size());
-    averageRatings.stream()
-        .sorted()
-        .forEach(
-            rating -> {
-              String movieID = rating.getItem();
-              System.out.printf(
-                  "%-4s %s%n     %s%n",
-                  rating.getValue(),
-                  MovieDatabase.getTitle(movieID),
-                  MovieDatabase.getGenres(movieID));
-            });
+    printRatingsList(
+        thirdRatings.getAverageRatingsByFilter(minimalRatings, new GenreFilter(genre)));
   }
 
   /**
@@ -104,20 +92,19 @@ public void printAverageRatingsByGenre(int minimalRatings, String genre) {
    * @param maxMinutes Maximum length of movies in minutes
    */
   public void printAverageRatingsByMinutes(int minimalRatings, int minMinutes, int maxMinutes) {
-    ArrayList<Rating> averageRatings =
+    printRatingsList(
         thirdRatings.getAverageRatingsByFilter(
-            minimalRatings, new MinutesFilter(minMinutes, maxMinutes));
-    System.out.printf("found %d movies%n", averageRatings.size());
-    averageRatings.stream()
-        .sorted()
-        .forEach(
-            rating -> {
-              String movieID = rating.getItem();
-              System.out.printf(
-                  "%-4s Time: %-3d %s%n",
-                  rating.getValue(),
-                  MovieDatabase.getMinutes(movieID),
-                  MovieDatabase.getTitle(movieID));
-            });
+            minimalRatings, new MinutesFilter(minMinutes, maxMinutes)));
+  }
+
+  /**
+   * Print a list of movies and their average ratings sorted by time
+   *
+   * @param minimalRatings Minimal number of ratings
+   * @param directors directors of the movies
+   */
+  public void printAverageRatingsByDirectors(int minimalRatings, String directors) {
+    printRatingsList(
+        thirdRatings.getAverageRatingsByFilter(minimalRatings, new DirectorsFilter(directors)));
   }
 }

Week3/src/Week3.java

@@ -5,11 +5,9 @@ public static void main(String[] args) {
     //    averageRatings.forEach(System.out::println);
     MovieRunnerWithFilters filters = new MovieRunnerWithFilters();
     filters.printAverageRatingsByYear(1, 2000);
-    System.out.println("---");
     filters.printAverageRatingsByGenre(1, "Crime");
-    System.out.println("---");
     filters.printAverageRatingsByMinutes(1, 110, 170);
-    System.out.println("---");
+    filters.printAverageRatingsByDirectors(1, "Charles Chaplin,Michael Mann,Spike Jonze");
 
     //    MovieRunnerWithFilters filtersFull =
     //            new MovieRunnerWithFilters("ratedmoviesfull.csv", "ratings.csv");