not working

Question

Hi,
I have been trying to make this work buy I am stuck... I don't see what is wrong so a hint would be highly appreciated... it gives the following error: 
"Both the reduce and filter methods have not been called yet."
Any idea?
```app.js
const purchaseItems = [
    {
        name: 'apples',
        dept: 'groceries',
        price: 2.49
    },
    {
        name: 'bread',
        dept: 'groceries',
        price: 2.99
    },
    {
        name: 'batteries',
        dept: 'electronics',
        price: 5.80
    },
    {
        name: 'eggs',
        dept: 'groceries',
        price: 3.99
    },
    {
        name: 't-shirts',
        dept: 'apparel',
        price: 9.99
    }
];
let groceryTotal;
// groceryTotal should be: 9.47
// Write your code below
let productsFromGrocery = dept => purchaseItems['dept'] === 'groceries';
let total = (sum, product) => sum + product['price'];
groceryTotal = purchaseItems
  .filter(productsFromGrocery)
  .reduce(total, 0)
  .toFixed(2);
```

Aaron Loften · Accepted Answer

Hello! I have an answer for ya. Okay, so I have the answer as to why yours failed. However, itll look confusing. Afterward, I provided a tip on how to keep it from being confusing in the future. :) Its kind of a naming convention.
First thing is that you called the wrong object inside of the "productsFromGrocey" variable. I found this out by testing the end result and saw your end result returned 0.00. I kinda cheated. lol. Tested on JsFiddle (https://jsfiddle.net/3cs6qh5L/11/)
Your code:
js
let productsFromGrocery = dept => purchaseItems['dept'] === 'groceries';

Should be:
js
let productsFromGrocery = dept => dept['dept'] === 'groceries';

The reason why you dont have to declare which object to pull the the "dept" from is because you are basically saying "For each object/array(key:value pair) inside of the object to be filtered(object.filter() or even better purchaseItems.filter()), we want to place in a temporary local variable named dept and for each dept, we want to grab the value for the key "dept". Move on to my next comment if this is confusing. :)
SECOND...Lets make this less confusing...
Instead of naming the key/value pair the same name as the key you are searching for, why not name it something more relevant to the entire object? Like this...
js
let productsFromGrocery = item => item['dept'] === 'groceries';

Notice, I avoided calling it "product." In Javascript, and all programming, product can mean a lot of things but usually refers to a math term. For this reason, I try to only use words that will not likely conflict but also are easy to read and describe the object. You could have also called it "thing" or "storeItem" or even "profitMaker" though I cant think of a better term than "item" right now.
BONUS!
Ive included a passing answer to that quiz that tries to keep it simple. :)
I did this because you already understand the logic, you just got tripped up on the wording in your code and lost your place, I think. Enjoy!
My code[Left out the part we shouldnt change ;)]:
```js
// groceryTotal should be: 9.47
// Write your code below
groceryTotal = purchaseItems.filter(item => item.dept == "groceries").reduce((sum, product) => sum + product.price, 0);
```

ludopuig · Answer

EyAaron: clap, clap, clap, clap, clap, clap, clap, clap, clap, clap, clap, clap
Thank you very much, very clear answer!!!

Welcome to the Treehouse Community

Looking to learn something new?

ludopuig

ludopuig

not working

2 Answers

Aaron Loften

Aaron Loften

ludopuig

ludopuig