Are Default Params in JavaScript bad? (Snack pack #4)

Are Default Params in JavaScript bad? (Snack pack #4)

Works Thoughts About

Not if it's still passed

Take this example:

image

I wrote this function well over two years ago. It just pops up a notification if the user makes a CRUD call. (Create, Read, Update Delete). The function only takes two defined parameters a message and a notification.

The notification parameter is a boolean.

notification = true // notify user by a pop-up
notification = false // inline-notification by form button, or in a defined space

Looking back at this function, it could be much better.

  • Instead of a notification boolean, it can be notification style as a string.
  • Call with the second parameter everywhere

Back to the question: Default Params in JS bad?

I think so if I'm not explicitly calling with all the params of a function. It's okay to have a fallback, but don't depend on it!

Why?

  • It's leaves technical debt.
  • How is another developer supposed to know what parameters are passed if they only see a partial picture?
  • My boolean parameter is bad in this case.

How I refactored this function:

image

My snack pack reads are intended for a quick read without any fluff or BS.

Feedback is always welcome.

Thought

Are Default Params in JavaScript bad? (Snack pack #4)
Apr 10, 2021
Overlooked design, blank states. (Snack Pack #3)
Mar 27, 2021
Readme for a Productive 2021
Dec 28, 2020
Better readme in 5 minutes. (Snack Pack #1)
Feb 28, 2021
Questions make you a better developer (Snack Pack #2)
Mar 7, 2021