Grundlegende Java-Bibliothek, Suchmethode funktioniert nicht

Question

Als Teil meiner Odysee auf Java zu lernen, haben wir eine Aufgabe, in der wir das Klassenbuch und die Klassenbibliothek implementieren müssen. Sie sollten bis zu 10 Bücher in einer Bibliothek speichern, außerdem sollten Sie die Möglichkeit haben, in der Bibliothek nach Büchern zu suchen. Jetzt ist mein Problem, dass meine Suchmethode nicht funktioniert. Vielleicht hat jemand hier eine Idee hat, das ist mein Code:

public class Library { 
    int Capacity = 10; 
    int Volume = 0; 
    Book[] storage = new Book[10]; 

    public Library() { 
     System.out.println("Hello, I am a library, which can store up to 10 books!"); 
     this.storage = new Book[10]; 
    } 

    public void add(Book book) { 
     if (Volume < Capacity) { 
      this.storage[Volume] = book; 
      System.out.println("I added the book " + book + "."); 
      Volume++; 
     } else if (Volume >= Capacity) System.out.println("The library is full!"); 

    } 

    public Book search(String title) { 
     String result = new String(); 
     for (int i = 0; i < this.Volume; i++) { 
      if (title.equals(this.storage[i].toString())) { 
       System.out.println("The book with the title " + title + " exists in the library!"); 
       result = this.storage[i].toString(); 
      } else { 
       System.out.println("The book with the title " + title + " does not exist in the library!"); 
       return null; 
      } 
     } 
     Book retBook = new Book(result); 
     return retBook; 
    } 
} 


public class Book { 
    String title; 

    public Book(String title){ 
     this.title = title; 
     System.out.println("Book " + title + " created.");} 

    public String toString(){ 
     return this.title; 
    }; 
} 

Vielen Dank für jede Hilfe!

Answer 1

Sie müssen das Buch zurück, wenn es gefunden wird, und null nur dann, wenn keine gefunden wird:

public Book search(String title) { 
    for (int i = 0; i < this.Volume; i++) { 
     if (title.equals(this.storage[i].toString())) { 
      System.out.println("The book with the title " + title + " exists in the library!"); 
      return this.storage[i]; 
     } 
    } 
    System.out.println("The book with the title " + title + " does not exist in the library!"); 
    return null; 
} 

Answer 2

Ihr Problem ist hier:

for (int i = 0; i < this.Volume; i++) { 
     ... 
     } else { 
      System.out.println("The book with the title " + title + " does not exist in the library!"); 
      // *** THIS LINE IS WRONG *** 
      return null; 
     } 

Hier können Sie eine Schleife durch alle Ihre Bücher versuchen, das zu finden, das passt (das ist die Schleife, was ist, nicht wahr?). Leider, was das tatsächlich tut, ist es null auf dem ersten Buch zurückgegeben, das nicht übereinstimmt.

Sie brauchen etwas wie folgt aus:

public Book search(String title) { 
    Book found = null; 
    for (int i = 0; i < this.Volume && book == null; i++) { 
     if (title.equals(this.storage[i].toString())) { 
      System.out.println("The book with the title " + title + " exists in the library!"); 
      found = this.storage[i]; 
     } 
    } 
    // Check if we found it AFTER the loop completes. 
    if (found == null) { 
     System.out.println("The book with the title " + title + " does not exist in the library!"); 
    } 
    return found; 
} 

Hinweis hier, wie wir überprüfen, ob wir das Buch nach gefunden haben wir auf all die Bücher geschaut haben (oder wir es gefunden).