Thread: method array list

i just want to make sure I did this correctly. Does it look right?

/** a method returning an ArrayList of Books whose price is less than a given number
 * @param searchDouble
 * @return 
 */
public ArrayList<Book> searchForPrice(double searchDouble)
{
    ArrayList<Book> searchResult = new ArrayList<>();
    for(Book lessBook : library)
    {
        if((lessBook.getTitle()).indexOf((int) searchDouble)!=-1)
            searchResult.add(lessBook);
    }
    searchResult.trimToSize();
    return searchResult;
    }
}

The rest of my code for the array List:

public class BookStore 
{
private ArrayList<Book> library;
 
/**default constructor
 * instantiates ArrayList of Books
 */
public BookStore()
{
    library=new ArrayList<>();
    library.add(new Book("Intro to Java", "James", 56.99));
    library.add(new Book("Advanced Java", "Green", 65.99));
    library.add(new Book("Java Servlets", "Brown", 75.99));
    library.add(new Book("Intro to HTML", "James", 29.49));
    library.add(new Book("Intro to Flash", "James", 34.99));
    library.add(new Book("Advanced HTML", "Green", 56.99));
    library.trimToSize();
}
 
/** toString
 * @return each book in library, one per line
 */
@Override
public String toString()
{
    String result = "";
    for(Book tempBook : library)
    {
        result += tempBook.toString()+"\n";
    }
    return result;
}

yes now create a tester class so you can see for yourself

Actually - wait ? i think it should be this?

public ArrayList<Book> searchForPrice(String searchString)
{
    ArrayList<Book> searchResult = new ArrayList<>();
    //converts search string to double
    double maxPrice = Double.parseDouble(searchString);
    for(Book lessBook : library)
    {
        //If itemPrice < maxPrice, add it to the list
        if(lessBook.getPrice()< maxPrice)
            searchResult.add(lessBook);
    }
    searchResult.trimToSize();
    return searchResult;
    }

You will find it recommended to abstract the details of the implementation by using the following approach:

// declaration
List<Book> library;
 
// initialization
library = new ArrayList<Book>();

Originally Posted by GregBrannon

You will find it recommended to abstract the details of the implementation by using the following approach:

// declaration
List<Book> library;
 
// initialization
library = new ArrayList<Book>();

So instead of saying

private ArrayList<Book> library;

write it as

private List<Book> library; 
library = new ArrayList<Book>();

Yes, or it can all be done in one line (but I'd normally initialize in the constructor):

private List<Book> library = new ArrayList<Book>();

when i change that it causes the arralist trimtosize to give an error:

   library.trimToSize();
}

says can't find it

--- Update ---

what would be better to use do you think?

/** a method returning an ArrayList of Books whose price is less than a given number
 * @param searchDouble
 * @return 
 */
public ArrayList<Book> searchForPrice(double searchDouble)
{
    ArrayList<Book> searchResult = new ArrayList<>();
    for(Book lessBook : library)
    {
        if((lessBook.getTitle()).indexOf((int) searchDouble)!=-1)
            searchResult.add(lessBook);
    }
    searchResult.trimToSize();
    return searchResult;
    }

or better to use:

public ArrayList<Book> searchForPrice(String searchString)
{
    ArrayList<Book> searchResult = new ArrayList<>();
    //converts search string to double
    double maxPrice = Double.parseDouble(searchString);
    for(Book lessBook : library)
    {
        //If itemPrice < maxPrice, add it to the list
        if(lessBook.getPrice()< maxPrice)
            searchResult.add(lessBook);
    }
    searchResult.trimToSize();
    return searchResult;
    }

You're right. To use methods specific to ArrayList, those not included in List, will require casting library to an ArrayList. A bit of a pain. Which should you use? Whichever you prefer. Sometimes you just gotta step around the experts while they shake their fingers at you and get on with the job.

Your second attempt seems correct to me, though I'm not sure why the method doesn't expect a double parameter for the max book price. Does it work?

i figured double cause that's what i have for the price in the class:

package natashaepperson_chapter_09_exercise_94;
 
public class Book {
private String title;
private String author;
private double price;
 
/**default constructor
 */
public Book()
{
    title = "";
    author = "";
    price = 0.0;
}
 
/** overloaded constructor
 * @param newTitle the value to assign to title
 * @param newauthor the value to assign to author
 * @param newPrice the value to assign to price
 */
public Book(String newTitle, String newAuthor, double newPrice)
{
    title = newTitle;
    author = newAuthor;
    price = newPrice;
}
 
/** getTitle method
 * @return the title
 */
public String getTitle()
{
    return title;
}
 
/** getAuthhor method
 * @return the author
 */
public String getAuthor()
{
    return author;
}
 
/**getPrice method
 * @return the price
 */
public double getPrice()
{
    return price;
    }
 
/** toString
 * @return title, author, and price
 */
@Override
public String toString()
{
    return ("title: "+title+"\t"
            +"author: "+author+"\t"
            +"price: "+price);
    }
}

Okay now I am trying to do the client test for it.
I have

public static void main(String[]args)
{
    BookStore bs = new BookStore();
 
    String keyword = JOptionPane.showInputDialog(null,
            "Enter a keyword ");
    System.out.println("Our book collection is: ");
    System.out.println(bs.toString());
 
    ArrayList<Book> results = bs.searchForTitle(keyword);
 
    System.out.println("The search results for "+keyword
            +" are: ");
    for(Book tempBook : results)
        System.out.println(tempBook.toString());
 
    /*Enter Price of Book
     * returns books lower than price entered
     */
    String Price = JOptionPane.showInputDialog(null,
            "Enter a Price ");
    ArrayList<Book> results = bs.searchForPrice(keyword);
    System.out.println("The search results for "+Price
            +" are: ");
    for(Book tempBook : results)
        System.out.println(tempBook.toString());
}
}

It's saying this part: ArrayList<Book> results = bs.searchForPrice(keyword);
is already defined in teh array above. But not sure how else to set it up so that it will search for price instead of searching teh same array?

Remove this: "ArrayList<Book>" from that line. You don't re-declare results, just change its value(s).

so if i fix it to

    String Price = JOptionPane.showInputDialog(null,
            "Enter a Price ");
 
    System.out.println("The search results for "+Price
            +" are: ");
    for(Book tempBook : results)
        System.out.println(tempBook.toString());

it's not calling the searchForPrice array in the class.
While it will give me a 2nd dialog it just repeats the same result as before eve though i put in say 30.00 and it should return any book under that amount

Where do you try to call searchForPrice()?

I would think the code you posted might look like:

String Price = JOptionPane.showInputDialog(null,
        "Enter a Price ");
 
results = searchForPrice( Price );  // Price should be price
 
System.out.println("The search results for "+Price
        +" are: ");
for(Book tempBook : results)
    System.out.println(tempBook.toString());

ahh so take out the arraylist comment since i already have it but keep the results and make it price!

though i had to add in the bs. for it to work- works correctly now too! YAY!!!

String Price = JOptionPane.showInputDialog(null,
            "Enter a Price ");
results = bs.searchForPrice(Price);  // Price should be price
 
System.out.println("The search results for "+Price
        +" are: ");
for(Book tempBook : results)
    System.out.println(tempBook.toString());

Now i gotta do the rest of the problem lmao
• a method returning the book with the lowest price in the library
• a method searching the library for Books of a given author and returning an ArrayList of such Books

Great! Glad it worked, and good job figuring out the bs. part. I wasn't sure what object the method was being called on, and I forgot to mention it.

so my 3 problems for the assignment were:
• a method returning the book with the lowest price in the library
• a method searching the library for Books of a given author and returning an ArrayList of such Books
• a method returning an ArrayList of Books whose price is less than a given number

I did 2 and 3. I'm not sure how to tackle 1 BECAUSE of 2 and 3. i have jpanel pops for the other two and the resolve correctly. I'm not sure what is the best way to do "return the book with the lowest price in the library" that won't mess up the other 2?

This is what I have so far and it all works correctly:

/*
94. Modify the Book Store and Book Search Engine classes from the chapter.
You should include the following additional methods and test them:
• a method returning the book with the lowest price in the library
• a method searching the library for Books of a given author and returning an ArrayList of such Books
• a method returning an ArrayList of Books whose price is less than a given number
Natasha Epperson
 */
package natashaepperson_chapter_09_exercise_94;
 
public class Book {
private String title;
private String author;
private double price;
 
/**default constructor
 */
public Book()
{
    title = "";
    author = "";
    price = 0.0;
}
 
/** overloaded constructor
 * @param newTitle the value to assign to title
 * @param newauthor the value to assign to author
 * @param newPrice the value to assign to price
 */
public Book(String newTitle, String newAuthor, double newPrice)
{
    title = newTitle;
    author = newAuthor;
    price = newPrice;
}
 
/** getTitle method
 * @return the title
 */
public String getTitle()
{
    return title;
}
 
/** getAuthhor method
 * @return the author
 */
public String getAuthor()
{
    return author;
}
 
/**getPrice method
 * @return the price
 */
public double getPrice()
{
    return price;
    }
 
/** toString
 * @return title, author, and price
 */
@Override
public String toString()
{
    return ("title: "+title+"\t"
            +"author: "+author+"\t"
            +"price: "+price);
    }
}

and for the search engine:

/*
 * BookSearchEngine class
 */
package natashaepperson_chapter_09_exercise_94;
import java.util.ArrayList;
import javax.swing.JOptionPane;
 
public class BookSearchEngine 
{
public static void main(String[]args)
{
    BookStore bs = new BookStore();
 
    String keyword = JOptionPane.showInputDialog(null,
            "Enter a keyword ");
    System.out.println("Our book collection is: ");
    System.out.println(bs.toString());
 
    ArrayList<Book> results = bs.searchForTitle(keyword);
 
    System.out.println("The search results for "+keyword
            +" are: ");
    for(Book tempBook : results)
        System.out.println(tempBook.toString());
/*Enter Author of book
 * searchs and returns Author of book
 */
String Author = JOptionPane.showInputDialog(null,
        "Enter Author ");
results = bs.searchforAuthor(Author);//Authors name
 
System.out.println("The search results for "+Author
        +" are: ");
for(Book tempBook : results)
    System.out.println(tempBook.toString());
 
/*Enter Price of Book
* returns books lower than price entered
*/
String Price = JOptionPane.showInputDialog(null,
            "Enter a Price ");
results = bs.searchForPrice(Price);  // Price should be price
 
System.out.println("The search results for "+Price
        +" are: ");
for(Book tempBook : results)
    System.out.println(tempBook.toString());
 
}
}

for the class

/*
 * BookStore Class
 */
package natashaepperson_chapter_09_exercise_94;
 
import java.util.ArrayList;
 
public class BookStore 
{
private ArrayList<Book> library;
 
/**default constructor
 * instantiates ArrayList of Books
 */
public BookStore()
{
    library=new ArrayList<>();
    library.add(new Book("Intro to Java", "James", 56.99));
    library.add(new Book("Advanced Java", "Green", 65.99));
    library.add(new Book("Java Servlets", "Brown", 75.99));
    library.add(new Book("Intro to HTML", "James", 29.49));
    library.add(new Book("Intro to Flash", "James", 34.99));
    library.add(new Book("Advanced HTML", "Green", 56.99));
    library.trimToSize();
}
 
/** toString
 * @return each book in library, one per line
 */
@Override
public String toString()
{
    String result = "";
    for(Book tempBook : library)
    {
        result += tempBook.toString()+"\n";
    }
    return result;
}
 
/** Generates list of books containing searchString
 * @param searchString the keyword to search for
 * @return the ArrayList of books containing the keyword
 */
public ArrayList<Book> searchForTitle(String searchString)
{
    ArrayList<Book> searchResult = new ArrayList<>();
    for(Book currentBook : library)
    {
        if((currentBook.getTitle()).indexOf(searchString)!=-1)
            searchResult.add(currentBook);
    }
    searchResult.trimToSize();
    return searchResult;
    }
/** a method returning an ArrayList of Books whose price is less than a given number
 * @param searchDouble
 * @return 
 */
public ArrayList<Book> searchForPrice(String searchString)
{
    ArrayList<Book> searchResult = new ArrayList<>();
    //converts search string to double
    double maxPrice = Double.parseDouble(searchString);
    for(Book lessBook : library)
    {
        //If itemPrice < maxPrice, add it to the list
        if(lessBook.getPrice()< maxPrice)
            searchResult.add(lessBook);
    }
    searchResult.trimToSize();
    return searchResult;
    }
public ArrayList<Book> searchforAuthor(String searchString)
{
    ArrayList<Book> searchResult = new ArrayList<>();
    for(Book thisAuthor : library)
    {
        if((thisAuthor.getAuthor()).indexOf(searchString)!=-1)
            searchResult.add(thisAuthor);
    }
    searchResult.trimToSize();
    return searchResult;
    }
}

I'm not sure how to tackle 1 BECAUSE of 2 and 3.

I think you mean you're unable to transition from the pattern of asking for data, responding with results. So for Problem 1, there is no input required. The program simply outputs the book with the lowest price. Create a method to find it, return it, and print it out.

Why does your program use JOptionPane methods to get data but uses the console to output the data? Why not use JOptionPane methods to present the data?[COLOR="Silver"]

i'm not sure. I was only following what the book did for the coding.
hrm okay I see what you mean.

This is the first java class I've ever taken and the last lol. It's a strange requirement for my mathematics degree. It's built into the core requirement for all science based degrees at my university. I am enjoying and wouldn't mind learning more programming but my plate is full to begin with lol