On Sun, Jul 17, 2016 at 8:21 PM, Rowan Collins <rowan.coll...@gmail.com> wrote:
> Hi All, > > In the discussion of escaping mechanisms, it's once again come up that > functions lack autoloading, and thus are hard to work with in larger code > bases. > > Previous solutions to this, most notably Anthony Ferrara's very thorough > RFC [1], have looked at adding new modes for the existing autoload > functions, or new functions alongside, to specify the type being autoloaded. > > A common response is that you can just use a class with static methods, > and thus leverage the existing autoload mechanism. But the introduction of > "static class" or "abstract final class" was rejected [2] in part with the > opposite justification: that you shouldn't need a class to hold static > methods now that we have namespaced functions. > > How about an alternative approach where a function inside a namespace can > be autoloaded using the existing callback, by using a reserved namespace > segment? So to autoload function "foo\bar()", the engine would construct a > string like "__function\foo\bar" or "foo\__function\bar", and pass that to > the registered autoloader stack. > > This shouldn't result in errors or misbehaviour from existing autoloaders, > it just won't find anything to load. An autoloader that knows how can then > use the namespace path to determine what to load, probably something like > "src/foo/functions.php". > > The focus on namespaced functions reflects the fact that > one-file-per-function is rare and somewhat unwieldy, so a call to load > "__function\foo" is unlikely to be that useful in practice. > > Note that, like many previous proposals, this could apply to namespaced > constants, too, using a token such as __constant. > > Any thoughts? Good idea, horrible idea? > I don't like this and strongly prefer handling function autoloading using either separate handler or mode flags, and not magic strings. I don't think this "solution" solves a problem that actually exists. The main problem with function autoloading is not integration, but loading order. Unqualified function calls in PHP have a global namespace fallback, i.e. a call like "foo()" in namespace "Bar" will first check if "Bar\foo()" exists and if it doesn't, try global "foo()" instead. How does function autoloading integrate in this? Option A: foo() in namespace Bar will a) Check if Bar\foo() exists b) Otherwise try to load 'Bar\foo' c) If that fails, check if foo() exists d) Otherwise try to load 'foo' e) If that fails, throw. This has the distinct disadvantage that any unqualified use of a function in a namespace will result in an autoloader call. As practically nobody qualifies all their calls, this would add a huge overhead to each internal function call (those are nowadays the only ones in the global namespace). I don't think this is realistic. As such: Option B: foo() in namespace Bar will a) Check if Bar\foo() exists b) Otherwise check if foo() exists c) Otherwise try to load 'Bar\foo' d) Otherwise try to load 'foo' e) If all fails, throw. This avoids the autoloading overhead when calling functions from the global namespace. However this also means that you cannot autoload a function with the same name as a global function through an unqualified call. In practice, this is probably less problematic than it might sound, because realistically function autoloading would likely operate on the namespace-level rather than the function level, i.e. if one function from a namespace is loaded all of them are, because they are all defined in the same file. In such a setting you would never run into this problem as there would be no unqualified calls to functions that have not been loaded yet. However, it *would* be an issue if you tried to, say, map each function to a separate file. Regards, Nikita
