Make Media abstract and add String imageUrl attribute to Media

MediaTo-SDP · Mar 9, 2023 · 53d58fc · 53d58fc
1 parent fcf9ae7
commit 53d58fc
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 6 deletions.
diff --git a/app/src/main/java/com/github/sdp/mediato/errorCheck/Preconditions.java b/app/src/main/java/com/github/sdp/mediato/errorCheck/Preconditions.java
@@ -35,10 +35,12 @@ public static void checkUser(User user){
      * @param mediaType
      * @param title
      * @param summary
+     * @param imageUrl
      */
-    public static void checkMedia(MediaType mediaType, String title, String summary){
+    public static void checkMedia(MediaType mediaType, String title, String summary, String imageUrl){
         Preconditions.checkTitle(title);
         Preconditions.checkSummary(summary);
+        Preconditions.checkImageURL(imageUrl);
         Objects.requireNonNull(mediaType);
     }
 
@@ -185,6 +187,14 @@ public static void checkSummary(String summary){
         checkNullOrEmptyString(summary, "summary");
     }
 
+    /**
+     * Checks if image url is valid
+     * @param imageUrl to be checked
+     * @TODO determine remaining criteria: format, existence etc...
+     */
+    public static void checkImageURL(String imageUrl){ checkNullOrEmptyString(imageUrl, "image url");}
+
+
     /**
      * Checks if a string is null or empty
      * @param string to be checked

diff --git a/app/src/main/java/com/github/sdp/mediato/model/media/Media.java b/app/src/main/java/com/github/sdp/mediato/model/media/Media.java
@@ -4,22 +4,27 @@
 
 import java.util.Objects;
 
-public class Media {
+public abstract class Media {
 
     private final MediaType mediaType;
     private final String title;
     private final String summary;
+    private final String imageUrl;
 
-    public Media(MediaType mediaType, String title, String summary) {
-        Preconditions.checkMedia(mediaType,title, summary);
+
+    public Media(MediaType mediaType, String title, String summary, String imageUrl) {
+        Preconditions.checkMedia(mediaType,title, summary, imageUrl);
         this.mediaType = mediaType;
         this.title = title;
         this.summary = summary;
+        this.imageUrl = imageUrl;
     }
 
     public MediaType getMediaType() {return mediaType;}
 
     public String getTitle() {return title;}
 
     public String getSummary() {return summary;}
+    public String getimageUrl() {return imageUrl;}
+
 }
diff --git a/app/src/main/java/com/github/sdp/mediato/model/media/Movie.java b/app/src/main/java/com/github/sdp/mediato/model/media/Movie.java
@@ -3,8 +3,8 @@
 import com.github.sdp.mediato.model.media.MediaType;
 
 public class Movie extends Media{
-    public Movie(MediaType mediaType, String title, String summary) {
-        super(MediaType.MOVIE, title, summary);
+    public Movie(MediaType mediaType, String title, String summary, String imageUrl) {
+        super(MediaType.MOVIE, title, summary, imageUrl);
     }
 
     @Override