Merge pull request #44 from ShengBin-101/shengbin-Test-v1.0

Fix handler to delete loans associated to book to be deleted

Author: ShengBin-101

src/main/java/bookkeeper/InputHandler.java

@@ -1,6 +1,7 @@
 package bookkeeper;
 
 import java.util.logging.Logger;
+
 import bookkeeper.exceptions.BookNotFoundException;
 import bookkeeper.exceptions.IncorrectFormatException;
 
@@ -17,6 +18,7 @@ public InputHandler() {
         this.loanList = new LoanList("Loan List");
         logger.info("InputHandler initialized");
     }
+
     public void askInput() {
         boolean isAskingInput = true;
         String userInputLine;
@@ -113,7 +115,15 @@ private void addBook(String[] commandArgs) throws IncorrectFormatException {
         }
         String[] bookArgs = InputParser.extractAddBookArgs(commandArgs[1]);
         assert bookArgs.length == 4 : "Book arguments should contain exactly 4 elements";
-        assert bookArgs[0] != null && !bookArgs[0].isEmpty() : "Book title cannot be null or empty";
+
+        // Trim whitespaces from the book title
+        String bookTitle = bookArgs[0].trim();
+
+        // Check if book already exists in the inventory
+        if (bookList.findBookByTitle(bookTitle) != null) {
+            System.out.println("Book already exists in inventory: " + bookTitle);
+            return;
+        }
 
         Book newBook = new Book(bookArgs[0], bookArgs[1], bookArgs[2], bookArgs[3]);
         bookList.addBook(newBook);
@@ -139,6 +149,7 @@ private void removeBook(String[] commandArgs) throws IncorrectFormatException, B
             System.out.println("Book not found in inventory: " + bookTitle);
         } else {
             assert toRemove.getTitle() != null : "Book to remove must have a valid title";
+            loanList.removeLoansByBook(toRemove);
             bookList.removeBook(toRemove);
             System.out.println("Removed book: " + toRemove.getTitle());
         }
@@ -147,6 +158,7 @@ private void removeBook(String[] commandArgs) throws IncorrectFormatException, B
     /**
      * Extract arguments needed to delete loan and delete loan
      * Checks if book and loan exist before deleting
+     *
      * @param commandArgs The parsed command arguments.
      * @throws IncorrectFormatException If the input format is invalid.
      * @throws BookNotFoundException    If the book is not found in the inventory.

src/main/java/bookkeeper/InputParser.java

@@ -12,11 +12,6 @@ public static String[] extractCommandArgs(String input) throws IncorrectFormatEx
         return commandArgs;
     }
 
-    public static String[] extractXXXX(String input) {
-        // Copy and Add Implementation
-        return null;
-    }
-
     public static String[] extractAddBookArgs(String input) throws IncorrectFormatException {
         String[] commandArgs = new String[4];
         String[] splitInput = input.trim().split("( a/)|( cat/)|( cond/)", 4);
@@ -93,7 +88,7 @@ public static String[] extractDeleteLoanArgs(String input) throws IncorrectForma
         for (int i = 0; i < splitInput.length; i++) {
             if (splitInput[i].isBlank()) {
                 throw new IncorrectFormatException("Invalid format for delete-loan. " +
-                        "Expected format: add-loan BOOK_TITLE n/BORROWER_NAME");
+                        "Expected format: delete-loan BOOK_TITLE n/BORROWER_NAME");
             }
             commandArgs[i] = splitInput[i].trim();
         }

src/main/java/bookkeeper/LoanList.java

@@ -58,4 +58,10 @@ public Loan findLoan(Book book, String borrower) {
         }
         return null;
     }
+
+    public void removeLoansByBook(Book book) {
+        assert book != null : "Book cannot be null";
+        loanList.removeIf(loan -> loan.getBook().equals(book));
+        logger.log(Level.INFO, "Removed all loans associated with book: {0}", book.getTitle());
+    }
 }