Is the key null in your code? No. So it's not wrong. I am pretty sure people won't deliberately pass null as the key. Null as a value is perfectly OK to store.
But ds.Get("table", someMethod()) would be "wrong" if someMethod() returned null. In that case, DataStore now substitutes the...