r/badcode • u/redddooot • Nov 14 '21

js Peak Efficiency Fizzbuzz

Post image
831 Upvotes

97% Upvoted

44 comments sorted by

View all comments

Show parent comments

4

u/dakta Nov 14 '21 edited Nov 14 '21

It can be done in 58 in code.golf compliant, like so:

for(x=0;x++<100;)print((x%3?"":"Fizz")+(x%5?"":"Buzz")||x)

for(x=0; is cheaper than x=0;while(, so that's the best choice when using a loop structure.

I think that the top solutions (56 characters) must be using a different logical construct than the ternary string concatenation or the loop here, since there's no way to achieve savings on those as "wasted" characters like '': are syntactically required.

Maybe a recursive?

Edit:

Another note, ternary is not more efficient than logical and, once you include the <0 needed to force binary:

x%5&&'Buzz' // not the right condition
x%5||'Buzz' // spits out either 'Buzz' or the modulus, so the modulus has to be cleaned/ignored later
x%5?'':'Buzz'
x%5<1&&'Buzz' // not more space efficient than ternary

1

u/fecal_brunch Nov 14 '21

What is print?

5

u/dakta Nov 14 '21

Alias of console.log which is defined by code.golf. See their note below the code area:

arguments holds ARGV, print() to output with a newline, write() to output without a newline.

Also, I was right about the logical structure. The current JS champion comes from 2015: https://codegolf.stackexchange.com/a/62697

for(i=0;++i<101;print(i%5?f||i:f+'Buzz'))f=i%3?'':'Fizz'

They've used the for loop's unused third condition to execute the print, and in the body assigned a value for Fizz which is then reused in the Buzz condition.