Specific property from object in Javascript

Question

Lets say i have this object and the first two have the property bought(how many of them the user bought)

var Drugs = {
    Extasy : {
        name : "Extasy",
        value: 0,
        bought : 2,
        hist : [],
        qty : 0
    },
    Hash : {
        name : "Hash",
        value: 0,
        hist : [],
        bought: 3,
        qty : 0
    },
    Adrenaline :  {
        name : "Adrenaline",
        value: 0,
        hist : [],
        qty : 0
    },
    Cocaine : {
        name : "Cocaine",
        value: 0,
        hist : [],
        qty : 0
    },
    Mdma : {
        name : "Mdma",
        value: 0,
        hist : [],
        qty : 0
    }
};

and i want to loop through the Drugs object in order to find out which of them has the bought property.

var outputs = '<li class="ui-li-has-count">';

for (var d in Drugs) {
    var dd = Drugs[d];

    if (Drugs.hasOwnProperty(dd.bought)) {
        // if (dd.bought != 'undefined' ){
        outputs+='<a  class="ui-btn ui-btn-icon-right ui-icon-carat-r" href="#myPopup" onclick="buyPopup(this.id)" id='+dd.name+'  >'+dd.name+'<span  class="ui-li-count">'+dd.bought+'</span> ';
    }
    else {
        outputs = false;
    }

    outputs += '</li></div>';
}

//getElementById
ast("panel1").innerHTML += outputs;

Even if my bought key has a property i always get false

hasOwnProperty is a method that only requires the name. So change dd.bought to just 'bought' — Caleb Lewis
– Caleb Lewis, Commented Jan 29, 2015 at 21:12

Bergi · Accepted Answer · 2015-01-29 21:15:46Z

1

You don't want to test the Drugs collection, you want to test the individual drug object. Also, hasOwnProperty does take the property name as a string, not the property value. So use

if (dd.hasOwnProperty("bought"))

Though in your case, a simple test like

if (dd.bought)

should suffice, especially as you want to treat non-existing properties (which resolve to undefined) just like 0 values.

answered Jan 29, 2015 at 21:15

Bergi

671k162 gold badges1k silver badges1.5k bronze badges

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

Comments

talemyn · Accepted Answer · 2015-01-29 21:24:44Z

0

If you want to find the property "bought", then you need to check for that word in the hasOwnProperty() call. So change this line:

if (Drugs.hasOwnProperty(dd.bought)){

. . . to this . . .

if (dd.hasOwnProperty("bought")){

In both of your test items, you are currently checking to see if the item has a property called "2" and "3", respectively, on the "Drugs" object (since that is what dd.bought equals at the time you are checking).

You should really also have this in your code before assigning dd, as well:

if (Drugs.hasOwnProperty(d)){

edited Jan 29, 2015 at 21:24

answered Jan 29, 2015 at 21:12

talemyn

8,0004 gold badges33 silver badges52 bronze badges

2 Comments

Orpheus Over a year ago

Problem being with that solution is that testing "Drugs" is testing that level. We need to test dd for the property 'bought', as 'bought' is a dd property, not a Drugs(obj) property.

talemyn Over a year ago

Apologies . . . I misread the question. I've updated my answer.

Collectives™ on Stack Overflow

Specific property from object in Javascript

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related