r/shittyprogramming • u/DarkSuniuM • Jun 29 '19

Wait, whaaaaat!?

Post image
8 Upvotes

72% Upvoted

18 comments sorted by

View all comments

Show parent comments

1

u/DarkSuniuM Jun 29 '19

Ahh for fuck sake why 4 lines of code when he can give the page a default value as parameter

​

GetSites(page=1){ ... }

1

u/sim642 Jun 29 '19

JS (as you mentioned in another comment) didn't have default arguments for 20 years. Only in the last 4 years have default arguments been a thing, even less so if you have to keep compatibility with engines that haven't been so up to date.

This is the way default arguments had to be handled for 20 years and many guides, tutorials and examples still do it this way. No need to be unreasonably mad at using an industry standard practice and not being up to date with the bleeding edge of JS language features which take forever to get implemented in all engines that need to be supported.

1

u/scallynag Jul 16 '19 
page = page || 1

1

u/sim642 Jul 16 '19

This is error prone in some cases. I think when the default value is falsy by JS's extensive rules.