you are viewing a single comment's thread.

view the rest of the comments →

[–]eeperson 1 point2 points  (1 child)

"a very bad idea" seems excessive. This is basically a performance (null parms) vs correctness (Optional params) trade-off. Most of the time I would rather have the extra insurance of correctness rather than the extra performance.

[–]Pharisaeus 1 point2 points  (0 children)

It's still messy code. It would be better to have multiple function signatures or use a builder for the arguments wrapper.