From ad6f136e0fcd7e57df926d6a18adc6ec76edd6b0 Mon Sep 17 00:00:00 2001 From: bennyLCK Date: Mon, 15 Apr 2024 17:44:08 +0800 Subject: [PATCH] Delete all dead code in classes --- .../model/article/UniqueArticleList.java | 31 ------------------- .../person/NameWithinArticlePredicate.java | 4 --- 2 files changed, 35 deletions(-) diff --git a/src/main/java/seedu/address/model/article/UniqueArticleList.java b/src/main/java/seedu/address/model/article/UniqueArticleList.java index 980dcd3e9d9..66a90d61ea4 100644 --- a/src/main/java/seedu/address/model/article/UniqueArticleList.java +++ b/src/main/java/seedu/address/model/article/UniqueArticleList.java @@ -35,14 +35,6 @@ public boolean contains(Article toCheck) { */ public void add(Article toAdd) { requireNonNull(toAdd); - - /* - if (contains(toAdd)) { - throw new DuplicatePersonException(); - } - - */ - internalList.add(toAdd); } @@ -55,16 +47,6 @@ public void setArticle(Article target, Article editedArticle) { int index = internalList.indexOf(target); - /* - if (index == -1) { - throw new PersonNotFoundException(); - } - - if (!target.isSameArticle(editedArticle) && contains(editedArticle)) { - throw new DuplicatePersonException(); - } - */ - internalList.set(index, editedArticle); } @@ -74,12 +56,6 @@ public void setArticle(Article target, Article editedArticle) { public void remove(Article toRemove) { requireNonNull(toRemove); internalList.remove(toRemove); - - /* - if (!internalList.remove(toRemove)) { - throw new PersonNotFoundException(); - } - */ } public void setArticles(UniqueArticleList replacement) { @@ -93,13 +69,6 @@ public void setArticles(UniqueArticleList replacement) { */ public void setArticles(List
articles) { requireAllNonNull(articles); - - /* - if (!personsAreUnique(persons)) { - throw new DuplicatePersonException(); - } - */ - internalList.setAll(articles); } diff --git a/src/main/java/seedu/address/model/person/NameWithinArticlePredicate.java b/src/main/java/seedu/address/model/person/NameWithinArticlePredicate.java index ac67e2d2023..af60394584a 100644 --- a/src/main/java/seedu/address/model/person/NameWithinArticlePredicate.java +++ b/src/main/java/seedu/address/model/person/NameWithinArticlePredicate.java @@ -22,10 +22,6 @@ public NameWithinArticlePredicate(Article article) { @Override public boolean test(Person person) { - /*return article.getAuthors().stream() - .anyMatch(author -> person.getName().fullName.equals(author.authorName)) || - article.getSources().stream() - .anyMatch(source -> person.getName().fullName.equals(source.sourceName));*/ return article.getPersons().contains(person); }