|
|
php.net | support | documentation | report a bug | advanced search | search howto | statistics | random bug | login |
[2018-11-03 16:12 UTC] fabien dot villepinte at gmail dot com
Description:
------------
The functions define(), defined() and constant() don't work the same way whether the name of the constant given as an argument is the name of an already defined constant or the name of a magic constant.
For instance :
define('PHP_EOL', 0); // returns false and emits a notice
define('__DIR__', 0); // returns true
__DIR__; // returns the path of the current directory
constant('__DIR__'); // returns the value of the user-defined constant, 0 in this case
There is nothing in the documentation mentioning this case, so I can't say if the current behavior is correct or not.
Test script:
---------------
<?php
var_dump(defined('__LINE__'));
var_dump(define('__LINE__', 'test'));
var_dump(constant('__LINE__'));
var_dump(__LINE__);
Actual result:
--------------
bool(false)
bool(true)
string(4) "test"
int(5)
PatchesPull RequestsHistoryAllCommentsChangesGit/SVN commits
|
|||||||||||||||||||||||||||
Copyright © 2001-2024 The PHP GroupAll rights reserved. |
Last updated: Thu Apr 25 14:01:31 2024 UTC |
To continue that: Which means that when run time comes around the names are free to be redefined as user constants. But the literal identifiers aren't in the code any more so the only way they can be accessed is through the constant('') construct. The compile-time evaluation of magic constants also means define('__DIR__', 0); is legal, but const __DIR__ = 0; is not, since the latter definition happens at compile time too. Maybe a check (c|sh)ould be made that users aren't trying to define() magic constants themselves? I'd suggest prepopulating the constants table with them only I don't know how something like constant('__LINE__') could work.> I'd suggest prepopulating the constants table with them only I > don't know how something like constant('__LINE__') could work. Prepopulating the constants table would require to update it frequently during run time to be consistent with the compile time constants. Besides being inefficient, it would also change “constant” values. It might be preferable to not allow any of the predefined magic constants (or maybe all constant names with two leading and trailing underscores) to be used with define(), defined() and constant(). Either way would break BC, though.I'm inclined to mark this won't fix. Magic constants are just keywords that have syntactical similarity to constants. I don't think we have any business checking that constant names do not match keywords. If we forbid define('__DIR__'), should we also forbid define('function')? Ultimately everything here works just fine in the sense that you can define that constant and also look it up using constant(). You can't access it with constant syntax because the __DIR__ keyword takes precedence, but there's any number of other reasons why such an access may not be possible. For example, you can also do define('123', 'foo') and then access it using constant('123'), but for obvious reasons it will not change the behavior of directly writing 123 in code.That is consistent with variable names: just as you can create a variable that has a name impossible to use as a literal (e.g., ${"$"}) so too you can create a constant that can't be referred to literally.