If a parameter isn't used, what should I pass? - The Old New Thing
If a parameter isn't used, what should I pass? - The Old New Thing
It doesn't matter what you pass, but if you have to ask, then just pass zero.
If a parameter isn't used, what should I pass? - The Old New Thing
It doesn't matter what you pass, but if you have to ask, then just pass zero.
I am a sky-high developer and know little of such low-level APIs. Please humor my ignorance: Isn’t it bad practice to write a god-method that sometimes uses these parameters, sometimes others? Isn’t this better refactored into multiple dedicated functions?
The answer, as with everything in software development, is that it depends.
A god method with 100 optional params that is usually bad practice. But a common pattern is to allow for an options object to be passed, and that object may contain 0-n supported parameters. This pattern is used everywhere, see graphql as a widely used library that is based on this.
Totes this. I've refactored far too many of these things in my day so I just about always reach for an options object if there's the slightest uncertainty about adding more params in the future.
I'm just guessing, but what about backwards compatibility? Or cross-system compatibility?
For example, something like a syscall that's existed for 20 years. Changing it would break old apps.
Of course you could just keep the now "old" syscall and add new methods that replicate it's behavior, but haven't you then introduced bloat? More ways to do the same thing, meaning (eventually) more bugs, more fragmentation, memory usage, etc.
Let's say you currently have an overloaded "god method" that is used for twenty different things. The proper thing to do would be to create twenty different interfaces that each do one specific thing and then call the syscall behind the scenes. That allows you to update and modernize your apps, allows for better testing of specific use cases, etc.
The original exposed syscall is deprecated and then eventually you force all the reliant code to adopt the proper new interface. The original overloaded code is still there, but locked up in a black box where no one has to worry about it.
Obviously taken to an extreme it's bad, but I think it's fine to have a function that can do one thing two or more different ways and ignore a certain parameter if one of the ways doesn't need it. I've done some programming against the Win32 API and this is what jumped to mind for me, and I think it's the typical case here. If I were designing from scratch I might split it into n functions that do it one way, but it's such a small difference I wouldn't fret over it. And of course making a change to the Windows API is an undertaking, probably not worth it in most cases.
Yes, but with things like syscalls it's easier to do this than require every high-level thing building on the syscall to be modified and recompiled. Very few people need to use such low-level APIs.
undefined
~~~ int dummy = 123; foobar(dummy); ~~~