нашел хорошое решение для removeByName

erenkov · erenkov · commit a55d6f4e880c · 2020-04-10T21:02:41.000+04:00
diff --git a/src/main/java/com/epam/izh/rd/online/entity/Author.java b/src/main/java/com/epam/izh/rd/online/entity/Author.java
@@ -18,7 +18,7 @@
  * 5) Переопределить методы equals и hashCode - используйте генерацию (не забывайте alt+inset)
  * 6) Переопределить метод toString с выводом всех полей (не забывайте alt+inset)
  */
-public class Author{
+public class Author {
     private String name;
     private String lastName;
     private LocalDate birthdate;
diff --git a/src/main/java/com/epam/izh/rd/online/repository/SimpleAuthorRepository.java b/src/main/java/com/epam/izh/rd/online/repository/SimpleAuthorRepository.java
@@ -14,41 +14,17 @@ public SimpleAuthorRepository(Author[] authors) {
         this.authors = authors;
     }
 
-    /**
-     * Метод должен сохранять автора в массив authors.
-     * Массив при каждом сохранении должен увеличиваться в размере ровно на 1.
-     * То есть он ровно того размера, сколько сущностей мы в него сохранили.
-     * <p>
-     * Если на вход для сохранения приходит автор, который уже есть в массиве (сохранен), то автор не сохраняется и
-     * метод возвращает false.
-     * <p>
-     * Можно сравнивать только по полному имени (имя и фамилия), считаем, что двух авторов
-     * с одинаковыми именем и фамилией быть не может.
-     * Подсказка - можно использовать для проверки метод findByFullName.
-     * <p>
-     * Если сохранение прошло успешно, метод должен вернуть true.
-     */
     @Override
     public boolean save(Author author) {
 
         if (findByFullName(author.getName(), author.getLastName()) == null) {
-//              Author[] array = Arrays.copyOf(authors, authors.length + 1);
-//              array[array.length - 1] = author;
-//              authors = array;
-
             authors = ArrayUtils.add(authors, new Author(author));
             return true;
         }
 
         return false;
     }
 
-    /**
-     * Метод должен находить в массиве authors автора по имени и фамилии (считаем, что двух авторов
-     * с одинаковыми именем и фамилией быть не может.)
-     * <p>
-     * Если автор с таким именем и фамилией найден - возвращаем его, если же не найден, метод должен вернуть null.
-     */
     @Override
     public Author findByFullName(String name, String lastname) {
 
@@ -76,8 +52,7 @@ public Author findByFullName(String name, String lastname) {
     public boolean remove(Author author) {
 
         if (findByFullName(author.getName(), author.getLastName()) != null) {
-
-            authors = ArrayUtils.remove(authors, ArrayUtils.indexOf(authors, author));
+            authors = ArrayUtils.removeElement(authors, author);
             return true;
         }
 
diff --git a/src/main/java/com/epam/izh/rd/online/repository/SimpleSchoolBookRepository.java b/src/main/java/com/epam/izh/rd/online/repository/SimpleSchoolBookRepository.java
@@ -64,23 +64,22 @@ public boolean removeByName(String name) {
 
         boolean result = false;
         SchoolBook[] tempSchoolBook = schoolBooks;
-        int i=0;
 
-        for (SchoolBook schoolBook : schoolBooks) {
-            if (schoolBook.getName().equals(name)) {
-                i = tempSchoolBook.length;
-                tempSchoolBook = ArrayUtils.remove(schoolBooks, ArrayUtils.indexOf(schoolBooks, schoolBook));
-                result = true;
-            }
-        }
+        schoolBooks=ArrayUtils.removeElements(schoolBooks, findByName(name));
 
-        if (result) {
-            if(i==1){
-                schoolBooks = new SchoolBook[0];
-            } else {
-                schoolBooks = tempSchoolBook;
-            }
-        }
+//        for (SchoolBook schoolBook : schoolBooks) {
+//            if (schoolBook.getName().equals(name)) {
+//
+//                if (tempSchoolBook.length == 1) {
+//                    tempSchoolBook = new SchoolBook[0];
+//                } else {
+//                    tempSchoolBook = ArrayUtils.removeElement(schoolBooks, schoolBook);
+//                }
+//                result = true;
+//            }
+//        }
+
+     //   schoolBooks = tempSchoolBook;
 
         return result;
     }
