PHP RFC: Lexical Scope Support for Anonymous Classes
- Version: 0.2
- Date: 2016-04-19
- Author: krakjoe
- Status: Under Discussion
- First Published at: http://wiki.php.net/rfc/lexical-anon
Introduction
Anonymous classes are cumbersome to use because they lack support for lexical scope [use()].
If the anonymous class has dependencies, they must be injected via the constructor, which is verbose, repetitive, and widens the margin for error considerably.
Proposal
We allow anonymous classes to import variables into their property table using the familiar syntax.
The following syntax is used:
new class [extends Parent] [implements Interface] [use(...)] {}
Where use is one or more variables, property references, or a references to either.
The following code:
$glow = 5; $foo = new class use($glow) { };
Is functionally equivalent to:
$glow = 5; $foo = new class($glow) { public function __construct($glow) { $this->glow = $glow; } private $glow; };
By default, imported variables shall be declared private members of the anonymous class unless the class declaration overrides this behaviour, as the following example shows:
$glow = 5; $foo = new class() use($glow) { public $glow; };
References are also accepted, so that the following code:
$glow = 5; $foo = new class() use(&$glow) { };
Is functionally equivalent to:
$glow = 5; $foo = new class($glow) { public function __construct(&$glow) { $this->glow =& $glow; } private $glow; };
Syntax Choices
Placing the [use(...)] anywhere else in the class declaration seems to be confusing, the current position seems the obvious choice.
It has been suggested that it would be better to have [use(...)] on each function declaration. This would lead to a lot of repetition, and fragmented code that is more difficult to reason about.
Another suggestion made is to allow symbols inline, so that the following code is legal:
class { public $prop = $var; }
This has some appealing advantages, such as using expression to initialize properties. But it also raises huge inconsistencies, why should this:
class { public $prop = new Foo(); }
Be allowed, while not allowing the same thing in normal classes.
Additionally, the following code would also have to be legal:
class { public $prop = &$this->prop; }
This is almost literally backwards.
If we are going to allow expressions for anonymous classes that we do not allow for normal classes, then it is better to leave that for another RFC.
What we are doing is only importing symbols, we don't need to invent a new way to initialize them.
The cognitive overhead of “anything use'd by the declaration is a member property” is almost nil.
Error Conditions
The same restrictions that apply to use() on function declarations apply:
- Must not be superglobal name
- Must not be
$this
The property reference must be well formed, and accessible in the lexical scope:
$object->property
A property name cannot be used twice:
use($prop, $prop)
and
use($prop, $this->prop)
Will both raise compile time errors:
Fatal error: Cannot use property name glow twice in /in/file on line 6
Backward Incompatible Changes
What breaks, and what is the justification for it?
Proposed PHP Version(s)
7.1
RFC Impact
To Existing Extensions
Trivial changes for new opcodes and new sequences.
To Opcache
Will need adjustment, rather trivial (if you are Dmitry)
Open Issues
Make sure there are no open issues when the vote starts!
Proposed Voting Choices
2/3 majority required, simple yes/no vote.
Patches and Tests
Implementation
After the project is implemented, this section should contain
- the version(s) it was merged to
- a link to the git commit(s)
- a link to the PHP manual entry for the feature
References
Links to external references, discussions or RFCs
Rejected Features
Keep this updated with features that were discussed on the mail lists.