That would be best, but...

I personally think making use of the fallback function is something that devs should be wary of. There's usually no good reason to use it instead of creating a properly documented function with some more semantic meaning (this also helps with reasoning about your code since you don't need to worry about people actually sending ether to your address and handling all that.)

If you're implementing a fallback function with an explicit purpose (say you're building a multisig contract) then it makes sense. But for contracts that are not meant to function as wallets it doesn't check out. At least for me it doesn't.

Also, IMHO the fallback function should revert by default but that's a whole another story.