JavaScript variable related [closed]

Question

Closed. This question needs details or clarity. It is not currently accepting answers.

Want to improve this question? As written, this question is lacking some of the information it needs to be answered. If the author adds details in comments, consider editing them into the question. Once there's sufficient detail to answer, vote to reopen the question.

Closed last year.

Improve this question

I've got this function that defines a random color set.

My issue is defining the variable (var target) to select the new color set from the kuler object.

kuler: {
    set0:['#499E8D','#85CC9F','#A4DEAB','#C8E8C7','#FFF6C9'],
    set1:['#0F2440','#2C3F59','#518C8C','#94BEAC','#D9CAAD'],
    set2:['#499E8D','#85CC9F','#A4DEAB','#C8E8C7','#FFF6C9'],
    set3:['#0F2440','#2C3F59','#518C8C','#94BEAC','#D9CAAD'],
    set4:['#499E8D','#85CC9F','#A4DEAB','#C8E8C7','#FFF6C9'],
    set5:['#0F2440','#2C3F59','#518C8C','#94BEAC','#D9CAAD'],
    set6:['#499E8D','#85CC9F','#A4DEAB','#C8E8C7','#FFF6C9']
},
setNewColourSet: function () {
    rn=Math.floor(Math.random()*5);
    for (i=0; i<4; i++){
        var target = "project.kuler.set"+rn+"["+i+"]";
        $('.kuler'+i).css('background-color',target);
        /* works *///  $('.kuler'+i).css('background-color',project.kuler.set4[i]);
    }
}

This question is similar to: Accessing an object property with a dynamically-computed name. If you believe it’s different, please edit the question, make it clear how it’s different and/or how the answers on that question are not helpful for your problem. — dumbass
– dumbass, Commented Oct 8, 2024 at 22:06

hvgotcodes · Accepted Answer · 2012-08-21 13:22:44Z

3

In think you were close -- just put the string within the square braces.

var target = project.kuler['set' + rn][i];

assuming rn is the number appended to 'set', and kuler is itself in an object literal called project.

Note the first brace (kuler['set'+rn]) is property access on the object literal kuler, and the second brace ([i])is array index access on an array....

answered Aug 21, 2012 at 13:22

hvgotcodes

121k33 gold badges209 silver badges237 bronze badges

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

1 Comment

etoxin Over a year ago

yeah, i need to wait 2mins before it will let me answer. :(

Mient-jan Stelling · Accepted Answer · 2012-08-21 13:28:38Z

1

kuler: {
    set0:['#499E8D','#85CC9F','#A4DEAB','#C8E8C7','#FFF6C9'],
    set1:['#0F2440','#2C3F59','#518C8C','#94BEAC','#D9CAAD'],
    set2:['#499E8D','#85CC9F','#A4DEAB','#C8E8C7','#FFF6C9'],
    set3:['#0F2440','#2C3F59','#518C8C','#94BEAC','#D9CAAD'],
    set4:['#499E8D','#85CC9F','#A4DEAB','#C8E8C7','#FFF6C9'],
    set5:['#0F2440','#2C3F59','#518C8C','#94BEAC','#D9CAAD'],
    set6:['#499E8D','#85CC9F','#A4DEAB','#C8E8C7','#FFF6C9']
},
setNewColourSet: function () {
    var rn = Math.floor(Math.random()*5);

    for (i=0; i<4; i++){
    $('.kuler'+i).css('background-color', this.kuler['set'+i][rn]);
    }
}

here you go

answered Aug 21, 2012 at 13:28

Mient-jan Stelling

5323 silver badges8 bronze badges

Comments

Aesthete · Accepted Answer · 2012-08-21 13:26:50Z

0

If you're going to have an object with numbered names like this, why not just have a list?

kuler: [
  ['#499E8D','#85CC9F','#A4DEAB','#C8E8C7','#FFF6C9'],
  ['#0F2440','#2C3F59','#518C8C','#94BEAC','#D9CAAD'],
  ['#499E8D','#85CC9F','#A4DEAB','#C8E8C7','#FFF6C9'],
  ['#0F2440','#2C3F59','#518C8C','#94BEAC','#D9CAAD'],
  ['#499E8D','#85CC9F','#A4DEAB','#C8E8C7','#FFF6C9'],
  ['#0F2440','#2C3F59','#518C8C','#94BEAC','#D9CAAD'],
  ['#499E8D','#85CC9F','#A4DEAB','#C8E8C7','#FFF6C9']
],

setNewColourSet: function () {
  rn=Math.floor(Math.random()*5);
  for (i=0; i<4; i++){        
    $('.kuler'+i).css('background-color',kuler[rn][i]);
  }
}

answered Aug 21, 2012 at 13:26

Aesthete

18.9k6 gold badges38 silver badges46 bronze badges

2 Comments

etoxin Over a year ago

thank you, this makes sense. unsure why i didn't do this in the first place.

Aesthete Over a year ago

Sometimes we over-engineer our solutions :)

Apurv · Accepted Answer · 2012-08-21 13:27:46Z

0

var target = "project.kuler.set"+rn+"["+i+"]";

You are assigning a string to a target.

I suppose project is an object and the kuler is a part of it so try with removing quotes;

answered Aug 21, 2012 at 13:27

Apurv

3092 silver badges10 bronze badges

Collectives™ on Stack Overflow

JavaScript variable related [closed]

4 Answers 4

1 Comment

Comments

2 Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

2 Comments

Comments

Linked

Related