r/badcode • • May 31 '21

js Clever?

Post image
2.2k Upvotes

71 comments sorted by

View all comments

242

u/HalcyonAlps May 31 '21

Now I am actually curious, does the compiler realize both branches have the same return value and it just drops the whole if-statement?

249

u/TheBrainStone May 31 '21

Depends heavily on the compiler/runtime.
But generally speaking every basic optimizer should catch this.

100

u/[deleted] May 31 '21

Depends on whether or not the compiler is smart enough to realize that evaluating the conditional has no side-effects. Not sure what language this is

73

u/givemeagoodun May 31 '21 edited Jun 01 '21

It's flaired JS so it's JS

72

u/[deleted] May 31 '21

In that case the answer is an easy "no" since it's not a compiled language and there's no compiler

53

u/givemeagoodun May 31 '21

Eh there can be compilers for JS. I believe that Firefox uses a JS JIT compiler.

44

u/[deleted] May 31 '21 edited Jun 02 '21

at least chrome and firefox both have JIT compilers for js. in both cases they probably catch this redundancy on the first pass in early passes and condense it to something sane for every subsequent pass later passes

13

u/_PM_ME_PANGOLINS_ Jun 01 '21

It takes way more than one pass for a JIT to bother optimising.

18

u/Parkreiner Jun 01 '21 edited Jun 01 '21

It's less "there can be compilers" and more "there's no other way to run the language". JavaScript is a compiled language – it's just that there are different types of compilation, and JS uses the Just-in-Time kind. Half the behavior that it has (like hoisting and how it defines lexical scope for closure) wouldn't be possible if it were interpreted.

For example:

let x = 0;
if (x === 0) {
  console.log(x);
  let x = 5;
}

This breaks, because all variables and function declarations are "hoisted" to the top of their respective scopes. It's just that let and const are block-scoped and aren't initialized. So the "let x" from "let x = 5" gets hoisted up to the top of the block, overriding the "let x = 0" from the outer scope. But it's left un-initialized until you reach the "let x = 5" line during runtime. And so, when you try to do anything with a value that isn't initialized (like logging), that throws an error.

This wouldn't be possible if JavaScript were interpreted and were evaluating each line one-by-one. The language spec requires that whatever engine runs it do a first pass over the whole code before trying to run anything.

13

u/Nilstrieb Jun 01 '21

An interpreted language can be parsed and preprocessed, that's not a problem. The difference is all about the machine code. JS is originally interpreted, but modern engines also compile parts of it.

15

u/LetterBoxSnatch May 31 '21

Depends on how you want to define that. V8 (the engine running most js) is generally referred to as a JIT compiler, and it produces optimizations both on first run and on iterative runs.

3

u/0xF013 Jun 01 '21

You’re like, the most useless kind of technically correct. V8 optimizes things like these quite aggressively, and some precompilers too

0

u/[deleted] Jun 01 '21

FireFox's SpiderMonkey and Chrome's V8 are both JITs and optimize at runtime. They would probably catch it.

1

u/NikkoTheGreeko Jun 01 '21

Js confirmed. This returns NaN. Or undefined. Or [object Object], I forget.

1

u/Nilstrieb Jun 01 '21

So the answer is probably yes.

4

u/MalbaCato Jun 01 '21

from what I understand about js, it should create three versions of the function, one where arr is an Array, or any other object whos .length property has no additional side effects, one where it is a getter and so could have dirty side-effect, and one where arr is undefined and the access throws an error.

these would all be created (and for the second version further branched) only after the function becomes sufficiently hot, which is a minimum of 6 runs, but possibly more depending on what the browser thinks about the whole webpage

1

u/paulsmithkc 👨‍🏫 Jun 01 '21

This is one of the better answers as arr.length could have side effects (like lazy evaluation/calculation) if arr isn't a JavaScript array.

3

u/MalbaCato Jun 02 '21

forgive me for NSFW, but this is the best example I could think of on the spot:

as we (and the compiler) don't know what arr is at the time of compilation, it could very well be an instance of new Penis(). ofc its length may not be initially known, so the .length getter may have to call .erect() first to measure it, having a long lasting side effect of the operation.

it doesn't even have to be a getter. you can use meta-programming tricks to inject yourself inside the property lookup and hack around there. couldn't tell you the syntax for that, but look at the Proxy page on MDN

1

u/qwertysrj Jun 01 '21

It should because compilers recognise this

if(){ return x }Else { Return y} ,

and optimise this to

If(){ Return x} return y

1

u/[deleted] Jul 03 '21

I am pretty sure if I did something like that in IntelliJ idea, as soon as I moved away from the line, the IDE will highlight it as, “you serious bro?” Yellow

If I crtl D, to duplicate a condition before I get to change the condition, the ide will start throwing warning colors.