returns a new ArrayList of MenuItems containing just those menu items that are currently available

fyidoctor :

This is what i have so far

i am confused on how to compare using a method i have also tried 1. items.get(i).available() == true and also using .equals

  public boolean available() { return myAvailability; } 

//method

  ArrayList<MenuItem> availableItems = new ArrayList<MenuItem>(items.size());
      int i = 0;
    while (i < availableItems.size())
    {
         boolean c = items.get(i).available();
        if (c == true)
        {
            availableItems.add(items.get(i));
            i++;
        }
        i++;

    }
    return  availableItems ;

//main

ArrayList<MenuItem> items = new ArrayList<MenuItem>();
        items.add( new MenuItem( "Big Bad Burger", 9.95, 2.15, 1, false ) );
        items.add( new MenuItem( "Cheeky Chicken", 5.95, 0.75, 1, true ) );
        items.add( new MenuItem( "Wild Wings", 5.95, 0.50, 0, false ) );
        items.add( new MenuItem( "Flying Fish", 15.95, 7.61, 1, false ) );
        items.add( new MenuItem( "Igloo Icecream", 1.95, 0.28, 2, true ) );

        ArrayList<MenuItem> available = availableItems( items );
        System.out.println(available);

        // should list:
        // Cheeky Chicken ($5.95) and Igloo Icecream ($1.95)
        for ( MenuItem item : available )
          System.out.println( item.menuString() );
Reputation Farmer :

The error is here:

while (i < availableItems.size())

since availableItems.size() is 0.

The solutions are:

  1. Use foreach loop, as in another answer
  2. Use removeIf:

    ArrayList<MenuItem> availableItems = new ArrayList<MenuItem>(items);
    availableItems.removeif(item -> !item.available());
    return availableItems;
    

Guess you like

Origin http://43.154.161.224:23101/article/api/json?id=114248&siteId=1