Replace toString with annotations

ls1intum · Aug 28, 2024 · 8a8381b · 8a8381b
1 parent 85f9e4c
commit 8a8381b
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 29 deletions.
diff --git a/...er/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/actor/Actor.java b/...er/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/actor/Actor.java
@@ -20,13 +20,15 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.Setter;
+import lombok.ToString;
 
 @Entity
 @Table(name = "actor")
 @Getter
 @Setter
 @NoArgsConstructor
 @JsonInclude(JsonInclude.Include.NON_EMPTY)
+@ToString
 public class Actor {
     /**
      * Unique identifier for a User entity.
@@ -57,13 +59,15 @@ public class Actor {
      */
     @OneToMany(cascade = CascadeType.ALL, mappedBy = "author")
     @JsonIgnore
+    @ToString.Exclude
     private Set<Pullrequest> pullrequests = new HashSet<>();;
 
     /**
      * The Comments of the User entity.
      */
     @OneToMany(cascade = CascadeType.ALL, mappedBy = "author")
     @JsonIgnore
+    @ToString.Exclude
     private Set<Comment> comments = new HashSet<>();;
 
     public void addComment(Comment comment) {
@@ -77,11 +81,4 @@ public void addPullrequest(Pullrequest pullrequest) {
             pullrequests.add(pullrequest);
         }
     }
-
-    @Override
-    public String toString() {
-        return "Actor{id=" + id + ", login=" + login + ", email=" + email + ", url="
-                + url + ", #pullrequests="
-                + pullrequests.size() + ", #comments=" + comments.size() + "}";
-    }
 }
diff --git a/...pplication-server/src/main/java/de/tum/in/www1/hephaestus/codereview/comment/Comment.java b/...pplication-server/src/main/java/de/tum/in/www1/hephaestus/codereview/comment/Comment.java
@@ -18,13 +18,15 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.Setter;
+import lombok.ToString;
 
 @Entity
 @Table(name = "comment")
 @Getter
 @Setter
 @NoArgsConstructor
 @JsonInclude(JsonInclude.Include.NON_EMPTY)
+@ToString
 public class Comment {
     /**
      * Unique identifier for a Comment entity.
@@ -62,6 +64,7 @@ public class Comment {
      */
     @ManyToOne(fetch = FetchType.LAZY)
     @JoinColumn(name = "author_id")
+    @ToString.Exclude
     private Actor author;
 
     /**
@@ -70,18 +73,12 @@ public class Comment {
     @ManyToOne(optional = false)
     @JoinColumn(name = "pullrequest_id", referencedColumnName = "id")
     @JsonIgnore
+    @ToString.Exclude
     private Pullrequest pullrequest;
 
     public Comment(String body, String createdAt, String updatedAt) {
         this.body = body;
         this.createdAt = createdAt;
         this.updatedAt = updatedAt;
     }
-
-    @Override
-    public String toString() {
-        return "Comment{id=" + id + ", githubId=" + githubId + ", body=" + body + ", createdAt=" + createdAt
-                + ", updatedAt=" + updatedAt + ", author=" + author.getLogin() + ", pullrequest="
-                + pullrequest.getTitle() + "}";
-    }
 }
diff --git a/...on-server/src/main/java/de/tum/in/www1/hephaestus/codereview/pullrequest/Pullrequest.java b/...on-server/src/main/java/de/tum/in/www1/hephaestus/codereview/pullrequest/Pullrequest.java
@@ -23,13 +23,15 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.Setter;
+import lombok.ToString;
 
 @Entity
 @Table(name = "pullrequest")
 @Getter
 @Setter
 @NoArgsConstructor
 @JsonInclude(JsonInclude.Include.NON_EMPTY)
+@ToString
 public class Pullrequest {
 
     /**
@@ -88,9 +90,11 @@ public class Pullrequest {
      */
     @ManyToOne(fetch = FetchType.LAZY)
     @JoinColumn(name = "author_id")
+    @ToString.Exclude
     private Actor author;
 
     @OneToMany(cascade = CascadeType.ALL, mappedBy = "pullrequest")
+    @ToString.Exclude
     private Set<Comment> comments = new HashSet<>();;
 
     /**
@@ -99,14 +103,6 @@ public class Pullrequest {
     @ManyToOne(fetch = FetchType.LAZY)
     @JoinColumn(name = "repository_id", referencedColumnName = "id")
     @JsonIgnore
+    @ToString.Exclude
     private Repository repository;
-
-    @Override
-    public String toString() {
-        return "Pullrequest{id=" + id + ", githubId=" + githubId + ", title=" +
-                title + ", url=" + url + ", state="
-                + state + ", createdAt=" + createdAt + ", updatedAt=" + updatedAt + ", mergedAt=" + mergedAt
-                + ", author=" + author.getLogin() + ", #comments=" + comments.size() + ", repository=" +
-                repository.getNameWithOwner() + "}";
-    }
 }
diff --git a/...tion-server/src/main/java/de/tum/in/www1/hephaestus/codereview/repository/Repository.java b/...tion-server/src/main/java/de/tum/in/www1/hephaestus/codereview/repository/Repository.java
@@ -17,12 +17,14 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.Setter;
+import lombok.ToString;
 
 @Entity
 @Table(name = "repository")
 @Getter
 @Setter
 @NoArgsConstructor
+@ToString
 public class Repository {
 
     /**
@@ -48,14 +50,9 @@ public class Repository {
     private String url;
 
     @OneToMany(cascade = CascadeType.ALL, mappedBy = "repository", fetch = FetchType.EAGER)
+    @ToString.Exclude
     private Set<Pullrequest> pullRequests = new HashSet<>();;
 
     @Column(name = "added_at")
     private Instant addedAt;
-
-    public String toString() {
-        return "Repository{id=" + id + ", name=" + name + ", nameWithOwner=" + nameWithOwner + ", description="
-                + description + ", url=" + url + ", #pullRequests=" + pullRequests.size() + ", addedAt=" + addedAt
-                + "}";
-    }
 }