如何使用使用JOptionPane的构造函数将对象从我的方法传递到空数组中？

Question

I'm having trouble figuring out how to store the objects from the constructor. so far all I get is one object and the remaining are all null. If someone can explain it to me so that a beginner can understand that would be much appreciated.

public class Bookstore 
{
    /*
    Main Class Bookstore to be made modular 
     */
    public static void main(String args[]) 
    {

        Book catalogue[] = new Book[3];

        int select;
        do
        {
            select = bookMenu();
            switch(select)
            {
            case 1:  
                int i =0;
                if(catalogue[i] != null)
                {
                    JOptionPane.showMessageDialog(null,"Test");
                    break;
                }
                catalogue[i] = addBook();

            case 2:     
                sortBook();     
            break;
            case 3:     
                searchBook(catalogue);   
            break;
            case 4:     
                displayBook(catalogue);  
            break;
            case 5:
            break;
            }   
        }
        while(select != 5);
    }
    public static int bookMenu()
    {
        int select;
        String menuOptions = "--Book store--\n"
                + "\n1. Add book to catalogue"
                + "\n2.Sort and display books by price"
                + "\n3. Search for a book by title" 
                + "\n4. Display all books"
                + "\n\n5. Exit";
        do
        {
            select = Integer.parseInt(JOptionPane.showInputDialog(menuOptions)); 
        }
        while(select < 1 || select > 5);
        return select;  
    }
    public static Book addBook()
    {
        int isbn;
        String title, author;
        Book catalogue = null;
        double price;
        for(int i=0; i<3;i++)
        {   
            isbn = Integer.parseInt(JOptionPane.showInputDialog
                                ("Enter Book ISBN or: "));
            title = JOptionPane.showInputDialog
                                ("Enter Book Title: ");
            author = JOptionPane.showInputDialog
                                ("Enter Book Author: ");
            price = Double.parseDouble(JOptionPane.showInputDialog
                                ("Enter Book Price: "));    
            catalogue = new Book(isbn, title, author, price);    
        }
        return catalogue;
    }
    public static void sortBook()
    {

    }
    public static void searchBook(Book catalogue[])//remain void
    {
        String searchValue = JOptionPane.showInputDialog("Enter the title of the book you are searching for");
        boolean found = true;
        for(int i=0; i<catalogue.length && catalogue[i] != null ; i++)          
        {
            if(searchValue.equalsIgnoreCase(catalogue[i].getTitle()))  
            {
                JOptionPane.showMessageDialog(null, "Book details: " + catalogue[i].toString()); 
                found = true; 
            }  
        }
        if(found == false)
        JOptionPane.showMessageDialog(null, "The title does not exist in the collection ");
    }
    public static void displayBook(Book catalogue[])//remain void
    {
        String output = "";
        for(Book bk:catalogue)
        {
        output += bk + "\n";
        }        
        JOptionPane.showMessageDialog(null, output);
    }
}

Answer 1

So, this is always false...

if(count == max)

You set the values immediately before that statement. Zero never equals ten.

Some IDEs would even point that out to you.

---

If you want to get a variable that can be used between methods, you need to learn variable scoping.

For example, using a static class variable

private static Book[] books;
private static final int  MAX_BOOKS = 10;
private static int count;

public static void main(String[] args) {
    books = getBooks();
    bookMenu();
}

public static Book[] getBooks() {
    if(count == MAX_BOOKS) {
        JOptionPane.showMessageDialog(null, "Catalogue full - cannot add any more books");
    } else {
        for(; count < MAX_BOOKS; count++) {
            books[count]= addBook();
        }
    } 
    return books;
}

---

Then, if you want to repeat the menu, use a loop. Don't place it at the end of every method.

Also searchBook(Book) isn't searching for a title. You want to pass a string to the method, not a Book class