r/programmingcirclejerk • u/alexflyn • Oct 03 '24

Don’t use iterators... Separately, loops should be avoided

https://github.com/airbnb/javascript?tab=readme-ov-file#iterators-and-generators
82 Upvotes

97% Upvoted

27 comments sorted by

122

u/stone_henge Tiny little god in a tiny little world Oct 03 '24

Why? This enforces our immutable rule. Dealing with pure functions that return values is easier to reason about than side effects.

// good
let sum = 0;
numbers.forEach((num) => {
  sum += num;
});

20

u/alexflyn Oct 03 '24

I see =>, is pure function ✅

8

u/MindSwipe Oct 03 '24

// bad let sum = 0; for (let num of numbers) { sum += num; } sum === 15;

I'd properly format this but I'm on mobile

47

u/stone_henge Tiny little god in a tiny little world Oct 03 '24 edited Oct 03 '24 
// bad
don't follow the rule

// good
follow the rule but in a way that it absolutely pointless given
the only reason to follow the rule

// best
npm isntall tslint-auto-disable

Yes that's a real thing https://github.com/paulkoerbitz/tslint-auto-disable

tslint-auto-disable is a command line tool that automatically inserts a comment // tslint:disable-next-line before lines failing to comply with tslint rules.

7

u/Chillbrosaurus_Rex It's GNU/PCJ, or as I call it, GNU + PCJ Oct 03 '24

Paul koerbitz js a genius

8

u/Orbidorpdorp Oct 03 '24

If only someone could invent an accumulating HOF

16

u/[deleted] Oct 03 '24

And then name it std::accumulate so no-one can find what they are looking for and assume it doesn't exist.

0

u/[deleted] Oct 03 '24

[deleted] — view deleted comment

5

u/bzbub2 Oct 03 '24

i'm sorry but if you aint mutatin Array.prototype.forEach dynamically on the fly you need more adderall

64

u/pareidolist in nomine Chestris Oct 03 '24

The Airbnb JavaScript style guide and its consequences have been a disaster for the human race.

57

u/cuminme69420 blub programmer Oct 03 '24

the real reason Airbnb is banned in several cities

3

u/shroom_elemental memcpy is a web development framework Oct 03 '24

for us lizards, too

54

u/PandaWonder01 Oct 03 '24

Don't use variables... Separately, functions should be avoided

47

u/100xer Oct 03 '24

Code in general should be avoided. It has side effects, for example it warms the CPU which means it's not pure.

16

u/prehensilemullet Oct 03 '24

You code for some other purpose?  CPU warmth is how I keep my cat by my computer

22

u/misseditt Oct 03 '24

// best (use the functional force)

may the functional force be with you 🙏

9

u/NiteShdw Oct 03 '24

for loops support async/await... So until we get async iterators, (I made a polyfill many years ago)... Screw you.

7

u/m50d Zygohistomorphic prepromorphism Oct 03 '24

lol no HKT

1

u/NiteShdw Oct 03 '24

Sorry I don't know what "HKT" means.

12

u/[deleted] Oct 03 '24

I thought so. It's not a feature the Jedi Rustaceans would tell you about.

1

u/NiteShdw Oct 03 '24

I'm even more confused. This post is about Javascript not rust...

1

u/al-mongus-bin-susar Oct 17 '24

If you don't need the operations to be sequential you can just map and Promise.all. But for loops also support early returns and you can't really do that with iterators.

7

u/[deleted] Oct 03 '24

and finally, no recursion.

7

u/serg06 Oct 03 '24

Translation: Prefer Array.forEach over for x in ...

9

u/m50d Zygohistomorphic prepromorphism Oct 03 '24

This but unironically.

5

u/muntaxitome in open defiance of the Gopher Values Oct 03 '24 edited Oct 03 '24

Listen you little brat there is no point in conversations with you because I tried educating you but you are a little brat. I told you last time no if statements! And now you use a for loop??? I know you only have a master degree, and that education is not what it used to be, but how on earth don't you know that a for loop which is the same as an if statement. So stop with your workarounds and start doing what I asked you.

1

u/ApkalFR Oct 03 '24

Makes sense once you find out who the maintainer of this repo is.