DEV Community

Discussion on: ✔||🤢 Commit or Vomit | Switch(true)

Collapse
 
siddharthshyniben profile image
Siddharth • Edited

This could easily have been an if/elseif/else.

You could have done:

if(someExpressionA) console.log('yes')
else if ((someExpressionB 
    && someExpressionC) || someExpressionD) console.log('nope');
else console.log('maybe');
Enter fullscreen mode Exit fullscreen mode
Collapse
 
jmdejager profile image
🐤🥇 Jasper de Jager

true, this is a short example. Apart from this example would you never commit a switch(true)?

Collapse
 
siddharthshyniben profile image
Siddharth • Edited

Nope 🤢

Collapse
 
jmdejager profile image
🐤🥇 Jasper de Jager

good to mention: the example was updated after this reply

Collapse
 
basbenik profile image
Bas van Baalen

I'm not sure it's because of the editor, but I would suggest to use more lines for more readability and easier comprehension.

if(someExpressionA) 
  console.log('yes')
else if ((someExpressionB && someExpressionC) || someExpressionD) 
  console.log('nope');
else 
  console.log('maybe');
Enter fullscreen mode Exit fullscreen mode

For me this already reduces the cognitive load a lot quickly see what can happen.

Collapse
 
siddharthshyniben profile image
Siddharth

Actually I typed my comment on mobile, so I didn't format it

Collapse
 
jmdejager profile image
🐤🥇 Jasper de Jager

Good point for a next Commit or Vomit! What do you think about switch(true) in general?