IsConstructor
Domenic Denicola
domenic at domenicdenicola.com
Wed Jun 11 12:27:12 PDT 2014
From: Rick Waldron <waldron.rick at gmail.com>
> Or maybe that's not necessary? Is it preferable to just throw when someone writes any of these:
I think it is indeed preferable, as would happen when using any other method (`this`-dependent function) without a `this`.
> (Note that `Array.isArray` doesn't cease to work correctly when aliased)
`Array.isArray` is not a method, but a function; it does not change behavior depending on its `this` value.
> With the IsConstructor or IsCallable guard, these "just work"; without any guard, they'll throw TypeError exceptions: "object is not a function" or "undefined is not a function" (in strict mode). Neither of these errors are very obvious.
I disagree that this is not obvious. This is the same error you always get when aliasing a method and trying to use it as a function.
> Of course this can all be fixed with .bind() or a "bind" operator, but it just seems unfortunate to throw out something that's not harming the spec in favor something that might be problematic in end user code.
This is a bit of a slippery-slope argument; the end result is that every method should have a default `this` value that it's "soft-bound" to, in order to coddle the users who might get confused.
More information about the es-discuss
mailing list