$.grep to check if object exists in array

Question

I am trying to build an array of objects and then either update the object or create a new one depending if one does not exist. This is my first time trying to use the $.grep function and I am not sure what I am doing wrong. Any Advice?

Here are the objects:

function Phase(phase, html, count) {
        this.phase = phase;
        this.html = html;
        this.count = count;
        this.porfolios = [];
    }

    function Portfolio(portfolio, html, count) {
        this.portfolio = portfolio;
        this.html = html;
        this.count = count;
    }

Here is the code in question....

var phases = [];
//Check Array of Phases 
result = $.grep(phases, function (e) { return e.phase == item.Phase; });

if (result.length == 0) 
{
     var newphase = Phase(item.Phase, itemhtml, 1)
     phases.push(newphase);
} 
else if (result.length == 1) 
{
 // update existing phase
 result[0].count ++;
 result[0].html += itemhtml;
}

What does it do or what doesn't it do? What is going wrong?

Richard de Wit
– Richard de Wit

2013-04-19 12:35:31 +00:00
Commented Apr 19, 2013 at 12:35 — Richard de Wit
– Richard de Wit, Commented Apr 19, 2013 at 12:35
Can you create a demo at jsfiddle

Arun P Johny
– Arun P Johny

2013-04-19 12:37:24 +00:00
Commented Apr 19, 2013 at 12:37 — Arun P Johny
– Arun P Johny, Commented Apr 19, 2013 at 12:37

Arun P Johny · Accepted Answer · 2013-04-19 12:36:36Z

1

var newphase = Phase(item.Phase, itemhtml, 1)

should be

var newphase = new Phase(item.Phase, itemhtml, 1)

answered Apr 19, 2013 at 12:36

Arun P Johny

389k68 gold badges532 silver badges532 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

BlueBird Over a year ago

yep, stupid mistake... sometimes it just takes another set of eyes i guess.

Guffa · Accepted Answer · 2013-04-19 12:37:04Z

1

You need to use the new keyword to create an object:

var newphase = new Phase(item.Phase, itemhtml, 1);

answered Apr 19, 2013 at 12:37

Guffa

703k111 gold badges760 silver badges1k bronze badges

Collectives™ on Stack Overflow

$.grep to check if object exists in array

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related