3

I'm making a lame text-based game, and I made an object player like so:

var player = {
    displayText: "<span>you</span>",
    currentPosition: 0,
    level: 1,
    health: function() { return 10 + (this.level * 15) },
    strength: function() { return this.level * 5 },
    hitRating: 4
}

My understanding is that you can give an object a function as a property.

However, when I alert(player.health) I get:

function() { return 10 + (this.level * 15) }

What am I doing wrong? Are you not able to declare a object property that way? Is there a way to auto-generate the value of player.health any time it's called later on?

Jan Turoň
  • 31,451
  • 23
  • 125
  • 169
Math chiller
  • 4,123
  • 6
  • 28
  • 44

3 Answers3

11

player.health is a function. To call a function, you have to put () after it:

alert(player.health());
Barmar
  • 741,623
  • 53
  • 500
  • 612
11

If you want to create property with accessor on JS object, proper way to do this is to use Object.defineProperty.

In your case:

// original object
var player = {
    displayText: "<span>you</span>",
    currentPosition: 0,
    level: 1,
    // health: function() { return 10 + (this.level * 15) },
    strength: function() { return this.level * 5 },
    hitRating: 4
}

// create property with accessor method
Object.defineProperty(player, "health", {
    get: function () {
        return 10 + (player.level * 15)
    }
})

// call the property
alert(player.health);  // 25
player.level++;
alert(player.health);  // 40
royhowie
  • 11,075
  • 14
  • 50
  • 67
Jan Turoň
  • 31,451
  • 23
  • 125
  • 169
4

You need to call the function, player.health()

Cole Pilegard
  • 582
  • 2
  • 11