PHP字符串常量过度使用？

Question

我有两个特殊情况,我不同意同事,是否应该使用常数.

我们使用一个大致类似Symfony 1.x的自制框架.

1. 初始代码是路由PHP配置文件中的路由,如下所示:

   $router->map('/some_url', array('module' => 'some_module', 'action' => 'some_action'));
   $router->map('/some_other_url', array('module' => 'some_module', 'action' => 'some_action'));
   // etc.
   

   同事将其改为:

   $router->map('/some_url', array(MODULE => 'some_module', ACTION => 'some_action'));
   $router->map('/some_other_url', array(MODULE => 'some_module', ACTION => 'some_action'));
   
   // + in constants.php file:
   define('MODULE', 'module');
   define('ACTION', 'action');
   

   IMO这是不断过度使用:如果"模块"或"动作"的概念被重命名,则必须在整个代码中重命名,或者写成字符串或常量.另外,上面定义的常量名称没有非常具体的含义,有利于命名冲突/混淆.

2. 初始代码示例:

   if (isset($_SESSION['unid']) && isset($_SESSION['login'])) { ... }
   

   由同事修改:

   if (isset($_SESSION[UNID]) && isset($_SESSION[LOGIN])) { ... }
   
   // + in a constants.php file:
   define('UNID', 'unid');
   define('LOGIN', 'login');
   

   在我们的应用程序中,那些会话变量名称unid并且login显然不太可能发生变化.尽管如此,如果声明常量在这里真的是一个很好的做法,我建议至少更准确的名称,例如FIELDNAME_UNID和FIELDNAME_LOGIN...

引入那些常常真正相关的常量(即命名应该改进),还是(我猜)完全没用？

谢谢.

编辑

几个月后,这里有一些来自constants.php文件的(令人难以置信的)行.我绝对觉得这是一个完全无用的混乱,类似于这个DailyWTF帖子.常量太多会导致常数死亡.

define('POST', 'POST');
define('GET', 'GET');

define('PROJECT', 'project');
define('APPLICATION', 'application');
define('MODULE', 'module');
define('ACTION', 'action');
define('ID', 'id');
define('SLUG', 'slug');
define('CONTROLLER', 'controller');
define('CONTENT', 'content');
define('AJAX', 'ajax');
define('EXECUTE', 'execute');
define('FORMAT', 'format');
define('BASE_HREF_CONSTANT', 'basehref');
define('UNID', 'unid');
define('USERNAME', 'username');
define('PASSWORD', 'password');
define('TEMPLATE', 'templates');
define('UNSECURE', 'unsecure');
define('MODE', 'mode');
define('MESSAGE', 'message');
define('TEMPORARY_SESSION', 'temporary_session');
define('ERRORMESSAGE', 'errormessage');
define('START_FROM', 'startfrom');
define('COUNT', 'count');

// and so on.

Answer 1

好处

• 拼写错误常量的后果应该触发E_NOTICE'使用未定义的常量',而拼写字符串文字不会提供这样的早期警告.
• 如果遵循其逻辑结论,代码中任何剩余的字符串文字应该是自然语言,因此识别要包装在国际化转换层中的字符串的任务变得更容易一些.

缺点

• 要求您定义所有常量,无论您是否需要它们.除非你定义了数以千计的瓶颈,否则不太可能是你的性能瓶颈!

Answer 2

使用这样的常量有一个有效的参数.

如果你不小心做了类似的事情:

$router->map('/some_url', array('moduel' => 'some_module', 'action' => 'some_action'));

它将以某种未定义的方式失败(注意拼写错误的"模块").

如果在涉及常量时出现拼写错误或拼写错误,PHP会发出通知,然后立即抓住它.

实际上拯救你的频率是一个争论的问题.就个人而言,我通常不认为值得这么麻烦.