The `removeFromCart()` instructions are not descriptive enough about js-basics-online-shopping-lab-bootcamp-prep-000 HOT 2 OPEN

The solutions branch solves this by using a for loop and mutating the cart array. However, it can be much cleaner if it doesn't require mutation:

function removeFromCart(item) {
  const newCart = cart.filter(el => !el.hasOwnProperty(item));

  if (newCart.length === cart.length) {
    console.log('That item is not in your cart.');
  }

  return newCart;
}

Making this sample code mutate is as simple as adding cart = newCart; immediately before the return, and I understand that Array.prototype functions may be outside the scope of this course, but they are still a valid way to solve the problem.

For clarity, I'm a Senior Frontend Engineer looking at these problems because my roommate is looking to apply to FlatIron School. I wanted to understand what he's working through so I could offer help if he needed it. I enjoy solving toy problems, so I decided to solve these. I was surprised when removeFromCart required mutation.

from js-basics-online-shopping-lab-bootcamp-prep-000.

I opened #1055 to update the test to pass if the element is removed from the returned array (and therefore is true to the instructions), whether the global cart variable is mutated or not.

from js-basics-online-shopping-lab-bootcamp-prep-000.