JavaScript append text using arrays

Question

I'm attempting to reuse the code from here Edit a variable within an array to create something simmilar but less complex.

I made the 'working' function:

var WorkT = function(gain,loss,message) {
    coins += this.gain;
    coins -= this.loss;
    this.message = message;
}


workT1 = new WorkT(30,0,'<span class="red">+ 30 Gold Coins');
workT2 = new WorkT(15,0,'<span class="red">+ 15 Gold Coins');
workT3 = new WorkT(80,0,'<span class="red">+ 80 Gold Coins');
workT4 = new WorkT(1,0,'<span class="red">+ 1 Gold Coin');

WorkTs = [workT1,workT2,workT3,workT4];

And I'm trying to call it later on in my code with this:

$('#output').html(WorkTs[Math.floor(Math.random() * 4)].WorkT());

But, when I click the button, nothing changes. Can anyone tell me why?

@YoannM whoops, you're right. forgot the 'S' in the array. Sorry, added that. — Asteria
– Asteria, Commented Oct 5, 2014 at 23:51

Yoann · Accepted Answer · 2014-10-05 23:53:26Z

2

Your WorkT instances have no WorkT() function.

You need to declare a function named WorkT (or else) inside your WorkT 'class' :

var WorkT = function(gain,loss,message) {
    //...
    this.work = function () {
        //Do Something.
    }
}

Or you won't be able to call it on your instances :

$('#output').html(WorkT[Math.floor(Math.random() * 4)].work());

It all depends on what you're trying to achieve here.

answered Oct 5, 2014 at 23:53

Yoann

3,0801 gold badge28 silver badges36 bronze badges

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

1 Comment

Asteria Over a year ago

Thank you, I can't believe I over looked that!

nisargjhaveri · Accepted Answer · 2014-10-06 00:01:15Z

0

There are lots of errors in your code.

What is the coins in the WorkT Constructor? Is it a global variable, or you wanted to do this.coins?
this.gain and this.loss does not exist. They are just gain and loss.
There is no function called WorkT in the class WorkT. You probably want to define the function, too.

Here is how the code will look like, assuming coins is a global variable.

var WorkT = function(gain,loss,message) {
    coins += gain;
    coins -= loss;
    this.message = message;

    this.work = function(){
        //Do something here.
    }
}

And,

$('#output').html(WorkTs[Math.floor(Math.random() * 4)].work());

answered Oct 6, 2014 at 0:01

nisargjhaveri

1,50912 silver badges23 bronze badges

Collectives™ on Stack Overflow

JavaScript append text using arrays

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related