Javascript if else loop?

Question

I'm sure there probably is, however I'm not sure what it's called so apologises if this is something super similar. I'm wondering if there's a faster way to code the following:

var b = "#ff0002";
var o = "#46c029";
var i = "#f2ec00";
var n = "#f64c98";
var g = "#52c6f3";

if(a==1){
    return b;
}else if(a==2){
    return o;
}else if(a==3){
    return i;
}else if(a==4){
    return n;
}else if(a==5){
    return g;
}

It's called a switch statement. Or potentially a lookup map would also work. — Taplar
– Taplar, Commented Jul 24, 2018 at 15:31
There's no loop here. Loops are for and while, where you do something repeatedly until some ending condition is met. — Barmar
– Barmar, Commented Jul 24, 2018 at 16:09

Jonas Wilms · Accepted Answer · 2018-07-24 15:31:41Z

14

Yeah, lookup array:

return [b, o, i, n, g][a - 1];

Not necessarily faster, but definetly shorter :)

answered Jul 24, 2018 at 15:31

Jonas Wilms

139k20 gold badges164 silver badges164 bronze badges

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

5 Comments

Larm Over a year ago

Amazing! So much cleaner, thank you. Yeah, I wasn't expecting it to be faster but knew it could be clean

Jonas Wilms Over a year ago

@liam glad to help :)

Larm Over a year ago

any reason the first letter wouldn't be getting a colour set? :/ I've included the entire script in case it's being caused by a different issue

Jonas Wilms Over a year ago

@liam in your original question, a was 1 to get b, but now your count starts with 0, in that case, just remove the - 1

Jonas Wilms Over a year ago

@liam no worries :)

Niladri Basu · Accepted Answer · 2018-07-24 15:51:49Z

2

If you have large number of strings to compare from use Object like this:

myObj = {1: '#ff0002', 2: '#46c029', 3: "#f2ec00", 4: "#f64c98", 5: "#52c6f3"}

console.log(myObj[3]);

If you are using ES6 you can use Map() like this:

const myMap = new Map([[1, '#ff0002'], [2, '#46c029'], [3, "#f2ec00"], [4, "#f64c98"], [5, "#52c6f3"]])

console.log(myMap.get(3)); // or any key

edited Jul 24, 2018 at 15:51

answered Jul 24, 2018 at 15:38

Niladri Basu

10.7k7 gold badges59 silver badges65 bronze badges

3 Comments

Larm Over a year ago

But what about 'boing' :P Nah, if this is actually faster I'd be interested, otherwise it's a lengthier script

Niladri Basu Over a year ago

@Liam if you don't have large number of strings/comparison go with the Jonas W's solution.

Larm Over a year ago

No worries, thank you! This is handy though as I started this thread to cover my future tracks when I expand into more numbers

Collectives™ on Stack Overflow

Javascript if else loop?

2 Answers 2

5 Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

5 Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related